Bug 8014

Summary: On the patron entry form hide "restricted until" field if "Restricted: No" is checked
Product: Koha Reporter: Owen Leonard <oleonard>
Component: PatronsAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Ian Walls <koha.sekjal>
Severity: minor    
Priority: P5 - low CC: chris, gmcharlt, paul.poulain
Version: 3.8   
Hardware: All   
OS: All   
URL: /cgi-bin/koha/members/memberentry.pl
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug 8014 - On the patron entry form hide "restricted until" field if "Restricted: No" is checked
Bug 8014 - On the patron entry form hide "restricted until" field if "Restricted: No" is checked

Description Owen Leonard 2012-04-27 12:52:04 UTC
On the patron entry form I think it would be clearer if the "restricted until" form field was hidden if "Restricted: No" is checked. This remove any ambiguity about whether the field can or should be filled in independently of the "Restricted" options.
Comment 1 Owen Leonard 2012-04-27 13:06:39 UTC Comment hidden (obsolete)
Comment 2 Jared Camins-Esakov 2012-05-13 11:45:05 UTC
Created attachment 9534 [details] [review]
Bug 8014 - On the patron entry form hide "restricted until" field if "Restricted: No" is checked

This patch adds JavaScript to automatically hide the "Restricted
unil" form field on the patron entry form if "Restricted: No" is
selected. If "Restricted: Yes" is chosen the "Until:" field
is shown and focus is given to it.

The patch also adds a note, "(optional)" after the "Until:" field
to make it clearer that a date is not required.

To test, open a patron record and try moving the selection
between "Restricted: No" and "Restricted: Yes." Confirm that
the "Until:" field is shown and hidden as expected. Confirm
that the restricted flag is correctly set or unset upon save,
including setting the flag with or without a date.

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Comment 3 Paul Poulain 2012-05-28 09:44:41 UTC
QA Comment : tiny patch, js and tt only, nothing specific to say

passed QA
Comment 4 Paul Poulain 2012-05-28 09:45:24 UTC
pushed, usefull for 3.8 I think, so versionning 3.8
Comment 5 Chris Cormack 2012-05-28 20:42:54 UTC
I agree useful for for 3.8.x, will be in 3.8.2