Bug 9933 - Changing/editing circulation rules for specific library affects all libraries
Summary: Changing/editing circulation rules for specific library affects all libraries
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: Main
Hardware: All All
: P5 - low blocker (vote)
Assignee: Bugs List
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-03-27 06:07 UTC by Katrin Fischer
Modified: 2019-06-27 09:24 UTC (History)
5 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 9933: Unable to edit or add rules for any branch except default (1.30 KB, patch)
2013-04-08 08:47 UTC, Chris Cormack
Details | Diff | Splinter Review
Bug 9933: Unable to edit or add rules for any branch except default (1.37 KB, patch)
2013-04-08 10:04 UTC, Mirko Tietgen
Details | Diff | Splinter Review
Bug 9933: Unable to edit or add rules for any branch except default (1.45 KB, patch)
2013-04-08 10:10 UTC, Mason James
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Katrin Fischer 2013-03-27 06:07:04 UTC
I think we might be missing a parameter for the branch here - when editing or adding circulation rules for a specific library the change is made to all libraries instead. 
Also upon saving the screen switches to all libraries and does not stay on the selected specific library.
Comment 1 Chris Cormack 2013-04-08 08:28:09 UTC
I concur, I can recreate this bug, working on a fix now
Comment 2 Chris Cormack 2013-04-08 08:47:12 UTC Comment hidden (obsolete)
Comment 3 Mirko Tietgen 2013-04-08 10:04:48 UTC Comment hidden (obsolete)
Comment 4 Mason James 2013-04-08 10:10:05 UTC
Created attachment 17277 [details] [review]
Bug 9933: Unable to edit or add rules for any branch except default

To test
1/ Select a branch in circ rules
2/ Edit or add a rule, note this change is made for default instead of
the library you selected
3/ Apply the patch
4/ Try 1 and 2 again

Signed-off-by: Mirko Tietgen <mirko@abunchofthings.net>
Works as expected.
Signed-off-by: Mason James <mtj@kohaaloha.com>
Comment 5 Mason James 2013-04-08 10:11:14 UTC
(In reply to comment #4)
> Created attachment 17277 [details] [review] [review]
> Bug 9933: Unable to edit or add rules for any branch except default
> 
> To test
> 1/ Select a branch in circ rules
> 2/ Edit or add a rule, note this change is made for default instead of
> the library you selected
> 3/ Apply the patch
> 4/ Try 1 and 2 again
> 
> Signed-off-by: Mirko Tietgen <mirko@abunchofthings.net>
> Works as expected.
> Signed-off-by: Mason James <mtj@kohaaloha.com>

works for me too, passing qa...
Comment 6 Jared Camins-Esakov 2013-04-15 12:32:03 UTC
This patch has been pushed to master and 3.12.x.