Last modified: 2012-10-23 20:36:01 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 T43305, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 41305 - PagedTiffHandler: Invalid message parameter
PagedTiffHandler: Invalid message parameter
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
PagedTiffHandler (Other open bugs)
unspecified
All All
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
http://test.wikipedia.org/wiki/File:W...
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-10-23 15:02 UTC by Sam Reed (reedy)
Modified: 2012-10-23 20:36 UTC (History)
2 users (show)

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


Attachments

Description Sam Reed (reedy) 2012-10-23 15:02:13 UTC
http://p.defau.lt/?JJBsKNNyIiaSPovUyKTXKw

2012-10-23 15:00:07 mw25 commonswiki: [4e470b44] /wiki/File:Weiss_-_Biographie_universelle_ou_dictionnaire_historique,_tome_2.tif?page=173   Exception from line 621 of /usr/local/apache/common-local/php-1.21wmf2/includes/Message.php: Invalid message parameter
#0 /usr/local/apache/common-local/php-1.21wmf2/includes/Message.php(595): Message->extractParam(Array)
#1 /usr/local/apache/common-local/php-1.21wmf2/includes/Message.php(451): Message->replaceParameters('$1 ?? $2 pixels...', 'before')
#2 /usr/local/apache/common-local/php-1.21wmf2/includes/Message.php(493): Message->toString()
#3 /usr/local/apache/common-local/php-1.21wmf2/extensions/PagedTiffHandler/PagedTiffHandler_body.php(516): Message->parse()
#4 /usr/local/apache/common-local/php-1.21wmf2/includes/filerepo/file/File.php(1745): PagedTiffHandler->getLongDesc(Object(LocalFile))
#5 /usr/local/apache/common-local/php-1.21wmf2/includes/ImagePage.php(330): File->getLongDesc()
#6 /usr/local/apache/common-local/php-1.21wmf2/includes/ImagePage.php(149): ImagePage->openShowImage()
#7 /usr/local/apache/common-local/php-1.21wmf2/includes/actions/ViewAction.php(37): ImagePage->view()
#8 /usr/local/apache/common-local/php-1.21wmf2/includes/Wiki.php(428): ViewAction->show()
#9 /usr/local/apache/common-local/php-1.21wmf2/includes/Wiki.php(305): MediaWiki->performAction(Object(ImagePage))
#10 /usr/local/apache/common-local/php-1.21wmf2/includes/Wiki.php(554): MediaWiki->performRequest()
#11 /usr/local/apache/common-local/php-1.21wmf2/includes/Wiki.php(447): MediaWiki->main()
#12 /usr/local/apache/common-local/php-1.21wmf2/index.php(59): MediaWiki->run()
#13 /usr/local/apache/common-local/live-1.5/index.php(3): require('/usr/local/apac...')
#14 {main}
Comment 1 Siebrand Mazeland 2012-10-23 15:03:07 UTC
Looking into this.
Comment 2 Siebrand Mazeland 2012-10-23 15:56:24 UTC
Niklas made Gerrit change #29594 which adds more informative debugging into to Message and Database class.

I submitted Gerrit change #29593 which refactors the code a bit and checks for all message parameters if they're not an array (otherwise throws exception in the class itself, logging which parameter was an array).
Comment 3 Siebrand Mazeland 2012-10-23 16:15:16 UTC
After deploying the patches sets from comment 2, added URL to reproduce and this backtrace:

Invalid message parameter: a:1:{s:3:"num";N;}

Backtrace:

#0 /home/wikipedia/common/php-1.21wmf2/includes/Message.php(595): Message->extractParam(Array)
#1 /home/wikipedia/common/php-1.21wmf2/includes/Message.php(451): Message->replaceParameters('$1 ?? $2 pixels...', 'before')
#2 /home/wikipedia/common/php-1.21wmf2/includes/Message.php(493): Message->toString()
#3 /home/wikipedia/common/php-1.21wmf2/extensions/PagedTiffHandler/PagedTiffHandler_body.php(516): Message->parse()
#4 /home/wikipedia/common/php-1.21wmf2/includes/filerepo/file/File.php(1745): PagedTiffHandler->getLongDesc(Object(ForeignDBFile))
#5 /home/wikipedia/common/php-1.21wmf2/includes/ImagePage.php(330): File->getLongDesc()
Comment 4 Sam Reed (reedy) 2012-10-23 16:32:23 UTC
2012-10-23 16:30:45 srv201 commonswiki: [6ac008dd] /wiki/File:Petitot_-_Collection_compl%C3%A8te_des_m%C3%A9moires_relatifs_%C3%A0_l%E2%80%99histoire_de_France,_tome_16.tif?page=97   Exception from line 621 of /usr/local/apache/common-local/php-1.21wmf2/includes/Message.php: Invalid message parameter: a:1:{s:3:"num";N;}
#0 /usr/local/apache/common-local/php-1.21wmf2/includes/Message.php(595): Message->extractParam(Array)
#1 /usr/local/apache/common-local/php-1.21wmf2/includes/Message.php(451): Message->replaceParameters('$1 ?? $2 pixels...', 'before')
#2 /usr/local/apache/common-local/php-1.21wmf2/includes/Message.php(493): Message->toString()
#3 /usr/local/apache/common-local/php-1.21wmf2/extensions/PagedTiffHandler/PagedTiffHandler_body.php(516): Message->parse()
#4 /usr/local/apache/common-local/php-1.21wmf2/includes/filerepo/file/File.php(1745): PagedTiffHandler->getLongDesc(Object(LocalFile))
#5 /usr/local/apache/common-local/php-1.21wmf2/includes/ImagePage.php(330): File->getLongDesc()
#6 /usr/local/apache/common-local/php-1.21wmf2/includes/ImagePage.php(149): ImagePage->openShowImage()
#7 /usr/local/apache/common-local/php-1.21wmf2/includes/actions/ViewAction.php(37): ImagePage->view()
#8 /usr/local/apache/common-local/php-1.21wmf2/includes/Wiki.php(428): ViewAction->show()
#9 /usr/local/apache/common-local/php-1.21wmf2/includes/Wiki.php(305): MediaWiki->performAction(Object(ImagePage))
#10 /usr/local/apache/common-local/php-1.21wmf2/includes/Wiki.php(554): MediaWiki->performRequest()
#11 /usr/local/apache/common-local/php-1.21wmf2/includes/Wiki.php(447): MediaWiki->main()
#12 /usr/local/apache/common-local/php-1.21wmf2/index.php(59): MediaWiki->run()
#13 /usr/local/apache/common-local/live-1.5/index.php(3): require('/usr/local/apac...')
#14 {main}
Comment 5 Siebrand Mazeland 2012-10-23 20:36:01 UTC
Gerrit change #29605 eventually fixed the fatal. I think we fixed a symptom and not the root cause this time. There's an issue with metadata values a bit deeper. My interest in this issue has ended with getting the fatal fixed, though.

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


Navigation
Links