https://bugzilla.wikimedia.org/show_bug.cgi?id=28224
--- Comment #3 from Reedy <[email protected]> 2011-03-24 21:06:30 UTC --- (In reply to comment #2) > But null is not an empty string. When the default is nothing, than > action=paraminfo should say that as it does for the long form with > ApiBase::PARAM_TYPE => 'string' > > action=protect and action=edit use 'reason' => '', that is useless than. action=undelete and action=protect you mean They pass the variables straight into something else, where the default parameter is $blah = '', so passing null isn't nice. 'foo' => '' in protect/undelete give <param name="reason" description="Reason for (un)protecting (optional)" type="string" default="" /> Which is correct. Would you rather I made those explicit? I can see some benefit switching => null to array( ApiBase::PARAM_TYPE => 'string' ), as it explicitally sets no default, and the default used is null. Is that what you were wanting originally? -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ Wikibugs-l mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
