Bugzilla – Attachment 102454 Details for
Bug 25069
AddressFormat="fr" behavior is broken
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 25069: Fix AddressFormat="fr" behavior
Bug-25069-Fix-AddressFormatfr-behavior.patch (text/plain), 5.62 KB, created by
Owen Leonard
on 2020-04-06 14:46:34 UTC
(
hide
)
Description:
Bug 25069: Fix AddressFormat="fr" behavior
Filename:
MIME Type:
Creator:
Owen Leonard
Created:
2020-04-06 14:46:34 UTC
Size:
5.62 KB
patch
obsolete
>From 13ba600c60ec6a71dc3eb162848b836f745e2f69 Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@bugs.koha-community.org> >Date: Mon, 6 Apr 2020 12:44:13 +0200 >Subject: [PATCH] Bug 25069: Fix AddressFormat="fr" behavior > >Something went wrong during a rebase of bug 13618 > commit dcd1f5d48c758aee17b6c6f069c6146b42efe117 > Bug 13618: Add html filters to all the variables > >Several changes related to AddressFormat are wrong: > >- [% IF Koha.Preference( 'AddressFormat' ) %] >- [% INCLUDE "member-main-address-style-${ Koha.Preference( 'AddressFormat' ) }.inc" %] >- [% ELSE %] >- [% INCLUDE 'member-main-address-style-us.inc' %] >- [% END %] >+ [% SWITCH Koha.Preference( 'AddressFormat' ) %] >+ [% CASE 'de' %] >+ [% INCLUDE 'member-main-address-style-de.inc' %] >+ [% CASE # us %] >+ [% INCLUDE 'member-main-address-style-us.inc' %] >+ [% END %] > >Test plan: >Create a patron with all the address fields filled >Play with the 3 option values of AddressFormat, and confirm that the address is displayed correctly >on the patron's view, and in the patron module (top left) > >Signed-off-by: Owen Leonard <oleonard@myacpl.org> >--- > .../intranet-tmpl/prog/en/includes/circ-menu.inc | 9 ++++----- > .../prog/en/modules/members/memberentrygen.tt | 20 ++++++++++---------- > 2 files changed, 14 insertions(+), 15 deletions(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/circ-menu.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/circ-menu.inc >index 7b3494268b..d665e81104 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/includes/circ-menu.inc >+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/circ-menu.inc >@@ -29,11 +29,10 @@ > > <ul class="patronbriefinfo"> > [% IF !(Koha.Preference('HidePersonalPatronDetailOnCirculation')) %] >- [% SWITCH Koha.Preference( 'AddressFormat' ) %] >- [% CASE 'de' %] >- [% INCLUDE 'member-display-address-style-de.inc' %] >- [% CASE # us %] >- [% INCLUDE 'member-display-address-style-us.inc' %] >+ [% IF Koha.Preference( 'AddressFormat' ) %] >+ [% INCLUDE "member-display-address-style-${ Koha.Preference( 'AddressFormat' ) }.inc" %] >+ [% ELSE %] >+ [% INCLUDE 'member-display-address-style-us.inc' %] > [% END %] > > [% IF ( patron.phone || patron.mobile || patron.phonepro ) %]<li class="patronphone"> >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 e4ba9ca31e..eec68264d9 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt >@@ -561,6 +561,8 @@ legend:hover { > [% UNLESS noaddress && noaddress2 && nocity && nostate && nozipcode && nocountry %] > [% IF Koha.Preference( 'AddressFormat' ) %] > [% INCLUDE "member-main-address-style-${ Koha.Preference( 'AddressFormat' ) }.inc" %] >+ [% ELSE %] >+ [% INCLUDE 'member-main-address-style-us.inc' %] > [% END %] > [% END # /UNLESS nostreet && nocity etc group%] > >@@ -726,22 +728,20 @@ legend:hover { > > [% IF ( step_6 ) %] > [% UNLESS noB_address && noB_address2 && noB_city && noB_zipcode && noB_state && noB_country &&nocontactnote && noB_phone && noB_email %] >- [% SWITCH Koha.Preference( 'AddressFormat' ) %] >- [% CASE 'de' %] >- [% INCLUDE 'member-alt-address-style-de.inc' %] >- [% CASE # us %] >+ [% IF Koha.Preference( 'AddressFormat' ) %] >+ [% INCLUDE "member-alt-address-style-${ Koha.Preference( 'AddressFormat' ) }.inc" %] >+ [% ELSE %] > [% INCLUDE 'member-alt-address-style-us.inc' %] >- [% END %] >+ [% END %] > [% END # UNLESS noB_address && noB_city && noB_state && noB_phone && noB_email %] > [% END # /IF ( step_6 ) %] > > [% IF ( step_2 ) %] > [% UNLESS noaltcontactsurname && noaltcontactfirstname && noaltcontactaddress1 && noaltcontactaddress2 && noaltcontactaddress3 && noaltcontactstate && noaltcontactzipcode && noaltcontactcountry && noaltcontactphone %] >- [% SWITCH Koha.Preference( 'AddressFormat' ) %] >- [% CASE 'de' %] >- [% INCLUDE 'member-alt-contact-style-de.inc' %] >- [% CASE # us %] >- [% INCLUDE 'member-alt-contact-style-us.inc' %] >+ [% IF Koha.Preference( 'AddressFormat' ) %] >+ [% INCLUDE "member-alt-contact-style-${ Koha.Preference( 'AddressFormat' ) }.inc" %] >+ [% ELSE %] >+ [% INCLUDE 'member-alt-contact-style-us.inc' %] > [% END %] > [% END # UNLESS noaltcontactsurname && noaltcontactfirstname etc %] > [% END # /IF ( step_2 ) %] >-- >2.11.0
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 25069
:
102436
|
102437
|
102454
|
102695
|
104349