What's the best way to handle/set the variable for user_mail_text?

For example, the profiler builder output is:

 variables[user_mail_cancel_confirm_body] = [user:name],

A request to cancel your account has been made at [site:name].

You may now cancel your account on [site:url-brief] by clicking this link or copying and pasting it into your browser:

[user:cancel-url]

NOTE: The cancellation of your account is not reversible.

This link expires in one day and nothing will happen if it is not used.

--  [site:name]

Only [user:name] is inputted into the database.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Robin Millette’s picture

Version: 7.x-1.0-alpha2 » 7.x-1.x-dev
Category: support » bug
Status: Active » Needs review
FileSize
1.91 KB

I am using var_export($var, true) in this patch to quote all values. I could special case it only for multi-line variables, but it's not necessary I think.

See the API function drupal_parse_info_format() for the accepted multi-line syntax.

btopro’s picture

@Robin Millette can you reroll this patch now that the ignore list function has your other commit in it to include a drupal_alter. This sounds like a good method for fixing oversight in building initially.

Robin Millette’s picture

Oups, I don't know how those couple of lines got in at the end. Here's a new patch.

Robin Millette’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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