Bug 14614: Multiple URLs (856) in cart/list email are broken
authorMarc Véron <veron@veron.ch>
Thu, 24 Mar 2016 10:59:27 +0000 (11:59 +0100)
committerBrendan A Gallagher <brendan@bywatersolutions.com>
Thu, 24 Mar 2016 15:32:48 +0000 (15:32 +0000)
To reproduce:
- Add multiple subfields 856 u to a bilbio
- Add this biblio to a cart and send it as mail
  (from Staff client and from OPAC)
Result: Links in mail body are broken

To test
- Apply patch
- Send carts again (from Staff client and from OPAC)
- Result: In mail body, links display separated with blank-pipe-blank
  like http://bla.com | http://blabla.com | http://blablabla.com
- Change one of the 856 u to not to be a link, e.g. äöü
- Send carts again
- Verify that in mail body äöü correctly display as text.

(Amended to make it work for OPAC as well, MV)

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Brendan A Gallagher <brendan@bywatersolutions.com>
koha-tmpl/intranet-tmpl/prog/en/modules/basket/sendbasket.tt
koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-sendbasket.tt

index c99b54e..adb47cd 100644 (file)
@@ -125,7 +125,7 @@ Your cart
             [% END %]
             [% IF ( BIBLIO_RESULT.url ) %]
             <span>
-                URL: [% BIBLIO_RESULT.url |url %]
+                URL: [% BIBLIO_RESULT.url %]
             </span>
             [% END %]
             </p>
index e3c88ab..7c7978f 100644 (file)
@@ -125,7 +125,7 @@ Your cart
             [% END %]
             [% IF ( BIBLIO_RESULT.url ) %]
             <span>
-                URL: [% BIBLIO_RESULT.url |url %]
+                URL: [% BIBLIO_RESULT.url %]
             </span>
             [% END %]
             </p>