my ($self, $cust_bill_pkg) = @_;
my $part_item = $cust_bill_pkg->part_X;
- my $location = $cust_bill_pkg->tax_location;
+ my $location = $cust_bill_pkg->tax_location
+ or return;
my $custnum = $self->{cust_main}->custnum;
push @{ $self->{items} }, $cust_bill_pkg;
- my @loc_keys = qw( district city county state country );
- my %taxhash = map { $_ => $location->get($_) } @loc_keys;
+ my %taxhash = map { $_ => $location->get($_) }
+ qw( district county state country );
+ # city names in cust_main_county are uppercase
+ $taxhash{'city'} = uc($location->get('city'));
$taxhash{'taxclass'} = $part_item->taxclass;
foreach my $cust_bill_pkg (@$taxables) {
- my $cust_pkg = $cust_bill_pkg->cust_pkg;
- my $part_pkg = $cust_bill_pkg->part_pkg;
my @new_exemptions;
my $taxable_charged = $cust_bill_pkg->setup + $cust_bill_pkg->recur
or next; # don't create zero-amount exemptions
$taxable_charged = 0;
}
+
+ if ( my $part_pkg = $cust_bill_pkg->part_pkg ) {
- if ( ($part_pkg->setuptax eq 'Y' or $tax_object->setuptax eq 'Y')
- and $cust_bill_pkg->setup > 0 and $taxable_charged > 0 ) {
+ if ( ($part_pkg->setuptax eq 'Y' or $tax_object->setuptax eq 'Y')
+ and $cust_bill_pkg->setup > 0 and $taxable_charged > 0 ) {
- push @new_exemptions, FS::cust_tax_exempt_pkg->new({
- amount => $cust_bill_pkg->setup,
- exempt_setup => 'Y'
- });
- $taxable_charged -= $cust_bill_pkg->setup;
+ push @new_exemptions, FS::cust_tax_exempt_pkg->new({
+ amount => $cust_bill_pkg->setup,
+ exempt_setup => 'Y'
+ });
+ $taxable_charged -= $cust_bill_pkg->setup;
+ }
- }
- if ( ($part_pkg->recurtax eq 'Y' or $tax_object->recurtax eq 'Y')
- and $cust_bill_pkg->recur > 0 and $taxable_charged > 0 ) {
+ if ( ($part_pkg->recurtax eq 'Y' or $tax_object->recurtax eq 'Y')
+ and $cust_bill_pkg->recur > 0 and $taxable_charged > 0 ) {
- push @new_exemptions, FS::cust_tax_exempt_pkg->new({
- amount => $cust_bill_pkg->recur,
- exempt_recur => 'Y'
- });
- $taxable_charged -= $cust_bill_pkg->recur;
+ push @new_exemptions, FS::cust_tax_exempt_pkg->new({
+ amount => $cust_bill_pkg->recur,
+ exempt_recur => 'Y'
+ });
+ $taxable_charged -= $cust_bill_pkg->recur;
+ }
}
$this_tax_cents = int($this_tax_cents);
}
+ my $locationnum;
+ if ( my $cust_pkg = $cust_bill_pkg->cust_pkg ) {
+ $locationnum = $cust_pkg->tax_locationnum;
+ } elsif ( $conf->exists('tax-ship_address') ) {
+ $locationnum = $cust_main->ship_locationnum;
+ } else {
+ $locationnum = $cust_main->bill_locationnum;
+ }
+
my $location = FS::cust_bill_pkg_tax_location->new({
- 'taxnum' => $tax_object->taxnum,
- 'taxtype' => ref($tax_object),
- 'cents' => $this_tax_cents,
- 'pkgnum' => $cust_bill_pkg->pkgnum,
- 'locationnum' => $cust_bill_pkg->cust_pkg->tax_locationnum,
+ 'taxnum' => $tax_object->taxnum,
+ 'taxtype' => ref($tax_object),
+ 'cents' => $this_tax_cents,
+ 'pkgnum' => $cust_bill_pkg->pkgnum,
+ 'locationnum' => $locationnum,
'taxable_cust_bill_pkg' => $cust_bill_pkg,
});
push @tax_links, $location;