Last modified: 2011-07-22 00:11:27 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 T18589, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 16589 - CoreParserFunctions error causes 500
CoreParserFunctions error causes 500
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Parser (Other open bugs)
1.14.x
All All
: Normal blocker with 1 vote (vote)
: ---
Assigned To: Nobody - You can work on this!
http://de.wikipedia.org/wiki/Vorlage:...
: patch
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-12-08 04:21 UTC by merl
Modified: 2011-07-22 00:11 UTC (History)
2 users (show)

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


Attachments
propsed fix, adds a call to $rev->getId() (558 bytes, patch)
2008-12-09 01:28 UTC, P.Copp
Details

Description merl 2008-12-08 04:21:12 UTC
After my change on the template the template site is sometimes not shown. In the most cases a blank page is returned with an 500er. Sometimes an error message with

PHP fatal error in /usr/local/apache/common-local/php-1.5/extensions/FlaggedRevs/FlaggedArticle.php line 1479: <br/> 
        Object of class Revision could not be converted to string

is returned. If you purge the page everything works fine for YOU (only!) for some time. Other users also see this bug some hours later, and have to purge once again. The template itself works fine when included into other sites and all links are set correctly.
Another user reported that this bug it not shown if not logged in. Discussion was done in irc and on http://de.wikipedia.org/w/index.php?title=Wikipedia:Administratoren/Anfragen&oldid=53885766#Archiv-Vorlage_sch.C3.BCtzen (in german).
Comment 1 P.Copp 2008-12-09 01:27:13 UTC
As it seems, this is not a bug in FlaggedRevs, but in CoreParserFunctions.php :
pagesize() contains the lines

>		$revid = Revision::newFromTitle($title);
>		$parser->mOutput->addTemplate($title, $id, $revid);

Instead of the actual revid it stuffs the whole Revision object into the parser output. Curiously this hasn't led to problems before.
Comment 2 P.Copp 2008-12-09 01:28:24 UTC
Created attachment 5566 [details]
propsed fix, adds a call to $rev->getId()
Comment 3 Aaron Schulz 2008-12-10 23:28:20 UTC
Fixed in r44433

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


Navigation
Links