Bug 5186

Summary: Setting a tax rate of zero for a vendor causes the tax value to become the default
Product: Koha Reporter: Robin Sheat <robin>
Component: AcquisitionsAssignee: Robin Sheat <robin>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: normal    
Priority: PATCH-Sent (DO NOT USE) CC: chris, katrin.fischer, nengard, paul.poulain
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: proposed patch
proposed patch
Rebased form of this patch that applies against v3.02.01
Patch that applies against master

Description Robin Sheat 2010-08-31 00:08:33 UTC
Instead, it should remain as zero, and if the field is made blank, then it should become the default.
Comment 1 Robin Sheat 2010-08-31 00:51:27 UTC
As part of this, the whole tax being ignored when creating a vendor thing will be fixed.
Comment 2 Robin Sheat 2010-08-31 03:04:37 UTC Comment hidden (obsolete)
Comment 3 Robin Sheat 2010-09-01 02:48:41 UTC Comment hidden (obsolete)
Comment 4 Robin Sheat 2010-09-01 02:52:31 UTC
From the commit message:

    Bug 5186 - allow tax rates to be set to zero
    
        This allows the tax rate for a vendor to be set to zero. Previously, a
        zero meant that the system default was used. Now, zero means no tax, and
        to have it be the default the field should simply be left empty.
        Additionally:
        * this will now show on the vendor display if the tax value is from the
          system default
        * this includes a database update that changes all the existing 0.00 tax
          settings to be NULL, which preserves existing behaviour.
        * this now saves the tax value supplied for new vendors
        * a message is shown telling people what'll happen if a field is
          left blank
        * tax is shown on a vendor even if it's zero, for clarity's sake (although
          all the other tax-related bits aren't)
Comment 5 Robin Sheat 2010-12-20 05:39:32 UTC
Created attachment 2886 [details] [review]
Rebased form of this patch that applies against v3.02.01

Patch against master coming up...
Comment 6 Robin Sheat 2010-12-20 06:18:07 UTC
Created attachment 2887 [details] [review]
Patch that applies against master

This version applies against master
Comment 7 Nicole C. Engard 2010-12-20 12:41:36 UTC
master signed off (not updating status since I didn't do them both)
Comment 8 Chris Cormack 2010-12-21 01:35:00 UTC
Master patch has been pushed to new/bug_5186 but not merged yet, can someone please test and let me know if its working ok.
Comment 9 MJ Ray (software.coop) 2011-01-07 11:13:13 UTC
This bug is mentioned in:
Bug 5186 - allow tax rates to	be set to zero (master) http://lists.koha-community.org/pipermail/koha-patches/2010-December/013433.html
Bug 5186 - allow tax to be set to zero	(3.2.1) http://lists.koha-community.org/pipermail/koha-patches/2010-December/013428.html
Bug 5186 - allow tax rates to be set to zero	(master) http://lists.koha-community.org/pipermail/koha-patches/2010-December/013429.html
Comment 10 Chris Cormack 2011-01-09 20:14:34 UTC
Merged to master, please test
Comment 11 MJ Ray (software.coop) 2011-02-04 09:13:41 UTC
This bug is mentioned in:
Bug 5186: Follow-up - Fixing wrong database	number http://lists.koha-community.org/pipermail/koha-patches/2011-January/013560.html
Comment 12 Paul Poulain 2011-04-04 09:31:15 UTC
tested, works as expected