[freeside-commits] branch FREESIDE_3_BRANCH updated. 342e1472c53f4718b6600fe52be3068f2caba61a

Mark Wells mark at 420.am
Thu Feb 12 11:02:08 PST 2015


The branch, FREESIDE_3_BRANCH has been updated
       via  342e1472c53f4718b6600fe52be3068f2caba61a (commit)
      from  3006575f9f09477ac2a72cf2b0a16d911cc09172 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 342e1472c53f4718b6600fe52be3068f2caba61a
Author: Mark Wells <mark at freeside.biz>
Date:   Thu Feb 12 11:01:46 2015 -0800

    unbreak fixed-amount discounts on quotations, #33099

diff --git a/FS/FS/quotation_pkg.pm b/FS/FS/quotation_pkg.pm
index 88147bc..6c8aa46 100644
--- a/FS/FS/quotation_pkg.pm
+++ b/FS/FS/quotation_pkg.pm
@@ -105,11 +105,8 @@ otherwise returns false.
 
 =cut
 
-use Data::Dumper; #XXX DEBUG
 sub insert {
   my ($self, %options) = @_;
-  warn Dumper($self);
-  warn Dumper(\%options);
 
   my $dbh = dbh;
   my $oldAutoCommit = $FS::UID::AutoCommit;
@@ -318,7 +315,7 @@ sub insert_discount {
   #my ($self, %options) = @_;
   my $self = shift;
 
-  my $cust_pkg_discount = FS::quotation_pkg_discount->new( {
+  my $quotation_pkg_discount = FS::quotation_pkg_discount->new( {
     'quotationpkgnum' => $self->quotationpkgnum,
     'discountnum'     => $self->discountnum,
     #for the create a new discount case
@@ -329,7 +326,7 @@ sub insert_discount {
     'setup'       => $self->discountnum_setup,
   } );
 
-  $cust_pkg_discount->insert;
+  $quotation_pkg_discount->insert;
 }
 
 sub _item_discount {
@@ -351,7 +348,7 @@ sub _item_discount {
     push @ext, $pkg_discount->description;
     $d->{setup_amount} -= $pkg_discount->setup_amount;
     $d->{recur_amount} -= $pkg_discount->recur_amount;
-  } 
+  }
   $d->{setup_amount} *= $self->quantity || 1;
   $d->{recur_amount} *= $self->quantity || 1;
   $d->{amount} = $d->{setup_amount} + $d->{recur_amount};
diff --git a/FS/FS/quotation_pkg_discount.pm b/FS/FS/quotation_pkg_discount.pm
index 19a7bce..87ac6bd 100644
--- a/FS/FS/quotation_pkg_discount.pm
+++ b/FS/FS/quotation_pkg_discount.pm
@@ -118,18 +118,6 @@ sub check {
 
 =back
 
-=item amount
-
-Returns the total amount of this discount (setup + recur), for compatibility
-with L<FS::cust_bill_pkg_discount>.
-
-=cut
-
-sub amount {
-  my $self = shift;
-  return $self->get('setup_amount') + $self->get('recur_amount');
-}
-
 =item description
 
 Returns a string describing the discount (for use on the quotation).

-----------------------------------------------------------------------

Summary of changes:
 FS/FS/quotation_pkg.pm          |    9 +++------
 FS/FS/quotation_pkg_discount.pm |   12 ------------
 2 files changed, 3 insertions(+), 18 deletions(-)




More information about the freeside-commits mailing list