Bug 13334

Summary: Add a warning when a course reserve is checked in
Product: Koha Reporter: simith.doliveira
Component: CirculationAssignee: Bugs List <koha-bugs>
Status: Failed QA --- QA Contact: Testopia <testopia>
Severity: normal    
Priority: P4 CC: catherine.dupuy-lenen, eric.begin, gmcharlt, kyle.m.hall, marjorie.barry-vila, martin.renvoize, philippe.blouin, tmisilo, veron
Version: MainKeywords: Academy
Hardware: All   
OS: All   
Change sponsored?: Sponsored Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function: holds
Attachments: Add a warning during the return of a document that was reserved to a course
This patch adds the following warning during the return of a document that was reserved to a course:

Description simith.doliveira 2014-11-25 14:14:58 UTC

    
Comment 1 simith.doliveira 2014-11-25 14:24:22 UTC
This patch adds the following warning during the return of a document that was reserved to a course:

"Please return 'title' to the 'course name' class reserve."  

Testing:

1) Define a document as a course reserve

see manual.koha-community.org/3.14/en/coursereserves.html

2) Checkout and checkin this document.
Validade the warning

Sponsored-by: CCSR ( http://www.ccsr.qc.ca )
Comment 2 simith.doliveira 2014-11-25 14:26:21 UTC Comment hidden (obsolete)
Comment 3 Owen Leonard 2014-11-26 18:58:15 UTC
+    <div class="error dialog" style="text-align:center;margin-right:0px;margin:auto;">

There's no reason to define a new message style in this patch. <div class="dialog message"> works perfectly well here.

The message text is not quite correct either. It refers to "class reserve" when the feature is called "course reserves."

I would suggest something like "[ title ] is part of the [ course ] course reserve." That text is informational without being specifically directional. I think it's better not to use the term "return" which is expected to be seen in this context with regard to transfers between branches.
Comment 4 simith.doliveira 2015-02-26 20:39:16 UTC
Created attachment 36212 [details] [review]
This patch adds the following warning during the return of a document that was reserved to a course:

"Please return 'title' to the 'course name' class reserve."

Testing:

1) Define a document as a course reserve

see manual.koha-community.org/3.14/en/coursereserves.html

2) Checkout and checkin this document.
Validade the warning
Comment 5 simith.doliveira 2015-02-26 20:41:34 UTC
The warning was changed to "[ title ] is part of the [ course ] course reserve."
Comment 6 Marc VĂ©ron 2015-03-01 19:00:02 UTC
It works as expected on returns.pl

However when I go to circulation.pl and check in using the button "Renow or check in selected items", I would expect a similar message as well (or some information in the "Checkouts" table e.g. in the column "Title")
Comment 7 catherine.dupuy-lenen 2024-09-09 10:10:31 UTC
returns.pl was working as expected, but it was 10 years ago... The last comment asking for a similar message when checking in several items at once seems to have  been discouraging.
But still, the initial work is interesting, this is really a lack. Could at least this process go until the end or would it have to start all over again to match with the latest versions ?
Comment 8 Katrin Fischer 2024-09-09 10:34:36 UTC
Hm, we do have the situation that when returning from the patron account directly, a lot of the alerts are not displaying, that would show on the normal returns page. I feel like this patch would not necessarily need to solve this.

I am not sure if Simith is still around to fix this, but maybe someone would be willing to adopt it.
Comment 9 Blou 2024-09-09 11:53:39 UTC
Seems to still be of value, but our sponsor for this patch is not using it anymore.   I do not see us (including simith) going back to it at this point.
Comment 10 Katrin Fischer 2024-09-09 11:55:58 UTC
(In reply to Blou from comment #9)
> Seems to still be of value, but our sponsor for this patch is not using it
> anymore.   I do not see us (including simith) going back to it at this point.

Thanks for the quick reply, Blou. I have updated the Assignee to reflect this.

I agree that a note on return would still be a nice thing.