[freeside-commits] branch master updated. 80d6c0b965e62795fa7990e82082be59c8a03087

[Ivan Kohler]

The branch, master has been updated
       via  80d6c0b965e62795fa7990e82082be59c8a03087 (commit)
      from  0566e27675c0150f00795587c8bb1123c6a3871b (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 80d6c0b965e62795fa7990e82082be59c8a03087
Author: Ivan Kohler <ivan at freeside.biz>
Date:   Thu Aug 2 13:11:48 2018 -0700

    fix editing employee pw, RT#32456

diff --git a/httemplate/edit/process/access_user.html b/httemplate/edit/process/access_user.html
index d589c620e..c27262017 100644
--- a/httemplate/edit/process/access_user.html
+++ b/httemplate/edit/process/access_user.html
@@ -44,8 +44,11 @@ sub post_new_object_callback {
   return '' unless length($cgi->param('_password'));
 
   my $password = scalar($cgi->param('_password'));
-  $access_user->is_password_allowed($password)
-    || $access_user->change_password_fields($password);
+  my $error = $access_user->is_password_allowed($password);
+  return $error if $error;
+
+  $access_user->change_password_fields($password);
+  '';
 }
 
 sub noerror_callback {

-----------------------------------------------------------------------

Summary of changes:
 httemplate/edit/process/access_user.html | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)