[Bug 35033] New: Add a validation for biblioitems in about/system information
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35033 Bug ID: 35033 Summary: Add a validation for biblioitems in about/system information Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: About Assignee: koha-bugs@lists.koha-community.org Reporter: philippe.blouin@inlibro.com QA Contact: testopia@bugs.koha-community.org In the About section, under the System information tab, we get validation between issues, borrowers, biblio, items, reserves tables, and their deleted/old counterpart. But there's no validation for biblioitems. And yes, on issues during creation, most probably, discrepencies do occur, as I've now discovered on 10 different clients. This patch will simply add the same test (and display) as the others, but for biblioitems. Testing 0) Create a simple biblio entry through cataloguing. Note the biblioitemnumber created. 1) Insert a dummy entry in deletedbiblioitem using the biblioitemnumber. An simple SQL will do insert into deletedbiblioitems select * from biblioitems where biblioitemnumber = GIVENbin; 2) Go to about, see there's no warning in systeminformation. 3) apply the patch, validate that an error appears. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35033 --- Comment #1 from Blou <philippe.blouin@inlibro.com> --- Created attachment 156852 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=156852&action=edit Bug 35033: Add a validation for biblioitems in about/system information In the About section, under the System information tab, we get validation between issues, borrowers, biblio, items, reserves tables, and their deleted/old counterpart. But there's no validation for biblioitems. This patch will simply add the same test (and display) as the others, but for biblioitems. Testing 0) Create a simple biblio entry through cataloguing. Note the biblioitemnumber created. 1) Insert a dummy entry in deletedbiblioitem using the biblioitemnumber. An simple SQL will do insert into deletedbiblioitems select * from biblioitems where biblioitemnumber = GIVENbin; 2) Go to about, see there's no warning in systeminformation. 3) apply the patch, validate that an error appears. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35033 Blou <philippe.blouin@inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |caroline.cyr-la-rose@inlibr | |o.com, | |marie-luce.laflamme@inlibro | |.com, | |philippe.blouin@inlibro.com Patch complexity|--- |Trivial patch Assignee|koha-bugs@lists.koha-commun |philippe.blouin@inlibro.com |ity.org | Status|NEW |Needs Signoff -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35033 Blou <philippe.blouin@inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #156852|0 |1 is obsolete| | --- Comment #2 from Blou <philippe.blouin@inlibro.com> --- Created attachment 156853 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=156853&action=edit Bug 35033: Add a validation for biblioitems in about/system information In the About section, under the System information tab, we get validation between issues, borrowers, biblio, items, reserves tables, and their deleted/old counterpart. But there's no validation for biblioitems. This patch will simply add the same test (and display) as the others, but for biblioitems. Testing 0) Create a simple biblio entry through cataloguing. Note the biblioitemnumber created. 1) Insert a dummy entry in deletedbiblioitem using the biblioitemnumber. An simple SQL will do insert into deletedbiblioitems select * from biblioitems where biblioitemnumber = GIVENbin; 2) Go to about, see there's no warning in systeminformation. 3) apply the patch, validate that an error appears. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35033 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35033 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #156853|0 |1 is obsolete| | --- Comment #3 from David Nind <david@davidnind.com> --- Created attachment 157621 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=157621&action=edit Bug 35033: Add a validation for biblioitems in about/system information In the About section, under the System information tab, we get validation between issues, borrowers, biblio, items, reserves tables, and their deleted/old counterpart. But there's no validation for biblioitems. This patch will simply add the same test (and display) as the others, but for biblioitems. Testing 0) Create a simple biblio entry through cataloguing. Note the biblioitemnumber created. 1) Insert a dummy entry in deletedbiblioitem using the biblioitemnumber. An simple SQL will do insert into deletedbiblioitems select * from biblioitems where biblioitemnumber = GIVENbin; 2) Go to about, see there's no warning in systeminformation. 3) apply the patch, validate that an error appears. Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35033 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|enhancement |minor -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35033 Victor Grousset/tuxayo <victor@tuxayo.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #157621|0 |1 is obsolete| | --- Comment #4 from Victor Grousset/tuxayo <victor@tuxayo.net> --- Created attachment 158098 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=158098&action=edit Bug 35033: Add a validation for biblioitems in about/system information In the About section, under the System information tab, we get validation between issues, borrowers, biblio, items, reserves tables, and their deleted/old counterpart. But there's no validation for biblioitems. This patch will simply add the same test (and display) as the others, but for biblioitems. Testing 0) Create a simple biblio entry through cataloguing. Note the biblioitemnumber created. 1) Insert a dummy entry in deletedbiblioitem using the biblioitemnumber. An simple SQL will do insert into deletedbiblioitems select * from biblioitems where biblioitemnumber = GIVENbin; 2) Go to about, see there's no warning in systeminformation. 3) apply the patch, validate that an error appears. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35033 --- Comment #5 from Victor Grousset/tuxayo <victor@tuxayo.net> --- Created attachment 158099 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=158099&action=edit Bug 35033: (QA follow-up): tidy up code -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35033 Victor Grousset/tuxayo <victor@tuxayo.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA CC| |victor@tuxayo.net --- Comment #6 from Victor Grousset/tuxayo <victor@tuxayo.net> --- Works, makes sense, QA script happy, code looks good, passing QA :) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35033 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to master Version(s)| |23.11.00 released in| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35033 --- Comment #7 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Pushed to master for 23.11. Nice work everyone, thanks! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35033 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to master |Pushed to stable CC| |fridolin.somers@biblibre.co | |m Version(s)|23.11.00 |23.11.00,23.05.06 released in| | --- Comment #8 from Fridolin Somers <fridolin.somers@biblibre.com> --- Pushed to 23.05.x for 23.05.06 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35033 Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to stable |Pushed to oldstable Version(s)|23.11.00,23.05.06 |23.11.00,23.05.06,22.11.12 released in| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35033 --- Comment #9 from Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> --- Nice work everyone! Pushed to oldstable for 22.11.x -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35033 Magnus Enger <magnus@libriotech.no> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=38617 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35033 Lisette Scheer <lisette@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |40740 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40740 [Bug 40740] Validation for biblioitems in about/system information joins on biblionumber not biblioitemnumber -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org