mark: A photo of Mark kneeling on top of the Taal Volcano in the Philippines. It was a long hike. (Default)
Mark Smith ([staff profile] mark) wrote in [site community profile] changelog2010-03-17 05:24 am

[dw-free] X-poster 'remember password' doesn't remember

[commit: http://hg.dwscoalition.org/dw-free/rev/66b1e6e3bf90]

http://bugs.dwscoalition.org/show_bug.cgi?id=2443

Save passwords when new xpost accounts are created.

Patch by [personal profile] afuna.

Files modified:
  • htdocs/manage/externalaccount.bml
--------------------------------------------------------------------------------
diff -r 44f1dea92afd -r 66b1e6e3bf90 htdocs/manage/externalaccount.bml
--- a/htdocs/manage/externalaccount.bml	Wed Mar 17 05:22:54 2010 +0000
+++ b/htdocs/manage/externalaccount.bml	Wed Mar 17 05:24:52 2010 +0000
@@ -328,7 +328,7 @@ sub create_external_account {
     if ($ok) {
         # if the user requested that we don't save their password, then
         # don't save their password.
-        $opts{password} = "" if $opts{savepassword};
+        $opts{password} = "" unless $opts{savepassword};
 
         my $new_acct = DW::External::Account->create($u, \%opts);
         # FIXME add error if create fails.
--------------------------------------------------------------------------------