https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21248 --- Comment #1 from paxed <pasi.kallinen@joensuu.fi> --- Created attachment 78011 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=78011&action=edit Bug 21248: Fix COinS carp in MARC details page on unknown record Looking up a nonexistent record in the MARC details -page causes the COinS to carp. The fix is trivial, just move the COinS call couple lines, after the record existence check. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21248 M. Tompsett <mtompset@hotmail.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=21248 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |BLOCKED CC| |m.de.rooy@rijksmuseum.nl -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21248 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #78451|0 |1 is obsolete| | --- Comment #3 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 78644 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=78644&action=edit Bug 21248: Fix COinS carp in MARC details page on unknown record Looking up a nonexistent record in the MARC details -page causes the COinS to carp. The fix is trivial, just move the COinS call couple lines, after the record existence check. TEST PLAN --------- 1) reset_all for kohadevbox 2) log in to staff client - admin/admin 3) search catalogue for green 4) tweak the top URL to have a bad biblionumber (eg. 9999) 5) cat /var/log/koha/kohadev/plack-err.log -- floody! with errors. 6) echo | sudo tee /var/log/koha/kohadev/plack-err.log 7) restart_all 8) repeat steps 2 through 5. -- just nice starting messages, not floody. 9) run koha qa test tools Signed-off-by: Mark Tompsett <mtompset@hotmail.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21248 Nick Clemens <nick@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |nick@bywatersolutions.com Status|Passed QA |Pushed to Master --- Comment #4 from Nick Clemens <nick@bywatersolutions.com> --- Awesome work all! Pushed to master for 18.11 -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21248 Fridolin SOMERS <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|Pushed to Stable |RESOLVED CC| |fridolin.somers@biblibre.co | |m --- Comment #6 from Fridolin SOMERS <fridolin.somers@biblibre.com> --- Pushed to 17.11.x for 17.11.10 -- You are receiving this mail because: You are watching all bug changes.