Bugzilla – Attachment 19967 Details for
Bug 10349
Don't show empty Descriptions/Title notes tabs in OPAC and staff
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 10349 - Don't show empty Descriptions/Title notes tabs in OPAC and staff
Bug-10349---Dont-show-empty-DescriptionsTitle-note.patch (text/plain), 3.60 KB, created by
Mirko Tietgen
on 2013-07-27 18:57:07 UTC
(
hide
)
Description:
Bug 10349 - Don't show empty Descriptions/Title notes tabs in OPAC and staff
Filename:
MIME Type:
Creator:
Mirko Tietgen
Created:
2013-07-27 18:57:07 UTC
Size:
3.60 KB
patch
obsolete
>From 37e851b85bbf3ae3736471034b1db5b2674f1b55 Mon Sep 17 00:00:00 2001 >From: Owen Leonard <oleonard@myacpl.org> >Date: Fri, 31 May 2013 12:05:49 -0400 >Subject: [PATCH] Bug 10349 - Don't show empty Descriptions/Title notes tabs in OPAC and staff > >The descriptions/title notes tab appears on the detail page in both >staff client and OPAC even if there are no notes. This is probably a >relic of the pre-T:T days when it wasn't possible to use || in an IF. >This patch adds a check for the various variables which might trigger >the display of the tab. > >To test, apply the patch and view records in the OPAC and staff client >which do and do not have title notes attached (whether that be in the >MARC reocrd or in the biblio.notes column). In the OPAC Syndetics >content should also be tested if possible. The descriptions/title notes >tab should only appear if there is content. > >Signed-off-by: Mirko Tietgen <mirko@abunchofthings.net> >--- > .../prog/en/modules/catalogue/detail.tt | 8 ++++++-- > koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt | 8 +++++++- > 2 files changed, 13 insertions(+), 3 deletions(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt >index b3a4240..97474d4 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt >@@ -358,7 +358,7 @@ function verify_images() { > [% ELSE %] > <li><a href="#holdings">Holdings</a></li> > [% END %] >-<li><a href="#description">Descriptions</a></li> >+[% IF ( MARCNOTES || notes ) %]<li><a href="#description">Descriptions</a></li>[% END %] > [% IF ( subscriptionsnumber ) %]<li><a href="#subscriptions">Subscriptions</a></li>[% END %] > [% IF ( FRBRizeEditions ) %][% IF ( XISBNS ) %]<li><a href="#editions">Editions</a></li>[% END %][% END %] > [% IF ( LocalCoverImages ) %][% IF ( localimages || CAN_user_tools_upload_local_cover_images ) %]<li><a href="#images">Images</a></li>[% END %][% END %] >@@ -596,7 +596,9 @@ function verify_images() { > [% END %] > </div> > [% END %] >- >+ >+[% IF ( MARCNOTES || notes ) %] >+ > <div id="description"> > <div class="content_set"> > >@@ -612,6 +614,8 @@ function verify_images() { > </div> > </div> > >+[% END %] >+ > [% IF ( subscriptionsnumber ) %] > <div id="subscriptions"> > <div class="yui-g"> >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 9226dec..8ee48ba 100644 >--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt >+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt >@@ -925,7 +925,9 @@ YAHOO.util.Event.onContentReady("furtherm", function () { > [% IF (SeparateHoldings) %] > <li><a href="#otherholdings">Other holdings ( [% otheritemloop.size || 0 %] )</a></li> > [% END %] >-<li id="tab_descriptions"> <a href="#descriptions">Title notes</a></li> >+[% IF ( MARCNOTES || notes || ( SyndeticsEnabled && SyndeticsSummary && SYNDETICS_SUMMARY )) %] >+ <li id="tab_descriptions"> <a href="#descriptions">Title notes</a></li> >+[% END %] > [% IF ( SYNDETICS_TOC ) %] > <li id="tab_toc"> <a href="#toc">TOC</a></li> > [% END %] >@@ -1063,6 +1065,8 @@ YAHOO.util.Event.onContentReady("furtherm", function () { > </div> > [% END %] > >+[% IF ( MARCNOTES || notes || ( SyndeticsEnabled && SyndeticsSummary && SYNDETICS_SUMMARY )) %] >+ > <div id="descriptions"> > <div class="content_set"> > >@@ -1090,6 +1094,8 @@ YAHOO.util.Event.onContentReady("furtherm", function () { > </div> > </div> > >+[% END %] >+ > [% IF ( SyndeticsEnabled ) %] > > [% IF ( SyndeticsTOC ) %][% IF ( SYNDETICS_TOC ) %] >-- >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 10349
:
18553
|
19967
|
20604