Attachment 25155 Details for Bug 11680 – [PASSED QA] Bug 11680 - tax rate changes on edit

[patch] [PASSED QA] Bug 11680 - tax rate changes on edit

PASSED-QA-Bug-11680---tax-rate-changes-on-edit.patch (text/plain), 1.59 KB, created by Katrin Fischer on 2014-02-09 16:24:31 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2014-02-09 16:24:31 UTC

Size: 1.59 KB

patch

obsolete

>From 180153f615ee6cde6c093491b1ccaf49d0c3c67f Mon Sep 17 00:00:00 2001
>From: Jacek Ablewicz <abl@biblos.pk.edu.pl>
>Date: Wed, 5 Feb 2014 11:16:14 +0100
>Subject: [PATCH] [PASSED QA] Bug 11680 - tax rate changes on edit
>
>If the tax rates in Acquisitions -> gist system preference are entered
>with trailing zeroes, given order tax rate value may not be correctly
>handled on order edit.
>
>To test:
>
>1) insert some tax rates with trailing zeroes in gist
>system preference (e.g: '0|0.080|0.12|0.20|0.23')
>2) place some new orders, choosing 8.0% 12.0% 20.0% 23.0%
>as gst rate
>3) try to modify them
>4) note that orders with 12.0% and 23.0% tax rates are
>preserving previously choosen rates on edit, while
>the ones with 8.0% and 20.0% do not (they are defaulting
>to the first defined tax rate)
>5) apply the patch
>6) repeat 2) and 3)
>7) all tax rates configured in system prefrence shall now
>behave properly while editing orders
>
>Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
>All tests pass
>
>Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
>Confirmed the problem and that this patch fixes it.
>Problem also exists for editing the default tax rate of a vendor.
>---
> acqui/neworderempty.pl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/acqui/neworderempty.pl b/acqui/neworderempty.pl
>index 9fdf032..2fc6100 100755
>--- a/acqui/neworderempty.pl
>+++ b/acqui/neworderempty.pl
>@@ -355,7 +355,7 @@ $template->param(
> 
> # get option values for gist syspref
> my @gst_values = map {
>-    option => $_
>+    option => $_ + 0.0
> }, split( '\|', C4::Context->preference("gist") );
> 
> $template->param(
>-- 
>1.8.3.2

Actions: View | Diff | Splinter Review

Attachments on bug 11680: 25048 | 25059 | 25155 | 25169 | 26452 | 26453