From: Christopher Burger <burgerc@freeside.biz>
Date: Tue, 27 Mar 2018 18:20:11 +0000 (-0400)
Subject: RT# 33362 - fixed Argument 1d is not numeric when trying to discount daily recuring... 
X-Git-Url: http://git.freeside.biz/gitweb/?a=commitdiff_plain;h=a140ffb44cd6ce9f5346e337eac68a4f874b7675;p=freeside.git

RT# 33362 - fixed Argument 1d is not numeric when trying to discount daily recuring packages
---

diff --git a/FS/FS/part_pkg/discount_Mixin.pm b/FS/FS/part_pkg/discount_Mixin.pm
index 5bcf561c2..f634913cc 100644
--- a/FS/FS/part_pkg/discount_Mixin.pm
+++ b/FS/FS/part_pkg/discount_Mixin.pm
@@ -92,7 +92,10 @@ sub calc_discount {
     # $chg_months: the number of months we are charging recur for
     # $months: $chg_months or the months left on the discount, whchever is less
 
-    my $chg_months = $cust_pkg->part_pkg->freq || 1;
+    my $chg_months = 1;
+    unless ($cust_pkg->part_pkg->freq !~ /^\d+$/) {
+      $chg_months = $cust_pkg->part_pkg->freq || 1;
+    }
     if ( defined($param->{'months'}) ) { # then override
       $chg_months = $param->{'months'};
     }