Voting starts in March for the Drupal Association Board election.
This patch does this, which makes this awesome module even more awesome!
for reference, http://code.iamcal.com/php/rfc822/full_regexp.txt has a more complete regex pattern for extracting the email from the 'to/from' fields. might add that for better token rfc822 support, though in an included file. also, the 'recipient' token will only grab the first recipient if more than one is given.
also, i cleaned up much of the code, ensuring t()'s and check_plain/markups all around.
and added media_logger_load($mlid), but i didn't drop that into hook_menu. maybe a followup patch, maybe not. :P
that's all for now.
This adds a condition to match against the mail key, making rules integration even more useful.
I'm not seeing an option for this in the Event choices on Rules, Triggered Rule add form. Should it be there?
I tried installing this -- got many errors on the patch. Then I notice that it looks like most, if not all, of these changes are incorporated in the 6.x-1.x-dev version?
From the straight 6.x-1.x-dev version (with no patch installed) I got the following error when looking at Triggers for Mail Logging:
# warning: array_merge() [function.array-merge]: Argument #1 is not an array in /var/www/jobs.kemri/sites/all/modules/mail_logger/mail_logger.module on line 261.
The patch from comment #4 is fine but the committer forgot to include the additional file mail_logger.rules.inc. This is where all rules conditions and events are defined. Compiled a new patch with the new file removing already committed changes.
gnindl, support for other modules is a feature request.
Issue #727210 by gnindl - Add Rules, Actions, and Token support.
Applied your patch.
The patch committed contains a CVS Id tag, and the header in the mail_logger.css file claims it is an "Install and update functions for Mail Logger module."
Issue #727210 - Clean up last patch.
Fixed it. Thanks.
Automatically closed - issue fixed for 2 weeks with no activity.
Drupal is a registered trademark of Dries Buytaert.