[Bug 29970] New: 500 Error when a record cannot be displayed
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29970 Bug ID: 29970 Summary: 500 Error when a record cannot be displayed Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Cataloging Assignee: koha-bugs@lists.koha-community.org Reporter: nick@bywatersolutions.com QA Contact: testopia@bugs.koha-community.org CC: m.de.rooy@rijksmuseum.nl When a record cannot be decoded Koha shows a 500 error to the user. The code throws a Koha::Exceptions::Metadata::Invalid which is recorded in the plack-error We should catch the exception and provide a better message to the user. Often, these records will load in the editor, we should provide a link to the user so they can attempt to fix the record Alternately, we should attempt to assume the record is encoded in a different format (marc8) and attempt to convert uft-8 -- 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=29970 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer@bsz-bw.de, | |kyle@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29970 Valerie <valerie@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |valerie@bywatersolutions.co | |m --- Comment #1 from Valerie <valerie@bywatersolutions.com> --- This would be much more helpful than the 500 Error. -- 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=29970 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff Depends on| |11175, 28734 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11175 [Bug 11175] Show the parent record's component parts in the detailed views https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28734 [Bug 28734] Koha::Biblio->get_marc_notes should parse authorised values -- 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=29970 --- Comment #2 from Nick Clemens <nick@bywatersolutions.com> --- Created attachment 130104 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=130104&action=edit Bug 29970: eval processing of MARC record to avoid crash detail.pl displays a warning at the top of the page when a record contains parsing errors Bug 28734 and bug 11175 added additional parsing of the MARC record, but they did not catch the thrown errors. The existing pattern in the script is to eval responses to avoid crashing This patch wraps the new calls in eval{} To test: 1 - Find a record in the staff catalog 2 - Edit the record 3 - In the 520 notes field, add an ASCII escape character (27 decimal, 1b hex) On ubuntu Ctrl+Shift+u, then 1b, enter 4 - Save the record 5 - 500 Error 6 - Apply patch 7 - Reload 8 - The record displays with a warning at the top -- 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=29970 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |andrew@bywatersolutions.com 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=29970 Jonathan Druart <jonathan.druart+koha@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jonathan.druart+koha@gmail. | |com --- Comment #3 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- I don't understand, we were supposed to have fixed that few days ago, with bug 29690. If the problem still persists, please have a look at the solution there (test $marc_record) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29970 Jonathan Druart <jonathan.druart+koha@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |DUPLICATE Status|Needs Signoff |RESOLVED --- Comment #4 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- You patch does not apply, I think it's a duplicate. *** This bug has been marked as a duplicate of bug 29690 *** -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org