[freeside-commits] branch FREESIDE_3_BRANCH updated. a6278360ced92d64a69a137494432d5b58a171ee

Jonathan Prykop jonathan at 420.am
Thu Apr 28 13:06:57 PDT 2016


The branch, FREESIDE_3_BRANCH has been updated
       via  a6278360ced92d64a69a137494432d5b58a171ee (commit)
      from  d39884aa7ef13f26a39e6db0c305b98aa61943e1 (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 a6278360ced92d64a69a137494432d5b58a171ee
Author: Jonathan Prykop <jonathan at freeside.biz>
Date:   Thu Apr 28 15:06:14 2016 -0500

    RT#42235: Strict password requirments are not disabled for signup portal

diff --git a/FS/FS/Password_Mixin.pm b/FS/FS/Password_Mixin.pm
index fc2e03e..7f5484c 100644
--- a/FS/FS/Password_Mixin.pm
+++ b/FS/FS/Password_Mixin.pm
@@ -55,7 +55,8 @@ sub is_password_allowed {
     $cust_main = $cust_pkg->cust_main if $cust_pkg;
   }
   warn "is_password_allowed: no customer could be identified" if !$cust_main;
-  return '' if $cust_main && $conf->config_bool('password-insecure', $cust_main->agentnum);
+  my $agentnum = $cust_main ? $cust_main->agentnum : undef; #selfservice signup has non-inserted pkg, no workaround
+  return '' if $conf->config_bool('password-insecure', $agentnum);
 
   # basic checks using Data::Password;
   # options for Data::Password

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

Summary of changes:
 FS/FS/Password_Mixin.pm |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)




More information about the freeside-commits mailing list