The next hold request showing on the detail.pl page next to items is useless and should be removed or fixed. The information for requests on an item is misleading. It shows the next item level hold rather than showing the next hold it will fulfill. I understand how this might be useful in some obscure circumstances, but for the most part it leads staff to believe that this is the next hold it will fill, and that is not the case. Case in point...we have a new item, only one in the collection. 33 next available holds, and 1 item level hold at the very end of the list, going to a different library. With all the other holds ahead of it, and the fact that we have local holds priority enabled, this looks very wrong, and leads staff to confusion and panic. It might be helpful to some libraries to see that there is a specific hold on a particular item, but around here, most libraries will view this as misleading. There should be a setting to change the behavior, or turn it off.
Does it concern 17.05 only? I do not understand what you are referring to on master, can you provide screenshots?
Created attachment 76145 [details] Misleading Hold Info Screenshot of misleading hold. Shows the only item-level hold in the 35 current holds. This is not who the item will be going to when checked in. Would prefer to not show this information if it is meaningless to us.
I realize that it may be impossible to display next available holds, but if there are next available holds and local hold priorities, I fail to see how showing the next item level hold helps.
Created attachment 76216 [details] Misleading Hold Info - In Transit I noticed today that there is also misleading hold info on the 'Holds' table on moremembers.pl and circulation.pl. It will list "In Transit from X branch since Y" even if that patron is not the next patron. This also only happens with item level holds. I'm happy to fill out a different bug, but I thought it might be related based on how the information is getting pulled. Lisette Scheer
I would be happy to have a look at this one if someone can provide a step-by-step plan to recreate the problem.
(In reply to Jonathan Druart from comment #5) > I would be happy to have a look at this one if someone can provide a > step-by-step plan to recreate the problem. Place several next available holds on a single item record. Then place an item level hold on the item. Look at the detail.pl page. The item will show the item level hold. Many staff interpret this to mean this is where the hold is going next. It is preferred if it just said "There is an item level hold on this item", without any specific details.
Created attachment 95476 [details] [review] Bug 20948: Simplify existing code No changes expected here.
Created attachment 95477 [details] [review] Bug 20948: Only display item-level hold info if first hold On the item list of the catalogue detail page we display next item-level hold info even if it is not the next hold. That leads to confusion as it will not necessarily be the accurate info. This patch makes the following changes: - Display the item-level hold only if it is the next hold (priority == 1) - Display "There is an item level hold on this item (priority=X)" if there is at least 1 item-level hold placed on this item Test plan: - Place several next available holds on a single item record - Place an item level hold on the item - Go the the biblio detail page => Without this patch, the item will show the item level hold => With the patch you see "There is an item level hold on this item" - Check the item in, confirm the hold => No changes, the item will show the waiting hold info - Cancel all the holds except the item-level one => No changes, the item will how the item-level one
Created attachment 95636 [details] [review] Bug 20948: Simplify existing code No changes expected here. Signed-off-by: Lisette Scheer <lisetteslatah@gmail.com>
Created attachment 95637 [details] [review] Bug 20948: Only display item-level hold info if first hold On the item list of the catalogue detail page we display next item-level hold info even if it is not the next hold. That leads to confusion as it will not necessarily be the accurate info. This patch makes the following changes: - Display the item-level hold only if it is the next hold (priority == 1) - Display "There is an item level hold on this item (priority=X)" if there is at least 1 item-level hold placed on this item Test plan: - Place several next available holds on a single item record - Place an item level hold on the item - Go the the biblio detail page => Without this patch, the item will show the item level hold => With the patch you see "There is an item level hold on this item" - Check the item in, confirm the hold => No changes, the item will show the waiting hold info - Cancel all the holds except the item-level one => No changes, the item will how the item-level one Signed-off-by: Lisette Scheer <lisetteslatah@gmail.com>
FAIL koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt FAIL filters missing_filter at line 365 ( There is an item level hold on this item (priority = [% hold.priority %]).) Work on qa tools ?
There is an item level hold on this item (priority = 2). Personally, I think we'd better not show this at all. Please explain why we need this.
(In reply to Marcel de Rooy from comment #12) > There is an item level hold on this item (priority = 2). > > Personally, I think we'd better not show this at all. Please explain why we > need this. I used Christopher's suggestion: (In reply to Christopher Brannon from comment #6) > It is preferred if it just said "There is an item level hold on this item", > without any specific details.
(In reply to Jonathan Druart from comment #13) > (In reply to Marcel de Rooy from comment #12) > > There is an item level hold on this item (priority = 2). > > > > Personally, I think we'd better not show this at all. Please explain why we > > need this. > > I used Christopher's suggestion: > > (In reply to Christopher Brannon from comment #6) > > It is preferred if it just said "There is an item level hold on this item", > > without any specific details. Well, Christopher is invited to reply too.
(In reply to Marcel de Rooy from comment #12) > There is an item level hold on this item (priority = 2). > > Personally, I think we'd better not show this at all. Please explain why we > need this. I would have suggested removing it altogether, but there are those that want to know if there are item level holds on items. Particularly if items are lost or missing or damaged. That way they can easily see them and do something about the holds.
(In reply to Christopher Brannon from comment #15) > I would have suggested removing it altogether, but there are those that want > to know if there are item level holds on items. Particularly if items are > lost or missing or damaged. That way they can easily see them and do > something about the holds. This is why I would find the message helpful as well. Are there any performance issues from having it show?
Is this similar/related to Bug 19288?
(In reply to Katrin Fischer from comment #17) > Is this similar/related to Bug 19288? Yes, at first glance they refer to the same problem.
(In reply to Lisette Scheer from comment #16) > (In reply to Christopher Brannon from comment #15) > > I would have suggested removing it altogether, but there are those that want > > to know if there are item level holds on items. Particularly if items are > > lost or missing or damaged. That way they can easily see them and do > > something about the holds. > > This is why I would find the message helpful as well. Are there any > performance issues from having it show? Absolutely not.
Created attachment 95870 [details] [review] Bug 20948: Simplify existing code No changes expected here. Signed-off-by: Lisette Scheer <lisetteslatah@gmail.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Created attachment 95871 [details] [review] Bug 20948: Only display item-level hold info if first hold On the item list of the catalogue detail page we display next item-level hold info even if it is not the next hold. That leads to confusion as it will not necessarily be the accurate info. This patch makes the following changes: - Display the item-level hold only if it is the next hold (priority == 1) - Display "There is an item level hold on this item (priority=X)" if there is at least 1 item-level hold placed on this item Test plan: - Place several next available holds on a single item record - Place an item level hold on the item - Go the the biblio detail page => Without this patch, the item will show the item level hold => With the patch you see "There is an item level hold on this item" - Check the item in, confirm the hold => No changes, the item will show the waiting hold info - Cancel all the holds except the item-level one => No changes, the item will how the item-level one Signed-off-by: Lisette Scheer <lisetteslatah@gmail.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
*** Bug 19288 has been marked as a duplicate of this bug. ***
Nice work everyone! Pushed to master for 20.05
Created attachment 96137 [details] [review] Bug 20948: Add missing html filter
Thanks for catching that Jonathan.. you beat me to it :)
Pushed to 19.11.x branch for 19.11.02 Thanks, joy
backported to 19.05.x for 19.05.07
Backported to 18.11.x for 18.11.13