Attachment 7571 Details for Bug 7392 – Bug 7345: Follow up: Adding the new export option to staff interface

[patch] Bug 7345: Follow up: Adding the new export option to staff interface

Bug-7345-Follow-up-Adding-the-new-export-option-to.patch (text/plain), 22.14 KB, created by Jared Camins-Esakov on 2012-02-10 18:54:11 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jared Camins-Esakov

Created: 2012-02-10 18:54:11 UTC

Size: 22.14 KB

patch

obsolete

>From ac649b03b5f04e211b1843d5403695efb68073bf Mon Sep 17 00:00:00 2001
>From: Katrin Fischer <Katrin.Fischer.83@web.de>
>Date: Sun, 1 Jan 2012 20:32:47 +0100
>Subject: [PATCH] Bug 7345: Follow up: Adding the new export option to staff interface
>Content-Type: text/plain; charset="UTF-8"
>
>NOTE: Patch depends on changes made for bug 7345 for the "marcstd" option.
>
>It adds a new system preference 'OpacExportOptions':
>
>The new system preference allows to give a list of | separated
>values to define, which download options are offered in OPAC.
>Possible options are listed in the system preference editor
>and include the new export option 'marcstd' from bug 7345.
>
>The sequence of the download options in the system preference will
>be the sequence in the OPAC pull down.
>
>If left empty, the export option will be hidden from OPAC.
>
>This patch creates a new include file for the OPAC with all the
>code that is common to the three detail views (regular, MARC, and
>ISBD).
>
>As a side effect of creating this include file, some inconsistencies
>in OpacSearchForTitleIn were fixed. {BIBLIONUMBER} and {CONTROLNUMBER}
>were broken for ISBD and MARC view before and should now be replaced
>with the correct values in all tabs.
>
>Patch makes changes according to QA comments:
>- entering an unknown format code will generate no entry in the OPAC pull down
>- small change to the description of opaclayoutstylesheet system preference
>- Updating text only for description of opaccolorstylesheet and
>  opaclayoutstylesheet.
>
>Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>Resolved merge conflicts on sysprefs.sql and updatedatabase.pl.
>Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
>Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
>
>http://bugs.koha-community.org/show_bug.cgi?id=7392
>---
> installer/data/mysql/sysprefs.sql                  |    1 +
> installer/data/mysql/updatedatabase.pl             |    8 +++
> .../prog/en/modules/admin/preferences/opac.pref    |   13 +++-
> .../prog/en/includes/opac-detail-sidebar.inc       |   62 ++++++++++++++++++++
> .../opac-tmpl/prog/en/modules/opac-ISBDdetail.tt   |   56 +-----------------
> .../opac-tmpl/prog/en/modules/opac-MARCdetail.tt   |   55 +-----------------
> koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt |   55 +-----------------
> opac/opac-ISBDdetail.pl                            |    9 +++
> opac/opac-MARCdetail.pl                            |   22 +++++--
> opac/opac-detail.pl                                |    5 ++
> 10 files changed, 115 insertions(+), 171 deletions(-)
> create mode 100644 koha-tmpl/opac-tmpl/prog/en/includes/opac-detail-sidebar.inc
>
>diff --git a/installer/data/mysql/sysprefs.sql b/installer/data/mysql/sysprefs.sql
>index 5e3d79d..6cf4df4 100644
>--- a/installer/data/mysql/sysprefs.sql
>+++ b/installer/data/mysql/sysprefs.sql
>@@ -334,3 +334,4 @@ INSERT INTO `systempreferences` (variable,value,explanation,options,type) VALUES
> INSERT INTO `systempreferences` (variable,value,explanation,options,type) VALUES ('LocalCoverImages','0','Display local cover images on intranet details pages.','1','YesNo');
> INSERT INTO `systempreferences` (variable,value,explanation,options,type) VALUES ('AllowMultipleCovers','0','Allow multiple cover images to be attached to each bibliographic record.','1','YesNo');
> INSERT INTO systempreferences (variable,value,explanation,options,type) VALUES('BorrowerRenewalPeriodBase', 'now', 'Set whether the borrower renewal date should be counted from the dateexpiry or from the current date ','dateexpiry|now','Choice');
>+INSERT INTO systempreferences (variable,value,explanation,options,type) VALUES('OpacExportOptions','bibtex|dc|marcxml|marc8|utf8|marcstd|mods|ris','Define export options available on OPAC detail page.','','free');
>diff --git a/installer/data/mysql/updatedatabase.pl b/installer/data/mysql/updatedatabase.pl
>index 099dfb9..3901573 100755
>--- a/installer/data/mysql/updatedatabase.pl
>+++ b/installer/data/mysql/updatedatabase.pl
>@@ -4663,6 +4663,14 @@ ENDOFRENEWAL
>     print "Upgrade to $DBversion done (Added a system preference to allow renewal of Patron account either from todays date or from existing expiry date in the patrons account.)\n";
>     SetVersion($DBversion);
> }
>+
>+$DBversion = "3.06.03.XXX";
>+if (C4::Context->preference("Version") < TransformToNum($DBversion)) {
>+    $dbh->do("INSERT INTO systempreferences (variable,value,explanation,options,type) VALUES('OpacExportOptions','bibtex|dc|marcxml|marc8|utf8|marcstd|mods|ris','Define available export options on OPAC detail page.','','free');");
>+    print "Upgrade to $DBversion done (Bug 7345: Add system preference OpacExportOptions.)\n";
>+    SetVersion($DBversion);
>+}
>+
> =head1 FUNCTIONS
> 
> =head2 DropAllForeignKeys($table)
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/opac.pref b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/opac.pref
>index d6ae2b2..f47f6b6 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/opac.pref
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/opac.pref
>@@ -130,12 +130,12 @@ OPAC:
>             - Include the additional CSS stylesheet
>             - pref: opaccolorstylesheet
>               class: file
>-            - to override specified settings from the default stylesheet. Enter the filename (if the file is in the server's css directory) or a complete URL beginning with <code>http://</code> (if the file lives on a remote server). Leave blank to disable.
>+            - to override specified settings from the default stylesheet (leave blank to disable). Enter a filename or a complete URL beginning with <code>http://</code> (if the file lives on a remote server). Please note that if you enter a filename, the file should be in the css subdirectory for each active theme and language within the Koha templates directory.
>         -
>-            - Use the CSS stylesheet /css/
>+            - Use the CSS stylesheet
>             - pref: opaclayoutstylesheet
>               class: file
>-            - on all pages in the OPAC, instead of the default (leave blank to disable).
>+            - on all pages in the OPAC, instead of the default (leave blank to disable). Please enter filename only. The file should be in the css subdirectory for each active theme and language within the Koha templates directory.
>         -
>             - Use the remote CSS stylesheet
>             - pref: opacstylesheet
>@@ -208,6 +208,13 @@ OPAC:
>                   Results: "Results page only"
>                   Both: "Both Details and Results pages"
>             - 'Note: The corresponding OPACXSLT option must be turned on.'
>+        -
>+            - 'List export options that should be available from OPAC detail page separated by |:'
>+            - pref: OpacExportOptions
>+              class: multi
>+            - '<br />Note: Available options are: BIBTEX (<code>bibtex</code>), Dublin Core (<code>dc</code>),'
>+            - 'MARCXML (<code>marcxml</code>), MARC-8 encoded MARC (<code>marc8</code>), Unicode/UTF-8 encoded MARC (<code>utf8</code>),'
>+            - 'Unicode/UTF-8 encoded MARC without local use (9xx, x9x, xx9) fields and subfields (<code>marcstd</code>), MODS (<code>mods</code>), RIS (<code>ris</code>)'
>     Features:
>         -
>             - pref: opacuserlogin
>diff --git a/koha-tmpl/opac-tmpl/prog/en/includes/opac-detail-sidebar.inc b/koha-tmpl/opac-tmpl/prog/en/includes/opac-detail-sidebar.inc
>new file mode 100644
>index 0000000..a563542
>--- /dev/null
>+++ b/koha-tmpl/opac-tmpl/prog/en/includes/opac-detail-sidebar.inc
>@@ -0,0 +1,62 @@
>+<ul id="action">
>+    [% UNLESS ( norequests ) %]
>+        [% IF ( opacuserlogin ) %]
>+            [% IF ( RequestOnOpac ) %]
>+                [% IF ( AllowOnShelfHolds ) %]
>+                    <li><a class="reserve" href="/cgi-bin/koha/opac-reserve.pl?biblionumber=[% biblionumber %]">Place Hold</a></li>
>+                [% ELSE %]
>+                    [% IF ( ItemsIssued ) %]
>+                        <li><a class="reserve" href="/cgi-bin/koha/opac-reserve.pl?biblionumber=[% biblionumber %]">Place Hold</a></li>
>+                    [% END %]
>+                [% END %]
>+            [% END %]
>+        [% END %]
>+    [% END %]
>+    <li><a class="print" href="#" onclick="window.print();">Print</a></li>
>+    [% IF ( virtualshelves ) %]
>+        [% IF ( opacuserlogin ) %][% IF ( loggedinusername ) %]
>+            <li><a class="addtoshelf" href="/cgi-bin/koha/opac-addbybiblionumber.pl?biblionumber=[% biblionumber %]" onclick="Dopop('opac-addbybiblionumber.pl?biblionumber=[% biblionumber %]'); return false;">
>+                Save to Your Lists
>+            </a></li>
>+        [% END %][% END %]
>+    [% END %]
>+    [% IF ( opacbookbag ) %]
>+        <li><a class="addtocart" href="#" onclick="addRecord('[% biblionumber %]'); return false;">Add to Your Cart</a></li>
>+    [% END %]
>+    <li style="display:none;"><a href="#" id="furthersearches">More searches</a></li>
>+</ul>
>+
>+[% IF ( OPACSearchForTitleIn ) %]
>+    <div id="furtherm" class="yuimenu">
>+    <div class="bd">
>+    <h4>Search for this title in:</h4>
>+    <ul class="first-of-type">
>+        [% OPACSearchForTitleIn %]
>+    </ul>
>+    </div>
>+    </div>
>+[% END %]
>+
>+[% IF ( export_options.size ) %]
>+    <div id="export" class="detailtagcell">
>+        <form method="get" action="/cgi-bin/koha/opac-export.pl">
>+        <label for="format">Save Record:</label>
>+        <select name="format" id="format">
>+        [% FOREACH option IN export_options %]
>+            [% SWITCH option %]
>+                [% CASE 'bibtex' %]<option value="bibtex">BIBTEX</option>
>+                [% CASE 'dc' %]<option value="dc">Dublin Core (XML)</option>
>+                [% CASE 'marcxml' %]<option value="marcxml">MARCXML</option>
>+                [% CASE 'marc8' %]<option value="marc8">MARC (non-Unicode/MARC-8)</option>
>+                [% CASE 'utf8' %]<option value="utf8">MARC (Unicode/UTF-8)</option>
>+                [% CASE 'marcstd' %]<option value="utf8">MARC (Unicode/UTF-8, Standard)</option>
>+                [% CASE 'mods' %]<option value="mods">MODS (XML)</option>
>+                [% CASE 'ris' %]<option value="ris">RIS</option>
>+            [% END %]
>+        [% END %]
>+        </select>
>+        <input type="hidden" name="op" value="export" />
>+        <input type="hidden" name="bib" value="[% biblionumber %]" />
>+        <input type="submit" name="save" value="Go" /></form>
>+    </div>
>+[% END %]
>diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-ISBDdetail.tt b/koha-tmpl/opac-tmpl/prog/en/modules/opac-ISBDdetail.tt
>index f9984a7..97b64ad 100644
>--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-ISBDdetail.tt
>+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-ISBDdetail.tt
>@@ -46,63 +46,9 @@
> 	</div>
> 	<div class="yui-u">
> 	<div id="ulactioncontainer" class="container">
>-<ul id="action">
> 
>-[% UNLESS ( norequests ) %]
>-        [% IF ( opacuserlogin ) %]
>-		[% IF ( RequestOnOpac ) %]
>-			[% IF ( AllowOnShelfHolds ) %]
>-				<li><a class="reserve" href="/cgi-bin/koha/opac-reserve.pl?biblionumber=[% biblionumber %]">Place Hold</a></li>
>-			[% ELSE %]
>-				[% IF ( ItemsIssued ) %]
>-					<li><a class="reserve" href="/cgi-bin/koha/opac-reserve.pl?biblionumber=[% biblionumber %]">Place Hold</a></li>
>-				[% END %]
>-			[% END %]
>+[% INCLUDE 'opac-detail-sidebar.inc' %]
> 
>-        	[% END %]
>-	[% END %]
>-[% END %]
>-        <li><a class="print" href="#" onclick="window.print();">Print</a></li>
>-[% IF ( virtualshelves ) %]
>-   [% IF ( opacuserlogin ) %] [% IF ( loggedinusername ) %]
>-        <li><a class="addtoshelf" href="/cgi-bin/koha/opac-addbybiblionumber.pl?biblionumber=[% biblionumber %]" onclick="Dopop('opac-addbybiblionumber.pl?biblionumber=[% biblionumber %]'); return false;">
>-            Save to Your Lists
>-        </a></li>
>-    [% END %][% END %]
>-[% END %]
>-        [% IF ( opacbookbag ) %]<li><a class="addtocart" href="#" onclick="addRecord('[% biblionumber %]'); return false;">
>-            Add to Your Cart
>-        </a></li>[% END %]
>-		<li style="display:none;"><a href="#" id="furthersearches">More searches</a></li>
>-</ul>
>-
>-[% IF ( OPACSearchForTitleIn ) %]
>-<div id="furtherm" class="yuimenu">
>-<div class="bd">
>-<h4>Search for this title in:</h4>
>-<ul class="first-of-type">
>-
>-[% OPACSearchForTitleIn %]
>-
>-</ul>
>-</div>
>-</div>
>-[% END %]
>-
>-		<div id="export">
>-		<form method="get" action="/cgi-bin/koha/opac-export.pl">
>-		<label for="format">Save Record:</label>
>-	    <select name="format" id="format">
>-		<option value="">-- Choose Format --</option>
>- 	    <option value="bibtex">BibTex</option>
>-        <option value="dc">Dublin Core (XML)</option>
>-        <option value="marcxml">MARCXML</option>
>-        <option value="marc8">MARC (non-Unicode/MARC-8)</option>
>-        <option value="utf8">MARC (Unicode/UTF-8)</option>
>-        <option value="mods">MODS (XML)</option>
>- 	    <option value="ris">RIS</option>    
>-		</select><input type="hidden" name="op" value="export" /><input type="hidden" name="bib" value="[% biblionumber %]" />
>-<input type="submit" name="save" value="Go" /></form></div>
> </div>
> </div>
> </div>
>diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-MARCdetail.tt b/koha-tmpl/opac-tmpl/prog/en/modules/opac-MARCdetail.tt
>index 284154e..f3d2c91 100644
>--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-MARCdetail.tt
>+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-MARCdetail.tt
>@@ -201,62 +201,9 @@ $(document).ready(function(){
>     </div>
>     <div class="yui-u">
>     <div id="ulactioncontainer" class="container">
>-<ul id="action">
> 
>-[% UNLESS ( norequests ) %]
>-        [% IF ( opacuserlogin ) %]
>-        [% IF ( RequestOnOpac ) %]
>-            [% IF ( AllowOnShelfHolds ) %]
>-                <li><a class="reserve" href="/cgi-bin/koha/opac-reserve.pl?biblionumber=[% biblionumber %]">Place Hold</a></li>
>-            [% ELSE %]
>-                [% IF ( ItemsIssued ) %]
>-                    <li><a class="reserve" href="/cgi-bin/koha/opac-reserve.pl?biblionumber=[% biblionumber %]">Place Hold</a></li>
>-                [% END %]
>-            [% END %]
>-
>-            [% END %]
>-    [% END %]
>-[% END %]
>-        <li><a class="print" href="#" onclick="window.print();">
>-            Print
>-        </a></li>
>-[% IF ( virtualshelves ) %]
>-   [% IF ( opacuserlogin ) %] [% IF ( loggedinusername ) %]
>-        <li><a class="addtoshelf" href="/cgi-bin/koha/opac-addbybiblionumber.pl?biblionumber=[% biblionumber %]" onclick="Dopop('opac-addbybiblionumber.pl?biblionumber=[% biblionumber %]'); return false;">
>-            Save to Your Lists
>-        </a></li>
>-    [% END %][% END %]
>-[% END %]
>-        [% IF ( opacbookbag ) %]<li><a class="addtocart" href="#" onclick="addRecord('[% biblionumber %]'); return false;">
>-            Add to Your Cart
>-        </a></li>[% END %]
>-        <li style="display:none;"><a href="#" id="furthersearches">More searches</a></li>
>-</ul>
>-
>-[% IF ( OPACSearchForTitleIn ) %]
>-<div id="furtherm" class="yuimenu">
>-<div class="bd">
>-<h4>Search for this title in:</h4>
>-<ul class="first-of-type">
>-
>-[% OPACSearchForTitleIn %]
>-
>-</ul>
>-</div>
>-</div>
>-[% END %]
>+[% INCLUDE 'opac-detail-sidebar.inc' %]
> 
>-        <div id="export">
>-        <form method="get" action="/cgi-bin/koha/opac-export.pl">
>-        <label for="format">Save Record:</label>
>-        <select name="format" id="format">
>-        <option value="">-- Choose Format --</option>
>-        <option value="mods">MODS (XML)</option>
>-        <option value="dc">Dublin Core (XML)</option>
>-        <option value="marcxml">MARCXML</option>
>-        <option value="marc8">MARC (non-Unicode/MARC-8)</option>
>-        <option value="utf8">MARC (Unicode/UTF-8)</option>    </select><input type="hidden" name="op" value="export" /><input type="hidden" name="bib" value="[% biblionumber %]" />
>-<input type="submit" name="save" value="Go" /></form></div>
> </div>
> </div>
> </div>
>diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt b/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt
>index dd1c33f..5a2dc09 100644
>--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt
>+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt
>@@ -1035,61 +1035,8 @@ YAHOO.util.Event.onContentReady("furtherm", function () {
>         [% END %]
>     </div>
> [% END %]
>-<ul id="action">
>-
>-[% UNLESS ( norequests ) %]
>-        [% IF ( opacuserlogin ) %]
>-		[% IF ( RequestOnOpac ) %]
>-			[% IF ( AllowOnShelfHolds ) %]
>-	            		<li><a class="reserve" href="/cgi-bin/koha/opac-reserve.pl?biblionumber=[% biblionumber %]">Place Hold</a></li>
>-			[% ELSE %]
>-				[% IF ( ItemsIssued ) %]
>-		            		<li><a class="reserve" href="/cgi-bin/koha/opac-reserve.pl?biblionumber=[% biblionumber %]">Place Hold</a></li>
>-				[% END %]
>-			[% END %]
>-        	[% END %]
>-	[% END %]
>-[% END %]
>-        <li><a class="print" href="#" onclick="window.print();">Print</a></li>
>-[% IF ( virtualshelves ) %]
>-    [% IF ( opacuserlogin ) %][% IF ( loggedinusername ) %]
>-        <li><a class="addtoshelf" href="/cgi-bin/koha/opac-addbybiblionumber.pl?biblionumber=[% biblionumber %]" onclick="Dopop('opac-addbybiblionumber.pl?biblionumber=[% biblionumber %]'); return false;">
>-            Save to Your Lists
>-        </a></li>
>-    [% END %][% END %]
>-[% END %]
>-        [% IF ( opacbookbag ) %]<li><a class="addtocart" href="#" onclick="addRecord('[% biblionumber %]'); return false;">Add to Your Cart</a></li>[% END %]
>-		<li style="display:none;"><a href="#" id="furthersearches">More searches</a></li>
>-</ul>
>-
>-[% IF ( OPACSearchForTitleIn ) %]
>-<div id="furtherm" class="yuimenu">
>-<div class="bd">
>-<h4>Search for this title in:</h4>
>-<ul class="first-of-type">
>-
>-[% OPACSearchForTitleIn %]
> 
>-</ul>
>-</div>
>-</div>
>-[% END %]
>-
>-        <div id="export" class="detailtagcell">
>-        <form method="get" action="/cgi-bin/koha/opac-export.pl">
>-        <label for="format">Save Record:</label>
>-        <select name="format" id="format">
>-        <option value="">-- Choose Format --</option>
>-        <option value="bibtex">BIBTEX</option>
>-        <option value="dc">Dublin Core (XML)</option>
>-        <option value="marcxml">MARCXML</option>
>-        <option value="marc8">MARC (non-Unicode/MARC-8)</option>
>-        <option value="utf8">MARC (Unicode/UTF-8)</option>    
>-        <option value="mods">MODS (XML)</option>
>-        <option value="ris">RIS</option>
>-		</select><input type="hidden" name="op" value="export" /><input type="hidden" name="bib" value="[% biblionumber %]" />
>-<input type="submit" name="save" value="Go" /></form>
>-        </div>
>+[% INCLUDE 'opac-detail-sidebar.inc' %]
> 
>         [% IF ( NovelistSelectProfile ) %] [% IF ( NovelistSelectView == 'right') %]
>          <div id="NovelistSelect">
>diff --git a/opac/opac-ISBDdetail.pl b/opac/opac-ISBDdetail.pl
>index c80d41c..3373787 100755
>--- a/opac/opac-ISBDdetail.pl
>+++ b/opac/opac-ISBDdetail.pl
>@@ -159,13 +159,22 @@ $template->param(
>     reviews             => $reviews,
> );
> 
>+#Export options
>+my $OpacExportOptions=C4::Context->preference("OpacExportOptions");
>+my @export_options = split(/\|/,$OpacExportOptions);
>+$template->{VARS}->{'export_options'} = \@export_options;
>+
> #Search for title in links
>+my $marccontrolnumber   = GetMarcControlnumber   ($record, $marcflavour);
>+
> if (my $search_for_title = C4::Context->preference('OPACSearchForTitleIn')){
>     $dat->{author} ? $search_for_title =~ s/{AUTHOR}/$dat->{author}/g : $search_for_title =~ s/{AUTHOR}//g;
>     $dat->{title} =~ s/\/+$//; # remove trailing slash
>     $dat->{title} =~ s/\s+$//; # remove trailing space
>     $dat->{title} ? $search_for_title =~ s/{TITLE}/$dat->{title}/g : $search_for_title =~ s/{TITLE}//g;
>     $isbn ? $search_for_title =~ s/{ISBN}/$isbn/g : $search_for_title =~ s/{ISBN}//g;
>+    $marccontrolnumber ? $search_for_title =~ s/{CONTROLNUMBER}/$marccontrolnumber/g : $search_for_title =~ s/{CONTROLNUMBER}//g;
>+    $search_for_title =~ s/{BIBLIONUMBER}/$biblionumber/g;
>  $template->param('OPACSearchForTitleIn' => $search_for_title);
> }
> 
>diff --git a/opac/opac-MARCdetail.pl b/opac/opac-MARCdetail.pl
>index f408cd2..519791c 100755
>--- a/opac/opac-MARCdetail.pl
>+++ b/opac/opac-MARCdetail.pl
>@@ -270,13 +270,25 @@ if(C4::Context->preference("ISBD")) {
> 	$template->param(ISBD => 1);
> }
> 
>+#Export options
>+my $OpacExportOptions=C4::Context->preference("OpacExportOptions");
>+my @export_options = split(/\|/,$OpacExportOptions);
>+$template->{VARS}->{'export_options'} = \@export_options;
>+
> #Search for title in links
>+my $marcflavour  = C4::Context->preference("marcflavour");
>+my $dat = TransformMarcToKoha( $dbh, $record );
>+my $isbn = GetNormalizedISBN(undef,$record,$marcflavour);
>+my $marccontrolnumber   = GetMarcControlnumber   ($record, $marcflavour);
>+
> if (my $search_for_title = C4::Context->preference('OPACSearchForTitleIn')){
>-    $biblio->{author} ? $search_for_title =~ s/{AUTHOR}/$biblio->{author}/g : $search_for_title =~ s/{AUTHOR}//g;
>-    $biblio->{title} =~ s/\/+$//; # remove trailing slash
>-    $biblio->{title} =~ s/\s+$//; # remove trailing space
>-    $biblio->{title} ? $search_for_title =~ s/{TITLE}/$biblio->{title}/g : $search_for_title =~ s/{TITLE}//g;
>-    $biblio->{isbn} ? $search_for_title =~ s/{ISBN}/$biblio->{isbn}/g : $search_for_title =~ s/{ISBN}//g;
>+    $dat->{author} ? $search_for_title =~ s/{AUTHOR}/$dat->{author}/g : $search_for_title =~ s/{AUTHOR}//g;
>+    $dat->{title} =~ s/\/+$//; # remove trailing slash
>+    $dat->{title} =~ s/\s+$//; # remove trailing space
>+    $dat->{title} ? $search_for_title =~ s/{TITLE}/$dat->{title}/g : $search_for_title =~ s/{TITLE}//g;
>+    $isbn ? $search_for_title =~ s/{ISBN}/$isbn/g : $search_for_title =~ s/{ISBN}//g;
>+    $marccontrolnumber ? $search_for_title =~ s/{CONTROLNUMBER}/$marccontrolnumber/g : $search_for_title =~ s/{CONTROLNUMBER}//g;
>+    $search_for_title =~ s/{BIBLIONUMBER}/$biblionumber/g;
>  $template->param('OPACSearchForTitleIn' => $search_for_title);
> }
> 
>diff --git a/opac/opac-detail.pl b/opac/opac-detail.pl
>index c4d8218..dd41bbb 100755
>--- a/opac/opac-detail.pl
>+++ b/opac/opac-detail.pl
>@@ -891,6 +891,11 @@ if (C4::Context->preference("OPACURLOpenInNewWindow")) {
>     $template->param(covernewwindow => 'false');
> }
> 
>+#Export options
>+my $OpacExportOptions=C4::Context->preference("OpacExportOptions");
>+my @export_options = split(/\|/,$OpacExportOptions);
>+$template->{VARS}->{'export_options'} = \@export_options;
>+
> #Search for title in links
> my $marccontrolnumber   = GetMarcControlnumber   ($record, $marcflavour);
> 
>-- 
>1.7.2.5

Actions: View | Diff | Splinter Review

Attachments on bug 7392: 7000 | 7002 | 7003 | 7157 | 7158 | 7159 | 7170 | 7171 | 7571 | 7608 | 7609 | 7610 | 7616 | 10339