Summary: | Circulation restrictions dont work when other withdrawn reasons are used | ||
---|---|---|---|
Product: | Koha | Reporter: | Savitra Sirohi <savitra.sirohi> |
Component: | Circulation | Assignee: | 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
Created attachment 5350 [details] [review] Bug id:6783 - Circulation restrictions dont work when other withdrawn reasons are used From a documentation standpoint - can you explain this further? What circulation restrictions are you talking about? How are you setting them up? Nicole, for withdrawn items regular circ transactions -- checkout, checkin and renewals are not allowed. This change extends those same restrictions to other withdrawn reasons. Oh! Okay, that I understand. I'll try and test this now. 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> Easy to read patch, makes "Withdrawn" behave more like other non-circ statuses. Marking as Passed QA. Pushed, please test Seems to work. Closing. |