Bug 15210

Summary: Novelist js throws an error if no ISBN
Product: Koha Reporter: Nick Clemens (kidclamp) <nick>
Component: OPACAssignee: Nick Clemens (kidclamp) <nick>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: brendan, f.demians, gwilliams, jdemuth, julian.maurice
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 15210 - Novelist throws an error if no ISBN
Bug 15210 - Novelist throws an error if no ISBN
Bug 15210 - Novelist throws an error if no ISBN
Bug 15210 - Novelist throws an error if no ISBN

Description Nick Clemens (kidclamp) 2015-11-18 16:48:51 UTC
If novelist is enabled and a record has no ISBN the js throws an error which subsequently breaks any custom code from opacuserjs.
Comment 1 Nick Clemens (kidclamp) 2015-11-18 16:59:25 UTC Comment hidden (obsolete)
Comment 2 Chris Cormack 2015-12-17 22:40:38 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2015-12-18 09:55:03 UTC Comment hidden (obsolete)
Comment 4 Brendan Gallagher 2015-12-30 03:56:15 UTC
Created attachment 46064 [details] [review]
Bug 15210 - Novelist throws an error if no ISBN

This patch makes opac-detail.tt check for an ISBN before attempting to load Novelist content and prevents a js error which breaks other code.

To test:
1 - Enable novelist
2 - Add a visible change to OPACUserJS eg: $(document).ready(function() {$("body").css("background-color","pink");});
3 - Load a record with no ISBN
4 - Note you do not see a color change in the body and that there is an error in the console
5 - Apply patch
6 - Reload the record
7 - You should see the color change and no error in the js

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>

Signed-off-by: Brendan A Gallagher <brendan@bywatersolutions.com>
Comment 5 Brendan Gallagher 2015-12-30 03:56:48 UTC
pushed to master - thanks!
Comment 6 Julian Maurice 2015-12-31 09:15:29 UTC
Patch pushed to 3.22.x, will be in 3.22.2
Comment 7 Frédéric Demians 2015-12-31 18:28:38 UTC
This patch has been pushed to 3.20.x, will be in 3.20.8.