I think it would be cool, if the admin could configure the mailtext, like the standard mail texts in drupal core.

There would be no problem for me to write the patch, but I want to know whether this is a good idea.

CommentFileSizeAuthor
#2 passquickset-532140.patch2.71 KBdawehner

Comments

solipsist’s picture

solipsist commented

Nice idea! I think it would be best to place the textarea for this email text in the user settings form, where every other email can be customized.

dawehner’s picture

dawehner
Primary language German
commented
Status: Active » Needs review
StatusFileSize
new passquickset-532140.patch2.71 KB

here is the first patch.

solipsist’s picture

solipsist commented
Status: Needs review » Fixed

Committed!

Status: Fixed » Closed (fixed)

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

EnekoAlonso-1’s picture

EnekoAlonso-1 commented

There is a bug in the code, line 128, once the email body gets overriden, the module is sending the subject, not the body field.

Bad code:

  $message['body'] = t(variable_get('passquickset_password_generated_subject', _passquickset_notify_mail('generated_body')), $variables, $language->language);

Fix:

  $message['body'] = t(variable_get('passquickset_password_generated_body', _passquickset_notify_mail('generated_body')), $variables, $language->language);
EnekoAlonso-1’s picture

EnekoAlonso-1 commented
Category: feature » bug
Status: Closed (fixed) » Needs work

Changed status to needs work.

solipsist’s picture

solipsist commented
Status: Needs work » Fixed

Already fixed with issue #1026934: Can't change Body of Email. Thanks for reporting!

Status: Fixed » Closed (fixed)

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