| Summary: | Holds to pull should honor syspref AllowHoldsOnDamagedItems | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Dobrica Pavlinusic <dpavlin> |
| Component: | Hold requests | Assignee: | Dobrica Pavlinusic <dpavlin> |
| Status: | CLOSED FIXED | QA Contact: | Martin Renvoize (ashimema) <martin.renvoize> |
| Severity: | normal | ||
| Priority: | P5 - low | CC: | caroline.cyr-la-rose, fridolin.somers, gmcharlt, martin.renvoize, mglavica, nick |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| Change sponsored?: | --- | Patch complexity: | Trivial patch |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: | ||
| Circulation function: | |||
| Attachments: |
Bug 21320: Holds to pull should honor syspref AllowHoldsOnDamagedItems
Bug 21320: Holds to pull should honor syspref AllowHoldsOnDamagedItems Bug 21320: Holds to pull should honor syspref AllowHoldsOnDamagedItems |
||
|
Description
Dobrica Pavlinusic
2018-09-06 11:30:19 UTC
Created attachment 78471 [details] [review] Bug 21320: Holds to pull should honor syspref AllowHoldsOnDamagedItems Test plan: 1. Set syspref AllowHoldsOnDamagedItems to Allow 2. Create a hold on record with item which is damaged 3. Verify that it doesn't appear in report 4. Apply patch and verify that it's now visible Sorry bug fix has a bug, please ignore it for now :-) Created attachment 78472 [details] [review] Bug 21320: Holds to pull should honor syspref AllowHoldsOnDamagedItems Test plan: 1. Set syspref AllowHoldsOnDamagedItems to Allow 2. Create a hold on record with item which is damaged 3. Verify that it doesn't appear in report 4. Apply patch and verify that it's now visible Patch works as advertized :) Created attachment 80255 [details] [review] Bug 21320: Holds to pull should honor syspref AllowHoldsOnDamagedItems Test plan: 1. Set syspref AllowHoldsOnDamagedItems to Allow 2. Create a hold on record with item which is damaged 3. Verify that it doesn't appear in report 4. Apply patch and verify that it's now visible Signed-off-by: Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Trivial patch.. works as advertised and causes no regressions. I took the liberty of adding your SO line for you Caroline Cyr La Rose as it appeared to be missing. Passed QA Awesome work all! Pushed to master for 18.11 Pushed to 18.05.x for 18.05.05 Pushed to 17.11.x for 17.11.12 |