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

Krinkle <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #13 from Krinkle <[email protected]> ---
The underlying bug is in core. mw.Title is a mess right now that in some cases
is more strict and sometimes less strict than the Title.php equivalent.

I'm currently working on rewriting that core module.

Meanwhile, I think this NewPageFeed bug is caused by something no longer
catching the exception when constructing a mw.Title object from arbitrary input
(like page names). Or perhaps it never caught it to begin with.

Either way, though I don't like this pattern, in JS right now mw.Title throws
on invalid titles, just like Title.php returns null instead of a Title instance
from Title::newFromText if the input is invalid.


So where in PHP you need:

 $t = Title::newFromText();
 if ( $t !== null ) {
 } else {
 }

in JS you need:

 try {
  $t = new mw.Title();
 } catch (e) {
 }

-- 
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

Reply via email to