Takes all the arguments of calc_recur. Calculates and returns the amount
by which to reduce the recurring fee; also increments months used on the
-discount and generates an invoice detail describing it.
-
-If the configuration option 'discount-show_available' is enabled, and this
-package is eligible for a prepayment discount but doesn't have one, an
-invoice detail will be generated to describe the available discounts.
+discount.
=cut
my $br = $self->base_recur_permonth($cust_pkg, $sdate);
$br += $param->{'override_charges'} if $param->{'override_charges'};
-
+
my $tot_discount = 0;
#UI enforces just 1 for now, will need ordering when they can be stacked
#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,
$discount->months - $cust_pkg_discount->months_used )
: $chg_months;
- if(defined $param->{'setup_charge'}) {
+ if (defined $param->{'setup_charge'}) {
next unless $discount->setup;
- if ( $discount->percent ) {
+ if ( $discount->percent > 0 ) {
$amount = sprintf('%.2f', $discount->percent * $param->{'setup_charge'} / 100 );
$months = 1;
- }
- elsif ( $discount->amount && $discount->months == 1) {
+ } elsif ( $discount->amount > 0 && $discount->months == 1) {
$discount_left = $param->{'setup_charge'} - $discount->amount;
$amount = $param->{'setup_charge'} if $discount_left < 0;
$amount = $discount->amount if $discount_left >= 0;
# transfer remainder of discount, if any, to recur
$param->{'discount_left_recur'}{$discount->discountnum} =
0 - $discount_left if $discount_left < 0;
- }
- else {
+ } else {
next;
}
- }
- elsif ( defined $param->{'discount_left_recur'}{$discount->discountnum}
- && $param->{'discount_left_recur'}{$discount->discountnum} > 0) {
+ } elsif ( defined $param->{'discount_left_recur'}{$discount->discountnum}
+ && $param->{'discount_left_recur'}{$discount->discountnum} > 0
+ ) {
# use up transferred remainder of discount from setup
$amount = $param->{'discount_left_recur'}{$discount->discountnum};
$param->{'discount_left_recur'}{$discount->discountnum} = 0;
$months = 1;
- }
- elsif ( $discount->setup && $discount->months == 1 && $discount->amount ) {
+ } elsif ( $discount->setup
+ && $discount->months == 1
+ && $discount->amount > 0
+ ) {
next;
}
- my $error = $cust_pkg_discount->increment_months_used($months)
- if (defined $param->{'real_pkgpart'}
- && $cust_pkg->pkgpart == $param->{'real_pkgpart'}
- && ! defined $param->{'setup_charge'});
- die "error discounting: $error" if $error;
+ if ( ! defined $param->{'setup_charge'} ) {
+ 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;
- $amount = sprintf('%.2f', $amount);
+ $amount = min($amount, $br);
+ $amount *= $months;
+ }
+
+ $amount = sprintf('%.2f', $amount + 0.00000001 ); #so 1.005 rounds to 1.01
next unless $amount > 0;
# transfer remainder of discount, if any, to setup
- if ( $discount->setup && $discount->amount
+ if ( $discount->setup && $discount->amount > 0
&& (!$discount->months || $discount->months != 1)
- && !defined $param->{'setup_charge'}) {
- $discount_left = $br - $amount;
- $amount = $br if $discount_left < 0;
+ && !defined $param->{'setup_charge'}
+ )
+ {
+ $discount_left = $br - $amount;
+ if ( $discount_left < 0 ) {
+ $amount = $br;
$param->{'discount_left_setup'}{$discount->discountnum} =
- 0 - $discount_left if $discount_left < 0;
+ 0 - $discount_left;
+ }
}
#record details in cust_bill_pkg_discount
'months' => $months,
};
push @{ $param->{'discounts'} }, $cust_bill_pkg_discount;
+ $tot_discount += $amount;
#add details on discount to invoice
- my $money_char = $conf->config('money_char') || '$';
- $months = sprintf('%.2f', $months) if $months =~ /\./;
+ # no longer! this is now done during rendering based on the existence
+ # of the cust_bill_pkg_discount record
+ #
+ #my $money_char = $conf->config('money_char') || '$';
+ #$months = sprintf('%.2f', $months) if $months =~ /\./;
+
+ #my $d = 'Includes ';
+ #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 );
- 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;
- push @$details, $d;
-
- $tot_discount += $amount;
- }
-
- if (!@cust_pkg_discount and $conf->exists('discount-show_available') ) {
- push @$details, $self->available_discounts;
}
sprintf('%.2f', $tot_discount);
}
-=item available_discounts
-
-Returns a list of details decribing the available prepayment discounts
-for this package.
-
-=cut
-
-sub available_discounts {
- my $self = shift;
- return if $self->freq ne '1';
- my @details;
- my $money_char = FS::Conf->new->config('money_char') || '$';
- my @discounts = map { $_->discount } $self->part_pkg_discount;
- # probably the most logical way to arrange these
- foreach my $discount (sort { $a->months <=> $b->months } @discounts) {
- my $months = $discount->months;
- my $amount;
- if ( $discount->amount > 0 ) {
- $amount = $money_char . sprintf('%.2f', $discount->amount);
- }
- elsif ( $discount->percent ) {
- $amount = $discount->percent .'%';
- }
- else { #?
- next;
- }
- push @details, "Prepay $months months for $amount discount."
- # better way to display this?
- # if it's a problem, we'll add discount.invoice_text or something
- # for the customer-visible text line.
- }
- return @details;
-}
-
-
1;