Last modified: 2013-09-05 23:28:52 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T54683, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 52683 - "Old" is broken: Error: mw.Title: Could not parse title ".digamma"
"Old" is broken: Error: mw.Title: Could not parse title ".digamma"
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
PageCuration (Other open bugs)
master
All All
: High major (vote)
: ---
Assigned To: bsitu
:
Depends on: 51308
Blocks: 53504
  Show dependency treegraph
 
Reported: 2013-08-09 22:54 UTC by Oliver Keyes
Modified: 2013-09-05 23:28 UTC (History)
9 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
Screenshot (32.75 KB, image/png)
2013-08-09 22:54 UTC, Oliver Keyes
Details

Description Oliver Keyes 2013-08-09 22:54:09 UTC
Created attachment 13082 [details]
Screenshot

See associated screenshot; if you go to the "old" view of Page Curation, it is, for some unknown reason, totally borked. Reported by multiple users.
Comment 1 Andre Klapper 2013-08-10 04:28:52 UTC
Cannot confirm with Firefox 22.0 on F19 on http://en.wikipedia.org/wiki/Special:NewPagesFeed though the "loading" animation next to the "Refresh list" button does not stop.

Exact URLs and browser info to reproduce welcome (in general), and/or links to forum threads.
Comment 2 Oliver Keyes 2013-08-10 04:31:38 UTC
Forum threads?

The exact URl is Special:NewPagesFeed; there's no real way to modify that URL. Firefox 22/Windows 7, Iceweasel 17/latest stable Debian release. I'll ask other users for their info.
Comment 3 Oliver Keyes 2013-08-10 06:33:54 UTC
Also on Chrome 28.0.1500.95/Windows 7.
Comment 4 Superzerocool 2013-08-10 06:39:00 UTC
I confirm the error. There is an error with javascript that renders mw.Title item.

Chrome 28.0, Fedora 19
Comment 6 Bartosz Dziewoński 2013-08-10 12:09:32 UTC

*** This bug has been marked as a duplicate of bug 51308 ***
Comment 7 Oliver Keyes 2013-08-10 13:19:35 UTC
Are we sure this is the same bug? It wasn't previously breaking on full-stop including pages.
Comment 8 Bartosz Dziewoński 2013-08-12 14:54:42 UTC
(In reply to comment #7)
> Are we sure this is the same bug? It wasn't previously breaking on full-stop
> including pages.

This seems rather clear:

(In reply to comment #5)
> Error: Error: mw.Title: Could not parse title ".digamma"
Comment 9 Oliver Keyes 2013-08-12 15:02:59 UTC
Sure, but again, we've had that bug live and present for months and it has never caused those effects before. I'm not particularly technical, but I'd suggest there are two possible scenarios here:

*The mw.Title parse failure is the source of the weird failure to display old pages - something that has never happened before, even with other failures in the same system to parse full stops;
*The two bugs are unrelated and occurring at the same time; the old page is loaded, generating the mw.Title parse error message, and then drops out of existence due to a second issue - something that would make sense since in testing the failure of the "old" view I've noticed that it's not failing to load the content: it loads as a FOUC and then vanishes from view.
Comment 10 Oliver Keyes 2013-08-15 03:29:56 UTC
Reopening; sorry, but side-effects of an existing bug do not simply reappear, independent of the initial bug, N months later, without any tweaks in the interim.
Comment 11 Oliver Keyes 2013-08-19 15:37:35 UTC
And reprioritising, too. This has left the suite totally broken.
Comment 12 Erik Moeller 2013-08-28 17:51:16 UTC
Not sure why Nischay is an assignee, he is no longer at WMF. Andre, could you check the default assignee list? Adding Benny who should be able to help with this.

Fabrice, this is a pretty critical new bug which we'll have to get sorted out ASAP.
Comment 13 Krinkle 2013-08-28 20:00:38 UTC
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) {
 }
Comment 14 Andre Klapper 2013-08-29 12:11:30 UTC
(In reply to comment #12 by Erik)
> Not sure why Nischay is an assignee, he is no longer at WMF. Andre, could you
> check the default assignee list?

Default assignee had been nobody@ already; Nischay was set manually.

(In reply to comment #13 by Krinkle)
> The underlying bug is in core.

I assume it's bug 51308, adding dependency.
Comment 15 Gerrit Notification Bot 2013-09-04 06:02:09 UTC
Change 82575 had a related patch set uploaded by Bsitu:
Get rid of some javascript mw.title related code

https://gerrit.wikimedia.org/r/82575
Comment 16 Gerrit Notification Bot 2013-09-05 21:45:46 UTC
Change 82575 merged by jenkins-bot:
Get rid of some javascript mw.title related code

https://gerrit.wikimedia.org/r/82575

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links