#UI enforces one or the other (for now? probably for good)
my $amount = 0;
$amount += $discount->amount
- if defined $param->{'real_pkgpart'} && $cust_pkg->pkgpart == $param->{'real_pkgpart'};
+ if $cust_pkg->pkgpart == $param->{'real_pkgpart'};
$amount += sprintf('%.2f', $discount->percent * $br / 100 );
- my $chg_months = $param->{'months'} || $cust_pkg->part_pkg->freq;
+ my $chg_months = defined($param->{'months'}) ?
+ $param->{'months'} :
+ $cust_pkg->part_pkg->freq;
my $months = $discount->months
? min( $chg_months,
}
if ( ! defined $param->{'setup_charge'} ) {
- my $error = $cust_pkg_discount->increment_months_used($months)
- if defined $param->{'real_pkgpart'}
- && $cust_pkg->pkgpart == $param->{'real_pkgpart'};
- die "error discounting: $error" if $error;
+ if ( $cust_pkg->pkgpart == $param->{'real_pkgpart'} ) {
+ push @{ $param->{precommit_hooks} }, sub {
+ my $error = $cust_pkg_discount->increment_months_used($months);
+ die "error discounting: $error" if $error;
+ };
+ }
$amount = min($amount, $br);
$amount *= $months;
$months = sprintf('%.2f', $months) if $months =~ /\./;
my $d = 'Includes ';
- $d .= 'setup ' if defined $param->{'setup_charge'};
- $d .= 'discount of '. $discount->description_short;
- $d .= " for $months month". ( $months!=1 ? 's' : '' ) unless defined $param->{'setup_charge'};
- $d .= ": $money_char$amount" if $months != 1 || $discount->percent;
+ my $format;
+
+ if ( $months eq '1' ) {
+ $d .= "discount of $money_char$amount";
+ $d .= " each" if $cust_pkg->quantity > 1;
+ $format = 'Undiscounted amount: %s%.2f';
+ } else {
+ $d .= 'setup ' if defined $param->{'setup_charge'};
+ $d .= 'discount of '. $discount->description_short;
+ $d .= " for $months months"
+ unless defined $param->{'setup_charge'};
+ $d .= ": $money_char$amount" if $discount->percent;
+ $format = 'Undiscounted monthly amount: %s%.2f';
+ }
+
push @$details, $d;
+ push @$details, sprintf( $format, $money_char, $br );
$tot_discount += $amount;
}