https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18736

Marcel de Rooy <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|BLOCKED                     |Failed QA

--- Comment #161 from Marcel de Rooy <[email protected]> ---
Test pass. QA pass.
Hmm. Not wanting to be too precise, but we still have some subtle differences
in some lines of the last patch. The first line rounds at the unit; with a
whole quantity, it will be rounded. The second line is not rounded since you
multiply with tax rate. The algorithm of tax = incl - excl is only used in the
first line.
The third line shows rounding at final stage.
+            $order->{tax_value_on_ordering} = (
get_rounded_price($order->{ecost_tax_included}) -
get_rounded_price($order->{ecost_tax_excluded}) ) * $order->{quantity};
+            $order->{tax_value_on_ordering} = $order->{quantity} *
get_rounded_price($order->{ecost_tax_excluded}) *
$order->{tax_rate_on_ordering};

+        my $subtotal = get_rounded_price( $left *
$data->{'ecost_tax_included'} );

Please explain why you do what you do.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to