Bug 7517

Summary: Patron category types not appearing as an <optgroup> when editing or creating a patron.
Product: Koha Reporter: Garry Collum <gcollum>
Component: TemplatesAssignee: Garry Collum <gcollum>
Status: CLOSED FIXED QA Contact: Ian Walls <koha.sekjal>
Severity: enhancement    
Priority: P5 - low CC: f.demians, julian.maurice, paul.poulain
Version: 3.8   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Proposed patch
Drop-down prior to patch
Drop-down after patch
Signed-off patch
Revised patch
Signed off revised patch

Description Garry Collum 2012-02-09 21:18:02 UTC
At one time the drop-down for borrower categories in the new patron would appear as an <optgroup>.  For example, you would have 'Child' and then your child category types, 'Adult' and then adult types, etc.  These are no longer displaying.
Comment 1 Garry Collum 2012-02-09 21:34:47 UTC Comment hidden (obsolete)
Comment 2 Garry Collum 2012-02-09 21:35:20 UTC
Created attachment 7545 [details]
Drop-down prior to patch
Comment 3 Garry Collum 2012-02-09 21:36:11 UTC
Created attachment 7546 [details]
Drop-down after patch
Comment 4 Frédéric Demians 2012-02-10 08:09:06 UTC Comment hidden (obsolete)
Comment 5 Ian Walls 2012-02-16 16:09:29 UTC
Template only change, most XHTML clean up and TT scope shift.  Marking a Passed QA.
Comment 6 Paul Poulain 2012-02-17 10:45:29 UTC
QA comment

This patch also does 2 un-announced changes:

-<input type="text" name="datedebarred" id="debarred" class="debarred" value="[% datedebarred %]" />
+<input type="text" name="datedebarred" id="datedebarred" class="debarred" value="[% datedebarred %]" />
and
-<textarea id="debarredcomment" name="debarredcomment" cols="55" rows="3" ">[% debarredcomment %]</textarea> 
+<textarea id="debarredcomment" name="debarredcomment" cols="55" rows="3">[% debarredcomment %]</textarea> 

While the 2nd one fixes a HTML mistake (trailing single " at the end of <textarea> I don't understand what the 1st is made for, and I think it should not be here.

and if it should, a few line below, we should also have 
-<input type="text" name="datedebarred" id="debarred" class="debarred" value="[% datedebarred %]" onclick="this.value=''" />
+<input type="text" name="datedebarred" id="datedebarred" class="debarred" value="[% datedebarred %]" onclick="this.value=''" />
Comment 7 Garry Collum 2012-02-25 20:23:32 UTC Comment hidden (obsolete)
Comment 8 Julian Maurice 2012-03-02 15:49:32 UTC
Created attachment 7990 [details] [review]
Signed off revised patch

The new patch still fix the initial problem and fix what has been pointed by Paul.
Signed-off.
Comment 9 Ian Walls 2012-03-05 18:48:57 UTC
Template only change, easy to read, marking as Passed QA.