Cannot register multiple functions for the same hook in extension registry
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Tgr
	May 13 2015, 4:14 PM

Description

I would expect this to work:

...
    "Hooks": {
        ...
        "LoadExtensionSchemaUpdates": [
            "Gather\\UpdaterHooks::onLoadExtensionSchemaUpdates",
            "Gather\\UpdaterHooks::onLoadExtensionSchemaUpdatesBC"
         ]
    }

but actually it will call Gather\UpdaterHooks::onLoadExtensionSchemaUpdates with the string "Gather\\UpdaterHooks::onLoadExtensionSchemaUpdatesBC" as the first parameter.

Details

	Subject	Repo	Branch	Lines +/-
	registration: Fix having multiple callbacks for a single hook	mediawiki/core	REL1_25	+32 -2
	registration: Fix having multiple callbacks for a single hook	mediawiki/core	master	+24 -2

Customize query in gerrit

Related Objects

Mentioned In: rMWccb28dc2548e: registration: Fix having multiple callbacks for a single hook
rMW2fcd5d790940: registration: Fix having multiple callbacks for a single hook

Event Timeline

Tgr created this task.May 13 2015, 4:14 PM

Tgr raised the priority of this task from to Needs Triage.

Tgr updated the task description. (Show Details)

Tgr added a project: MediaWiki-Configuration.

Tgr subscribed.

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMay 13 2015, 4:14 PM

The obvious workaround is to call one hook handler from the other.

Change 210761 had a related patch set uploaded (by Legoktm):
registration: Fix having multiple callbacks for a single hook

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

gerritbot added a project: Patch-For-Review.May 13 2015, 7:30 PM

Legoktm claimed this task.May 13 2015, 7:30 PM

Change 210761 merged by jenkins-bot:
registration: Fix having multiple callbacks for a single hook

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

Legoktm mentioned this in rMW2fcd5d790940: registration: Fix having multiple callbacks for a single hook.May 13 2015, 9:07 PM

Tgr closed this task as Resolved.May 13 2015, 9:13 PM

Change 231980 had a related patch set uploaded (by Legoktm):
registration: Fix having multiple callbacks for a single hook

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

Change 231980 merged by jenkins-bot:
registration: Fix having multiple callbacks for a single hook