Bug 27339

Summary: Display OPAC notes and OPAC messages on check-in
Product: Koha Reporter: Didier Gautheron <didier.gautheron>
Component: CirculationAssignee: Didier Gautheron <didier.gautheron>
Status: Failed QA --- QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: gmcharlt, kyle.m.hall, lucas, stina.hallin
Version: MainKeywords: release-notes-needed
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 27339: Display opac notes and opac messages on check-in
Bug 27339: Display opac notes and opac messages on check-in

Description Didier Gautheron 2021-01-05 08:55:38 UTC
Currently, Circulation note (borrowers.borrowernotes) and item notes (items.itemnotes and items.itemnotes_nonpublic) are displayed during check-in.
    
This patch adds Opac note (borrowers.opacnote) and Opac messages (messages table).
    
Test plan:
- Add an Opac note and one or several Opac messages to a patron.
- Checkout and check-in an item for this patron.
- Check that Opac note and Opac message(s) are not displayed on check-in.
- Apply the patch.
- Checkout and check-in an item for this patron.
- Check that Opac note and Opac message(s) are displayed on check-in.
Comment 1 Didier Gautheron 2021-01-05 10:15:47 UTC
Created attachment 114856 [details] [review]
Bug 27339: Display opac notes and opac messages on check-in

.
Adds Opac note (borrowers.opacnote) and Opac messages (messages table) during check-in.

Test plan:
1- Add an Opac note and one or several Opac messages to a patron.
2- Checkout and check-in an item for this patron.
3- Check that Opac note and Opac message(s) are not displayed on check-in.
4- Apply the patch.
5- Checkout and check-in an item for this patron.
6- Check that Opac note and Opac message(s) are displayed on check-in.
Comment 2 Lucas Gass 2021-01-22 21:40:59 UTC
Hi Didier,

I'm just curious why we display the OPAC message but not the staff internal - message here?
Comment 3 PTFS Europe Sandboxes 2021-02-01 10:11:44 UTC
Created attachment 116144 [details] [review]
Bug 27339: Display opac notes and opac messages on check-in

 .
Adds Opac note (borrowers.opacnote) and Opac messages (messages table) during check-in.

Test plan:
1- Add an Opac note and one or several Opac messages to a patron.
2- Checkout and check-in an item for this patron.
3- Check that Opac note and Opac message(s) are not displayed on check-in.
4- Apply the patch.
5- Checkout and check-in an item for this patron.
6- Check that Opac note and Opac message(s) are displayed on check-in.

Signed-off-by: Stina Hallin <stina.hallin@ub.lu.se>
Comment 4 Katrin Fischer 2021-02-07 11:52:53 UTC
Hi Didier, I can see that you were active on the patch, but the given author is Author: Matthias Meusburger <matthias.meusburger@biblibre.com>. Is this correct?
Comment 5 Katrin Fischer 2021-02-07 11:58:45 UTC
I like that you have given each note an individual class, that would make it possible to hide things for libraries as these notes could get a bit much for some depending on their use.

Now one note is still missing: The staff message. I think these might often be used to give some kind of history about the patron so could really turn into too much information here. I assume libraries are more likely to remove OPAC messages, that are no longer needed. So I am ok with that.
Comment 6 Katrin Fischer 2021-02-07 12:02:40 UTC
I have been looking at this a bit longer... and I'd suggest to create a new column for the OPAC notes in the table and add it to the column configuration.

With the current solution we cannot see the context of the notes, I think this would make things clearer. What do you think?
Comment 7 Stina Hallin 2021-02-08 08:26:35 UTC
I think your suggestion is good. You might end up with too many notes in one column, and some of them might not be important in the checkin workflow. Best if the coulmn only is visible if there are any notes to display.