Last modified: 2012-04-16 09:16:13 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 T30236, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 28236 - PHP fatal in UploadStash currently deployed on WMF wikis
PHP fatal in UploadStash currently deployed on WMF wikis
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Uploading (Other open bugs)
unspecified
All All
: Highest major with 1 vote (vote)
: ---
Assigned To: Nobody - You can work on this!
http://commons.wikimedia.org/wiki/Spe...
: code-update-regression
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-03-25 15:05 UTC by Mormegil
Modified: 2012-04-16 09:16 UTC (History)
5 users (show)

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


Attachments

Description Mormegil 2011-03-25 15:05:40 UTC
The wmf branch currently deployed on Wikimedia has broken [[Special:UploadStash]], ending in

  PHP fatal error in /usr/local/apache/common-local/php-1.17/includes/specials/SpecialUploadStash.php line 363:
  Call to undefined method HTMLForm::prepareForm()

The changes which got merged in at r79129 reference HTMLForm::prepareForm(), introduced in r78425, which has not been ported to the wmf branch.
Comment 1 Chad H. 2011-03-25 15:11:10 UTC
Merged the change in r84748 and sync'd it live, thanks for tracking this down :)
Comment 2 Mormegil 2011-03-25 15:15:08 UTC
(In reply to comment #1)
> Merged the change in r84748 and sync'd it live, thanks for tracking this down
> :)

OK, much better now:

PHP fatal error in /usr/local/apache/common-local/php-1.17/includes/specials/SpecialUploadStash.php line 369:
Call to undefined method UploadStash::listFiles() 

;-)
Comment 3 Mormegil 2011-03-26 17:37:51 UTC
In this case, it seems r78426 has been merged to SpecialUploadStash.php, but not to UploadStash.php in r79129… :-? Even though r78426 (nor any other upload-stash-related change) is not listed in the list of revisions allegedly merged in at r79129.
Comment 4 Neil Kandalgaonkar 2011-03-26 23:34:49 UTC
Hey there. Some context:

UploadStash::listFiles() is an undocumented feature. You can invoke it by visiting the Special:UploadStash url, without any subpage url appended, e.g. 

   http://commons.wikimedia.org/wiki/Special:UploadStash/

There are no links to this URL anywhere I am aware of. People may guess the URL by removing the file portion of certain URLs for image previews in UploadWizard. 

Mormegil's diagnosis seems correct, to fix it, just push the change r78426 for includes/upload/UploadStash.php. Here's the diff.

   r1=78425&r2=78426">http://svn.wikimedia.org/viewvc/mediawiki/trunk/phase3/includes/upload/UploadStash.php?&pathrev=78426&r1=78425&r2=78426

Some pushes for UploadWizard / UploadStash are already scheduled for next week. If it's not a crisis we can wait till then.
Comment 5 Roan Kattouw 2011-03-30 09:00:14 UTC
(In reply to comment #4)
> Some pushes for UploadWizard / UploadStash are already scheduled for next week.
> If it's not a crisis we can wait till then.
This is done now.

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


Navigation
Links