summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorIvan Kohler <ivan@freeside.biz>2017-02-08 11:48:26 -0800
committerIvan Kohler <ivan@freeside.biz>2017-02-08 11:48:26 -0800
commit9f3d40d71b076c664b38c1cfa4b60c12d42c473a (patch)
tree0c6277ade274cbe6efef3eb210eeb84c26a40950
parent360ddc75126ea4b87dfd7a3b98fac47ed03f0fd4 (diff)
fix recurring display w/discounts, RT#41844
-rw-r--r--FS/FS/cust_main/Packages.pm6
1 files changed, 5 insertions, 1 deletions
diff --git a/FS/FS/cust_main/Packages.pm b/FS/FS/cust_main/Packages.pm
index 4201cf869..fec9fd348 100644
--- a/FS/FS/cust_main/Packages.pm
+++ b/FS/FS/cust_main/Packages.pm
@@ -593,6 +593,8 @@ this customer that are active (recurring).
=cut
+#recurring_pkgs? different from cust_pkg idea of "active" which has
+# a setup vs not_yet_billed which doesn't
sub active_pkgs {
my $self = shift;
grep { my $part_pkg = $_->part_pkg;
@@ -608,6 +610,8 @@ are active (recurring).
=cut
+#ncancelled_recurring_pkgs? different from cust_pkg idea of "active" which has
+# a setup vs not_yet_billed which doesn't
sub ncancelled_active_pkgs {
my $self = shift;
grep { my $part_pkg = $_->part_pkg;
@@ -807,7 +811,7 @@ sub display_recurring {
my $discount = $cust_pkg_discount->discount;
#and only one of these for each
$pkg_amount -= $discount->amount;
- $pkg_amount -= $amount * $discount->percent/100;
+ $pkg_amount -= $pkg_amount * $discount->percent/100;
}
$pkg_amount *= ( $cust_pkg->quantity || 1 );