https://bugzilla.wikimedia.org/show_bug.cgi?id=30230

--- Comment #4 from Reedy <[email protected]> 2011-08-04 20:10:50 UTC ---
(In reply to comment #2)
> (In reply to comment #1)
> >         if ( !$title_obj ) {
> >             $title_obj = Title::newFromText( 'API' ); // default
> >         }
> > It can quite obviously be seen in the code too ;)
> 
> Not all people looking first into the code ;-)
> 
> There is also:
> 
> 'title' => array(
>   ApiBase::PARAM_DFLT => 'API',
> ),
> 
> Maybe make title and text required?

No need to make Title required, if it's not a variable in the template
expansion it is daft.

However, making text needed makes sense, then we don't pointlessly go through
the effort of trying to expand "", and hence, the api can die out earlier by
making it required...

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
You are on the CC list for the bug.

_______________________________________________
Wikibugs-l mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to