Bug 13261

Summary: Better check in message for patrons with indefinite restriction
Product: Koha Reporter: Marc Véron <veron>
Component: CirculationAssignee: Marc Véron <veron>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: didier.queneutte, f.demians, gmcharlt, jonathan.druart, katrin.fischer, kyle.m.hall, olli-antti.kivilahti, tomascohen
Version: Main   
Hardware: All   
OS: All   
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13532
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13025
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 13261 - Better check in message for patrons with indefinite restricition
Bug 13261 - Better check in message for patrons with indefinite restricition
[Follow-up] Bug 13261 - Better wording for check in message
Bug 13261 - Better check in message for patrons with indefinite restricition
[PASSED QA] Bug 13261 - Better check in message for patrons with indefinite restricition
Bug 13261: Remove warning if no parameter given

Description Marc Véron 2014-11-15 15:39:54 UTC
After checking in an item for patron with an indefinite restriction the checkin message reads:
Reminder: Patron was earlier restricted until 9999-12-31

It should be changed to:
Reminder: Patron has a restriction (no expiry date)
(Same message like after check out with restriction override)
Comment 1 Marc Véron 2014-11-15 16:18:29 UTC Comment hidden (obsolete)
Comment 2 Frédéric Demians 2014-11-15 18:42:47 UTC Comment hidden (obsolete)
Comment 3 Owen Leonard 2014-11-15 21:19:31 UTC
I would prefer something like "Patron has an indefinite restriction" Or "Patron is restricted indefinitely." I'm glad to see this worked on--this was a complaint at my library after our recent upgrade.
Comment 4 Tomás Cohen Arazi 2014-11-15 22:29:21 UTC
(In reply to Owen Leonard from comment #3)
> I would prefer something like "Patron has an indefinite restriction" Or
> "Patron is restricted indefinitely." I'm glad to see this worked on--this
> was a complaint at my library after our recent upgrade.

Maybe 'permanently'.
Comment 5 Owen Leonard 2014-11-16 00:17:56 UTC
> Maybe 'permanently'.

'Permanently' implies that the restriction can't be removed. 'Indefinitely' implies that the ending date is simply not known.
Comment 6 Marc Véron 2014-11-16 07:27:33 UTC Comment hidden (obsolete)
Comment 7 Marc Véron 2014-11-16 07:30:02 UTC
Reset to "Needs Signoff" for second patch.
Comment 8 Didier Queneutte 2014-11-17 15:05:40 UTC
Tested on Sandbox 1 (Unimarc)

When checking in after adding a restriction without expiry date, ends with a gateway time out (error 504). Check out and check in without restriction work, check in with restriction and expiry date works too.
Comment 9 Frédéric Demians 2014-11-17 16:43:41 UTC Comment hidden (obsolete)
Comment 10 Katrin Fischer 2014-11-17 21:41:13 UTC
Marc, could you add a test to cover the change to AddReturn?
Comment 11 Katrin Fischer 2014-11-17 21:41:56 UTC
Created attachment 33630 [details] [review]
[PASSED QA] Bug 13261 - Better check in message for patrons with indefinite restricition

This patch adds a better check in message for patrons with indefinite restriction.

To test:
Check out an item to a patron.
Add a manual restriction without expiry date to that patron.
Check in the item.

Without patch, the checkin message reads:
Reminder: Patron was earlier restricted until 9999-12-31

Apply patch and repeat steps above.
The message should now read:
Reminder: Patron has a restriction (no expiry date)

NOTE: Changed wording at two places following Owen's  suggestion.  New: "Patron
has an indefinite restriction"

Signed-off-by: Frederic Demians <f.demians@tamil.fr>

Thanks Marc for catching this case. I was thinking like you that the wording
sounded strange while playing with bug 13242. Merge the original patch and the
followup, containing a better wording, thanks to Owen comment.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described, no problems found.
Passes tests and QA script.
Comment 12 Tomás Cohen Arazi 2014-11-20 00:49:26 UTC
Patch pushed to master.

Thanks Marc!
Comment 13 Jonathan Druart 2014-11-26 08:45:10 UTC Comment hidden (obsolete)
Comment 14 Jonathan Druart 2014-11-26 08:46:00 UTC
Comment on attachment 33944 [details] [review]
Bug 13261: Remove warning if no parameter given

wrong bug report.
Comment 15 Olli-Antti Kivilahti 2015-01-05 11:45:54 UTC
Thanks for the patch Marc! Just stumbled upon this issue having barred some of our patrons :)