something thing that's annoyed me for a while, and I assume 99% of other users are the same.

drupal already knows my public email, why should I need to type it in again?

this patch adds the user's email into the compulsery email box as the default text. It can then be edited if needed.

CommentFileSizeAuthor
#5 project.inc_set_default_email.patch976 bytespfaocle
#3 project.inc.email_0.patch685 bytesmoggy
project.inc.email.patch701 bytesmoggy

Comments

nedjo’s picture

nedjo
he/him/his
Primary language English
commented

+1, good idea.

dries’s picture

dries commented

Looks good to me.

moggy’s picture

moggy commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new project.inc.email_0.patch685 bytes

updated to apply to latest cvs

pfaocle’s picture

pfaocle
He/Him
Primary language English
commented

+1, seems like a sensibile default?

Can't test till Forms API stuff is in project module, tho! (Hint, hint)

pfaocle’s picture

pfaocle
He/Him
Primary language English
commented
Title: Auto insert compulsery email » Auto insert compulsory email
Version: » x.y.z
StatusFileSize
new project.inc_set_default_email.patch976 bytes

Tested on HEAD, works only by setting $user as global at the top of the function. Revised patch attached.

moggy’s picture

moggy commented

Just tested this patch against the latest 4.6 release.

It applies fine, with an ofset of 6 lines, and appears to work fine.

Could this be applied to 4.6 please?

pfaocle’s picture

pfaocle
He/Him
Primary language English
commented

Anyone have any objections? Can we get this in?

nedjo’s picture

nedjo
he/him/his
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Thanks for the patch, applied an updated version.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)