Bugzilla – Attachment 8868 Details for
Bug 7720
Ambiguity in OPAC Details location.
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 7720 - Ambiguity in OPAC Details location.
Bug-7720---Ambiguity-in-OPAC-Details-location.patch (text/plain), 22.92 KB, created by
Kyle M Hall
on 2012-04-04 13:28:48 UTC
(
hide
)
Description:
Bug 7720 - Ambiguity in OPAC Details location.
Filename:
MIME Type:
Creator:
Kyle M Hall
Created:
2012-04-04 13:28:48 UTC
Size:
22.92 KB
patch
obsolete
>From ed10beaffbcb112bc7755daf8ff50b56ad758d7c Mon Sep 17 00:00:00 2001 >From: Kyle M Hall <kyle@bywatersolutions.com> >Date: Thu, 15 Mar 2012 10:27:12 -0400 >Subject: [PATCH] Bug 7720 - Ambiguity in OPAC Details location. > >Currently, in opac-detail.pl, there exists a column named 'Location'. >This column lists the name of the holding branch, and the item's >location description. This can cause confusion to borrowers, as >they may assume that the holding branch is the *owning* branch >(homebranch) of an item. > >This could cause a situation where a borrower waits for an >item to be returned to his or her library, only to find that >the library never owned that item, and it was transferred back >to it's homebranch. It could also lead a borrower to falsely >assume that his or her home library does not own a copy of a >particular item because the borrower does not see an his or her >home library listed for any of the items on the record. > >In addition, even when the holding branch is different >than the home branch, the item's shelving location is displayed, >even though that branch may not use that location. > >This commit makes the item details table equivilent to the intranet >details page by adding a "Home Library" column, which displays the >item's home library, as well as the shelving location. > >If singleBranchMode is enabled, this column disappears and the >"Location" column displays the shelving location only. > >This commit adds the system preference OpacLocationBranchToDisplay which >allows a library to control whether to display the holding library, >the home library, or both for the opac details page. > >http://bugs.koha-community.org/show_bug.cgi?id=7220 >--- > C4/Items.pm | 11 +- > installer/data/mysql/sysprefs.sql | 1 + > installer/data/mysql/updatedatabase.pl | 7 + > .../prog/en/modules/admin/preferences/opac.pref | 8 + > koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt | 209 +++++++++++--------- > opac/opac-detail.pl | 2 + > 6 files changed, 145 insertions(+), 93 deletions(-) > >diff --git a/C4/Items.pm b/C4/Items.pm >index 0b2f99b..b86e348 100644 >--- a/C4/Items.pm >+++ b/C4/Items.pm >@@ -1191,7 +1191,8 @@ sub GetItemsInfo { > itemtypes.notforloan as notforloan_per_itemtype, > holding.branchurl, > holding.branchname, >- holding.opac_info as branch_opac_info >+ holding.opac_info as branch_opac_info, >+ home.branchurl AS homebranchurl > FROM items > LEFT JOIN branches AS holding ON items.holdingbranch = holding.branchcode > LEFT JOIN branches AS home ON items.homebranch=home.branchcode >@@ -1243,6 +1244,14 @@ sub GetItemsInfo { > if ( my $bdata = $bsth->fetchrow_hashref ) { > $data->{'branchname'} = $bdata->{'branchname'}; > } >+ $bsth = $dbh->prepare( >+ "SELECT * FROM branches WHERE branchcode = ? >+ " >+ ); >+ $bsth->execute( $data->{'homebranch'} ); >+ if ( my $bdata = $bsth->fetchrow_hashref ) { >+ $data->{'homebranchname'} = $bdata->{'branchname'}; >+ } > $data->{'datedue'} = $datedue; > > # get notforloan complete status if applicable >diff --git a/installer/data/mysql/sysprefs.sql b/installer/data/mysql/sysprefs.sql >index 34ac684..1a8e7b8 100644 >--- a/installer/data/mysql/sysprefs.sql >+++ b/installer/data/mysql/sysprefs.sql >@@ -360,3 +360,4 @@ INSERT INTO `systempreferences` (variable,value,explanation,options,type) VALUES > INSERT INTO `systempreferences` (variable,value,explanation,options,type) VALUES ('SocialNetworks','1','Enable/Disable social networks links in opac detail pages','','YesNo'); > INSERT INTO systempreferences (variable,value,options,explanation,type) VALUES('SubscriptionDuplicateDroppedInput','','','List of fields which must not be rewritten when a subscription is duplicated (Separated by pipe |)','Free'); > INSERT INTO systempreferences (variable,value,options,explanation,type) VALUES ('AutoResumeSuspendedHolds', '1', NULL , 'Allow suspended holds to be automatically resumed by a set date.', 'YesNo'); >+INSERT INTO systempreferences` (variable,value,options,explanation,type) VALUES ('OpacLocationBranchToDisplay', 'holding', 'holding|home|both', 'In the OPAC, under location show which branch for Location in the record details.', 'Choice'); >diff --git a/installer/data/mysql/updatedatabase.pl b/installer/data/mysql/updatedatabase.pl >index c297677..d257faa 100755 >--- a/installer/data/mysql/updatedatabase.pl >+++ b/installer/data/mysql/updatedatabase.pl >@@ -5139,6 +5139,13 @@ if ( C4::Context->preference("Version") < TransformToNum($DBversion) ) { > SetVersion($DBversion); > } > >+$DBversion = "3.07.00.XXX"; >+if ( C4::Context->preference("Version") < TransformToNum($DBversion) ) { >+ $dbh->do("INSERT INTO systempreferences (variable,value,options,explanation,type) VALUES ('OpacLocationBranchToDisplay', 'holding', 'holding|home|both', 'In the OPAC, under location show which branch for Location in the record details.', 'Choice')"); >+ print "Upgrade to $DBversion done (Added system preference OpacLocationBranchToDisplay)\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 a871399..1f9b7b0 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 >@@ -84,6 +84,14 @@ OPAC: > no: "Don't show" > - the name of the patron that has an item checked out on item detail pages on the OPAC. > - >+ - Display the >+ - pref: OpacLocationBranchToDisplay >+ choices: >+ home: "home library" >+ holding: "holding library" >+ both: "home and holding libraries" >+ - for items on the OPAC record details page. >+ - > - pref: OpacKohaUrl > default: 0 > choices: >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 a5be324..79eab4b 100644 >--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt >+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt >@@ -41,19 +41,19 @@ > widgets : ['zebra'], > sortList: [[0,0]] > }); >- [% IF ( GoogleJackets ) %] >+ [% IF ( GoogleJackets ) %] > KOHA.Google.GetCoverFromIsbn([% covernewwindow %]); >- [% END %] >- [% IF OpenLibraryCovers %] >- KOHA.OpenLibrary.GetCoverFromIsbn(); >- [% END %] >- [% IF OPACLocalCoverImages %] >- KOHA.LocalCover.GetCoverFromBibnumber(true); >- [% END %] >+ [% END %] >+ [% IF OpenLibraryCovers %] >+ KOHA.OpenLibrary.GetCoverFromIsbn(); >+ [% END %] >+ [% IF OPACLocalCoverImages %] >+ KOHA.LocalCover.GetCoverFromBibnumber(true); >+ [% END %] > [% IF ( NovelistSelectProfile ) %] > novSelect.loadContentForISBN('[% normalized_isbn %]','[% NovelistSelectProfile %]', '[% NovelistSelectPassword %]', function(d){}); > [% END %] >- [% IF ( opacuserlogin ) %][% IF ( loggedinusername ) %][% IF ( TagsEnabled ) %] >+ [% IF ( opacuserlogin ) %][% IF ( loggedinusername ) %][% IF ( TagsEnabled ) %] > $(".tagbutton").click(KOHA.Tags.add_tag_button);[% END %][% END %][% END %] > [% IF ( busc ) %] > if (arrPagination.length > 0) { >@@ -203,19 +203,19 @@ function renderPagination(index, total, ul, highlIndex) > > YAHOO.util.Event.onContentReady("furtherm", function () { > $("#furtherm").css("display","block").css("visibility","hidden"); >- $("#furthersearches").parent().show(); >- var furthersearchesMenu = new YAHOO.widget.Menu("furtherm"); >- furthersearchesMenu.render(); >- furthersearchesMenu.cfg.setProperty("context", ["furthersearches", "tr", "br"]); >- furthersearchesMenu.subscribe("beforeShow",positionfurthersearchesMenu); >- furthersearchesMenu.subscribe("show", furthersearchesMenu.focus); >+ $("#furthersearches").parent().show(); >+ var furthersearchesMenu = new YAHOO.widget.Menu("furtherm"); >+ furthersearchesMenu.render(); >+ furthersearchesMenu.cfg.setProperty("context", ["furthersearches", "tr", "br"]); >+ furthersearchesMenu.subscribe("beforeShow",positionfurthersearchesMenu); >+ furthersearchesMenu.subscribe("show", furthersearchesMenu.focus); > function positionfurthersearchesMenu() { > furthersearchesMenu.align("tr", "br"); >- } >- YAHOO.util.Event.addListener("furthersearches", "click", furthersearchesMenu.show, null, furthersearchesMenu); >- YAHOO.widget.Overlay.windowResizeEvent.subscribe(positionfurthersearchesMenu); >+ } >+ YAHOO.util.Event.addListener("furthersearches", "click", furthersearchesMenu.show, null, furthersearchesMenu); >+ YAHOO.widget.Overlay.windowResizeEvent.subscribe(positionfurthersearchesMenu); > }); >- >+ > //]]> > </script> > [% IF ( opacuserlogin ) %][% IF ( loggedinusername ) %][% IF ( TagsEnabled ) %]<style type="text/css"> >@@ -393,25 +393,25 @@ YAHOO.util.Event.onContentReady("furtherm", function () { > > <!-- This puts the LTFL reviews in, and if TabbedView is not set, puts the remaining content above the Tabs instead of in them --> > [% IF ( LibraryThingForLibrariesID ) %] >- [% UNLESS ( LibraryThingForLibrariesTabbedView ) %] >- <div class="results_summary"><div id="ltfl_related" class="ltfl"></div></div> >- <div class="results_summary"><div id="ltfl_similars" class="ltfl"></div></div> >- <div class="results_summary"><div id="ltfl_tagbrowse" class="ltfl"></div></div> >- [% END %] >- <span class="results_summary"> >- <span class="label">Reviews from LibraryThing.com:</span> >+ [% UNLESS ( LibraryThingForLibrariesTabbedView ) %] >+ <div class="results_summary"><div id="ltfl_related" class="ltfl"></div></div> >+ <div class="results_summary"><div id="ltfl_similars" class="ltfl"></div></div> >+ <div class="results_summary"><div id="ltfl_tagbrowse" class="ltfl"></div></div> >+ [% END %] >+ <span class="results_summary"> >+ <span class="label">Reviews from LibraryThing.com:</span> > <span style="display: block;" class="ltfl_reviews"></span> > </span> > [% END %] > > <!--This grabs all of the lists a bib record appears in --> > [% IF ( GetShelves ) %] >- <span class="results_summary"><span class="label">List(s) this item appears in: </span> >- [% FOREACH GetShelve IN GetShelves %] >- <a href="/cgi-bin/koha/opac-shelves.pl?viewshelf=[% GetShelve.shelfnumber %]">[% GetShelve.shelfname %]</a> >- [% IF ( loop.last ) %][% ELSE %]|[% END %] >- [% END %] >- </span> >+ <span class="results_summary"><span class="label">List(s) this item appears in: </span> >+ [% FOREACH GetShelve IN GetShelves %] >+ <a href="/cgi-bin/koha/opac-shelves.pl?viewshelf=[% GetShelve.shelfnumber %]">[% GetShelve.shelfname %]</a> >+ [% IF ( loop.last ) %][% ELSE %]|[% END %] >+ [% END %] >+ </span> > [% END %] > > [% IF ( TagsEnabled ) %] >@@ -594,20 +594,20 @@ YAHOO.util.Event.onContentReady("furtherm", function () { > [% IF ( serialcollection ) %] > <div id="serialcollection"> > <table id="serialcollectiont"> >- <thead> >- <tr> >- <th id="serial_library">Library</th> >- <th id="seral_collection">Serial collection</th> >- </tr> >- </thead> >- <tbody> >- [% FOREACH serialcollection IN serialcollections %] >- <tr> >- <td>[% serialcollection.branch %]</td> >- <td>[% serialcollection.text %]</td> >- </tr> >- [% END %] >- </tbody> >+ <thead> >+ <tr> >+ <th id="serial_library">Library</th> >+ <th id="seral_collection">Serial collection</th> >+ </tr> >+ </thead> >+ <tbody> >+ [% FOREACH serialcollection IN serialcollections %] >+ <tr> >+ <td>[% serialcollection.branch %]</td> >+ <td>[% serialcollection.text %]</td> >+ </tr> >+ [% END %] >+ </tbody> > </table> > </div> > [% END %] >@@ -615,12 +615,19 @@ YAHOO.util.Event.onContentReady("furtherm", function () { > <div id="holdings"> > [% IF ( count ) %] > [% IF ( lotsofitems ) %] >- <p>This record has many physical items. <a href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% biblionumber %]&viewallitems=1#holdings">Click here to view them all.</a></p> >+ <p>This record has many physical items. <a href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% biblionumber %]&viewallitems=1#holdings">Click here to view them all.</a></p> > [% ELSE %] > <table id="holdingst"> > <thead><tr> > [% IF ( item_level_itypes ) %]<th id="item_itemtype">Item type</th>[% END %] >- <th id="item_location">Location</th> >+ [% IF ( OpacLocationBranchToDisplay == 'holding' || OpacLocationBranchToDisplay == 'both' || singleBranchMode ) %] >+ <th>Current Location</th> >+ [% END %] >+ [% UNLESS ( singleBranchMode ) %] >+ [% IF ( OpacLocationBranchToDisplay == 'home' || OpacLocationBranchToDisplay == 'both' ) %] >+ <th>Home Library</th> >+ [% END %] >+ [% END %] > [% IF ( itemdata_ccode ) %]<th id="item_ccode">Collection</th>[% END %] > <th id="item_callnumber">Call Number</th> > [% IF ( itemdata_enumchron ) %]<th id="item_enumchron">Vol Info</th>[% END %] >@@ -630,32 +637,50 @@ YAHOO.util.Event.onContentReady("furtherm", function () { > [% IF ( itemdata_itemnotes ) %]<th id="item_notes">Notes</th>[% END %] > <th id="item_datedue">Date Due</th> > </tr></thead> >- <tbody>[% FOREACH ITEM_RESULT IN ITEM_RESULTS %] >- <tr>[% IF ( item_level_itypes ) %]<td>[% UNLESS ( noItemTypeImages ) %][% IF ( ITEM_RESULT.imageurl ) %]<img src="[% ITEM_RESULT.imageurl %]" title="[% ITEM_RESULT.description %]" alt="[% ITEM_RESULT.description %]" />[% END %][% END %] [% ITEM_RESULT.description %]</td>[% END %] >- <td> >- [% UNLESS ( singleBranchMode ) %] >- <span class="[% ITEM_RESULT.branch_opac_info ? 'branch-info-tooltip-trigger' : '' %]"> >- [% IF ( ITEM_RESULT.branchurl ) %] >- <a href="[% ITEM_RESULT.branchurl %]">[% ITEM_RESULT.branchname %]</a> >- [% ELSE %] >- [% ITEM_RESULT.branchname %] >- [% END %] >- </span> >- <div class="branch-info-tooltip">[% ITEM_RESULT.branch_opac_info %]</div> >- [% END %] >- <span class="shelvingloc">[% ITEM_RESULT.location_description %]</span> >- </td> >- [% IF ( itemdata_ccode ) %]<td>[% ITEM_RESULT.ccode %]</td>[% END %] >- <td>[% IF ( ITEM_RESULT.itemcallnumber ) %] [% ITEM_RESULT.itemcallnumber %][% IF ( OPACShelfBrowser ) %] (<a href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% ITEM_RESULT.biblionumber %]&shelfbrowse_itemnumber=[% ITEM_RESULT.itemnumber %]#shelfbrowser">Browse Shelf</a>)[% END %][% END %]</td> >- [% IF ( itemdata_enumchron ) %]<td>[% ITEM_RESULT.enumchron %]</td>[% END %] >- [% IF ( itemdata_uri ) %]<td><a href="[% ITEM_RESULT.uri %]">[% ITEM_RESULT.uri %]</a></td>[% END %] >- [% IF ( itemdata_copynumber ) %]<td>[% ITEM_RESULT.copynumber %]</td>[% END %] >- <td>[% INCLUDE 'item-status.inc' item = ITEM_RESULT %]</td> >- [% IF ( itemdata_itemnotes ) %]<td>[% ITEM_RESULT.itemnotes %]</td>[% END %] >- <td>[% ITEM_RESULT.datedue | $KohaDates %]</td> >- </tr> >- [% END %]</tbody> >- </table> >+ <tbody>[% FOREACH ITEM_RESULT IN ITEM_RESULTS %] >+ <tr>[% IF ( item_level_itypes ) %]<td>[% UNLESS ( noItemTypeImages ) %][% IF ( ITEM_RESULT.imageurl ) %]<img src="[% ITEM_RESULT.imageurl %]" title="[% ITEM_RESULT.description %]" alt="[% ITEM_RESULT.description %]" />[% END %][% END %] [% ITEM_RESULT.description %]</td>[% END %] >+ [% UNLESS ( singleBranchMode ) %] >+ [% IF ( OpacLocationBranchToDisplay == 'holding' || OpacLocationBranchToDisplay == 'both' ) %] >+ <td> >+ <span class="[% ITEM_RESULT.branch_opac_info ? 'branch-info-tooltip-trigger' : '' %]"> >+ [% IF ( ITEM_RESULT.branchurl ) %] >+ <a href="[% ITEM_RESULT.branchurl %]">[% ITEM_RESULT.branchname %]</a> >+ [% ELSE %] >+ [% ITEM_RESULT.branchname %] >+ [% END %] >+ </span> >+ <div class="branch-info-tooltip">[% ITEM_RESULT.branch_opac_info %]</div> >+ >+ </td> >+ [% END %] >+ >+ [% IF ( OpacLocationBranchToDisplay == 'home' || OpacLocationBranchToDisplay == 'both' ) %] >+ <td> >+ <span class="[% ITEM_RESULT.branch_opac_info ? 'branch-info-tooltip-trigger' : '' %]"> >+ [% IF ( ITEM_RESULT.homebranchurl ) %] >+ <a href="[% ITEM_RESULT.homebranchurl %]">[% ITEM_RESULT.homebranchname %]</a> >+ [% ELSE %] >+ [% ITEM_RESULT.homebranchname %] >+ [% END %] >+ </span> >+ <span class="shelvingloc">[% ITEM_RESULT.location_description %]</span> >+ </td> >+ [% END %] >+ [% ELSE %] >+ <span class="shelvingloc">[% ITEM_RESULT.location_description %]</span> >+ [% END %] >+ >+ [% IF ( itemdata_ccode ) %]<td>[% ITEM_RESULT.ccode %]</td>[% END %] >+ <td>[% IF ( ITEM_RESULT.itemcallnumber ) %] [% ITEM_RESULT.itemcallnumber %][% IF ( OPACShelfBrowser ) %] (<a href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% ITEM_RESULT.biblionumber %]&shelfbrowse_itemnumber=[% ITEM_RESULT.itemnumber %]#shelfbrowser">Browse Shelf</a>)[% END %][% END %]</td> >+ [% IF ( itemdata_enumchron ) %]<td>[% ITEM_RESULT.enumchron %]</td>[% END %] >+ [% IF ( itemdata_uri ) %]<td><a href="[% ITEM_RESULT.uri %]">[% ITEM_RESULT.uri %]</a></td>[% END %] >+ [% IF ( itemdata_copynumber ) %]<td>[% ITEM_RESULT.copynumber %]</td>[% END %] >+ <td>[% INCLUDE 'item-status.inc' item = ITEM_RESULT %]</td> >+ [% IF ( itemdata_itemnotes ) %]<td>[% ITEM_RESULT.itemnotes %]</td>[% END %] >+ <td>[% ITEM_RESULT.datedue | $KohaDates %]</td> >+ </tr> >+ [% END %]</tbody> >+ </table> > [% END %] > [% ELSE %] > [% IF ( ALTERNATEHOLDINGS ) %] >@@ -695,7 +720,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () { > [% IF ( OPACAmazonEnabled ) %][% IF ( OPACAmazonCoverImages ) %][% IF ( NEXT_SHELF_BROWS.browser_normalized_isbn ) %] > <img border="0" src="http://images.amazon.com/images/P/[% NEXT_SHELF_BROWS.browser_normalized_isbn %].01._AA75_PU_PU-5_.jpg" alt="" />[% ELSE %]<span class="no-image">No cover image available</span>[% END %][% END %][% END %] > >- [% IF ( SyndeticsEnabled ) %][% IF ( SyndeticsCoverImages ) %][% IF ( content_identifier_exists ) %] >+ [% IF ( SyndeticsEnabled ) %][% IF ( SyndeticsCoverImages ) %][% IF ( content_identifier_exists ) %] > [% IF ( using_https ) %] > <img border="0" src="https://secure.syndetics.com/index.aspx?isbn=[% NEXT_SHELF_BROWS.browser_normalized_isbn %]/SC.GIF&client=[% SyndeticsClientCode %][% IF ( NEXT_SHELF_BROWS.browser_normalized_upc ) %]&upc=[% NEXT_SHELF_BROWS.browser_normalized_upc %][% END %][% IF ( NEXT_SHELF_BROWS.browser_normalized_oclc ) %]&oclc=[% NEXT_SHELF_BROWS.browser_normalized_oclc %][% END %]&type=xw10" alt="" /> > [% ELSE %]<img border="0" src="http://www.syndetics.com/index.aspx?isbn=[% NEXT_SHELF_BROWS.browser_normalized_isbn %]/SC.GIF&client=[% SyndeticsClientCode %][% IF ( NEXT_SHELF_BROWS.browser_normalized_upc ) %]&upc=[% NEXT_SHELF_BROWS.browser_normalized_upc %][% END %][% IF ( NEXT_SHELF_BROWS.browser_normalized_oclc ) %]&oclc=[% NEXT_SHELF_BROWS.browser_normalized_oclc %][% END %]&type=xw10" alt="" />[% END %] >@@ -796,13 +821,13 @@ YAHOO.util.Event.onContentReady("furtherm", function () { > [% FOREACH SYNDETICS_REVIEW IN SYNDETICS_REVIEWS %] > [% IF ( SYNDETICS_REVIEW.title ) %] > <h4>[% SYNDETICS_REVIEW.title %]</h4> >- [% FOREACH review IN SYNDETICS_REVIEW.reviews %] >+ [% FOREACH review IN SYNDETICS_REVIEW.reviews %] > >- [% IF ( review.content ) %] >- [% review.content %] >- [% END %] >+ [% IF ( review.content ) %] >+ [% review.content %] >+ [% END %] > >- [% END %] >+ [% END %] > [% END %] > [% END %] > </div> >@@ -883,25 +908,25 @@ YAHOO.util.Event.onContentReady("furtherm", function () { > > [% IF ( LibraryThingForLibrariesID ) %] > [% IF ( LibraryThingForLibrariesTabbedView ) %] >- <!-- Library Thing for Libraries Content --> >- <div id="LFTLSimilarItems"> >- <div class="content_set"> >- <!-- Uncommenting this span makes the font smaller in the tab for LTFL --> >+ <!-- Library Thing for Libraries Content --> >+ <div id="LFTLSimilarItems"> >+ <div class="content_set"> >+ <!-- Uncommenting this span makes the font smaller in the tab for LTFL --> > <!-- but breaks Xhtml validation --> >- <!-- <span class="results_summary">--> >- <div id="ltfl_related"></div> >- <div id="ltfl_similars"></div> >- <!-- </span>--> >- </div> >- </div> >- <div id="LTFLTagBrowse"> >+ <!-- <span class="results_summary">--> >+ <div id="ltfl_related"></div> >+ <div id="ltfl_similars"></div> >+ <!-- </span>--> >+ </div> >+ </div> >+ <div id="LTFLTagBrowse"> > <div class="content_set"> > <!-- <span class="results_summary"> --> > <div id="ltfl_tagbrowse" class="ltfl"></div> > <!-- </span> --> > </div> > </div> >- [% END %] >+ [% END %] > [% END %] > <!-- /Library Thing for Libraries Content --> > >diff --git a/opac/opac-detail.pl b/opac/opac-detail.pl >index a00e976..59aa155 100755 >--- a/opac/opac-detail.pl >+++ b/opac/opac-detail.pl >@@ -945,4 +945,6 @@ if (C4::Context->preference('OPACLocalCoverImages') == 1) { > $template->{VARS}->{localimages} = \@images; > } > >+$template->param('OpacLocationBranchToDisplay' => C4::Context->preference('OpacLocationBranchToDisplay') ); >+ > output_html_with_http_headers $query, $cookie, $template->output; >-- >1.7.2.5
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 7720
:
8213
|
8215
|
8216
|
8223
|
8224
|
8226
|
8229
|
8230
|
8237
|
8238
|
8243
|
8245
|
8825
|
8868
|
8878
|
9570
|
9762
|
10149
|
11091
|
11092
|
12389
|
13466
|
15237
|
15238
|
15241
|
15247
|
18622
|
19658
|
20217
|
21374
|
21375
|
22155
|
22156
|
28172
|
28173
|
28175