[Koha-patches] [PATCH] bug 2596: remove URL escaping added by previous patch

Tue, 16 Sep 2008 19:58:34 -0700 

If URL in an href comes from a MARC record, it should not
be URL-escaped - otherwise, link is interpreted as relative
to MARCdetail.pl.
---
 .../prog/en/modules/catalogue/MARCdetail.tmpl      |   20 ++++++++++----------
 1 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/MARCdetail.tmpl 
b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/MARCdetail.tmpl
index 267aa12..5669b7f 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/MARCdetail.tmpl
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/MARCdetail.tmpl
@@ -89,7 +89,7 @@ function Changefwk(FwkList) {
                     <span class="subfield-label">&nbsp;
                     <!-- TMPL_UNLESS name="hide_marc" --><!-- TMPL_VAR 
NAME="marc_subfield" --><!-- /TMPL_UNLESS -->
                     <span title="<!-- TMPL_VAR name="long_desc" -->"><!-- 
TMPL_VAR NAME="short_desc" --></span></span>
-                    <!-- TMPL_IF NAME="is_url" --><a href="<!-- TMPL_VAR 
NAME="marc_value" escape="url" -->"><!-- TMPL_VAR NAME="marc_value" 
escape="html" --></a><!-- TMPL_ELSE --><!-- TMPL_VAR NAME="marc_value" 
ESCAPE="html" --><!-- /TMPL_IF -->
+                    <!-- TMPL_IF NAME="is_url" --><a href="<!-- TMPL_VAR 
NAME="marc_value" -->"><!-- TMPL_VAR NAME="marc_value" escape="html" 
--></a><!-- TMPL_ELSE --><!-- TMPL_VAR NAME="marc_value" ESCAPE="html" --><!-- 
/TMPL_IF -->
                     <!-- TMPL_IF name="link" -->
                         <a 
href="/cgi-bin/koha/catalogue/search.pl?op=do_search&amp;idx=<!-- TMPL_VAR 
NAME="link" -->,phr&amp;type=intranet&amp;q=<!-- TMPL_VAR NAME="marc_value" 
ESCAPE="URL" -->">
                             <img border="0" src="<!-- TMPL_VAR 
NAME="interface" -->/<!-- TMPL_VAR NAME="theme" -->/images/filefind.png" 
height="15" title="Search on <!-- TMPL_VAR NAME="marc_value" ESCAPE="html" -->" 
alt="Search on <!-- TMPL_VAR NAME="marc_value" ESCAPE="html" -->" />
@@ -119,7 +119,7 @@ function Changefwk(FwkList) {
                     <span class="subfield-label">&nbsp;
                     <!-- TMPL_UNLESS name="hide_marc" --><b><!-- TMPL_VAR 
NAME="marc_subfield" --></b><!-- /TMPL_UNLESS -->
                     <span title="<!-- TMPL_VAR name="long_desc" -->"><!-- 
TMPL_VAR NAME="short_desc" --></span></span>
-                    <!-- TMPL_IF NAME="is_url" --><a href="<!-- TMPL_VAR 
NAME="marc_value" escape="url" -->"><!-- TMPL_VAR NAME="marc_value" 
escape="html" --></a><!-- TMPL_ELSE --><!-- TMPL_VAR NAME="marc_value" 
ESCAPE="html" --><!-- /TMPL_IF -->
+                    <!-- TMPL_IF NAME="is_url" --><a href="<!-- TMPL_VAR 
NAME="marc_value" -->"><!-- TMPL_VAR NAME="marc_value" escape="html" 
--></a><!-- TMPL_ELSE --><!-- TMPL_VAR NAME="marc_value" ESCAPE="html" --><!-- 
/TMPL_IF -->
                     <!-- TMPL_IF name="link" -->
                         <a 
href="/cgi-bin/koha/catalogue/search.pl?op=do_search&amp;idx=<!-- TMPL_VAR 
NAME="link" -->,phr&amp;type=intranet&amp;q=<!-- TMPL_VAR NAME="marc_value" 
ESCAPE="URL" -->">
                             <img border="0" src="<!-- TMPL_VAR 
NAME="interface" -->/<!-- TMPL_VAR NAME="theme" -->/images/filefind.png" 
height="15" title="Search on <!-- TMPL_VAR NAME="marc_value" ESCAPE="html" -->" 
alt="Search on <!-- TMPL_VAR NAME="marc_value" ESCAPE="html" -->" />
@@ -149,7 +149,7 @@ function Changefwk(FwkList) {
                     <span class="subfield-label">&nbsp;
                     <!-- TMPL_UNLESS name="hide_marc" --><b><!-- TMPL_VAR 
NAME="marc_subfield" --></b><!-- /TMPL_UNLESS -->
                     <span title="<!-- TMPL_VAR name="long_desc" -->"><!-- 
TMPL_VAR NAME="short_desc" --></span></span>
-                    <!-- TMPL_IF NAME="is_url" --><a href="<!-- TMPL_VAR 
NAME="marc_value" escape="url" -->"><!-- TMPL_VAR NAME="marc_value" 
escape="html" --></a><!-- TMPL_ELSE --><!-- TMPL_VAR NAME="marc_value" 
ESCAPE="html" --><!-- /TMPL_IF -->
+                    <!-- TMPL_IF NAME="is_url" --><a href="<!-- TMPL_VAR 
NAME="marc_value" -->"><!-- TMPL_VAR NAME="marc_value" escape="html" 
--></a><!-- TMPL_ELSE --><!-- TMPL_VAR NAME="marc_value" ESCAPE="html" --><!-- 
/TMPL_IF -->
                     <!-- TMPL_IF name="link" -->
                         <a 
href="/cgi-bin/koha/catalogue/search.pl?op=do_search&amp;idx=<!-- TMPL_VAR 
NAME="link" -->,phr&amp;type=intranet&amp;q=<!-- TMPL_VAR NAME="marc_value" 
ESCAPE="URL" -->">
                             <img border="0" src="<!-- TMPL_VAR 
NAME="interface" -->/<!-- TMPL_VAR NAME="theme" -->/images/filefind.png" 
height="15" title="Search on <!-- TMPL_VAR NAME="marc_value" ESCAPE="html" -->" 
alt="Search on <!-- TMPL_VAR NAME="marc_value" ESCAPE="html" -->" />
@@ -179,7 +179,7 @@ function Changefwk(FwkList) {
                     <span class="subfield-label">&nbsp;
                     <!-- TMPL_UNLESS name="hide_marc" --><b><!-- TMPL_VAR 
NAME="marc_subfield" --></b><!-- /TMPL_UNLESS -->
                     <span title="<!-- TMPL_VAR name="long_desc" -->"><!-- 
TMPL_VAR NAME="short_desc" --></span></span>
-                    <!-- TMPL_IF NAME="is_url" --><a href="<!-- TMPL_VAR 
NAME="marc_value" escape="url" -->"><!-- TMPL_VAR NAME="marc_value" 
escape="html" --></a><!-- TMPL_ELSE --><!-- TMPL_VAR NAME="marc_value" 
ESCAPE="html" --><!-- /TMPL_IF -->
+                    <!-- TMPL_IF NAME="is_url" --><a href="<!-- TMPL_VAR 
NAME="marc_value" -->"><!-- TMPL_VAR NAME="marc_value" escape="html" 
--></a><!-- TMPL_ELSE --><!-- TMPL_VAR NAME="marc_value" ESCAPE="html" --><!-- 
/TMPL_IF -->
                     <!-- TMPL_IF name="link" -->
                         <a 
href="/cgi-bin/koha/catalogue/search.pl?op=do_search&amp;idx=<!-- TMPL_VAR 
NAME="link" -->,phr&amp;type=intranet&amp;q=<!-- TMPL_VAR NAME="marc_value" 
ESCAPE="URL" -->">
                             <img border="0" src="<!-- TMPL_VAR 
NAME="interface" -->/<!-- TMPL_VAR NAME="theme" -->/images/filefind.png" 
height="15" title="Search on <!-- TMPL_VAR NAME="marc_value" ESCAPE="html" -->" 
alt="Search on <!-- TMPL_VAR NAME="marc_value" ESCAPE="html" -->" />
@@ -209,7 +209,7 @@ function Changefwk(FwkList) {
                     <span class="subfield-label">&nbsp;
                     <!-- TMPL_UNLESS name="hide_marc" --><b><!-- TMPL_VAR 
NAME="marc_subfield" --></b><!-- /TMPL_UNLESS -->
                     <span title="<!-- TMPL_VAR name="long_desc" -->"><!-- 
TMPL_VAR NAME="short_desc" --></span></span>
-                    <!-- TMPL_IF NAME="is_url" --><a href="<!-- TMPL_VAR 
NAME="marc_value" escape="url" -->"><!-- TMPL_VAR NAME="marc_value" 
escape="html" --></a><!-- TMPL_ELSE --><!-- TMPL_VAR NAME="marc_value" 
ESCAPE="html" --><!-- /TMPL_IF -->
+                    <!-- TMPL_IF NAME="is_url" --><a href="<!-- TMPL_VAR 
NAME="marc_value" -->"><!-- TMPL_VAR NAME="marc_value" escape="html" 
--></a><!-- TMPL_ELSE --><!-- TMPL_VAR NAME="marc_value" ESCAPE="html" --><!-- 
/TMPL_IF -->
                     <!-- TMPL_IF name="link" -->
                         <a 
href="/cgi-bin/koha/catalogue/search.pl?op=do_search&amp;idx=<!-- TMPL_VAR 
NAME="link" -->,phr&amp;type=intranet&amp;q=<!-- TMPL_VAR NAME="marc_value" 
ESCAPE="URL" -->">
                             <img border="0" src="<!-- TMPL_VAR 
NAME="interface" -->/<!-- TMPL_VAR NAME="theme" -->/images/filefind.png" 
height="15" title="Search on <!-- TMPL_VAR NAME="marc_value" ESCAPE="html" 
-->"> alt="Search on <!-- TMPL_VAR NAME="marc_value" ESCAPE="html" -->" />
@@ -239,7 +239,7 @@ function Changefwk(FwkList) {
                     <span class="subfield-label">&nbsp;
                     <!-- TMPL_UNLESS name="hide_marc" --><b><!-- TMPL_VAR 
NAME="marc_subfield" --></b><!-- /TMPL_UNLESS -->
                     <span title="<!-- TMPL_VAR name="long_desc" -->"><!-- 
TMPL_VAR NAME="short_desc" --></span></span>
-                    <!-- TMPL_IF NAME="is_url" --><a href="<!-- TMPL_VAR 
NAME="marc_value" escape="url" -->"><!-- TMPL_VAR NAME="marc_value" 
escape="html" --></a><!-- TMPL_ELSE --><!-- TMPL_VAR NAME="marc_value" 
ESCAPE="html" --><!-- /TMPL_IF -->
+                    <!-- TMPL_IF NAME="is_url" --><a href="<!-- TMPL_VAR 
NAME="marc_value" -->"><!-- TMPL_VAR NAME="marc_value" escape="html" 
--></a><!-- TMPL_ELSE --><!-- TMPL_VAR NAME="marc_value" ESCAPE="html" --><!-- 
/TMPL_IF -->
                     <!-- TMPL_IF name="link" -->
                         <a 
href="/cgi-bin/koha/catalogue/search.pl?op=do_search&amp;idx=<!-- TMPL_VAR 
NAME="link" -->,phr&amp;type=intranet&amp;q=<!-- TMPL_VAR NAME="marc_value" 
ESCAPE="URL" -->">
                             <img border="0" src="<!-- TMPL_VAR 
NAME="interface" -->/<!-- TMPL_VAR NAME="theme" -->/images/filefind.png" 
height="15" title="Search on <!-- TMPL_VAR NAME="marc_value" ESCAPE="html" -->" 
alt="Search on <!-- TMPL_VAR NAME="marc_value" ESCAPE="html" -->" />
@@ -269,7 +269,7 @@ function Changefwk(FwkList) {
                     <span class="subfield-label">&nbsp;
                     <!-- TMPL_UNLESS name="hide_marc" --><b><!-- TMPL_VAR 
NAME="marc_subfield" --></b><!-- /TMPL_UNLESS -->
                     <span title="<!-- TMPL_VAR name="long_desc" -->"><!-- 
TMPL_VAR NAME="short_desc" --></span></span>
-                    <!-- TMPL_IF NAME="is_url" --><a href="<!-- TMPL_VAR 
NAME="marc_value" escape="url" -->"><!-- TMPL_VAR NAME="marc_value" 
escape="html" --></a><!-- TMPL_ELSE --><!-- TMPL_VAR NAME="marc_value" 
ESCAPE="html" --><!-- /TMPL_IF -->
+                    <!-- TMPL_IF NAME="is_url" --><a href="<!-- TMPL_VAR 
NAME="marc_value" -->"><!-- TMPL_VAR NAME="marc_value" escape="html" 
--></a><!-- TMPL_ELSE --><!-- TMPL_VAR NAME="marc_value" ESCAPE="html" --><!-- 
/TMPL_IF -->
                     <!-- TMPL_IF name="link" -->
                         <a 
href="/cgi-bin/koha/catalogue/search.pl?op=do_search&amp;idx=<!-- TMPL_VAR 
NAME="link" -->,phr&amp;type=intranet&amp;q=<!-- TMPL_VAR NAME="marc_value" 
ESCAPE="URL" -->">
                             <img border="0" src="<!-- TMPL_VAR 
NAME="interface" -->/<!-- TMPL_VAR NAME="theme" -->/images/filefind.png" 
height="15" title="Search on <!-- TMPL_VAR NAME="marc_value" ESCAPE="html" -->" 
alt="Search on <!-- TMPL_VAR NAME="marc_value" ESCAPE="html" -->" />
@@ -299,7 +299,7 @@ function Changefwk(FwkList) {
                     <span class="subfield-label">&nbsp;
                     <!-- TMPL_UNLESS name="hide_marc" --><b><!-- TMPL_VAR 
NAME="marc_subfield" --></b><!-- /TMPL_UNLESS -->
                     <span title="<!-- TMPL_VAR name="long_desc" -->"><!-- 
TMPL_VAR NAME="short_desc" --></span></span>
-                    <!-- TMPL_IF NAME="is_url" --><a href="<!-- TMPL_VAR 
NAME="marc_value" escape="url" -->"><!-- TMPL_VAR NAME="marc_value" 
escape="html" --></a><!-- TMPL_ELSE --><!-- TMPL_VAR NAME="marc_value" 
ESCAPE="html" --><!-- /TMPL_IF -->
+                    <!-- TMPL_IF NAME="is_url" --><a href="<!-- TMPL_VAR 
NAME="marc_value" -->"><!-- TMPL_VAR NAME="marc_value" escape="html" 
--></a><!-- TMPL_ELSE --><!-- TMPL_VAR NAME="marc_value" ESCAPE="html" --><!-- 
/TMPL_IF -->
                     <!-- TMPL_IF name="link" -->
                         <a 
href="/cgi-bin/koha/catalogue/search.pl?op=do_search&amp;idx=<!-- TMPL_VAR 
NAME="link" -->,phr&amp;type=intranet&amp;q=<!-- TMPL_VAR NAME="marc_value" 
ESCAPE="URL" -->">
                             <img border="0" src="<!-- TMPL_VAR 
NAME="interface" -->/<!-- TMPL_VAR NAME="theme" -->/images/filefind.png" 
height="15" title="Search on <!-- TMPL_VAR NAME="marc_value" ESCAPE="html" -->" 
alt="Search on <!-- TMPL_VAR NAME="marc_value" ESCAPE="html" -->" />
@@ -329,7 +329,7 @@ function Changefwk(FwkList) {
                     <span class="subfield-label">&nbsp;
                     <!-- TMPL_UNLESS name="hide_marc" --><b><!-- TMPL_VAR 
NAME="marc_subfield" --></b><!-- /TMPL_UNLESS -->
                     <span title="<!-- TMPL_VAR name="long_desc" -->"><!-- 
TMPL_VAR NAME="short_desc" --></span></span>
-                    <!-- TMPL_IF NAME="is_url" --><a href="<!-- TMPL_VAR 
NAME="marc_value" escape="url" -->"><!-- TMPL_VAR NAME="marc_value" 
escape="html" --></a><!-- TMPL_ELSE --><!-- TMPL_VAR NAME="marc_value" 
ESCAPE="html" --><!-- /TMPL_IF -->
+                    <!-- TMPL_IF NAME="is_url" --><a href="<!-- TMPL_VAR 
NAME="marc_value" -->"><!-- TMPL_VAR NAME="marc_value" escape="html" 
--></a><!-- TMPL_ELSE --><!-- TMPL_VAR NAME="marc_value" ESCAPE="html" --><!-- 
/TMPL_IF -->
                     <!-- TMPL_IF name="link" -->
                         <a 
href="/cgi-bin/koha/catalogue/search.pl?op=do_search&amp;idx=<!-- TMPL_VAR 
NAME="link" -->,phr&amp;type=intranet&amp;q=<!-- TMPL_VAR NAME="marc_value" 
ESCAPE="URL" -->">
                             <img border="0" src="<!-- TMPL_VAR 
NAME="interface" -->/<!-- TMPL_VAR NAME="theme" -->/images/filefind.png" 
height="15" title="Search on <!-- TMPL_VAR NAME="marc_value" ESCAPE="html" -->" 
alt="Search on <!-- TMPL_VAR NAME="marc_value" ESCAPE="html" -->" />
@@ -359,7 +359,7 @@ function Changefwk(FwkList) {
                     <span class="subfield-label">&nbsp;
                     <!-- TMPL_UNLESS name="hide_marc" --><b><!-- TMPL_VAR 
NAME="marc_subfield" --></b><!-- /TMPL_UNLESS -->
                     <span title="<!-- TMPL_VAR name="long_desc" -->"><!-- 
TMPL_VAR NAME="short_desc" --></span></span>
-                    <!-- TMPL_IF NAME="is_url" --><a href="<!-- TMPL_VAR 
NAME="marc_value" escape="url" -->"><!-- TMPL_VAR NAME="marc_value" 
escape="html" --></a><!-- TMPL_ELSE --><!-- TMPL_VAR NAME="marc_value" 
ESCAPE="html" --><!-- /TMPL_IF -->
+                    <!-- TMPL_IF NAME="is_url" --><a href="<!-- TMPL_VAR 
NAME="marc_value" -->"><!-- TMPL_VAR NAME="marc_value" escape="html" 
--></a><!-- TMPL_ELSE --><!-- TMPL_VAR NAME="marc_value" ESCAPE="html" --><!-- 
/TMPL_IF -->
                     <!-- TMPL_IF name="link" -->
                         <a 
href="/cgi-bin/koha/catalogue/search.pl?op=do_search&amp;idx=<!-- TMPL_VAR 
NAME="link" -->,phr&amp;type=intranet&amp;q=<!-- TMPL_VAR NAME="marc_value" 
ESCAPE="URL" -->">
                             <img border="0" src="<!-- TMPL_VAR 
NAME="interface" -->/<!-- TMPL_VAR NAME="theme" -->/images/filefind.png" 
height="15" title="Search on <!-- TMPL_VAR NAME="marc_value" ESCAPE="html" -->" 
alt="Search on <!-- TMPL_VAR NAME="marc_value" ESCAPE="html" -->" />
-- 
1.5.5.GIT

_______________________________________________
Koha-patches mailing list
[email protected]
http://lists.koha.org/mailman/listinfo/koha-patches

Reply via email to