[Bug 8832] New: can't enter a tax rate on vendor or when ordering
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8832 Priority: P5 - low Change sponsored?: --- Bug ID: 8832 Assignee: koha-bugs@lists.koha-community.org Summary: can't enter a tax rate on vendor or when ordering Severity: major Classification: Unclassified OS: All Reporter: nengard@gmail.com Hardware: All Status: NEW Version: master Component: Acquisitions Product: Koha Created attachment 12536 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=12536&action=edit placing an order I can no longer enter a tax rate when adding a new vendor, editing a vendor or placing an order. It's always a pull down with 0% in it. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8832 --- Comment #1 from Nicole C. Engard <nengard@gmail.com> --- Created attachment 12537 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=12537&action=edit editing a vendor -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8832 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer@bsz-bw.de Depends on| |5335 --- Comment #2 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi Nicole, I am quite sure this is a problem introduced by bug 5335: - my $default_gst_rate = (C4::Context->preference('gist') * 100) || '0.0'; + # get option values from gist syspref + my @gst_values = map { + option => $_ + }, split( '\|', C4::Context->preference("gist") ); It seems it expects a | separated list of values now for the gist system preference. I think this maybe slipped in by accident as 5335 doesn't seem to include a database update of any sort. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8832 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|major |critical --- Comment #3 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Updating severity - this is a bad bug. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8832 --- Comment #4 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Ok, seems I am not fully awake yet, there is an updatedatebase in 5335, but it doesn't touch systempreferences. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8832 M. Tompsett <mtompset@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mtompset@hotmail.com --- Comment #5 from M. Tompsett <mtompset@hotmail.com> --- Yes, but the gist system preference is actually a character field, which I thought was bizarre when I was looking around there. Have you attempted to put 0|12|13|4|string into gist, nengard? -- By the way, that (because of string) should also trigger errors (if you select string), because we have no field validation on system preferences, as far as I know. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8832 --- Comment #6 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- It still seems like an undocumented change of behaviour that maybe belongs to a different development. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8832 --- Comment #7 from Nicole C. Engard <nengard@gmail.com> --- Created attachment 12548 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=12548&action=edit preference values Okay, this seems a bit silly to me, but most US libraries don't charge tax. I followed the preference instructions and entered data like in the picture (12%|6%|7%|5%) but then it appears way wrong in the pull down. So the preference needs to be updated. Also, what happens to those who have a tax rate already on a vendor but not in the preference and upgrade? More importantly what happens to those who had 0 in the gist? That used to hide all of the tax fields from acq making it easier for those who don't use tax rates .... I think that this change in the preference was not thought out and I unfortunately missed it and didn't complain before now. Nicole -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8832 --- Comment #8 from Nicole C. Engard <nengard@gmail.com> --- Created attachment 12549 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=12549&action=edit pull down after entering values in preference -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8832 --- Comment #9 from M. Tompsett <mtompset@hotmail.com> --- Actually, perhaps the list should be parsed for %'s and then not multiply by 100. So that way you could have 12% or 0.12 to generate the same result. ;) -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8832 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |paul.poulain@biblibre.com --- Comment #10 from Paul Poulain <paul.poulain@biblibre.com> --- (In reply to comment #9)
Actually, perhaps the list should be parsed for %'s and then not multiply by 100. So that way you could have 12% or 0.12 to generate the same result. ;)
you're right here: it's written on the syspref description: "Enter in Numeric form 0.12 for 12%. First is default. If you want more than 1 value, please separate with |" (In reply to comment #6)
It still seems like an undocumented change of behaviour that maybe belongs to a different development.
Katrin, I don't know what you mean here: the main purpose of bug 5335 was to have tax rate at order line level, instead of bookseller level. In France, books have a 5.5% VAT, while CD/DVD have a 19.6%. That was a big pain for libraries. The only problem I see here is that, if you have no tax rates defined, you just have '0'. Maybe a follow-up could display, if the syspref is empty "No tax rate entered, go to syspref gist to define possible tax rate" Also note that, now, the bookseller tax rate is just the *default* one. When you enter an order line, you can always choose between all taxes listed in the syspref.
Also, what happens to those who have a tax rate already on a vendor but not in the preference and upgrade? If you have already a tax that is one of the tax in the syspref, nothing specific. If you have a tax rate that is not in the syspref, then you'll be in trouble when ordering, because you'll just get the "0"
Updating the gist syspref is something you *must* do when upgrading. We could do a follow-up that does something like: UPDATE syspref value= join '|', @{SELECT distinct gist FROM aqbooksellers} (ie = put in the syspref all the distinct values that exist in the booksellers tax rates) -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8832 --- Comment #11 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi Paul, sorry for being confusing. I think there are different things to think about here: 1) If the system preference is empty, the tax information and configuration fields should be hidden from display. There are libraries which don't want to use the feature and it worked like this before - so it's a regression for them. 2) We need a good update path for libraries. If we require now a list of values the update has to take care of that and prefill the list. From Nicole's description with the pull down I think that at the moment if you update your vendor the tax value will be lost if the value is not in the gist system preference. I didn't really test the feature yet, so this is mostly based on looking at the code and on the descriptions from others. Hope I am not too far off. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8832 --- Comment #12 from Paul Poulain <paul.poulain@biblibre.com> --- (In reply to comment #11)
1) If the system preference is empty, the tax information and configuration fields should be hidden from display. There are libraries which don't want to use the feature and it worked like this before - so it's a regression for them. OK, I hesitated between "display that the syspref is empty" or "ignore the gst". I think that libraries that want a gst will complain that there's none ( ;-) ) so OK with hiding the gst for libraries that have the syspref empty.
2) We need a good update path for libraries. If we require now a list of values the update has to take care of that and prefill the list. From Nicole's description with the pull down I think that at the moment if you update your vendor the tax value will be lost if the value is not in the gist system preference. yes, that's why i'm proposing to fill it automatically with all distinct values
follow-up easy to do. Will do it quickly (Jonathan is afk for 2 more week) that exist in aqbooksellers before the update.
I didn't really test the feature yet, so this is mostly based on looking at the code and on the descriptions from others. Hope I am not too far off. not too far, and agreeing with my proposal, so it's fine ;-)
-- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8832 --- Comment #13 from Nicole C. Engard <nengard@gmail.com> --- (In reply to comment #12)
so OK with hiding the gst for libraries that have the syspref empty.
+1, also if the pref is 0 (as mine was) it should either hide the tax fields or it should be changed to empty. At one point 0 was the default we were supposed to have in there to hide the tax fields.
yes, that's why i'm proposing to fill it automatically with all distinct values that exist in aqbooksellers before the update.
That would be a nice option, that way people don't lose the settings they have in place already. Nicole -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8832 --- Comment #14 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 12815 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=12815&action=edit Bug 8832: Update the gist pref and hide the tax rate value if not needed The gist system preference will be updated with the existing values and with the supplier's gstrate values. The "Tax rate" part becomes hidden if the syspref is not filled. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8832 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff CC| |jonathan.druart@biblibre.co | |m -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8832 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |jonathan.druart@biblibre.co |ity.org |m -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8832 --- Comment #15 from Paul Poulain <paul.poulain@biblibre.com> --- (In reply to comment #14)
Created attachment 12815 [details] Bug 8832: Update the gist pref and hide the tax rate value if not needed
The gist system preference will be updated with the existing values and with the supplier's gstrate values. The "Tax rate" part becomes hidden if the syspref is not filled.
QA comment: * Jonathan, in your patch, you set the version to 062 instead of XXX * I haven't tested the patch (waiting for nicole or someone else signoff), but it looks very wisely written: if the syspref has been filled, what was in is preserved, only values that are in aqbookseller.gist and not in the syspref are added. Well done ! -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8832 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #12815|0 |1 is obsolete| | --- Comment #16 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 12911 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=12911&action=edit Bug 8832: Update the gist pref and hide the tax rate value if not needed The gist system preference will be updated with the existing values and with the supplier's gstrate values. The "Tax rate" part becomes hidden if the syspref is not filled. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8832 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #12911|0 |1 is obsolete| | --- Comment #17 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 13234 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=13234&action=edit [SIGNED-OFF] Bug 8832: Update the gist pref and hide the tax rate value if not needed The gist system preference will be updated with the existing values and with the supplier's gstrate values. The "Tax rate" part becomes hidden if the syspref is not filled. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8832 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off CC| |kyle@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8832 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA Version|master |3.10 --- Comment #18 from Paul Poulain <paul.poulain@biblibre.com> --- QA comments: * small patch, despite it's size, the largest part of it is reindenting after addind a if * passes koha-qa.pl * critical problem, need to be fixed, no string change passed QA -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8832 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master CC| |jcamins@cpbibliography.com --- Comment #19 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- This patch has been pushed to master. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8832 --- Comment #20 from Paul Poulain <paul.poulain@biblibre.com> --- Patch pushed to branch 3.10.x -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8832 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to Master |RESOLVED CC| |chris@bigballofwax.co.nz Resolution|--- |FIXED --- Comment #21 from Chris Cormack <chris@bigballofwax.co.nz> --- Released in 3.10.0 -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org