Last modified: 2012-08-04 21:10:53 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 T38395, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 36395 - Article Feedback Page - Some filters do not load comments, and show an error message
Article Feedback Page - Some filters do not load comments, and show an error ...
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
ArticleFeedbackv5 (Other open bugs)
unspecified
All All
: High major (vote)
: ---
Assigned To: mjackson
:
Depends on:
Blocks: 39043
  Show dependency treegraph
 
Reported: 2012-05-01 20:36 UTC by Fabrice Florin
Modified: 2012-08-04 21:10 UTC (History)
1 user (show)

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


Attachments
Feedback Page - Hidden Filter Loading Error for Obama page (150.61 KB, image/png)
2012-05-01 20:54 UTC, Fabrice Florin
Details

Description Fabrice Florin 2012-05-01 20:36:29 UTC
Some filters on the Feedback Page do not load and show this error message:

'Error loading feedback'

This problem occurs when I select 'Hidden' or 'Oversighted' in the 'Showing' drop-down menu on either prototype or production, such as this low-traffic page on prototype:

http://prototype.wikimedia.org/release-en/Special:ArticleFeedbackv5/Golden-crowned_Sparrow

... or this high-traffic page on production:
https://en.wikipedia.org/wiki/Special:ArticleFeedbackv5/Barack_Obama

Both Oliver and I could reproduce the problem on the Obama page today, with vastly different configurations and connections (he's on Win XP with Firefox in the UK, I'm on Mac Safari in the WMF office in CA).

One theory is that it may be related to the number of posts hidden, though we're not sure if that's the real issue.

For example, we cannot view hidden posts on the Obama page (with 185 hidden posts), but I can view them on the China page (with only 27 hidden posts):
https://en.wikipedia.org/wiki/Special:ArticleFeedbackv5/China

However, the Mitt Romney (with only 32 hidden posts) does not show hidden posts either:
http://en.wikipedia.org/wiki/Special:ArticleFeedbackv5/Mitt_Romney

Another theory is that it may be related to the number of oversighted items: all the pages listed above have more than 1 oversighted item -- except for the China page, which has 0 oversighted items.

This is a serious enough issue that we should attempt to solve it for this deployment, if possible.
Comment 1 Fabrice Florin 2012-05-01 20:54:32 UTC
Created attachment 10493 [details]
Feedback Page - Hidden Filter Loading Error for Obama page

Here's a screenshot showing this problem on the Obama page. I also have screenshots for Mitt Romney and Golden Sparrow, if you like.
Comment 2 Reha Sterbin 2012-05-01 22:36:39 UTC
Here's the error message from prototype:

Exception Caught: Non-string key given","*":"#0 \/srv\/org\/wikimedia\/prototype\/wikis\/rc\/includes\/GlobalFunctions.php(1426): MessageCache->get(NULL, true, Object(Language))#1 \/srv\/org\/wikimedia\/prototype\/wikis\/rc\/includes\/GlobalFunctions.php(1551): wfMsgGetKey(NULL, true, Object(Language), false)#2 \/srv\/org\/wikimedia\/prototype\/wikis\/rc\/languages\/Language.php(709): wfMsgExt(NULL, Array)#3 \/srv\/org\/wikimedia\/prototype\/wikis\/rc\/languages\/Language.php(731): Language->getMessageFromDB(NULL)#4 \/srv\/org\/wikimedia\/prototype\/wikis\/rc\/languages\/Language.php(989): Language->getMonthName(false)#5 \/srv\/org\/wikimedia\/prototype\/wikis\/rc\/languages\/Language.php(1819): Language->sprintfDate('j F Y', false)#6 \/srv\/org\/wikimedia\/prototype\/wikis\/rc\/extensions\/ArticleFeedbackv5\/api\/ApiArticleFeedbackv5Utils.php(340): Language->date('??????????????')#7 \/srv\/org\/wikimedia\/prototype\/wikis\/rc\/extensions\/ArticleFeedbackv5\/ArticleFeedbackv5.render.php(117): ApiArticleFeedbackv5Utils::renderStatusLine('autohide', '0', '??????????????')#8 \/srv\/org\/wikimedia\/prototype\/wikis\/rc\/extensions\/ArticleFeedbackv5\/api\/ApiViewFeedbackArticleFeedbackv5.php(58): ArticleFeedbackv5Render->run(Array)
#9 \/srv\/org\/wikimedia\/prototype\/wikis\/rc\/includes\/api\/ApiQuery.php(266): ApiViewFeedbackArticleFeedbackv5->execute()
#10 \/srv\/org\/wikimedia\/prototype\/wikis\/rc\/includes\/api\/ApiMain.php(705): ApiQuery->execute()
#11 \/srv\/org\/wikimedia\/prototype\/wikis\/rc\/includes\/api\/ApiMain.php(360): ApiMain->executeAction()
#12 \/srv\/org\/wikimedia\/prototype\/wikis\/rc\/includes\/api\/ApiMain.php(344): ApiMain->executeActionWithErrorHandling()
#13 \/srv\/org\/wikimedia\/prototype\/wikis\/rc\/api.php(117): ApiMain->execute()
#14 {main}

I think we must have some places in the database where "??????????????" is in the last status timestamp field.
Comment 3 Reha Sterbin 2012-05-01 22:40:59 UTC
NB: the string is not actually "??????????????" but a series of characters that look like question marks.
Comment 4 Fabrice Florin 2012-05-01 23:10:39 UTC
Thanks, Reha.

Do we know which field and/or record in the DB could be causing this problem? 

Is this something that could have been introduced when we ran the SQL script for adding relevance scores to older posts last week? 

If so, could this be solved by asking Roan to re-run an SQL script for us? 

Or is this something you could solve on your end?

Please let us know what you think can be done on our end.
Comment 5 Reha Sterbin 2012-05-02 14:21:11 UTC
The problem is in the last status timestamp field, so the relevance score wouldn't have affected it.  I'm afraid I don't know how to determine which rows are causing the problem -- using the binary field for a timestamp is a non-standard MediaWiki variant, and I'm not quite sure how to search for a series of messed-up unicode characters -- but Roan is much more familiar with this usage and could probably figure it out.
Comment 6 Yoni Shostak 2012-05-04 18:05:58 UTC
mike, reha - please make sure the SQL deploy script is updated and the bug is no longer reproducible.
in this case, close the bug
Comment 7 Fabrice Florin 2012-05-04 19:37:01 UTC
Sorry, I didn't see the last note from Yoni above, so I just re-opened this bug.

Also, on the off chance that fixing this bug introduced new issues, I would like to point out that we now are having $ signs show up again on the central activity log:

https://en.wikipedia.org/w/index.php?title=Special%3ALog&type=articlefeedbackv5

Not sure if it's related to the recent SQL fix by Roan, but wanted to mention it, just in case.

Oliver is opening up a separate bug for this.
Comment 8 Reha Sterbin 2012-05-09 00:50:08 UTC
The log issues are unrelated.  Marking fixed.

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


Navigation
Links