From 98406923bb2d6a7632d5c8f10525c398efa594bf Mon Sep 17 00:00:00 2001
From: Jesse Weaver <jesse.weaver@liblime.com>
Date: Thu, 17 Jul 2008 13:39:22 -0500
Subject: [PATCH] fix for bug 2282: Biblio framework subfields...

This fixes the tabs plugin's view of the % subfield by adding another template var called urisubfieldcode that is "pct" when the subfieldcode is "%".

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
---
 admin/marc_subfields_structure.pl                      |  1 +
 .../en/modules/admin/marc_subfields_structure.tmpl     | 10 +++++-----
 2 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/admin/marc_subfields_structure.pl b/admin/marc_subfields_structure.pl
index 3f82b99d08..473e50a381 100755
--- a/admin/marc_subfields_structure.pl
+++ b/admin/marc_subfields_structure.pl
@@ -207,6 +207,7 @@ if ( $op eq 'add_form' ) {
           . $data->{'tagsubfield'}
           . "\" id=\"tagsubfield\" />";
         $row_data{subfieldcode} = $data->{'tagsubfield'} eq '@'?'_':$data->{'tagsubfield'};
+        $row_data{urisubfieldcode} = $row_data{subfieldcode} eq '%' ? 'pct' : $row_data{subfieldcode};
         $row_data{liblibrarian} = CGI::escapeHTML( $data->{'liblibrarian'} );
         $row_data{libopac}      = CGI::escapeHTML( $data->{'libopac'} );
         $row_data{seealso}      = CGI::escapeHTML( $data->{'seealso'} );
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/marc_subfields_structure.tmpl b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/marc_subfields_structure.tmpl
index ff4bfb9f31..5a0c060ee0 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/marc_subfields_structure.tmpl
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/marc_subfields_structure.tmpl
@@ -68,10 +68,10 @@ function displayMoreConstraint(numlayer){
         <!-- TMPL_LOOP NAME="loop" -->
             <!-- TMPL_IF NAME="new_subfield" -->
                 <!-- TMPL_IF EXPR="tagfield >=10" --><!-- display new only for > 10 fields -->
-                <li><a href="/cgi-bin/koha/admin/marc_subfields_structure.pl#sub<!-- TMPL_VAR NAME="subfieldcode" -->field" title="<!-- TMPL_VAR NAME="liblibrarian" -->">New</a></li>
+                <li><a href="/cgi-bin/koha/admin/marc_subfields_structure.pl#sub<!-- TMPL_VAR NAME="urisubfieldcode" -->field" title="<!-- TMPL_VAR NAME="liblibrarian" -->">New</a></li>
                 <!-- /TMPL_IF -->
             <!-- TMPL_ELSE -->
-                <li><a href="/cgi-bin/koha/admin/marc_subfields_structure.pl#sub<!-- TMPL_VAR NAME="subfieldcode" -->field" title="<!-- TMPL_VAR NAME="liblibrarian" -->">
+                <li><a href="/cgi-bin/koha/admin/marc_subfields_structure.pl#sub<!-- TMPL_VAR NAME="urisubfieldcode" -->field" title="<!-- TMPL_VAR NAME="liblibrarian" -->">
                     <!-- TMPL_VAR NAME="subfieldcode" -->
                </a></li>
             <!-- /TMPL_IF --></li>
@@ -79,7 +79,7 @@ function displayMoreConstraint(numlayer){
     </ul>
     
     <!-- TMPL_LOOP NAME="loop" --> 
-        <div id="sub<!-- TMPL_VAR NAME="subfieldcode" -->field">
+        <div id="sub<!-- TMPL_VAR NAME="urisubfieldcode" -->field">
         
                <fieldset class="rows"><legend>Basic constraints</legend>
                 	<ol>
@@ -96,8 +96,8 @@ function displayMoreConstraint(numlayer){
                 	</ol>
             </fieldset>
 			
-            <a href="javascript:displayMoreConstraint('more<!-- TMPL_VAR NAME="subfieldcode" -->')">display more constraints</a>
-            <div id="more<!-- TMPL_VAR NAME="subfieldcode" -->" class="content_hidden" style="clear: both">
+            <a href="javascript:displayMoreConstraint('more<!-- TMPL_VAR NAME="urisubfieldcode" -->')">display more constraints</a>
+            <div id="more<!-- TMPL_VAR NAME="urisubfieldcode" -->" class="content_hidden" style="clear: both">
              <fieldset class="rows" id="advanced-constraints">
 				<legend>Advanced constraints:</legend>
                 <ol><li><label for="defaultvalue">Default value:</label>
-- 
2.20.1