[freeside-commits] branch FREESIDE_4_BRANCH updated. 2c8cfec03cf66102f7a182d0bb6bd04e2628dc6d

[Ivan]

The branch, FREESIDE_4_BRANCH has been updated
       via  2c8cfec03cf66102f7a182d0bb6bd04e2628dc6d (commit)
      from  5c756345550e03e47740545c46fb93a2aad1d631 (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 2c8cfec03cf66102f7a182d0bb6bd04e2628dc6d
Author: Ivan Kohler <ivan at freeside.biz>
Date:   Mon Apr 25 17:41:59 2016 -0700

    fix cust_main-require_address2 vs cust_location, RT#41184

diff --git a/FS/FS/Conf.pm b/FS/FS/Conf.pm
index 31133d2..2f107c1 100644
--- a/FS/FS/Conf.pm
+++ b/FS/FS/Conf.pm
@@ -3282,7 +3282,7 @@ and customer address. Include units.',
   {
     'key'         => 'cust_main-require_address2',
     'section'     => 'UI',
-    'description' => 'Second address field is required (on service address only, if billing and service addresses differ).  Also enables "Unit" labeling of address2 on customer view and edit pages.  Useful for multi-tenant applications.  See also: address2-search',
+    'description' => 'Second address field is required.  Also enables "Unit" labeling of address2 on customer view and edit pages.  Useful for multi-tenant applications.  See also: address2-search', # service address only part not working in the modern world, see #41184  (on service address only, if billing and service addresses differ)
     'type'        => 'checkbox',
   },
 
diff --git a/FS/FS/cust_location.pm b/FS/FS/cust_location.pm
index f38e8ef..0dec065 100644
--- a/FS/FS/cust_location.pm
+++ b/FS/FS/cust_location.pm
@@ -381,10 +381,12 @@ sub check {
     $self->censustract("$1.$2");
   }
 
-  if ( $conf->exists('cust_main-require_address2') and 
-       !$self->ship_address2 =~ /\S/ ) {
-    return "Unit # is required";
-  }
+  #yikes... this is ancient, pre-dates cust_location and will be harder to
+  # implement now... how do we know this location is a service location from
+  # here and not a billing? we can't just check locationnums, we might be new :/
+  return "Unit # is required"
+    if $conf->exists('cust_main-require_address2')
+    && ! $self->address2 =~ /\S/;
 
   # tricky...we have to allow for the customer to not be inserted yet
   return "No prospect or customer!" unless $self->prospectnum 

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

Summary of changes:
 FS/FS/Conf.pm          |    2 +-
 FS/FS/cust_location.pm |   10 ++++++----
 2 files changed, 7 insertions(+), 5 deletions(-)