The trigger module is not listed between the dependencies of this module, and if somebody using this module without the trigger module, will receive this error message: Fatal error: Call to undefined function _trigger_get_hook_aids() in /....../modules/mail_logger/mail_logger.module on line 235

Comments

Ela’s picture

Ela commented

Yes, I got the same error since I was not using trigger.

deekayen’s picture

deekayen commented
Status: Active » Fixed

I added the dependency to the .info file in DRUPAL-6--1.

Status: Fixed » Closed (fixed)

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

dalin’s picture

dalin
he/him
Primary language English
Location 🇨🇦, 🇲🇽, 🌍
commented
Status: Closed (fixed) » Active

A better fix might be to conditionally call the trigger and actions only if they exist.

fizk’s picture

fizk commented
Issue summary: View changes
Status: Active » Closed (fixed)

dalin, you can open another issue if you'd like to explore your suggestion further.

fizk’s picture

fizk commented

Scratch that, I've added a module_exists() check and removed the dependency on Trigger in the .info.