User "Catrope" posted a comment on MediaWiki.r89581.

Full URL: 
https://secure.wikimedia.org/wikipedia/mediawiki/wiki/Special:Code/MediaWiki/89581#c19669
Commit summary:

* (bug 27716) Make a method to do checking of 0 or 1 of the parameters 
existence (like requireOnlyOneParameter), but without needing one of the 
parameters

Comment:

<pre>
+        * Die if more than one of a certain set of parameters is set and not 
false.
+        *
+        * @param $params array
</pre>
Should document that varargs is being used here, and that <code>$params</code> 
means something entirely different in 
<code>getRequireMaxOneParameterErrorMessage()</code> (which BTW is the craziest 
method name ever).

_______________________________________________
MediaWiki-CodeReview mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview

Reply via email to