To recreate: - create a course in course reserves, add an item to it. - confirm your item shows Available for its status on opac-course-details.pl - give your item a notforloan status - reload opac-course-details.pl, confirm its still "available" - give your item a restricted user status - reload opac-course-details.pl, confirm its still "available" - give your item a lost, damaged, or withdrawn status - reload opac-course-details.pl, these statuses *are* reflected Filing a separate bug for the staff client.
Created attachment 123232 [details] [review] Bug 28762: Handle different item statuses in course details To test: 1. create a course in course reserves, add an item to it. 2. confirm your item shows Available for its status on course-details.pl 3. edit your item to be withdrawn, lost, damaged, notforloan, and restricted use 4. reload course-details.pl, confirm it still shows available 5. apply patch 6. repeat step 3 with each of the statuses and make sure it correctly shows on course-details.pl
Created attachment 123235 [details] [review] Bug 28761: better handle notforloan and restricted status in item.status.inc To test: 1. create a course in course reserves, add an item to it. 2. confirm your item shows Available for its status on opac-course-details.pl 3. give your item a notforloan status 4. reload opac-course-details.pl, confirm its still "available" 5. give your item a restricted user status 6. reload opac-course-details.pl, confirm its still "available" 7. give your item a lost, damaged, or withdrawn status 8. reload opac-course-details.pl, these statuses *are* reflected 9. apply patch 10. check notforloan and restricted, they should be reflect the correct status now
Created attachment 123246 [details] [review] Bug 28761: better handle notforloan and restricted status in item.status.inc To test: 1. create a course in course reserves, add an item to it. 2. confirm your item shows Available for its status on opac-course-details.pl 3. give your item a notforloan status 4. reload opac-course-details.pl, confirm its still "available" 5. give your item a restricted user status 6. reload opac-course-details.pl, confirm its still "available" 7. give your item a lost, damaged, or withdrawn status 8. reload opac-course-details.pl, these statuses *are* reflected 9. apply patch 10. check notforloan and restricted, they should be reflect the correct status now Signed-off-by: Owen Leonard <oleonard@myacpl.org>
This patch also affects the details page on the opac, and the cart Before this patch a restricted item shows as: Available (Restricted Access) After it is: Restricted Access
Created attachment 123351 [details] [review] Bug 28761: better handle notforloan and restricted status in item.status.inc To test: 1. create a course in course reserves, add an item to it. 2. confirm your item shows Available for its status on opac-course-details.pl 3. give your item a notforloan status 4. reload opac-course-details.pl, confirm its still "available" 5. give your item a restricted user status 6. reload opac-course-details.pl, confirm its still "available" 7. give your item a lost, damaged, or withdrawn status 8. reload opac-course-details.pl, these statuses *are* reflected 9. apply patch 10. check notforloan and restricted, they should be reflect the correct status now 11. restricted status should NOT cause the item to become unavailable 11. The item-status.inc also effects opac-detail and opac-cart 12. look at those and make sure everything looks correct and statuses are properly reflected
(In reply to Nick Clemens from comment #4) > This patch also affects the details page on the opac, and the cart > > Before this patch a restricted item shows as: > Available (Restricted Access) > > After it is: > Restricted Access Thanks Nick, I erroneously set the itemavailable flag to 0 for restricted items. They should be considered available. Reset the status of this one to NSO.
I have signed off on this : Kelly McElligott kelly@bywatersolutions.com
I noticed this the other day.. should we also be falling back to item-type level not for loan values when the local not for loan is not set or set to allow loans.. which value should take precedence if both are set? Otherwise, this looks pretty reasonable to me.
(In reply to Martin Renvoize from comment #8) > I noticed this the other day.. should we also be falling back to item-type > level not for loan values when the local not for loan is not set or set to > allow loans.. which value should take precedence if both are set? > > Otherwise, this looks pretty reasonable to me. Not sure I understand the question correctly, do you mean item vs. record leven not for loan? Then I think if both are set, we should use the description from item level. To me it makes sense that the more specific should win.
There is a problem here with item type 'not for loan' set: Without the patch: - Mark an item type as not for loan in administration - Make sure one item of that item type is on course reserve - Check the detail page, it shows as: not for loan - Check course reserves: it shows as: available With the patch: - It shows as Available on both detail and course reserves page The correct behaviour would be to show "not for loan" on both, so this introduces a regression. Lucas - when you follow-up, can you please add Kelly's sign-off line? So we don't forget about it.
Is this still on your list Lucas?