X-Git-Url: http://git.freeside.biz/gitweb/?a=blobdiff_plain;f=httemplate%2Fbrowse%2Fcust_main_county.cgi;h=26a3e21b9662c0bbdb3b8659eef01a0d2254674d;hb=9a8399783bb9d87ef662b4371bebe983d2781dce;hp=d94e892ef4ace89e2fbb07f9338402ac111c8a85;hpb=0b94e40c533288be69a4fe60da36a385d31eff7f;p=freeside.git
diff --git a/httemplate/browse/cust_main_county.cgi b/httemplate/browse/cust_main_county.cgi
index d94e892ef..26a3e21b9 100755
--- a/httemplate/browse/cust_main_county.cgi
+++ b/httemplate/browse/cust_main_county.cgi
@@ -10,7 +10,9 @@
'table' => 'cust_main_county',
'hashref' => $hashref,
'order_by' =>
- 'ORDER BY country, state, county, taxclass',
+ 'ORDER BY country, state, county, city, '.
+ 'district, taxclass, '.
+ "COALESCE(taxname, '')",
},
'count_query' => $count_query,
'header' => \@header,
@@ -46,18 +48,19 @@ my $exempt_sub = sub {
};
my $cs_oldrow;
-my $cell_style;
+my $cell_style = '';
my $cell_style_sub = sub {
my $row = shift;
if ( $cs_oldrow ne $row ) {
if ( $cs_oldrow ) {
if ( $cs_oldrow->country ne $row->country ) {
- $cell_style = 'border-top:1px solid #000000';
+ $cell_style = 'border-top:2px solid #000000';
} elsif ( $cs_oldrow->state ne $row->state ) {
- $cell_style = 'border-top:1px solid #cccccc'; #default?
- } elsif ( $cs_oldrow->state eq $row->state ) {
- #$cell_style = 'border-top:dashed 1px dark gray';
- $cell_style = 'border-top:1px dashed #cccccc';
+ $cell_style = 'border-top:1px solid #888888';
+ } elsif ( $cs_oldrow->county ne $row->county ) {
+ $cell_style = 'border-top:1px solid #cccccc';
+ } else {
+ $cell_style = '';
}
}
$cs_oldrow = $row;
@@ -105,37 +108,85 @@ sub expand_link {
'';
}
+sub add_link {
+ my %param = @_;
+
+ #if ( $ex_oldrow eq $param{'row'} ) {
+ # return '';
+ #} else {
+ # $ex_oldrow = $param{'row'};
+ #}
+
+ my %below = ( 'county' => 'city',
+ 'state' => 'county',
+ );
+ my $what = $below{ $param{'col' } };
+
+ my $taxnum = $param{'row'}->taxnum;
+ my $url = "${p}edit/cust_main_county-add.cgi?taxnum=$taxnum;what=$what";
+
+ ''.
+ include( '/elements/popup_link.html',
+ 'label' => $param{'label'},
+ 'action' => $url,
+ 'actionlabel' => $param{'desc'},
+ 'height' => 420,
+ #default# 'width' => 540,
+ #default# 'color' => '#333399',
+ ).
+ '';
+}
+
sub collapse_link {
my %param = @_;
my $row = $param{'row'};
my $col = $param{'col'};
- return ''
- if $col eq 'county' and $row->city
- || qsearch({
- 'table' => 'cust_main_county',
- 'hashref' => {
- 'country' => $row->country,
- 'state' => $row->state,
- 'city' => { op=>'!=', value=>'' },
- },
- 'order_by' => 'LIMIT 1',
- });
-
- my %above = ( 'city' => 'county',
- 'county' => 'state',
+# return ''
+# if $col eq 'state' and $row->city
+# || qsearch({
+# 'table' => 'cust_main_county',
+# 'hashref' => {
+# 'country' => $row->country,
+# 'state' => $row->state,
+# 'city' => { op=>'!=', value=>'' },
+# },
+# 'order_by' => 'LIMIT 1',
+# });
+
+ my %below = ( 'county' => 'city',
+ 'state' => 'county',
);
#XXX can still show the link when you have some counties broken down into
#cities and others not :/
my $taxnum = $param{'row'}->taxnum;
- my $url = "${p}edit/process/cust_main_county-collapse.cgi?$taxnum";
- $url = "javascript:collapse_areyousure('$url', '$col', '$above{$col}')";
+ my $url = "${p}edit/process/cust_main_county-collapse.cgi?taxnum=$taxnum;".
+ 'country='. uri_escape($cgi->param('country')). ';'.
+ 'state='. uri_escape($cgi->param('state')). ';'.
+ 'county='. uri_escape($cgi->param('county'));
+ $url = "javascript:collapse_areyousure('$url', '$below{$col}', '$col')";
qq($param{'label'});
}
+sub remove_link {
+ my %param = @_;
+
+ my $row = $param{'row'};
+ my $col = $param{'col'};
+
+ my $taxnum = $param{'row'}->taxnum;
+ my $url = "${p}edit/process/cust_main_county-remove.cgi?taxnum=$taxnum;".
+ 'country='. uri_escape($cgi->param('country')). ';'.
+ 'state='. uri_escape($cgi->param('state')). ';'.
+ 'county='. uri_escape($cgi->param('county'));
+ $url = "javascript:remove_areyousure('$url', '$col')";
+
+ qq($param{'label'});
+
+}
sub separate_taxclasses_link {
my( $row ) = @_;
@@ -165,11 +216,11 @@ my $html_init = <
-
- Click on add states to specify a country's tax rates by state or province.
-
Click on add counties to specify a state's tax rates by county, or remove counties to remove per-county tax rates.
-
Click on add cities to specify a county's tax rates by city, or remove cities to remove per-city tax rates.
END
$html_init .= "
Click on separate taxclasses to specify taxes per taxclass."
@@ -278,7 +329,7 @@ if ( scalar(@counties) > 1 ) {
'empty_data_label' => '(none)',
'empty_data_value' => '__NONE__',
'disable_empty' => 0,
- 'disable_countyupdate' => 1,
+ 'disable_cityupdate' => 1,
);
} else {
$html_posttotal .=
@@ -291,7 +342,7 @@ $html_posttotal .= ' )';
my $bulk_popup_link =
include( '/elements/popup_link_onclick.html',
- 'action' => "${p}edit/bulk-cust_main_county.html?MAGIC_taxnum_MAGIC",
+ 'action' => "${p}edit/bulk-cust_main_county.html?taxnum=MAGIC_taxnum_MAGIC",
'actionlabel' => 'Bulk add new tax',
'nofalse' => 1,
'height' => 420,
@@ -324,7 +375,7 @@ my $html_foot = < 1920 ) { // IE 2083 URL limit
alert('Too many selections'); // should do some session thing...
return false;
@@ -356,7 +408,9 @@ my $html_foot = <unselect all |
toggle all
-Add new tax to selected
+Add new tax to selected
+|
+Bulk edit selected
END
@@ -388,49 +442,134 @@ if ( $taxclass ) {
$cell_style = '';
-my @header = ( 'Country', 'State/Province', 'County', 'City' );
-my @header2 = ( '', '', '', '', );
-my @links = ( '', '', '', '', );
-my @link_onclicks = ( '', '', '', '', );
-my $align = 'llll';
+my @header = ( 'Country', 'State/Province', 'County', 'City', '' );
+# last column is 'district', but usually unused
+my @header2 = ( '', '', '', '', '' );
+my @links = ( '', '', '', '', '' );
+my @link_onclicks = ( '', '', '', '', '' );
+my $align = 'lllll';
+
+my %seen_country = ();
+my %seen_state = ();
+my %seen_county = ();
my @fields = (
sub { my $country = shift->country;
- code2country($country). " ($country)";
+ return '' if $seen_country{$country}++;
+ FS::geocode_Mixin->code2country($country). " ($country)";
},
- sub { state_label($_[0]->state, $_[0]->country).
- ( $_[0]->state
- ? ''
- : ' '. expand_link( desc => 'Add States',
- row => $_[0],
- label => 'add states',
- )
- )
+
+ #state
+ sub { my $label = $seen_state{$_[0]->country}->{$_[0]->state}++
+ ? '' : state_label($_[0]->state, $_[0]->country);
+
+ my $countylinks = ( $_[0]->county && $label )
+ ? ' '. add_link(
+ desc => 'Add more counties',
+ col => 'state',
+ label=> 'add more counties',
+ row => $_[0],
+ cgi => $cgi,
+ ).
+ ' '. collapse_link(
+ col => 'state',
+ label=> 'remove all counties',
+ row => $_[0],
+ cgi => $cgi,
+ )
+ : '';
+
+ my $addlink =
+ ( $_[0]->state
+ ? ''
+ : ' '. expand_link( desc => 'Add States',
+ row => $_[0],
+ label => 'add states',
+ cgi => $cgi,
+ )
+ );
+
+ $label.$countylinks.$addlink;
},
- sub { $_[0]->county
- ? $_[0]->county. ' '.
- collapse_link( col => 'county',
- label=> 'remove counties',
- row => $_[0],
- )
- : '(all) '.
+
+ #county
+ sub { my $label =
+ $seen_county{$_[0]->country}->{$_[0]->state}->{$_[0]->county}++
+ ? '' : $_[0]->county;
+
+ my $citylinks = '';
+ if ( $label ) {
+ $citylinks = $_[0]->city
+ ? ' '. add_link(
+ desc => 'Add more cities',
+ col => 'county',
+ label=> 'add more cities',
+ row => $_[0],
+ cgi => $cgi,
+ ).
+ ' '. collapse_link(
+ col => 'county',
+ label=> 'remove all cities',
+ row => $_[0],
+ cgi => $cgi,
+ )
+ : ' '. remove_link( col => 'county',
+ label=> 'remove county',
+ row => $_[0],
+ cgi => $cgi,
+ );
+ }
+
+ $_[0]->county
+ ? $label.$citylinks
+ : '(all) '.
expand_link( desc => 'Add Counties',
row => $_[0],
- label => 'add counties',
+ label => 'add counties',
+ cgi => $cgi,
);
},
- sub { $_[0]->city
- ? $_[0]->city. ' '.
- collapse_link( col => 'city',
- label=> 'remove cities',
- row => $_[0],
- )
- : '(all) '.
- expand_link( desc => 'Add Cities',
- row => $_[0],
- label => 'add cities',
+
+ #city
+ sub {
+ my $r = shift;
+ if ( $r->city ) {
+
+ if ( $r->taxclass #but if it has a taxclass, can't remove
+ or $r->district ) { # or a district
+ $r->city;
+ } else {
+ $r->city. ' '.
+ remove_link( col => 'city',
+ label=> 'remove city',
+ row => $r,
+ cgi => $cgi,
);
+ }
+ } else {
+ '(all) '.
+ expand_link( desc => 'Add Cities',
+ row => $r,
+ label => 'add cities',
+ cgi => $cgi,
+ );
+ }
+ },
+
+ #district
+ sub {
+ my $r = shift;
+ if ( $r->district ) {
+ $r->district . ' '.
+ remove_link( col => 'district',
+ label=> 'remove district',
+ row => $r,
+ cgi => $cgi,
+ );
+ }
+ # manually editing districts is not exactly intended
},
+
);
my @color = (
@@ -443,10 +582,24 @@ my @color = (
if ( $conf->exists('enable_taxclasses') ) {
push @header, qq!Tax class (add new)!;
push @header2, '(per-package classification)';
- push @fields, sub { $_[0]->taxclass || '(all) '.
- separate_taxclasses_link($_[0], 'Separate Taxclasses').
- 'separate taxclasses'
- };
+ push @fields, sub {
+ my $r = shift;
+ if ( $r->taxclass ) {
+ $r->taxclass;
+ } else {
+ my $sql = 'SELECT COUNT(*) FROM cust_main_county
+ WHERE country = ? AND state = ? AND county = ?
+ AND city = ? AND taxclass IS NOT NULL';
+ if ( FS::Record->scalar_sql($sql, map $r->$_,
+ qw( country state county city) ) ) {
+ '(none)';
+ } else {
+ '(all) '.
+ separate_taxclasses_link($r, 'Separate Taxclasses').
+ 'separate taxclasses';
+ }
+ }
+ };
push @color, sub { shift->taxclass ? '000000' : '999999' };
push @links, '';
push @link_onclicks, '';
@@ -473,7 +626,8 @@ my $cb_sub = sub {
my $cust_main_county = shift;
if ( $cb_oldrow ) {
- if ( $cb_oldrow->city ne $cust_main_county->city
+ if ( $cb_oldrow->district ne $cust_main_county->district
+ || $cb_oldrow->city ne $cust_main_county->city
|| $cb_oldrow->county ne $cust_main_county->county
|| $cb_oldrow->state ne $cust_main_county->state
|| $cb_oldrow->country ne $cust_main_county->country