Last modified: 2013-10-05 16:48:58 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 T56628, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 54628 - GlobalBlockingBlockedIpMsg hook passed a string instead of an array
GlobalBlockingBlockedIpMsg hook passed a string instead of an array
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
GlobalBlocking (Other open bugs)
master
All All
: High major (vote)
: ---
Assigned To: Nobody - You can work on this!
:
: 54658 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-09-26 09:58 UTC by Antoine "hashar" Musso (WMF)
Modified: 2013-10-05 16:48 UTC (History)
7 users (show)

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


Attachments

Description Antoine "hashar" Musso (WMF) 2013-09-26 09:58:56 UTC
Stack trace on beta:

[26-Sep-2013 08:03:52] Catchable fatal error: Argument 2 passed to wfRunHooks() must be an array, string given, called in /data/project/apache/common-local/php-master/extensions/GlobalBlocking/GlobalBlocking.class.php on line 79 and defined at /data/project/apache/common-local/php-master/includes/GlobalFunctions.php on line 3897
Server: i-0000031b
Method: GET
URL: http://simple.wikipedia.beta.wmflabs.org/w/index.php?title=User:Pipeo15&action=edit
Backtrace:
#0 /data/project/apache/common-local/php-master/includes/GlobalFunctions.php(3897): wfRunHooks()
#1 /data/project/apache/common-local/php-master/extensions/GlobalBlocking/GlobalBlocking.class.php(79): wfRunHooks('GlobalBlockingB...', 'globalblocking-...')
#2 /data/project/apache/common-local/php-master/extensions/GlobalBlocking/GlobalBlocking.class.php(23): GlobalBlocking::getUserBlockErrors(Object(User), 'XXX.XXX.XXX.XXX')
#3 [internal function]: GlobalBlocking::getUserPermissionsErrors(Object(Title), Object(User), 'edit', '')
#4 /data/project/apache/common-local/php-master/includes/Hooks.php(199): call_user_func_array('GlobalBlocking:...', Array)
#5 /data/project/apache/common-local/php-master/includes/GlobalFunctions.php(3898): Hooks::run('getUserPermissi...', Array)
#6 /data/project/apache/common-local/php-master/includes/Title.php(1843): wfRunHooks('getUserPermissi...', Array)
#7 /data/project/apache/common-local/php-master/includes/Title.php(2235): Title->checkPermissionHooks('edit', Object(User), Array, true, false)
#8 /data/project/apache/common-local/php-master/includes/Title.php(1717): Title->getUserPermissionsErrorsInternal('edit', Object(User), true)
#9 /data/project/apache/common-local/php-master/includes/EditPage.php(451): Title->getUserPermissionsErrors('edit', Object(User))
#10 /data/project/apache/common-local/php-master/includes/EditPage.php(384): EditPage->getEditPermissionErrors()
#11 /data/project/apache/common-local/php-master/includes/actions/EditAction.php(50): EditPage->edit()
#12 /data/project/apache/common-local/php-master/includes/Wiki.php(441): EditAction->show()
#13 /data/project/apache/common-local/php-master/includes/Wiki.php(305): MediaWiki->performAction(Object(Article), Object(Title))
#14 /data/project/apache/common-local/php-master/includes/Wiki.php(591): MediaWiki->performRequest()
#15 /data/project/apache/common-local/php-master/includes/Wiki.php(460): MediaWiki->main()
#16 /data/project/apache/common-local/php-master/index.php(55): MediaWiki->run()
#17 /data/project/apache/common-local/w/index.php(3): require('/data/project/a...')
#18 {main}


The related code in GlobalBlocking extension:


  // Allow site customization of blocked message.
  $blockedIpMsg = 'globalblocking-ipblocked';
  wfRunHooks( 'GlobalBlockingBlockedIpMsg', &$blockedIpMsg );


Probably caused by https://gerrit.wikimedia.org/r/#/c/84333/


 -  return $result = array( 'globalblocking-ipblocked',
 +  // Allow site customization of blocked message.
 +  $blockedIpMsg = 'globalblocking-ipblocked';
 +  wfRunHooks( 'GlobalBlockingBlockedIpMsg', &$blockedIpMsg );
 +  return $result = array( $blockedIpMsg,


CCing Nemo Bis and Parent5446
Comment 1 Antoine "hashar" Musso (WMF) 2013-09-26 10:02:39 UTC
The first occurrence of that backtrace on the beta cluster happened on [23-Sep-2013 04:29:21] UTC.
Comment 2 Tyler Romeo 2013-09-26 14:57:34 UTC
Strange, this worked on my test wiki. Guess I missed something.
Comment 3 Gerrit Notification Bot 2013-09-26 15:30:57 UTC
Change 86107 had a related patch set uploaded by Parent5446:
Fix error in call to wfRunHooks()

https://gerrit.wikimedia.org/r/86107
Comment 4 Gerrit Notification Bot 2013-09-26 15:37:56 UTC
Change 86109 had a related patch set uploaded by Parent5446:
Fix error in call to wfRunHooks()

https://gerrit.wikimedia.org/r/86109
Comment 5 Sam Reed (reedy) 2013-09-26 18:45:43 UTC
*** Bug 54658 has been marked as a duplicate of this bug. ***
Comment 6 Gerrit Notification Bot 2013-09-26 18:46:25 UTC
Change 86107 merged by jenkins-bot:
Fix error in call to wfRunHooks()

https://gerrit.wikimedia.org/r/86107
Comment 7 Gerrit Notification Bot 2013-09-26 18:46:47 UTC
Change 86109 merged by jenkins-bot:
Fix error in call to wfRunHooks()

https://gerrit.wikimedia.org/r/86109
Comment 8 Gerrit Notification Bot 2013-09-26 19:45:42 UTC
Change 86161 had a related patch set uploaded by Reedy:
Fix error in call to wfRunHooks()

https://gerrit.wikimedia.org/r/86161
Comment 9 Gerrit Notification Bot 2013-09-26 19:46:40 UTC
Change 86161 merged by jenkins-bot:
Fix error in call to wfRunHooks()

https://gerrit.wikimedia.org/r/86161
Comment 10 Gerrit Notification Bot 2013-09-27 18:57:19 UTC
Change 86291 had a related patch set uploaded by Reedy:
Fix error in call to wfRunHooks()

https://gerrit.wikimedia.org/r/86291
Comment 11 Gerrit Notification Bot 2013-09-27 19:08:01 UTC
Change 86291 merged by Chad:
Fix error in call to wfRunHooks()

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

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


Navigation
Links