Bugzilla – Attachment 33791 Details for
Bug 13321
Fix tax and prices calculation
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 13321: Fix the prices calculation method
Bug-13321-Fix-the-prices-calculation-method.patch (text/plain), 25.66 KB, created by
Jonathan Druart
on 2014-11-21 16:05:46 UTC
(
hide
)
Description:
Bug 13321: Fix the prices calculation method
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2014-11-21 16:05:46 UTC
Size:
25.66 KB
patch
obsolete
>From e52799ab97f92a0fa22bd503250532002129afee Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@biblibre.com> >Date: Mon, 17 Nov 2014 12:06:09 +0100 >Subject: [PATCH] Bug 13321: Fix the prices calculation method > >Well, we have finally arrived here \o/ > >The method where the prices are calculated uses the equations listed on >the wiki page (http://wiki.koha-community.org/wiki/GST_Rewrite_RFC). > >The ecost is calculated from the rrp (using the discount and the tax >rate). That's why we removed the ability to edit this value. > >That's why we remove the ability to edit the ecost on ordering in a >previous commit (bug 12840). > >The total is now calculated in the scripts. That's why this patch >removes lines in the test file. > >In C4::Acquisition::populate_order_with_prices, the calculation on >receiving must depend on the 'invoiceincgst' supplier parameter, and not >listincgst (which is used on ordering). >It also removes the rounding errors, now we store "exact" values in DB >(10^-6). >The values will be displayed using the Price TT plugin it will round the >values for us. >--- > C4/Acquisition.pm | 70 ++++---- > acqui/basket.pl | 6 +- > acqui/basketgroup.pl | 4 - > acqui/finishreceive.pl | 20 +-- > acqui/invoice.pl | 11 +- > acqui/parcel.pl | 4 +- > .../prog/en/modules/acqui/orderreceive.tt | 9 +- > t/Prices.t | 179 +++------------------ > 8 files changed, 75 insertions(+), 228 deletions(-) > >diff --git a/C4/Acquisition.pm b/C4/Acquisition.pm >index 6fefa25..931cbef 100644 >--- a/C4/Acquisition.pm >+++ b/C4/Acquisition.pm >@@ -2815,9 +2815,6 @@ sub GetBiblioCountByBasketno { > return $sth->fetchrow; > } > >-# This is *not* the good way to calcul prices >-# But it's how it works at the moment into Koha >-# This will be fixed later. > # Note this subroutine should be moved to Koha::Acquisition::Order > # Will do when a DBIC decision will be taken. > sub populate_order_with_prices { >@@ -2834,48 +2831,59 @@ sub populate_order_with_prices { > my $discount = $order->{discount}; > $discount /= 100 if $discount > 1; > >- $order->{rrp} = Koha::Number::Price->new( $order->{rrp} )->round; >- $order->{ecost} = Koha::Number::Price->new( $order->{ecost} )->round; > if ($ordering) { > if ( $bookseller->{listincgst} ) { >+ # The user entered the rrp tax included > $order->{rrp_tax_included} = $order->{rrp}; >- $order->{rrp_tax_excluded} = Koha::Number::Price->new( >- $order->{rrp_tax_included} / ( 1 + $order->{tax_rate} ) )->round; >- $order->{ecost_tax_included} = $order->{ecost}; >- $order->{ecost_tax_excluded} = Koha::Number::Price->new( >- $order->{ecost} / ( 1 + $order->{tax_rate} ) )->round; >- $order->{tax_value} = Koha::Number::Price->new( >- ( $order->{ecost_tax_included} - $order->{ecost_tax_excluded} ) * >- $order->{quantity} )->round; >+ >+ # rrp tax excluded = rrp tax included / ( 1 + tax rate ) >+ $order->{rrp_tax_excluded} = $order->{rrp_tax_included} / ( 1 + $order->{tax_rate} ); >+ >+ # ecost tax excluded = rrp tax excluded * ( 1 - discount ) >+ $order->{ecost_tax_excluded} = $order->{rrp_tax_excluded} * ( 1 - $discount ); >+ >+ # ecost tax included = rrp tax included ( 1 - discount ) >+ $order->{ecost_tax_included} = $order->{rrp_tax_included} * ( 1 - $discount ); > } > else { >+ # The user entered the rrp tax excluded > $order->{rrp_tax_excluded} = $order->{rrp}; >- $order->{rrp_tax_included} = Koha::Number::Price->new( >- $order->{rrp} * ( 1 + $order->{tax_rate} ) )->round; >- $order->{ecost_tax_excluded} = $order->{ecost}; >- $order->{ecost_tax_included} = Koha::Number::Price->new( >- $order->{ecost} * ( 1 + $order->{tax_rate} ) )->round; >- $order->{tax_value} = Koha::Number::Price->new( >- ( $order->{ecost_tax_included} - $order->{ecost_tax_excluded} ) * >- $order->{quantity} )->round; >+ >+ # rrp tax included = rrp tax excluded * ( 1 - tax rate ) >+ $order->{rrp_tax_included} = $order->{rrp_tax_excluded} * ( 1 + $order->{tax_rate} ); >+ >+ # ecost tax excluded = rrp tax excluded * ( 1 - discount ) >+ $order->{ecost_tax_excluded} = $order->{rrp_tax_excluded} * ( 1 - $discount ); >+ >+ # ecost tax included = rrp tax excluded * ( 1 - tax rate ) * ( 1 - discount ) >+ $order->{ecost_tax_included} = >+ $order->{rrp_tax_excluded} * >+ ( 1 + $order->{tax_rate} ) * >+ ( 1 - $discount ); > } >+ >+ # tax value = quantity * ecost tax excluded * tax rate >+ $order->{tax_value} = $order->{quantity} * $order->{ecost_tax_excluded} * $order->{tax_rate}; > } > > if ($receiving) { >- if ( $bookseller->{listincgst} ) { >- $order->{unitprice_tax_included} = Koha::Number::Price->new( $order->{unitprice} )->round; >- $order->{unitprice_tax_excluded} = Koha::Number::Price->new( >- $order->{unitprice_tax_included} / ( 1 + $order->{tax_rate} ) )->round; >+ if ( $bookseller->{invoiceincgst} ) { >+ # The user entered the unit price tax included >+ $order->{unitprice_tax_included} = $order->{unitprice}; >+ >+ # unit price tax excluded = unit price tax included / ( 1 + tax rate ) >+ $order->{unitprice_tax_excluded} = $order->{unitprice_tax_included} / ( 1 + $order->{tax_rate} ); > } > else { >- $order->{unitprice_tax_excluded} = Koha::Number::Price->new( $order->{unitprice} )->round; >- $order->{unitprice_tax_included} = Koha::Number::Price->new( >- $order->{unitprice_tax_excluded} * ( 1 + $order->{tax_rate} ) )->round; >+ # The user entered the unit price tax excluded >+ $order->{unitprice_tax_excluded} = $order->{unitprice}; >+ >+ # unit price tax included = unit price tax included * ( 1 + tax rate ) >+ $order->{unitprice_tax_included} = $order->{unitprice_tax_excluded} * ( 1 + $order->{tax_rate} ); > } >- $order->{tax_value} = Koha::Number::Price->new( >- ( $order->{unitprice_tax_included} - $order->{unitprice_tax_excluded} ) >- * $order->{quantityreceived} )->round; > >+ # tax value = quantity * unit price tax excluded * tax rate >+ $order->{tax_value} = $order->{quantity} * $order->{unitprice_tax_excluded} * $order->{tax_rate}; > } > > return $order; >diff --git a/acqui/basket.pl b/acqui/basket.pl >index 93ae6c3..f4705be 100755 >--- a/acqui/basket.pl >+++ b/acqui/basket.pl >@@ -349,9 +349,6 @@ if ( $op eq 'delete_confirm' ) { > $template->param( uncertainprices => 1 ); > } > >- $line->{total_tax_excluded} = Koha::Number::Price->new( $line->{ecost_tax_excluded} * $line->{quantity} )->format; >- $line->{total_tax_included} = Koha::Number::Price->new( $line->{ecost_tax_included} * $line->{quantity} )->format; >- > push @books_loop, $line; > > $foot{$$line{tax_rate}}{tax_rate} = $$line{tax_rate}; >@@ -449,6 +446,9 @@ sub get_order_infos { > $line{basketno} = $basketno; > $line{budget_name} = $budget->{budget_name}; > >+ $line{total_tax_included} = $line{ecost_tax_included} * $line{quantity}; >+ $line{total_tax_excluded} = $line{ecost_tax_excluded} * $line{quantity}; >+ > if ( $line{uncertainprice} ) { > $line{rrp_tax_excluded} .= ' (Uncertain)'; > } >diff --git a/acqui/basketgroup.pl b/acqui/basketgroup.pl >index 8bf21e4..ecf0d36 100755 >--- a/acqui/basketgroup.pl >+++ b/acqui/basketgroup.pl >@@ -82,10 +82,6 @@ sub BasketTotal { > } else { > $total = $total + ( $order->{ecost_tax_excluded} * $order->{quantity} ); > } >- if ($bookseller->{invoiceincgst} && ! $bookseller->{listincgst} && ( $bookseller->{tax_rate} // C4::Context->preference("gist") )) { >- my $gst = $bookseller->{tax_rate} // C4::Context->preference("gist"); >- $total = $total * ( $gst / 100 +1); >- } > } > $total .= " " . ($bookseller->{invoiceprice} // 0); > return $total; >diff --git a/acqui/finishreceive.pl b/acqui/finishreceive.pl >index 685e00c..b156b7c 100755 >--- a/acqui/finishreceive.pl >+++ b/acqui/finishreceive.pl >@@ -54,8 +54,6 @@ my $invoiceno = $invoice->{invoicenumber}; > my $datereceived = $invoice->{shipmentdate}; > my $booksellerid = $input->param('booksellerid'); > my $cnt = 0; >-my $ecost = $input->param('ecost'); >-my $rrp = $input->param('rrp'); > my $bookfund = $input->param("bookfund"); > my $order = GetOrder($ordernumber); > my $new_ordernumber = $ordernumber; >@@ -83,25 +81,9 @@ if ($quantityrec > $origquantityrec ) { > } > > $order->{order_internalnote} = $input->param("order_internalnote"); >- $order->{rrp} = $rrp; >- $order->{ecost} = $ecost; > $order->{unitprice} = $unitprice; > >- # FIXME is it still useful? > my $bookseller = Koha::Acquisition::Bookseller->fetch({ id => $booksellerid }); >- if ( $bookseller->{listincgst} ) { >- if ( not $bookseller->{invoiceincgst} ) { >- $order->{rrp} = $order->{rrp} * ( 1 + $order->{tax_rate} ); >- $order->{ecost} = $order->{ecost} * ( 1 + $order->{tax_rate} ); >- $order->{unitprice} = $order->{unitprice} * ( 1 + $order->{tax_rate} ); >- } >- } else { >- if ( $bookseller->{invoiceincgst} ) { >- $order->{rrp} = $order->{rrp} / ( 1 + $order->{tax_rate} ); >- $order->{ecost} = $order->{ecost} / ( 1 + $order->{tax_rate} ); >- $order->{unitprice} = $order->{unitprice} / ( 1 + $order->{tax_rate} ); >- } >- } > > $order = C4::Acquisition::populate_order_with_prices( > { >@@ -170,7 +152,7 @@ ModItem( > booksellerid => $booksellerid, > dateaccessioned => $datereceived, > price => $unitprice, >- replacementprice => $rrp, >+ replacementprice => $order->{rrp}, > replacementpricedate => $datereceived, > }, > $biblionumber, >diff --git a/acqui/invoice.pl b/acqui/invoice.pl >index 5880edb..562b480 100755 >--- a/acqui/invoice.pl >+++ b/acqui/invoice.pl >@@ -142,7 +142,6 @@ foreach my $order (@$orders) { > > push @foot_loop, map {$_} values %foot; > >-my $format = "%.2f"; > my $budgets = GetBudgets(); > my @budgets_loop; > my $shipmentcost_budgetid = $details->{shipmentcost_budgetid}; >@@ -169,11 +168,11 @@ $template->param( > orders_loop => \@orders_loop, > foot_loop => \@foot_loop, > total_quantity => $total_quantity, >- total_tax_excluded => sprintf( $format, $total_tax_excluded ), >- total_tax_included => sprintf( $format, $total_tax_included ), >- total_tax_value => sprintf( $format, $total_tax_value ), >- total_tax_excluded_shipment => sprintf( $format, $total_tax_excluded + $details->{shipmentcost}), >- total_tax_included_shipment => sprintf( $format, $total_tax_included + $details->{shipmentcost}), >+ total_tax_excluded => $total_tax_excluded, >+ total_tax_included => $total_tax_included, >+ total_tax_value => $total_tax_value, >+ total_tax_excluded_shipment => $total_tax_excluded + $details->{shipmentcost}, >+ total_tax_included_shipment => $total_tax_included + $details->{shipmentcost}, > invoiceincgst => $bookseller->{invoiceincgst}, > currency => GetCurrency()->{currency}, > budgets_loop => \@budgets_loop, >diff --git a/acqui/parcel.pl b/acqui/parcel.pl >index 1711daf..9353e74 100755 >--- a/acqui/parcel.pl >+++ b/acqui/parcel.pl >@@ -150,8 +150,8 @@ for my $order ( @orders ) { > $line{budget} = GetBudgetByOrderNumber( $line{ordernumber} ); > $foot{$line{tax_rate}}{tax_rate} = $line{tax_rate}; > $foot{$line{tax_rate}}{tax_value} += $line{tax_value}; >- $total_tax_excluded += Koha::Number::Price->new( $line{ecost_tax_excluded} * $line{quantity} )->format; >- $total_tax_included += Koha::Number::Price->new( $line{ecost_tax_included} * $line{quantity} )->format; >+ $total_tax_excluded += $line{unitprice_tax_excluded} * $line{quantity}; >+ $total_tax_included += $line{unitprice_tax_included} * $line{quantity}; > > my $suggestion = GetSuggestionInfoFromBiblionumber($line{biblionumber}); > $line{suggestionid} = $suggestion->{suggestionid}; >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/orderreceive.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/orderreceive.tt >index d115cc3..8321f93 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/orderreceive.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/orderreceive.tt >@@ -321,13 +321,14 @@ > </div> > [% END %][%# IF (AcqCreateItemReceiving) %] > </li> >- <li><label for="rrp">Replacement cost: </label><input type="text" size="20" name="rrp" id="rrp" value="[% rrp %]" /></li> >- <li><label for="ecost">Budgeted cost: </label><input type="text" size="20" name="ecost" id="ecost" value="[% ecost %]" /></li> >+ >+ <li><label for="rrp">Replacement cost: </label>[% rrp | $Price %]</li> >+ <li><label for="ecost">Budgeted cost: </label>[% ecost | $Price %]</li> > <li><label for="cost">Actual cost:</label> > [% IF ( unitprice ) %] >- <input type="text" size="20" name="cost" id="cost" value="[% unitprice %]" /> >+ <input type="text" size="20" name="cost" id="cost" value="[% unitprice | $Price %]" /> > [% ELSE %] >- <input type="text" size="20" name="cost" id="cost" value="[% ecost %]" /> >+ <input type="text" size="20" name="cost" id="cost" value="[% ecost | $Price %]" /> > [% END %]</li> > <li><label for="order_internalnote">Internal note: </label><textarea name="order_internalnote" width="40" rows="8" >[% order_internalnote %]</textarea></li> > <li><label for="order_vendornote">Vendor note: </label><span>[% order_vendornote %]</span></li> >diff --git a/t/Prices.t b/t/Prices.t >index 2b8b296..ea5ae2e 100644 >--- a/t/Prices.t >+++ b/t/Prices.t >@@ -21,7 +21,7 @@ my $today; > for my $currency_format ( qw( US FR ) ) { > t::lib::Mocks::mock_preference( 'CurrencyFormat', $currency_format ); > subtest 'Configuration 1: 0 0' => sub { >- plan tests => 12; >+ plan tests => 8; > $bookseller_module->mock( > 'fetch', > sub { >@@ -53,7 +53,6 @@ for my $currency_format ( qw( US FR ) ) { > } > ); > >- # Note that this configuration is correct \o/ > compare( > { > got => $order_0_0->{rrp_tax_included}, >@@ -94,22 +93,6 @@ for my $currency_format ( qw( US FR ) ) { > field => 'tax_value' > } > ); >- compare( >- { >- got => $order_0_0->{total_tax_included}, >- expected => 154.98, >- conf => '0 0', >- field => 'total_tax_included' >- } >- ); >- compare( >- { >- got => $order_0_0->{total_tax_excluded}, >- expected => 147.60, >- conf => '0 0', >- field => 'total_tax_excluded' >- } >- ); > > $order_0_0 = C4::Acquisition::populate_order_with_prices( > { >@@ -119,7 +102,6 @@ for my $currency_format ( qw( US FR ) ) { > } > ); > >- # Note that this configuration is correct \o/ > compare( > { > got => $order_0_0->{unitprice_tax_included}, >@@ -144,26 +126,10 @@ for my $currency_format ( qw( US FR ) ) { > field => 'tax_value' > } > ); >- compare( >- { >- got => $order_0_0->{total_tax_included}, >- expected => 154.98, >- conf => '0 0', >- field => 'total_tax_included' >- } >- ); >- compare( >- { >- got => $order_0_0->{total_tax_excluded}, >- expected => 147.60, >- conf => '0 0', >- field => 'total_tax_excluded' >- } >- ); > }; > > subtest 'Configuration 1: 1 1' => sub { >- plan tests => 12; >+ plan tests => 8; > $bookseller_module->mock( > 'fetch', > sub { >@@ -195,8 +161,6 @@ for my $currency_format ( qw( US FR ) ) { > } > ); > >- # Note that this configuration is *not* correct >- # tax_value should be 7.03 instead of 7.02 > compare( > { > got => $order_1_1->{rrp_tax_included}, >@@ -232,27 +196,11 @@ for my $currency_format ( qw( US FR ) ) { > compare( > { > got => $order_1_1->{tax_value}, >- expected => 7.02, >+ expected => 7.03, > conf => '1 1', > field => 'tax_value' > } > ); >- compare( >- { >- got => $order_1_1->{total_tax_included}, >- expected => 147.60, >- conf => '1 1', >- field => 'total_tax_included' >- } >- ); >- compare( >- { >- got => $order_1_1->{total_tax_excluded}, >- expected => 140.58, >- conf => '1 1', >- field => 'total_tax_excluded' >- } >- ); > > $order_1_1 = C4::Acquisition::populate_order_with_prices( > { >@@ -261,8 +209,7 @@ for my $currency_format ( qw( US FR ) ) { > receiving => 1, > } > ); >- # Note that this configuration is *not* correct! >- # tax_value should be 7.03 >+ > compare( > { > got => $order_1_1->{unitprice_tax_included}, >@@ -282,31 +229,15 @@ for my $currency_format ( qw( US FR ) ) { > compare( > { > got => $order_1_1->{tax_value}, >- expected => 7.02, >+ expected => 7.03, > conf => '1 1', > field => 'tax_value' > } > ); >- compare( >- { >- got => $order_1_1->{total_tax_included}, >- expected => 147.60, >- conf => '1 1', >- field => 'total_tax_included' >- } >- ); >- compare( >- { >- got => $order_1_1->{total_tax_excluded}, >- expected => 140.58, >- conf => '1 1', >- field => 'total_tax_excluded' >- } >- ); > }; > > subtest 'Configuration 1: 1 0' => sub { >- plan tests => 12; >+ plan tests => 8; > $bookseller_module->mock( > 'fetch', > sub { >@@ -319,11 +250,11 @@ for my $currency_format ( qw( US FR ) ) { > biblionumber => $biblionumber_1_0, > quantity => 2, > listprice => 82.000000, >- unitprice => 73.804500, >+ unitprice => 70.290000, > quantityreceived => 2, > basketno => $basketno_1_1, > invoiceid => $invoiceid_1_1, >- rrp => 82.01, >+ rrp => 82.00, > ecost => 73.80, > tax_rate => 0.0500, > discount => 10.0000, >@@ -338,14 +269,10 @@ for my $currency_format ( qw( US FR ) ) { > } > ); > >- # Note that this configuration is *not* correct! >- # rrp_tax_included should be 82 (what we inserted!) >- # tax_value should be 7.03 instead of 7.02 >- > compare( > { > got => $order_1_0->{rrp_tax_included}, >- expected => 82.01, >+ expected => 82, > conf => '1 0', > field => 'rrp_tax_included' > } >@@ -377,27 +304,11 @@ for my $currency_format ( qw( US FR ) ) { > compare( > { > got => $order_1_0->{tax_value}, >- expected => 7.02, >+ expected => 7.03, > conf => '1 0', > field => 'tax_value' > } > ); >- compare( >- { >- got => $order_1_0->{total_tax_included}, >- expected => 147.60, >- conf => '1 0', >- field => 'total_tax_included' >- } >- ); >- compare( >- { >- got => $order_1_0->{total_tax_excluded}, >- expected => 140.58, >- conf => '1 0', >- field => 'total_tax_excluded' >- } >- ); > > $order_1_0 = C4::Acquisition::populate_order_with_prices( > { >@@ -406,8 +317,7 @@ for my $currency_format ( qw( US FR ) ) { > receiving => 1, > } > ); >- # Note that this configuration is *not* correct! >- # gstvalue should be 7.03 >+ > compare( > { > got => $order_1_0->{unitprice_tax_included}, >@@ -427,31 +337,15 @@ for my $currency_format ( qw( US FR ) ) { > compare( > { > got => $order_1_0->{tax_value}, >- expected => 7.02, >+ expected => 7.03, > conf => '1 0', > field => 'tax_value' > } > ); >- compare( >- { >- got => $order_1_0->{total_tax_included}, >- expected => 147.60, >- conf => '1 0', >- field => 'total_tax_included' >- } >- ); >- compare( >- { >- got => $order_1_0->{total_tax_excluded}, >- expected => 140.58, >- conf => '1 0', >- field => 'total_tax_excluded' >- } >- ); > }; > > subtest 'Configuration 1: 0 1' => sub { >- plan tests => 12; >+ plan tests => 8; > $bookseller_module->mock( > 'fetch', > sub { >@@ -464,7 +358,7 @@ for my $currency_format ( qw( US FR ) ) { > biblionumber => $biblionumber_0_1, > quantity => 2, > listprice => 82.000000, >- unitprice => 73.800000, >+ unitprice => 77.490000, > quantityreceived => 2, > basketno => $basketno_1_1, > invoiceid => $invoiceid_1_1, >@@ -483,12 +377,11 @@ for my $currency_format ( qw( US FR ) ) { > } > ); > >- # Note that this configuration is correct \o/ > compare( > { > got => $order_0_1->{rrp_tax_included}, > expected => 86.10, >- conf => '1 0', >+ conf => '0 1', > field => 'rrp_tax_included' > } > ); >@@ -496,7 +389,7 @@ for my $currency_format ( qw( US FR ) ) { > { > got => $order_0_1->{rrp_tax_excluded}, > expected => 82.00, >- conf => '1 0', >+ conf => '0 1', > field => 'rrp_tax_excluded' > } > ); >@@ -504,7 +397,7 @@ for my $currency_format ( qw( US FR ) ) { > { > got => $order_0_1->{ecost_tax_included}, > expected => 77.49, >- conf => '1 0', >+ conf => '0 1', > field => 'ecost_tax_included' > } > ); >@@ -512,7 +405,7 @@ for my $currency_format ( qw( US FR ) ) { > { > got => $order_0_1->{ecost_tax_excluded}, > expected => 73.80, >- conf => '1 0', >+ conf => '0 1', > field => 'ecost_tax_excluded' > } > ); >@@ -520,26 +413,10 @@ for my $currency_format ( qw( US FR ) ) { > { > got => $order_0_1->{tax_value}, > expected => 7.38, >- conf => '1 0', >+ conf => '0 1', > field => 'tax_value' > } > ); >- compare( >- { >- got => $order_0_1->{total_tax_included}, >- expected => 154.98, >- conf => '1 0', >- field => 'total_tax_included' >- } >- ); >- compare( >- { >- got => $order_0_1->{total_tax_excluded}, >- expected => 147.60, >- conf => '1 0', >- field => 'total_tax_excluded' >- } >- ); > > $order_0_1 = C4::Acquisition::populate_order_with_prices( > { >@@ -548,7 +425,7 @@ for my $currency_format ( qw( US FR ) ) { > receiving => 1, > } > ); >- # Note that this configuration is correct >+ > compare( > { > got => $order_0_1->{unitprice_tax_included}, >@@ -573,22 +450,6 @@ for my $currency_format ( qw( US FR ) ) { > field => 'tax_value' > } > ); >- compare( >- { >- got => $order_0_1->{total_tax_included}, >- expected => 154.98, >- conf => '0 1', >- field => 'total_tax_included' >- } >- ); >- compare( >- { >- got => $order_0_1->{total_tax_excluded}, >- expected => 147.60, >- conf => '0 1', >- field => 'total_tax_excluded' >- } >- ); > }; > } > >-- >2.1.0
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 13321
:
33789
|
33790
|
33791
|
33792
|
33793
|
34432
|
35152
|
35165
|
47489
|
47490
|
47491
|
47492
|
47493
|
47494
|
48867
|
48868
|
48869
|
48870
|
48871
|
48872
|
51080
|
51081
|
51082
|
51083
|
51084
|
51085
|
51643
|
51644
|
51645
|
51646
|
51647
|
51648
|
51649
|
52046
|
55168
|
55415
|
55416
|
55417
|
55418
|
55419
|
55420
|
55421
|
55422
|
55423
|
55427
|
56408
|
56422
|
56423
|
56424
|
56425
|
56426
|
56427
|
56428
|
56429
|
56430
|
56431
|
56432
|
56898
|
56899
|
56900
|
56901
|
56902
|
56903
|
56904
|
56905
|
56906
|
56907
|
56908
|
56909
|
56930
|
56959