[Bug 7726] New: AllowItemsOnHoldCheckout has columns in wrong order in sysprefs.sql
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7726 Priority: P5 - low Change sponsored?: --- Bug ID: 7726 CC: gmcharlt@gmail.com Assignee: paul.poulain@biblibre.com Summary: AllowItemsOnHoldCheckout has columns in wrong order in sysprefs.sql QA Contact: koha.sekjal@gmail.com Severity: trivial Classification: Unclassified OS: All Reporter: magnus@enger.priv.no Hardware: All Status: NEW Version: master Component: System Administration Product: Koha installer/data/mysql/sysprefs.sql has this, on line 341: INSERT INTO systempreferences (variable,value,options,explanation,type) VALUES ('AllowItemsOnHoldCheckout',0,'Do not generate RESERVE_WAITING and RESERVED warning when checking out items reserved to someone else. This allows self checkouts for those items.','','YesNo'); The order of the columns should be switched to this: (variable,value,explanation,options,type) The problem is also present in updatedatabase.pl: $dbh->do("INSERT INTO `systempreferences` (variable,value,options,explanation,type) VALUES ('AllowItemsOnHoldCheckout',0,'Do not generate RESERVE_WAITING and RESERVED warning when checking out items reserved to someone else. This allows self checkouts for those items.','','YesNo')"); This does probably not have any practical implications (hence severity = trivial), since the "explanation" is not used anymore and "options" is not used when the type is YesNo (I think?). -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7726 --- Comment #1 from Paul Poulain <paul.poulain@biblibre.com> --- Created attachment 8643 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=8643&action=edit Bug 7726: explanation and options that were in the wrong order I also removed the ` mysql-ism -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7726 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7726 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7726 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |chris@bigballofwax.co.nz --- Comment #2 from Chris Cormack <chris@bigballofwax.co.nz> --- (In reply to comment #1)
Created attachment 8643 [details] Bug 7726: explanation and options that were in the wrong order
I also removed the ` mysql-ism
If these files are to become non database specific, moving them out of the mysql dir makes sense. Removing mysqlisms from mysql specific files doesn't seem like a hugely useful thing to do. In this case it doesn't break anything, in the case of removing them all from the mysql/kohastructure.sql it renders that file unable to create a database. So a blanket policy of removing them under all circumstances is dangerous. Making updatedatabase.pl db agnostic makes sense, making the schemas .. not so much. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7726 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #8643|0 |1 is obsolete| | --- Comment #3 from Chris Cormack <chris@bigballofwax.co.nz> --- Created attachment 9039 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=9039&action=edit Bug 7726: explanation and options that were in the wrong order I also removed the ` mysql-ism Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7726 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7726 Ian Walls <koha.sekjal@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #4 from Ian Walls <koha.sekjal@gmail.com> --- Updates an old DBRev to fix the issue, rather than updating the values in the existing database. The result, for anyone who has upgraded since 3.07.00.012, will not fix their database values. This is exceedingly minor, since these values are not displayed in the syspref editor, but better do fix this completely than only partially. Marking Failed QA -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7726 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off --- Comment #5 from Paul Poulain <paul.poulain@biblibre.com> --- (In reply to comment #4)
Updates an old DBRev to fix the issue, rather than updating the values in the existing database. The result, for anyone who has upgraded since 3.07.00.012, will not fix their database values.
This is exceedingly minor, since these values are not displayed in the syspref editor, but better do fix this completely than only partially.
Marking Failed QA
I this had been released I would agree. But since it hasn't, I disagree. And: * I don't have time to provide a new patch * releasing with this mistake (in less than 2 weeks) would be a shame So, I think this patch should pass QA, pushed, and I can move it to "assigned" again, for a more complete fix later. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7726 M. de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA CC| |m.de.rooy@rijksmuseum.nl --- Comment #6 from M. de Rooy <m.de.rooy@rijksmuseum.nl> --- QA Comment: I would not object to push it. It resolves a bug (the sooner, the better). I agree with Ian that you could also send a followup to fix installations that already past this point. Passed QA -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7726 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |ASSIGNED Version|master |rel_3_8 --- Comment #7 from Paul Poulain <paul.poulain@biblibre.com> --- Patch pushed to master before 3.8 release. Switching back to ASSIGNED though, to provide a fix for installations already updated -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7726 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RESOLVED CC| |katrin.fischer@bsz-bw.de Resolution|--- |FIXED --- Comment #8 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- This has been pushed to 3.8, we didn't get any complaints and it would only have affected people installing an unstable release for a short amount of tim - I am closing this. -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org