This patch does this, which makes this awesome module even more awesome!


aaron’s picture

for reference, 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.

aaron’s picture

also, i cleaned up much of the code, ensuring t()'s and check_plain/markups all around.

aaron’s picture

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.

aaron’s picture

This adds a condition to match against the mail key, making rules integration even more useful.

litwol’s picture

Status:Needs review» Fixed


kentr’s picture

I'm not seeing an option for this in the Event choices on Rules, Triggered Rule add form. Should it be there?

barckhoff’s picture

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.

barckhoff’s picture

Status:Fixed» Needs work
gnindl’s picture

Version:6.x-1.x-dev» 6.x-1.1
new2.84 KB

The patch from comment #4 is fine but the committer forgot to include the additional file This is where all rules conditions and events are defined. Compiled a new patch with the new file removing already committed changes.

gnindl’s picture

Category:feature» bug
Status:Needs work» Needs review

Changed status

fizk’s picture

Category:Bug report» Feature request
Issue summary:View changes

gnindl, support for other modules is a feature request.