Last modified: 2014-10-24 07:53:26 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 T74441, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 72441 - Argument 1 passed to Flow\Content\Content::onShowMissingArticle() must be an instance of Article, instance of Wikibase\ViewItemAction in Content.php
Argument 1 passed to Flow\Content\Content::onShowMissingArticle() must be an ...
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
WikidataRepo (Other open bugs)
unspecified
All All
: Unprioritized normal (vote)
: ---
Assigned To: Wikidata bugs
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-10-23 18:43 UTC by Aude
Modified: 2014-10-24 07:53 UTC (History)
7 users (show)

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


Attachments

Description Aude 2014-10-23 18:43:09 UTC
when viewing a (non-existent) page in the item namespace, with Wikibase Repo + Flow enabled together.

[23-Oct-2014 18:40:32 UTC] PHP Catchable fatal error:  Argument 1 passed to Flow\Content\Content::onShowMissingArticle() must be an instance of Article, instance of Wikibase\ViewItemAction given in /Library/WebServer/Documents/php-master/extensions/Flow/includes/Content/Content.php on line 25
[23-Oct-2014 18:40:32 UTC] PHP Stack trace:                                                               
[23-Oct-2014 18:40:32 UTC] PHP   1. {main}() /Library/WebServer/Documents/php-master/index.php:0    
[23-Oct-2014 18:40:32 UTC] PHP   2. MediaWiki->run() /Library/WebServer/Documents/php-master/index.php:46
[23-Oct-2014 18:40:32 UTC] PHP   3. MediaWiki->main() /Library/WebServer/Documents/php-master/includes/MediaWiki.php:435
[23-Oct-2014 18:40:32 UTC] PHP   4. MediaWiki->performRequest() /Library/WebServer/Documents/php-master/includes/MediaWiki.php:584
[23-Oct-2014 18:40:32 UTC] PHP   5. MediaWiki->performAction() /Library/WebServer/Documents/php-master/includes/MediaWiki.php:282[23-Oct-2014 18:40:32 UTC] PHP   6. Wikibase\ViewEntityAction->show() /Library/WebServer/Documents/php-master/includes/MediaWiki.php:414
[23-Oct-2014 18:40:32 UTC] PHP   7. Wikibase\ViewEntityAction->displayMissingEntity() /Library/WebServer/Documents/php-master/extensions/WikidataBuild/extensions/Wikibase/repo/includes/actions/ViewEntityAction.php:124
[23-Oct-2014 18:40:32 UTC] PHP   8. wfRunHooks() /Library/WebServer/Documents/php-master/extensions/WikidataBuild/extensions/Wikibase/repo/includes/actions/ViewEntityAction.php:245
[23-Oct-2014 18:40:32 UTC] PHP   9. Hooks::run() /Library/WebServer/Documents/php-master/includes/GlobalFunctions.php:4021[23-Oct-2014 18:40:32 UTC] PHP  10. call_user_func_array:{/Library/WebServer/Documents/php-master/includes/Hooks.php:206}() /Library/WebServer/Documents/php-master/includes/Hooks.php:206
[23-Oct-2014 18:40:32 UTC] PHP  11. Flow\Content\Content::onShowMissingArticle() /Library/WebServer/Documents/php-master/includes/Hooks.php:206
Comment 1 Erik Bernhardson 2014-10-23 18:49:34 UTC
At a minimum, it looks like https://www.mediawiki.org/wiki/Manual:Hooks/ShowMissingArticle needs to be updated to say that the first argument can be literlly anything, and not an article as implied(but not guaranteed)
Comment 2 Erik Bernhardson 2014-10-23 18:53:09 UTC
Additionally, i'm not sure how i feel about passing a non-article through this hook.  If this hook had been created yesterday instead of in 1.16 i'm almost certain it would have gotten the Article type hint.
Comment 3 Gerrit Notification Bot 2014-10-23 18:55:59 UTC
Change 168383 had a related patch set uploaded by Aude:
Provide Article object for ShowMissingArticle hook

https://gerrit.wikimedia.org/r/168383
Comment 4 Gerrit Notification Bot 2014-10-24 07:53:26 UTC
Change 168383 merged by jenkins-bot:
Provide Article object for ShowMissingArticle hook

https://gerrit.wikimedia.org/r/168383

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


Navigation
Links