Bug 5956 - Highlight holds on check-in
Summary: Highlight holds on check-in
Status: In Discussion
Alias: None
Product: Koha
Classification: Unclassified
Component: Circulation (show other bugs)
Version: master
Hardware: All All
: P5 - low enhancement (vote)
Target Milestone: ---
Assignee: Kyle M Hall
QA Contact: Bugs List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-03-25 22:07 UTC by Jared Camins-Esakov
Modified: 2016-08-19 08:47 UTC (History)
6 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Bot Control: ---
When did the bot last check this:
Who signed the patch off:
Text to go in the release notes:
Version(s) released in:


Attachments
5956 Highlight holds on chech-in (2.38 KB, patch)
2014-06-16 07:15 UTC, Viktor Sarge
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jared Camins-Esakov 2011-03-25 22:07:08 UTC
When a book is checked in that is on hold, it would be helpful if the title showed up in the list of checked in books in red.  When you are checking in a large number of books it is easy to go by the hold message if you are just checking the books in.  Highlighting the hold as you highlight the overdue would be a big help.
Comment 1 Katrin Fischer 2013-03-24 18:09:48 UTC
There is no highlighting for reserved books on the return screen in current versions of Koha.
Comment 2 Viktor Sarge 2014-06-16 07:15:03 UTC
Created attachment 28858 [details] [review]
5956 Highlight holds on chech-in

This patch adds a flag in returns.pl if the returned books have reservervations.
The flag is picked up by returns.tt and displays a warning using the "Note" field and CSS-class already in place.

Note that this differs from the atcual suggestion from Jared, but using the note field instead of making the title red felt that it kept it more consistent.

Apologies if the commit message hints at the fact that I'm still learning Git.
Comment 3 Viktor Sarge 2014-06-16 07:17:57 UTC
Test plan: 

* Return both reserved and not reserved books and verify that the reserved books are not highlighted
* Install the patch
* Return both reserved and not reserved books again and verify that reserved books are now highlighted in the "Notes" field.
Comment 4 Viktor Sarge 2014-06-19 08:59:27 UTC
Please note - I see that I omitted "Bug" as the first word of the commit. That will make it impossible to sign of on the bug from the sandboxes. I'm not yet clear on how to resubmit a patch correctly but will be careful to write correctly formated commits in the future.
Comment 5 Nick Clemens 2014-07-03 17:51:47 UTC
I tested on a sandbox and found that the highlighting (note in red, not highlighting of row) only occurs on confirmation of a hold.

I think the intention is that if someone checks in a reserved book, doesn't see the message, then checks in a book with no reserves, they would still see the highlighting even without having confirmed the hold.
Comment 6 Viktor Sarge 2014-07-03 19:40:59 UTC
Thanks for catching this Nick - that makes revisiting the code a must. I'll look into it but as I'm on vacation right now it might be a while, so if feel free if anyone sees where the patch is lacking. 

(In reply to Nick Clemens from comment #5)
> I tested on a sandbox and found that the highlighting (note in red, not
> highlighting of row) only occurs on confirmation of a hold.
> 
> I think the intention is that if someone checks in a reserved book, doesn't
> see the message, then checks in a book with no reserves, they would still
> see the highlighting even without having confirmed the hold.