Two minor tweaks
| Project: | Mail editor |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
Jump to:
Since you now maintain a {mail_edit_registry} table, I'd like you to join the {mail_edit} table with the registry before retrieving the rows for building the admin/build/mail-edit page. This will allow other modules to store additional template pieces in {mail_edit}, instead of having to create their own table with only a record or two and having to maintain two different access mechanisms.
Also, I'd like to be able to use %variables like
$mailkeys['node-nid'] = t('Notifications for %Pages subscriptions', array('%Pages' => t('Pages')));
for the descriptions. This doesn't work so well when the user tries to edit the descriptions (they contain <em> tags). So I'd remove the tags before placing the description strings into the description edit field. This means the user will lose the emphasizing, but at least he had it when the mailkey first showed up.
The attached patch makes these two changes.
| Attachment | Size |
|---|---|
| mail_edit.patch | 1.29 KB |

#1
Do you intend to add those requested features as part of this patch later on ? i'm asking because you've assigned this issue to yourself. by the way i would love to get your help with those features. i'm swamped.
#2
It's already done. The two features are just two tiny code changes in the patch.
Sorry to confuse you...
#3
What does it take for you to commit that simple patch?
#4
time mostly.
http://drupal.org/cvs?commit=144936
#5
Thanks!
#6
Automatically closed -- issue fixed for two weeks with no activity.