Bugzilla – Attachment 95943 Details for
Bug 14543
Order lines updated that have a tax rate not in gist will have tax rate set to 0!
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 14543: Order lines updated that have a tax rate not in gist will have tax rate set to 0!
Bug-14543-Order-lines-updated-that-have-a-tax-rate.patch (text/plain), 2.66 KB, created by
Jonathan Druart
on 2019-12-03 11:21:47 UTC
(
hide
)
Description:
Bug 14543: Order lines updated that have a tax rate not in gist will have tax rate set to 0!
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2019-12-03 11:21:47 UTC
Size:
2.66 KB
patch
obsolete
>From 98342b69e6970a1897c4113447e7d3913b204fe3 Mon Sep 17 00:00:00 2001 >From: Kyle M Hall <kyle@bywatersolutions.com> >Date: Thu, 16 Jul 2015 12:14:42 -0400 >Subject: [PATCH] Bug 14543: Order lines updated that have a tax rate not in > gist will have tax rate set to 0! > >If the tax rate for a given order line has been removed from the system >preference gist, that tax rate will show in the pulldown as 0. If a >librarian then saves the order line the original tax rate will be >overwritten with 0%! > >Test Plan: >1) Find an order line >2) Remove the tax rate set for that order line from the syspref gist >3) Edit the order line >4) Note the tax rate is 0 >5) Apply this patch >6) Edit the order line >7) Note the tax rate now displays with a warning >--- > .../prog/en/modules/acqui/neworderempty.tt | 20 +++++++++++++------- > 1 file changed, 13 insertions(+), 7 deletions(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/neworderempty.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/neworderempty.tt >index c2d0a1cb12..2b37b5f98a 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/neworderempty.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/neworderempty.tt >@@ -391,13 +391,19 @@ > <li> > <label for="tax_rate">Tax rate: </label> > <select name="tax_rate" id="tax_rate" onchange="updateCosts();"> >- [% FOREACH gst IN gst_values %] >- [% IF ( gst.option == tax_rate ) %] >- <option value="[% gst.option | html %]" selected="selected">[% gst.option * 100 | html %]%</option> >- [% ELSE %] >- <option value="[% gst.option | html %]">[% gst.option * 100 | html %]%</option> >- [% END %] >- [% END %] >+ [% SET gst_found = 0 %] >+ [% FOREACH gst IN gst_values %] >+ [% IF ( gst.option == tax_rate ) %] >+ <option value="[% gst.option | html %]" selected="selected">[% gst.option * 100 | html %]%</option> >+ [% SET gst_found = 1 %] >+ [% ELSE %] >+ <option value="[% gst.option | html %]">[% gst.option * 100 | html %]%</option> >+ [% END %] >+ [% END %] >+ >+ [% IF !gst_found %] >+ <option value="[% gstrate %]" selected="selected">[% gstrate * 100 | html %]%</option> >+ [% END %] > </select> > [% ELSE %] > <input type="hidden" name="tax_rate" value="0" /> >-- >2.11.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 14543
:
41033
|
95942
|
95943
|
105181
|
105404
|
105629
|
105630