gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Bug 41116: (follow-up) Some more fixes for consistency 

- There was a mix of "patronphone" and "patron-phone" and I fixed it to
  standardize on "patron-phone" (and patron-email) which I find easier
  to read.
- I made similar changes to the branchtransfers template
- Corrected the TT filter on mailto links from "html" to "uri"

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Laura_Escamilla <laura.escamilla@bywatersolutions.com>
Signed-off-by: Pedro Amorim <pedro.amorim@openfifth.co.uk>
This commit is contained in:
Owen Leonard 2025-11-21 07:42:22 -05:00 committed by Pedro Amorim
parent 1b2dd6ab54
commit 56928506ae
No known key found for this signature in database
GPG Key ID: 06B75BA7F3894D55
2 changed files with 15 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
koha-tmpl/intranet-tmpl/prog/en/modules/circ/branchtransfers.tt
View File

@ -98,11 +98,11 @@
</li>
[% INCLUDE display_holdpatron_address %]
[% IF ( patron.phone ) %]
<li> [% patron.phone | html %]</li>
<li class="patron-phone"> [% patron.phone | html %]</li>
[% END %]
[% IF ( patron.email ) %]
<li><a id="boremail" href="mailto:[% patron.email | html %]">[% patron.email | html %]</a></li>
<li class="patron-email"><a href="mailto:[% patron.email | uri %]">[% patron.email | html %]</a></li>
[% END %]
[% IF ( patron.is_debarred ) %]

26
koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt
View File

@ -84,15 +84,15 @@
[% INCLUDE display_holdpatron_address %]
[% IF ( patron.phone ) %]
<li class="patronphone">[% patron.phone | html %]</li>
<li class="patron-phone">[% patron.phone | html %]</li>
[% END %]
[% IF ( patron.email ) %]
<li class="patronemail">
<li class="patron-email">
[% IF ( transfertodo ) %]
[% patron.email | html %]
[% ELSE %]
<a href="mailto:[% patron.email | html %]">[% patron.email | html %]</a>
<a href="mailto:[% patron.email | uri %]">[% patron.email | html %]</a>
[% END %]
</li>
[% END %]
@ -704,10 +704,10 @@
</li>
[% INCLUDE display_holdpatron_address %]
[% IF ( patron.phone ) %]
<li class="patronphone"> [% patron.phone | html %]</li>
<li class="patron-phone"> [% patron.phone | html %]</li>
[% END %]
[% IF ( patron.email ) %]
<li class="patronemail"><a href="mailto:[% patron.email | html %]">[% patron.email | html %]</a></li>
<li class="patron-email"><a href="mailto:[% patron.email | uri %]">[% patron.email | html %]</a></li>
[% END %]
[% IF ( patron.smsalertnumber ) %]
@ -923,11 +923,11 @@
[% END %]
[% IF ( patron.email ) %]
<li class="patronemail">
<li class="patron-email">
[% IF ( transfertodo ) %]
[% patron.email | html %]
[% ELSE %]
<a href="mailto:[% patron.email | html %]">[% patron.email | html %]</a>
<a href="mailto:[% patron.email | uri %]">[% patron.email | uri %]</a>
[% END %]
</li>
[% END %]
@ -1037,15 +1037,15 @@
[% INCLUDE display_holdpatron_address patron=recall.patron %]
[% IF ( recall.patron.phone ) %]
<l >[% recall.patron.phone | html %]</li>
<li class="patron-phone">[% recall.patron.phone | html %]</li>
[% END %]
[% IF ( recall.patron.email ) %]
<li>
<li class="patron-email">
[% IF ( transfertodo ) %]
[% recall.patron.email | html %]
[% ELSE %]
<a id="boremail" href="mailto:[% recall.patron.email | html %]">[% recall.patron.email | html %]</a>
<a href="mailto:[% recall.patron.email | uri %]">[% recall.patron.email | html %]</a>
[% END %]
</li>
[% END %]
@ -1134,15 +1134,15 @@
[% INCLUDE display_holdpatron_address patron=recall.patron %]
[% IF ( recall.patron.phone ) %]
<li>[% recall.patron.phone | html %]</li>
<li class="patron-phone">[% recall.patron.phone | html %]</li>
[% END %]
[% IF ( recall.patron.email ) %]
<li>
<li class="patron-email">
[% IF ( transfertodo ) %]
[% recall.patron.email | html %]
[% ELSE %]
<a id="boremail" href="mailto:[% recall.patron.email | html %]">[% recall.patron.email | html %]</a>
<a href="mailto:[% recall.patron.email | uri %]">[% recall.patron.email | html %]</a>
[% END %]
</li>
[% END %]