A wrong value is removed from the values and a empty array is threated like a valid value. For core you can see this with prop=revisions. When giving an empty rvprop= or a rvprop= with wrong value, that results in the same output, because no of the properties are requested. Requesting no properties makes no sense, but is valid at the moment (no need for b/c).

In my opinion is there no problem, because it is not required that a value must be set and than the empty value is ok.

But setting PARAM_REQUIRED = true has no effect. That is the bug, in my opinion.

(In reply to comment #1)

But setting PARAM_REQUIRED = true has no effect. That is the bug, in my
opinion.

Commited gerrit 52987 for this

(In reply to comment #2)
Commited Gerrit change #52987 for this

Abandoned for the moment, does not work as aspected.

Still there is a patch in gerrit that somebody could pick up, hence restoring patch-in-gerrit, keyword.

In T41830#440226, @Aklapper wrote:

Still there is a patch in gerrit that somebody could pick up, hence restoring patch-in-gerrit, keyword.

While there was a patch, it's not something someone could "pick up". The approach there turned out to be unfeasible, and while the review there has useful information for someone writing a new patch it would in fact be an entirely new patch.