Last modified: 2013-08-01 18:54:14 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 T54368, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 52368 - VisualEditor.php attempts to register a hook handler that was removed
VisualEditor.php attempts to register a hook handler that was removed
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
MediaWiki integration (Other open bugs)
unspecified
All All
: Highest normal
: VE-deploy-2013-08-15
Assigned To: Ori Livneh
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-08-01 04:26 UTC by jeremyb
Modified: 2013-08-01 18:54 UTC (History)
6 users (show)

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


Attachments

Description jeremyb 2013-08-01 04:26:39 UTC
VisualEditor.php attempts to register a hook handler that was removed (summary credit goes to Ori)

stack: http://paste.tstarling.com/p/GuTJEM.html

Tim says:

> var_dump($wgHooks['BeforeWelcomeCreation']);
array(2) {
  [0]=>
  string(42) "VisualEditorHooks::onBeforeWelcomeCreation"
  [1]=>
  string(44) "GettingStartedHooks::onBeforeWelcomeCreation"
}

> print is_callable($wgHooks['BeforeWelcomeCreation'][0]);

> print is_callable($wgHooks['BeforeWelcomeCreation'][1]);
1
Comment 1 jeremyb 2013-08-01 04:30:03 UTC
Ia4e2a7854b3a2a5cbbe4da
Comment 2 Ori Livneh 2013-08-01 04:47:54 UTC
I removed the BeforeWelcomeCreation handler in I8be6198a6 but not the statement in VisualEditor.php that registered it.

1.22wmf11 had not yet been updated to I8be6198a6, so it didn't need to be fixed. 1.22wmf12 did need a fix, but there was a merged change separating the deployed commit and my fix: change I1cd9e2bea. I didn't know if it was safe to deploy and no one from the VE team was around, so I created a cut a 'bug/52368' branch based on the deployed commit, cherry picked the fix on top, and synced that.

My apologies to the VisualEditor team and to the users that were affected.

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


Navigation
Links