Bug 6783

Summary: Circulation restrictions dont work when other withdrawn reasons are used
Product: Koha Reporter: Savitra Sirohi <savitra.sirohi>
Component: CirculationAssignee: Meenakshi <meenakshi.r>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: enhancement    
Priority: PATCH-Sent (DO NOT USE) CC: chris, gmcharlt, koha.sekjal, kohapatch, nengard
Version: 3.6   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug id:6783 - Circulation restrictions dont work when other withdrawn reasons are used
[SIGNED-OFF] Bug id:6783 - Circulation restrictions dont work when other withdrawn reasons are used

Description Savitra Sirohi 2011-08-25 04:49:56 UTC
Circulation restrictions work only when withdrawn reason (value 1) is selected. If ones sets up other withdrawn reasons using authorized values, and selects one of them (e.g. value 2), circulation restrictions don't work. This changes fixes this problem.
Comment 1 Amit Gupta 2011-09-08 05:57:59 UTC Comment hidden (obsolete)
Comment 2 Nicole C. Engard 2011-09-08 13:37:03 UTC
From a documentation standpoint - can you explain this further?  What circulation restrictions are you talking about? How are you setting them up?
Comment 3 Savitra Sirohi 2011-09-09 04:54:52 UTC
Nicole, for withdrawn items regular circ transactions -- checkout, checkin and renewals are not allowed. This change extends those same restrictions to other withdrawn reasons.
Comment 4 Nicole C. Engard 2011-09-10 22:10:49 UTC
Oh! Okay, that I understand. I'll try and test this now.
Comment 5 Nicole C. Engard 2011-09-10 22:17:32 UTC
Created attachment 5394 [details] [review]
[SIGNED-OFF] Bug id:6783 - Circulation restrictions dont work when other withdrawn reasons are used

Circulation restrictions work only when withdrawn reason (value 1) is selected.
If ones sets up other withdrawn reasons using authorized values, and selects
one of them (e.g. value 2), circulation restrictions don't work. This changes
fixes this problem.

Signed-off-by: Amit Gupta <amit.gupta@osslabs.biz>
Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Comment 6 Ian Walls 2011-09-11 00:30:14 UTC
Easy to read patch, makes "Withdrawn" behave more like other non-circ statuses.  Marking as Passed QA.
Comment 7 Chris Cormack 2011-09-11 09:42:09 UTC
Pushed, please test
Comment 8 Jared Camins-Esakov 2012-05-23 12:52:56 UTC
Seems to work. Closing.