Attachment 159114 Details for Bug 35356 – Bug 35356: Show SMS labels only when data exists

[patch] Bug 35356: Show SMS labels only when data exists

Bug-35356-Show-SMS-labels-only-when-data-exists.patch (text/plain), 2.47 KB, created by Lucas Gass (lukeg) on 2023-11-17 21:41:58 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Lucas Gass (lukeg)

Created: 2023-11-17 21:41:58 UTC

Size: 2.47 KB

patch

obsolete

>From c635934750f503b38119a8e693b4c92baa8aa395 Mon Sep 17 00:00:00 2001
>From: Lucas Gass <lucas@bywatersolutions.com>
>Date: Fri, 17 Nov 2023 21:39:02 +0000
>Subject: [PATCH] Bug 35356: Show SMS labels only when data exists
>
>To test:
>APPLY PATCH
>1. Set SMSSendDriver = Email
>2. Admin > SMS cellular providers, have at least 1.
>3. Go to the members/moremember.pl
>4. If the patron has no SMSnumber or SMS provider then niether of the labels should appear.
>5. Add both a SMSnumber and SMS provider, make sure the data appears correctly on members/moremember.pl.
>---
> .../prog/en/modules/members/moremember.tt | 12 +++++++-----
> 1 file changed, 7 insertions(+), 5 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 3a1278e3cfe..095c6e35dc7 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt
>@@ -354,11 +354,13 @@
> [% IF Koha.Preference('SMSSendDriver') %]
> <div class="rows">
> <ol>
>- <li>
>- SMS number:
>- <a href="sms:[% patron.smsalertnumber | uri %]">[% patron.smsalertnumber | html %]</a>
>- </li>
>- [% IF Koha.Preference( 'SMSSendDriver') == 'Email' %]
>+ [% IF patron.smsalertnumber %]
>+ <li>
>+ SMS number:
>+ <a href="sms:[% patron.smsalertnumber | uri %]">[% patron.smsalertnumber | html %]</a>
>+ </li>
>+ [% END %]
>+ [% IF Koha.Preference( 'SMSSendDriver') == 'Email' && patron.sms_provider.name %]
> <li>
> SMS provider:
> [% patron.sms_provider.name | html %]
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 35356: 159114 | 159117 | 159214