summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorC.J. Adams-Collier <cjac@colliertech.org>2014-09-11 13:14:50 -0700
committerC.J. Adams-Collier <cjac@colliertech.org>2014-09-11 13:19:21 -0700
commitab6e4f9b42a6e7fe2031ad5f2443ef6a3584be1e (patch)
tree0e5f4a8a1a410eadfdfd6c66f2d48e1cdc411d0d
parentd87d064dcbc96f948f7ce18b9b36ab839d35e9f5 (diff)
FS RT #27189 - only printing 'each' if package quantity > 1 ; removed now-obsolete checks for pluralization of 'month'
-rw-r--r--FS/FS/part_pkg/discount_Mixin.pm7
1 files changed, 4 insertions, 3 deletions
diff --git a/FS/FS/part_pkg/discount_Mixin.pm b/FS/FS/part_pkg/discount_Mixin.pm
index 0063567dd..be0200c77 100644
--- a/FS/FS/part_pkg/discount_Mixin.pm
+++ b/FS/FS/part_pkg/discount_Mixin.pm
@@ -171,14 +171,15 @@ sub calc_discount {
my $format;
if ( $months eq '1' ) {
- $d .= "discount of $money_char$amount each";
+ $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 month". ( $months!=1 ? 's' : '' )
+ $d .= " for $months months"
unless defined $param->{'setup_charge'};
- $d .= ": $money_char$amount" if $months != 1 || $discount->percent;
+ $d .= ": $money_char$amount" if $discount->percent;
$format = 'Undiscounted monthly amount: %s%.2f';
}