Last modified: 2012-11-29 12:43:08 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 T42639, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 40639 - Diff between .js- and .css-revisions broken
Diff between .js- and .css-revisions broken
Status: VERIFIED FIXED
Product: MediaWiki extensions
Classification: Unclassified
WikidataRepo (Other open bugs)
master
All All
: High blocker (vote)
: ---
Assigned To: Wikidata bugs
:
Depends on:
Blocks: 38622
  Show dependency treegraph
 
Reported: 2012-09-30 14:21 UTC by jeblad
Modified: 2012-11-29 12:43 UTC (History)
3 users (show)

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


Attachments

Description jeblad 2012-09-30 14:21:08 UTC
This is not verified in test or dev, only at a local repo.

Create [[Mediawiki:Vector.js]] and [[Mediawiki:Vector.css]] and make additional revisjons. Then take a diff between the two revisions. The following was then spit out from a localhost. It seems to be identical for both pages. The following is from the css page.

Notice: Undefined property: DifferenceEngine::$mContentObject in /home/john/Workspace/core/includes/diff/DifferenceEngine.php on line 525 Call Stack: 0.0004 648480

 1. {main}() /home/john/Workspace/core/index.php:0 0.0309 4170776
 2. MediaWiki->run() /home/john/Workspace/core/index.php:59 0.0310 4170776
 3. MediaWiki->main() /home/john/Workspace/core/includes/Wiki.php:446 0.0884 7562064
 4. MediaWiki->performRequest() /home/john/Workspace/core/includes/Wiki.php:536 0.0990 8477808
 5. MediaWiki->performAction() /home/john/Workspace/core/includes/Wiki.php:304 0.1040 8495416
 6. ViewAction->show() /home/john/Workspace/core/includes/Wiki.php:427 0.1040 8495416
 7. Article->view() /home/john/Workspace/core/includes/actions/ViewAction.php:37 0.1068 8509528
 8. Article->showDiffPage() /home/john/Workspace/core/includes/Article.php:553 0.1271 9460728
 9. DifferenceEngine->showDiffPage() /home/john/Workspace/core/includes/Article.php:823 0.5088 21291408
10. DifferenceEngine->renderNewRevision() /home/john/Workspace/core/includes/diff/DifferenceEngine.php:415

Fatal error: Call to a member function getParserOutput() on a non-object in /home/john/Workspace/core/includes/diff/DifferenceEngine.php on line 525 Call Stack: 0.0004 648480
 1. {main}() /home/john/Workspace/core/index.php:0 0.0309 4170776
 2. MediaWiki->run() /home/john/Workspace/core/index.php:59 0.0310 4170776
 3. MediaWiki->main() /home/john/Workspace/core/includes/Wiki.php:446 0.0884 7562064
 4. MediaWiki->performRequest() /home/john/Workspace/core/includes/Wiki.php:536 0.0990 8477808
 5. MediaWiki->performAction() /home/john/Workspace/core/includes/Wiki.php:304 0.1040 8495416
 6. ViewAction->show() /home/john/Workspace/core/includes/Wiki.php:427 0.1040 8495416
 7. Article->view() /home/john/Workspace/core/includes/actions/ViewAction.php:37 0.1068 8509528
 8. Article->showDiffPage() /home/john/Workspace/core/includes/Article.php:553 0.1271 9460728
 9. DifferenceEngine->showDiffPage() /home/john/Workspace/core/includes/Article.php:823 0.5088 21291408
10. DifferenceEngine->renderNewRevision() /home/john/Workspace/core/includes/diff/DifferenceEngine.php:415
Comment 1 jeblad 2012-10-04 09:37:46 UTC
Confirmed in Wikidata Test.

http://wikidata-test-repo.wikimedia.de/w/index.php?title=MediaWiki:Vector.js&curid=11919&diff=36813&oldid=36812

Fatal error: Call to a member function getParserOutput() on a non-object in /var/www/wikidata-test-repo.wikimedia.de/w/includes/diff/DifferenceEngine.php on line 525
Comment 2 jeblad 2012-10-04 10:02:29 UTC
Seems like something either kills the mContentObject in the hook
'ShowRawCssJs', or its not properly initialized, or it is simply wrong (its
never used anywhere else).

Line 525 is
 $po = $this->mContentObject->getParserOutput();
Comment 3 Daniel Kinzler 2012-10-04 10:13:36 UTC
Fixed in I8801d49a
Comment 4 jeblad 2012-10-04 13:04:21 UTC
Confirmed fix after a pull on core.
Comment 5 Anja Jentzsch 2012-11-29 12:43:08 UTC
Verified in Wikidata demo time for sprint 18

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


Navigation
Links