Attachment 26215 Details for Bug 8760 – Bug 8760 - Phone number Field tags are reversed in patron record

[patch] Bug 8760 - Phone number Field tags are reversed in patron record

Bug-8760---Phone-number-Field-tags-are-reversed-in.patch (text/plain), 2.63 KB, created by Katrin Fischer on 2014-03-12 17:57:39 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2014-03-12 17:57:39 UTC

Size: 2.63 KB

patch

obsolete

>From d889e52871d1cb8b41f3cee15c01dcefe4a012b8 Mon Sep 17 00:00:00 2001
>From: Jesse Maseto <jesse@bywatersolutions.com>
>Date: Wed, 12 Mar 2014 10:41:54 -0400
>Subject: [PATCH] Bug 8760 - Phone number Field tags are reversed in patron
> record
>
>Adjusted fields to display correctly. Now the fields on the
>memberentry page match the memberdetails page.
>
>To test:
>Select a patron account that you can edit their phone number information
>
>Fill in all 3 phone number fields:
>
>	 Primary phone:  #1
>         Secondary phone: #2
>         Other phone: #3
>
>Then view the member details page. This will display #1,#3,#2.
>
>Apply patch and should display correctly.
>
>http://bugs.koha-community.org/show_bug.cgi?id=8760
>Signed-off-by: Owen Leonard <oleonard@myacpl.org>
>---
> .../prog/en/modules/members/memberentrygen.tt        | 20 ++++++++++----------
> 1 file changed, 10 insertions(+), 10 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt
>index 720193f..66fbf91 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt
>@@ -585,27 +585,27 @@
> 
>     </li>
>         [% END %]
>-        [% UNLESS nophonepro %]
>+        [% UNLESS nomobile %]
>     <li>
>-      [% IF ( mandatoryphonepro ) %]
>-      <label for="phonepro" class="required">
>+      [% IF ( mandatorymobile ) %]
>+      <label for="mobile" class="required">
>       [% ELSE %]
>-      <label for="phonepro">
>+      <label for="mobile">
>       [% END %]
>       Secondary phone: </label>
>-    <input type="text" id="phonepro" name="phonepro" value="[% phonepro %]" class="[% focusAction %]" />
>-	  [% IF ( mandatoryphonepro ) %]<span class="required">Required</span>[% END %]
>+    <input type="text" id="mobile" name="mobile" value="[% mobile %]" class="[% focusAction %]" />
>+      [% IF ( mandatorymobile ) %]<span class="required">Required</span>[% END %]
>     </li>
>         [% END %]
>         [% UNLESS nomobile %]
>     <li>
>-      [% IF ( mandatorymobile ) %]
>-      <label for="mobile" class="required">
>+      [% IF ( mandatoryphonepro ) %]
>+      <label for="phonepro" class="required">
>       [% ELSE %]
>-      <label for="mobile">
>+      <label for="phonepro">
>       [% END %]
>       Other phone: </label>
>-        <input type="text" id="mobile" name="mobile" value="[% mobile %]" class="[% focusAction %]" />
>+        <input type="text" id="phonepro" name="phonepro" value="[% phonepro %]" class="[% focusAction %]" />
> 	  [% IF ( mandatorymobile ) %]<span class="required">Required</span>[% END %]
>     </li>
>         [% END %]
>-- 
>1.8.3.2

Actions: View | Diff | Splinter Review

Attachments on bug 8760: 12139 | 12140 | 24722 | 24723 | 24724 | 26184 | 26190 | 26215