I am using the same E-Mail account on several Drupal Sites and want to process incoming update status notifications by my Redmine installation. It is possible to assign the incoming E-Mail to a preconfigured Project (the website represented in the Redmine project tracking tool). When adding some lines to the incoming E-Mail, one can override the predefined project. See http://www.redmine.org/wiki/1/RedmineReceivingEmails for details.

So I need a possibility to add some more textlines to the mail that drupal send on status update notifications. I added a textarea that can hold several lines, each will be added to the mailbody by hook_mail_alter. Please review the attached patch and add it to the module.

CommentFileSizeAuthor
#4 addinfo-992964-3.patch2.85 KBMcGo
#1 update_advanced-992964.patch2.72 KBMcGo

Comments

McGo’s picture

McGo commented
StatusFileSize
new update_advanced-992964.patch2.72 KB

Upsie. Forgot the patch.

McGo’s picture

McGo commented

This could obviously also be used for explaining some details on updating the site to the mail recipient, like a link to the internal site wiki or some standards on SLA... Whatever. I'd really like to see it in the module.

stborchert’s picture

stborchert
he/him
Primary language German
commented
Status: Active » Reviewed & tested by the community

Very good idea. Looks good and works like a charme.

(very) minor code style issue:
$param_list = explode("\n", variable_get('update_advanced_additional_mailtext',''));
There is a space missing before ''.

McGo’s picture

McGo commented
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new addinfo-992964-3.patch2.85 KB

rerolled the patch (git), please review and set to rtbc

stborchert’s picture

stborchert
he/him
Primary language German
commented
Status: Needs review » Reviewed & tested by the community

Same as above without code style issue :)

steffenr’s picture

steffenr
Primary language German
Location Bremen
commented

@McGo: thx for your patch - i just tried it in a test installation - works "like charme" for me too and should be added ...

Leeteq’s picture

Leeteq commented
Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Reviewed & tested by the community » Active

This is a very practical addition to this module.
No activity in this thread for a year, can it be adapted to 7.x too?

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
commented

Perhaps switch this issue over at Update Status Detailed Email that seems more appropriate for this?

dww’s picture

dww
we/he/they
commented
Project: Update Status Advanced Settings » Update Status Detailed Email

Yup, good idea. Since this other project exists specifically for doing email stuff, I'm happy to move all such feature requests from update_advanced over there. I'd rather update_advanced didn't get into business duplicating the functionality of update_detailed_email.

Thanks,
-Derek

xurizaemon’s picture

xurizaemon
he/him
Location Ōtepoti, Aotearoa 🏝
commented
Issue summary: View changes
Status: Active » Closed (works as designed)

The patch here is not for this module, but I'm open to such a contribution. Note that #2028091: Override Update Manager's update_mail function -- i.e. change the text of the email's body is basically the same feature request.

If a patch is submitted or the feature is sponsored then I'm happy to consider this going in. It's been over a year since this was updated so I'm going to close this for inactivity in the meantime, don't be shy about bringing it back. I'm just cleaning up the queue of a long-neglected module ;)