diff options
author | Mark Wells <mark@freeside.biz> | 2013-03-06 15:04:14 -0800 |
---|---|---|
committer | Mark Wells <mark@freeside.biz> | 2013-03-06 15:04:14 -0800 |
commit | 0704f3f641deb525399521368f1e6ba9ff5b71cd (patch) | |
tree | a300eb157ac9c34b6fdae154d02d42ce28131506 /FS/FS/UI | |
parent | 5fe75497ce60689c128d93a3f1c9339e7c5e4e49 (diff) |
customer location columns in more reports, #940
Diffstat (limited to 'FS/FS/UI')
-rw-r--r-- | FS/FS/UI/Web.pm | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/FS/FS/UI/Web.pm b/FS/FS/UI/Web.pm index 91d2034dd..d6f183d8e 100644 --- a/FS/FS/UI/Web.pm +++ b/FS/FS/UI/Web.pm @@ -378,6 +378,9 @@ Otherwise, this function will assume the field is named "custnum". If the argument isn't present at all, the join will just say "USING (custnum)", which might work. +As a special case, if TABLE is 'cust_main', only the joins to cust_location +will be returned. + LOCATION_TABLE is an optional table name to use for joining ship_location, in case your query also includes package information and you want the "service address" columns to reflect package addresses. @@ -392,11 +395,12 @@ sub join_cust_main { ($location_table, $locationnum) = split(/\./, $location_table); $locationnum ||= 'locationnum'; - my $sql = ' LEFT JOIN cust_main '; + my $sql = ''; if ( $cust_table ) { - $sql .= "ON (cust_main.custnum = $cust_table.$custnum)"; + $sql = " LEFT JOIN cust_main ON (cust_main.custnum = $cust_table.$custnum)" + unless $cust_table eq 'cust_main'; } else { - $sql .= "USING (custnum)"; + $sql = " LEFT JOIN cust_main USING (custnum)"; } if ( !@cust_fields or grep /^bill_/, @cust_fields ) { |