Very easy feature request - increase the size of the e-mail subject box on the admin/user/invite/settings page. Right now it is limited to 30 characters, but using the token module with replacements you can very easily fill up the box. For example, including a single content profile field such as [profile-field_givenname-raw] already takes up 29 of the 30 available characters.
This would be done via editing invite_admin.inc and changing the #size in the following area of the code:
$form['email']['invite_subject'] = array(
'#type' => 'textfield',
'#title' => t('Subject'),
'#default_value' => variable_get('invite_subject', t('[inviter-raw] has sent you an invite!')),
'#size' => 30,
'#maxlength' => 64,
'#description' => t('Type the default subject of the invitation e-mail.') .' '. t('Use the syntax [token] if you want to insert a replacement pattern.'),
'#required' => TRUE,
);
I would suggest changing it to something fairly lengthy, such as 90 characters to allow for token replacement space. Thank you.
Comment | File | Size | Author |
---|---|---|---|
#8 | 496894-subject.patch | 1.41 KB | smk-ka |
#2 | invite.patch | 756 bytes | seutje |
Comments
Comment #1
rjbrown99 CreditAttribution: rjbrown99 commentedActually it's probably #maxlength and #size, but I think you get the idea.
Comment #2
seutje CreditAttribution: seutje commented+1
it's only maxlength, size is just the width of the box, which should fall back to the default imo
patch attached removes maxlength and size altogether
Comment #3
rjbrown99 CreditAttribution: rjbrown99 commentedComment #4
andypost+1 to remove limit. As rfc 2821 and 2822 says - subject is unstructured field and have no limit but
rfc2822.txt 2.2.3. Long Header Fields
Comment #5
chrisdawson CreditAttribution: chrisdawson commentedI'd also like to vote for this patch to be committed, please
Comment #8
smk-ka CreditAttribution: smk-ka commentedIf we increase the length of the admin field, we need to also increase the length in the front-end. Committed the attached patch. Thanks to the spammer for bringing this issue to my attention.