Bug 18119

Summary: Bug 17988 broke cataloging javascript functionality
Product: Koha Reporter: Josef Moravec <josef.moravec>
Component: CatalogingAssignee: Josef Moravec <josef.moravec>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: trivial    
Priority: P5 - low CC: jonathan.druart+koha, katrin.fischer, kyle, m.de.rooy, mtj, synapse.ova
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 17988    
Bug Blocks: 17780    
Attachments: Bug 18119: Fix comment in cataloguing.js
Bug 18119: Fix comment in cataloguing.js

Description Josef Moravec 2017-02-15 07:05:35 UTC
See comment 8 on bug 17988
Comment 1 Josef Moravec 2017-02-15 07:08:49 UTC
Blocks 17780, because it is not testable without this patch
Comment 2 Josef Moravec 2017-02-15 07:09:18 UTC
Created attachment 60237 [details] [review]
Bug 18119: Fix comment in cataloguing.js

Test plan:
Go to cataloging, and try something which depends on javascript -
collapse/uncollapse fields, open authority search window, ...
    -> without patch it is not working
    -> with patch it is working correctly
Comment 3 Jonathan Druart 2017-02-15 07:50:15 UTC
Created attachment 60240 [details] [review]
Bug 18119: Fix comment in cataloguing.js

Test plan:
Go to cataloging, and try something which depends on javascript -
collapse/uncollapse fields, open authority search window, ...
    -> without patch it is not working
    -> with patch it is working correctly

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 4 Jonathan Druart 2017-02-15 07:50:52 UTC
Thanks Josef!
Skipping QA as it's  very easy one.
Comment 5 Kyle M Hall 2017-02-15 11:13:12 UTC
Pushed to master for 17.05, thanks Josef!
Comment 6 Katrin Fischer 2017-02-19 19:47:57 UTC
Decided to push last patch from 17988 and then this on top, so that master and 16.11.x have all patches.

This patch has been pushed to 16.11.x and will be in 16.11.04.
Comment 7 Marcel de Rooy 2017-02-24 08:02:34 UTC
16.05 still needs this one
Comment 8 Mason James 2017-02-24 23:10:56 UTC
(In reply to Marcel de Rooy from comment #7)
> 16.05 still needs this one

Pushed to 16.05.x, for 16.05.10 release