ChangeOptions(): A previous change retained empty fields by passing

cgi.FieldStorage constructor keep_blank_values=1.  Without this, it
isn't possible to set a text field to the empty string.  However, this
breaks admin password changing on the General page because the test
was for key presence.  We now also check that the key's value is
non-empty (meaning empty admin passwords are no longer allowed).

Also, when changing the admin password, make sure that the new
password isn't empty, and string.strip() the new and confirm
password.

Finally, use AddErrorMessage() more consistently.

AddErrorMessage(): make the tag (i.e. "Warning: ") settable via
arguments.

0 files changed, 0 insertions, 0 deletions