Part of #675046: Make sure all hooks in D7 have documentation

hook_trigger_info_alter is invoked in function _trigger_get_all_info() in modules/trigger/trigger.module but is currently undocmented. Patch coming shortly.

CommentFileSizeAuthor
#1 776728.patch1.81 KBjhodgdon

Comments

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Active » Needs review
StatusFileSize
new 776728.patch1.81 KB

Here's a patch...

So, I can't figure out why someone would really want to alter the trigger info, since the only thing you can change is the label the user sees. The ability to alter it was introduced in the trigger mddule overhaul #525540: trigger.module and includes/actions.inc need overhaul and as far as I can see, wasn't really discussed.

Because of this, the example is a bit lame. But, anyway, it's documented...

robeano’s picture

robeano commented

#1: 776728.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, 776728.patch, failed testing.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented

That test failure in RDF was definitely not caused by this patch...

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Needs work » Needs review

#1: 776728.patch queued for re-testing.

cwgordon7’s picture

cwgordon7 commented
Status: Needs review » Reviewed & tested by the community

Looks good.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD! w00t!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.