[Bug 26950] New: Checkin failure messages are unclear or too specific
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26950 Bug ID: 26950 Summary: Checkin failure messages are unclear or too specific Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: koha-bugs@lists.koha-community.org Reporter: nick@bywatersolutions.com QA Contact: testopia@bugs.koha-community.org Currently checkins can fail for one of two known reasons: 1 - Duplicate issue ids 2 - Anonymous borrower configuration issues In the first case we refer to the 'data problems' or 'data corrupted' section of the about page. All errors are on the 'System information' tab and this is what we should mention In the second case we refer to the data problems section, but information is in another section on the tab. Additionally, during checkin we issue an 'Error' but on the about page we call it a 'Warning' we should upgrade to error and specify what is broken -- 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=26950 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- 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=26950 --- Comment #1 from Nick Clemens <nick@bywatersolutions.com> --- Created attachment 113246 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=113246&action=edit Bug 26950: Make Anonymous patron warnings 'errors' and refer to the system information tab on about.pl Currently checkins can fail for one of two known reasons: 1 - Duplicate issue ids 2 - Anonymous borrower configuration issues In the first case we refer to the 'data problems' or 'data corrupted' section of the about page. All errors are on the 'System information' tab and this is what we should mention In the second case we refer to the data problems section, but information is in another section on the tab. Additionally, during checkin we issue an 'Error' but on the about page we call it a 'Warning' we should upgrade to error and specify what is broken Additionally this patch fixes the warning that should display while issuing to a patron if anonymous is not set and patron has selected privacy To test: 1 - Enable OPACPrivacy 2 - Set anonymousborrower to '0' 3 - Log in a a patron on OPAC and set your privacy to never 4 - Issue a book to that patron in the staff client 5 - Note there is no warning 6 - Return the book 7 - Note the circulation page displays an error and the checkin fails and refers to 'data problems' 8 - Apply patch 9 - Issue another book to patron 10 - Note warning on the circulation page 11 - Try to check in a book again 12 - Check the new message makes sense 13 - Click the link to 'about.pl' check that errors are displayed 14 - Check the logs 15 - Confirm updated error message makes sense -- 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=26950 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |liz@catalyst.net.nz Assignee|koha-bugs@lists.koha-commun |nick@bywatersolutions.com |ity.org | -- 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=26950 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC|liz@catalyst.net.nz |liz@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26950 Fridolin SOMERS <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |fridolin.somers@biblibre.co | |m --- Comment #2 from Fridolin SOMERS <fridolin.somers@biblibre.com> --- Testing it.
2 - Set anonymousborrower to '0' Syspref is 'AnonymousPatron', I will change it on signoff
-- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26950 Fridolin SOMERS <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |String patch 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=26950 Fridolin SOMERS <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #113246|0 |1 is obsolete| | --- Comment #3 from Fridolin SOMERS <fridolin.somers@biblibre.com> --- Created attachment 113300 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=113300&action=edit Bug 26950: Make Anonymous patron warnings 'errors' and refer to the system information tab on about.pl Currently checkins can fail for one of two known reasons: 1 - Duplicate issue ids 2 - Anonymous borrower configuration issues In the first case we refer to the 'data problems' or 'data corrupted' section of the about page. All errors are on the 'System information' tab and this is what we should mention In the second case we refer to the data problems section, but information is in another section on the tab. Additionally, during checkin we issue an 'Error' but on the about page we call it a 'Warning' we should upgrade to error and specify what is broken Additionally this patch fixes the warning that should display while issuing to a patron if anonymous is not set and patron has selected privacy To test: 1 - Enable system preference OPACPrivacy 2 - Set system preference AnonymousPatron to '0' 3 - Log in a a patron on OPAC and set your privacy to never 4 - Issue a book to that patron in the staff client 5 - Note there is no warning 6 - Return the book 7 - Note the circulation page displays an error and the checkin fails and refers to 'data problems' 8 - Apply patch 9 - Issue another book to patron 10 - Note warning on the circulation page 11 - Try to check in a book again 12 - Check the new message makes sense 13 - Click the link to 'about.pl' check that errors are displayed 14 - Check the logs 15 - Confirm updated error message makes sense Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26950 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #113300|0 |1 is obsolete| | --- Comment #4 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 115028 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115028&action=edit Bug 26950: Make Anonymous patron warnings 'errors' and refer to the system information tab on about.pl Currently checkins can fail for one of two known reasons: 1 - Duplicate issue ids 2 - Anonymous borrower configuration issues In the first case we refer to the 'data problems' or 'data corrupted' section of the about page. All errors are on the 'System information' tab and this is what we should mention In the second case we refer to the data problems section, but information is in another section on the tab. Additionally, during checkin we issue an 'Error' but on the about page we call it a 'Warning' we should upgrade to error and specify what is broken Additionally this patch fixes the warning that should display while issuing to a patron if anonymous is not set and patron has selected privacy To test: 1 - Enable system preference OPACPrivacy 2 - Set system preference AnonymousPatron to '0' 3 - Log in a a patron on OPAC and set your privacy to never 4 - Issue a book to that patron in the staff client 5 - Note there is no warning 6 - Return the book 7 - Note the circulation page displays an error and the checkin fails and refers to 'data problems' 8 - Apply patch 9 - Issue another book to patron 10 - Note warning on the circulation page 11 - Try to check in a book again 12 - Check the new message makes sense 13 - Click the link to 'about.pl' check that errors are displayed 14 - Check the logs 15 - Confirm updated error message makes sense Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26950 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |martin.renvoize@ptfs-europe | |.com Status|Signed Off |Passed QA --- Comment #5 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Trivial improvement... Passing QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26950 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to master Version(s)| |21.05.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=26950 --- Comment #6 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Pushed to master for 21.05, thanks to everybody involved! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26950 --- Comment #7 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 115116 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=115116&action=edit Bug 26950: Fix SIP/Message.t t/db_dependent/SIP/Message.t .. 2/9 # Failed test 'DBIx error on duplicate issue_id' # at t/db_dependent/SIP/Message.t line 643. # found warning: DBIx::Class::Storage::DBI::_dbh_execute(): DBI Exception: DBD::mysql::st execute failed: Duplicate entry '1' for key 'PRIMARY' [for Statement "INSERT INTO `old_issues` ( `auto_renew`, `auto_renew_error`, `borrowernumber`, `branchcode`, `date_due`, `issue_id`, `issuedate`, `issuer_id`, `itemnumber`, `lastreneweddate`, `note`, `notedate`, `noteseen`, `onsite_checkout`, `renewals`, `returndate`, `timestamp`, `unseen_renewals`) VALUES ( ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ? )" with ParamValues: 0=0, 1=undef, 2=57, 3='m_HHGh', 4=undef, 5=1, 6=undef, 7=undef, 8=973, 9=undef, 10=undef, 11=undef, 12=undef, 13=0, 14=0, 15='2021-01-14 16:12:47', 16='2021-01-13 16:12:48', 17=0] at /kohadevbox/koha/Koha/Object.pm line 169 # found carped warning: The checkin for the following issue failed, Please go to the about page and check all messages on the 'System information' to see if there are configuration / data issues (Duplicate ID)$VAR1 = { # expected to find warning: (?^u:Duplicate entry) # expected to find warning: (?^u:data corrupted) # Looks like you failed 1 test of 35. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26950 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart@bugs.koha-c | |ommunity.org --- Comment #8 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Last patch pushed to master. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26950 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|Pushed to master |RESOLVED --- Comment #9 from Fridolin Somers <fridolin.somers@biblibre.com> --- Enhancement not pushed to 20.11.x -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26950 Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |caroline.cyr-la-rose@inlibr | |o.com -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org