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: Lucas Gass <lucas@bywatersolutions.com>
This commit is contained in:
Owen Leonard 2025-11-21 07:42:22 -05:00 committed by Lucas Gass
parent bf451c7fe9
commit 5ba8ab39ab
No known key found for this signature in database
GPG Key ID: 60B447398B0D36CA
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 %]