From 8f6ef0bab2783650b964949825a142d1784fd7a1 Mon Sep 17 00:00:00 2001
From: Zoe Bennett <zoebennett1308@gmail.com>
Date: Tue, 16 Jan 2018 03:06:59 +0000
Subject: [PATCH] Bug 19818: Added id's to the li tags for labels

To test:
1)Go to the screen: /cgi-bin/koha/members/moremember.pl
2)On the right of the screen, right click one of the labels below 'Library use' etc, and Inspect Element
3) View <li> tags which should now each have a unique <id> tag for each
---
 .../prog/en/modules/members/moremember.tt          | 52 +++++++++++-----------
 1 file changed, 26 insertions(+), 26 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt
index 180d941..a3f8a77 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt
@@ -392,12 +392,12 @@ function validate1(date) {
  <h3>Library use</h3>
 <div class="rows"> 
 <ol>
-    <li><span class="label">Card number: </span>[% cardnumber %]</li>
-	<li><span class="label">Borrowernumber: </span> [% borrowernumber %]</li>
-    <li><span class="label">Category: </span>[% categoryname %] ([% categorycode %])</li>
-    <li><span class="label">Registration date: </span>[% dateenrolled | $KohaDates %]</li>
+    <li id ="cardnumber"><span class="label">Card number: </span>[% cardnumber %]</li>
+	<li id="borrowernumber"><span class="label">Borrowernumber: </span> [% borrowernumber %]</li>
+    <li id="category"><span class="label">Category: </span>[% categoryname %] ([% categorycode %])</li>
+    <li id="registrationdate"><span class="label">Registration date: </span>[% dateenrolled | $KohaDates %]</li>
     
-    <li><span class="label">Expiration date: </span>
+    <li id="expirationdate"><span class="label">Expiration date: </span>
     [% IF ( was_renewed ) %]
             <strong class="reregistrinfo">[% dateexpiry | $KohaDates %]</strong>
     [% ELSE %]
@@ -405,15 +405,15 @@ function validate1(date) {
     [% END %]
     </li>
     
-    <li><span class="label">Library: </span>[% branchname %]</li>
+    <li id="library"><span class="label">Library: </span>[% branchname %]</li>
 
-    [% IF ( OPACPrivacy ) %]<li><span class="label">Privacy Pref:</span>
+    [% IF ( OPACPrivacy ) %]<li id="privacypref"><span class="label">Privacy Pref:</span>
          [% IF ( privacy0 ) %]Forever[% END %]
          [% IF ( privacy1 ) %]Default[% END %]
          [% IF ( privacy2 ) %]Never[% END %]
     </li>[% END %]
 
-    <li>
+    <li id="showcheckouts">
         <span class="label">Show checkouts to guarantor</span>
         [% IF privacy_guarantor_checkouts %]
             Yes
@@ -422,23 +422,23 @@ function validate1(date) {
         [% END %]
     </li>
 
-    [% IF ( sort1 ) %]<li><span class="label">Sort field 1:</span>[% AuthorisedValues.GetByCode('Bsort1', sort1) |html %]</li>[% END %]
-    [% IF ( sort2 ) %]<li><span class="label">Sort field 2:</span>[% AuthorisedValues.GetByCode('Bsort2', sort2) |html %]</li>[% END %]
-    <li><span class="label">Username: </span>[% userid |html  %]</li>
-    <li><span class="label">Password: </span>
+    [% IF ( sort1 ) %]<li id="sortfield1"><span class="label">Sort field 1:</span>[% AuthorisedValues.GetByCode('Bsort1', sort1) |html %]</li>[% END %]
+    [% IF ( sort2 ) %]<li id="sortfield2"><span class="label">Sort field 2:</span>[% AuthorisedValues.GetByCode('Bsort2', sort2) |html %]</li>[% END %]
+    <li id="username"><span class="label">Username: </span>[% userid |html  %]</li>
+    <li id="password"><span class="label">Password: </span>
     [% IF ( password ) %]
         *******
     [% ELSE %]
         <span class="problem"><a href="/cgi-bin/koha/members/member-password.pl?member=[% borrowernumber %]">Undefined</a></span>
     [% END %] 
     </li>
-    [% IF ( borrowernotes ) %]<li><span class="label">Circulation note: </span>[% borrowernotes |html %]</li>[% END %]
-    [% IF ( opacnote ) %]<li><span class="label">OPAC note:</span>[% opacnote |html %]</li>[% END %]
+    [% IF ( borrowernotes ) %]<li id="circulationnote"><span class="label">Circulation note: </span>[% borrowernotes |html %]</li>[% END %]
+    [% IF ( opacnote ) %]<li id="opacnote"><span class="label">OPAC note:</span>[% opacnote |html %]</li>[% END %]
     [% IF Koha.Preference( 'NorwegianPatronDBEnable' ) == 1 %]
         [% IF ( sync == 1 ) %]
-            <li><span class="label">Activate sync: </span>Yes</li>
-            [% IF ( syncstatus ) %]<li><span class="label">Sync status: </span>[% syncstatus %]</li>[% END %]
-            [% IF ( lastsync ) %]<li><span class="label">Last sync: </span>[% lastsync | $KohaDates %]</li>[% END %]
+            <li id="activatesync"><span class="label">Activate sync: </span>Yes</li>
+            [% IF ( syncstatus ) %]<li id="syncstatus"><span class="label">Sync status: </span>[% syncstatus %]</li>[% END %]
+            [% IF ( lastsync ) %]<li id="lastsync"><span class="label">Last sync: </span>[% lastsync | $KohaDates %]</li>[% END %]
         [% ELSE %]
             <li><span class="label">Activate sync: </span>No</li>
         [% END %]
@@ -485,15 +485,15 @@ function validate1(date) {
 
  <div id="patron-alternative-contact" style="padding-top: 1em;">
  <h3>Alternative contact</h3>
-   <div class="rows"> <ol><li><span class="label">Surname: </span>[% altcontactsurname | html %]</li>
-    <li><span class="label">First name: </span>[% altcontactfirstname | html %]</li>
-    <li><span class="label">Address: </span>[% altcontactaddress1 | html %]</li>
-    <li><span class="label">Address 2: </span>[% altcontactaddress2 | html %]</li>
-	<li><span class="label">City: </span>[% altcontactaddress3 | html %]</li>
-    [% IF ( altcontactstate ) %]<li><span class="label">State: </span>[% altcontactstate | html %]</li>[% END %]
-    <li><span class="label">ZIP/Postal code: </span>[% altcontactzipcode | html %]</li>
-	[% IF ( altcontactcountry ) %]<li><span class="label">Country: </span>[% altcontactcountry | html %]</li>[% END %]
-    [% IF ( altcontactphone ) %]<li><span class="label">Phone: </span><a href="tel:[% altcontactphone | url %]">[% altcontactphone | html %]</a></li>[% END %]
+   <div class="rows"> <ol><li id="surname"><span class="label">Surname: </span>[% altcontactsurname | html %]</li>
+    <li id="firstname"><span class="label">First name: </span>[% altcontactfirstname | html %]</li>
+    <li id="address"><span class="label">Address: </span>[% altcontactaddress1 | html %]</li>
+    <li id="address2"><span class="label">Address 2: </span>[% altcontactaddress2 | html %]</li>
+	<li id="city"><span class="label">City: </span>[% altcontactaddress3 | html %]</li>
+    [% IF ( altcontactstate ) %]<li id="state"><span class="label">State: </span>[% altcontactstate | html %]</li>[% END %]
+    <li id="zip/postalcode"><span class="label">ZIP/Postal code: </span>[% altcontactzipcode | html %]</li>
+	[% IF ( altcontactcountry ) %]<li id="country"><span class="label">Country: </span>[% altcontactcountry | html %]</li>[% END %]
+    [% IF ( altcontactphone ) %]<li id="phone"><span class="label">Phone: </span><a href="tel:[% altcontactphone | url %]">[% altcontactphone | html %]</a></li>[% END %]
     </ol></div>
 </div>
 <div class="action"><a href="memberentry.pl?op=modify&amp;borrowernumber=[% borrowernumber %]&amp;step=2">Edit</a></div>
-- 
2.1.4