I don't see why trigger module should be a hard dependency of this module. The mail logging can be done without trigger, as far as I can see from the code.

Trigger module is only used in mail_logger_mail_logger(), and that could be wrapped in a module_exists().

CommentFileSizeAuthor
#1 1239070.mail_logger.trigger-dep.patch1.34 KBjoachim

Comments

joachim’s picture

Status: Active » Needs review
StatusFileSize
new1.34 KB

Here's a patch.

fizk’s picture

Issue summary: View changes
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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