Bug 10439

Summary: Cannot modify the notice's library
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: NoticesAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P5 - low CC: bgkriegel, chrish, gmcharlt, nengard, srdjan, tomascohen
Version: Main   
Hardware: All   
OS: All   
URL: tools/letter.pl
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 10439: FIX modify the library for a notice.
[SIGNED-OFF] Bug 10439: FIX modify the library for a notice.
[PASSED QA] Bug 10439: FIX modify the library for a notice.

Description Jonathan Druart 2013-06-10 12:13:41 UTC
To reproduce:
- switch on the the IndependentBranches pref
- go on Home › Tools › Notices & Slips 
- click on a edit link
- change the library and save
 => changes are not saved.
Comment 1 Jonathan Druart 2013-06-10 12:15:18 UTC Comment hidden (obsolete)
Comment 2 Srdjan Jankovic 2013-06-11 03:46:51 UTC Comment hidden (obsolete)
Comment 3 Srdjan Jankovic 2013-06-11 03:47:41 UTC
It actually affects "New Notice" as well
Comment 4 Katrin Fischer 2013-06-16 16:50:50 UTC
Created attachment 19044 [details] [review]
[PASSED QA] Bug 10439: FIX modify the library for a notice.

2 javascript functions should be apply only in a specific case, not for all
views.

Test plan:
- switch on the IndependentBranches pref
- try to change the library for a notice. Before the patch, no change is
applied.

Signed-off-by: Srdjan <srdjan@catalyst.net.nz>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>

Note: only a superlibrarian can change the library or create new
notices for every library. This bug can only be reproduced with
superlibrarian permissions.

Tested with independentbranches on/off.
All tests and QA script pass.
Comment 5 Galen Charlton 2013-06-17 02:00:22 UTC
Pushed to master.  Thanks, Jonathan!
Comment 6 Tomás Cohen Arazi 2013-07-02 18:00:17 UTC
This patch has been pushed to 3.12.x, will be in 3.12.2.

Thanks Jonathan!
Comment 7 Bernardo Gonzalez Kriegel 2013-07-13 18:17:36 UTC
Pushed to 3.10.x, will be in 3.10.8
Comment 8 Chris Hall 2013-07-18 09:13:56 UTC
Pushed to 3.8.x, will be in 3.8.15