Hello Sam!
Your webmail displays an error message when changing of user's
password fails. However it doesn't display any message when
changing password or updating preferences succeeds. I think it
can be a bit confusing for users. My a little patch corrects it.
Have a nice day!
Pawel Tecza
--- courier-0.53.1/webmail/pref.c-orig 2006-04-21 12:27:23.000000000 +0200
+++ courier-0.53.1/webmail/pref.c 2006-04-21 13:22:32.000000000 +0200
@@ -337,6 +337,8 @@
}
savemailinglists(cgi("mailinglists"));
+
+ printf("%s\n", getarg("PREFSOK"));
}
if (*cgi("do.changepwd"))
@@ -357,6 +359,10 @@
{
printf("%s\n", getarg("PWDERR"));
}
+ else
+ {
+ printf("%s\n", getarg("PWDOK"));
+ }
}
#if 0
--- courier-0.53.1/webmail/images/sqwebmail.css-orig 2006-04-21
12:40:09.000000000 +0200
+++ courier-0.53.1/webmail/images/sqwebmail.css 2006-04-21 12:43:23.000000000
+0200
@@ -453,6 +453,12 @@
margin-bottom: 2em;
}
+/* Success message */
+
+.success { color: #00c000; font-size: 150% }
+
+.small-success { color: #00c000 }
+
/* Outer border in readmsg page */
.box-border-1 { background-color: #cccccc }
--- courier-0.53.1/webmail/html/en-us/preferences.html-orig 2006-04-21
13:27:07.000000000 +0200
+++ courier-0.53.1/webmail/html/en-us/preferences.html 2006-04-21
12:41:45.000000000 +0200
@@ -13,7 +13,9 @@
<body>
<!--
-[#$PWDERR=<p class="error">ERROR: Unable to change password. Possible reasons
why you cannot change the password are:</p><ul class="error"><li>You did not
enter the old password correctly</li><li>You did not enter the new password
twice correctly</li><li>Your new password is too short, or is based on a
dictionary word, and short/dictionary-based passwords are not
allowed</li><li>Your new password is the same as one or more previous passwords,
or the minimum time between password changes has not yet elapsed</li></ul>#]
+[#$PWDERR=<p class="error">ERROR: Unable to change password. Possible reasons
why you cannot change the password are:</p><ul class="error"><li>You did not
enter the old password correctly</li><li>You did not enter the new password
twice correctly</li><li>Your new password is too short, or is based on a
dictionary word, and short/dictionary-based passwords are not
allowed</li><li>Your new password contains forbidden characters</li></ul>#]
+[#$PWDOK=<p class="success">Your password was changed successfully!</p>#]
+[#$PREFSOK=<p class="success">Your preferences was updated successfully!</p>#]
[#$DATE=Date#]
[#$SENDER=Sender#]
[#$SUBJECT=Subject#]
3 messages in net.sourceforge.lists.courier-sqwebmail[sqwebmail] Messages for password/pre...
.diff
