Summary | Unclear error message in compose window when profile's from address is invalid |
Queue | IMP |
Queue Version | 4.2 |
Type | Enhancement |
State | Resolved |
Priority | 1. Low |
Owners | chuck (at) horde (dot) org |
Requester | steinhof (at) umich (dot) edu |
Created | 06/17/2008 (6204 days ago) |
Due | |
Updated | 06/25/2008 (6196 days ago) |
Assigned | |
Resolved | 06/25/2008 (6196 days ago) |
Milestone | |
Patch | Yes |
Assigned to Chuck Hagenbuch
State ⇒ Resolved
Patch ⇒ Yes
http://cvs.horde.org/diff.php/imp/docs/CHANGES?r1=1.1144&r2=1.1145&ty=u
http://cvs.horde.org/diff.php/imp/lib/Identity/imp.php?r1=1.78&r2=1.79&ty=u
address in the identities page? That would make the most sense.
addresses because we migrated our data before the upgrade script
validated those addresses.
New Attachment: compose_from_validation2.patch
message should be extended earlier in the code, in getFromLine().
Also, please use getMessage() instead of accessing the message
property directly.
does so in getFromLine().
address in the identities page? That would make the most sense.
State ⇒ Feedback
preference page is not so good in this very case. Opening the
preference page in a new window is bad usability (see the category
color popup, it's horrible). And since the compose window can be both,
in the horde frame, or in a popup window, we can't target the current
window either.
I think it would be better to only tell the user what to do. And the
message should be extended earlier in the code, in getFromLine().
Also, please use getMessage() instead of accessing the message
property directly.
State ⇒ New
New Attachment: compose_from_validation.patch
Patch ⇒ No
Milestone ⇒
Queue ⇒ IMP
Summary ⇒ Unclear error message in compose window when profile's from address is invalid
Type ⇒ Enhancement
Priority ⇒ 1. Low
the user is prevented from sending mail and is given this error
message: "Validation failed for: Joe User <joe@example.com>" This can
be confusing for the user and it would be nice to have a more helpful
error message.
I've attached a patch with extends the error message to read:
"Validation failed for: Joe User <joe@example.com> Your From address
is not a valid email address. This can be fixed in your <a
href="/horde/services/prefs.php?app=imp&group=identities">Personal
Information</a> options page." This allows users to immediately jump
to the place where they can fix the problem.
(Is there a better way to link to the "Personal Information" pref page?)