Bugzilla – Attachment 53072 Details for
Bug 16530
Add a circ sidebar navigation menu
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 16530: Adding a circ sidebar navigation menu and circSidebar syspref to activate/deactivate
Bug-16530-Adding-a-circ-sidebar-navigation-menu-an.patch (text/plain), 34.37 KB, created by
Aleisha Amohia
on 2016-07-04 05:31:07 UTC
(
hide
)
Description:
Bug 16530: Adding a circ sidebar navigation menu and circSidebar syspref to activate/deactivate
Filename:
MIME Type:
Creator:
Aleisha Amohia
Created:
2016-07-04 05:31:07 UTC
Size:
34.37 KB
patch
obsolete
>From c3aca29fd27dfc0d4e66f97d61c08ab468389cb1 Mon Sep 17 00:00:00 2001 >From: Aleisha <aleishaamohia@hotmail.com> >Date: Tue, 17 May 2016 04:35:46 +0000 >Subject: [PATCH] Bug 16530: Adding a circ sidebar navigation menu and > circSidebar syspref to activate/deactivate > >Edit: Fast cataloging link should now show on all pages, removed offline circulation links >Edit 2: Creates the syspref to activate or deactivate the sidebar > >This adds a sidebar to >circ/branchoverdues.tt >circ/circulation.tt (I also fixed up some of the indentation in this file to make it easier to see where the divs started and ended) >circ/renew.tt >circ/returns.tt >circ/selectbranchprinter.tt >circ/transferstoreceive.tt >circ/view_holdsqueue.tt >circ/waitingreserves.tt >circ/branchtransfers.tt > >To test: >1) Confirm syspref circSidebar is activated >2) Go to all of the above pages and confirm the sidebar menu shows up >3) Confirm fast cataloguing link and transfer link are there >4) Trigger any error messages you can possibly think of (i.e. on renew.pl: barcode does not exist). Confirm that this does not mess up the layout of the page >5) Go to a user account page, Check out tab. (Since this is a circ/circulation.pl page). Ensure the circ nav sidebar doesn't show up (confirm it looks as it usually does) >6) Deactivate circSidebar >7) Confirm pages all look normal > >Sponsored-by: Catalyst IT > >Signed-off-by: Jan Kissig <jkissig@th-wildau.de> >--- > circ/branchoverdues.pl | 9 +- > circ/branchtransfers.pl | 10 +- > circ/renew.pl | 10 +- > circ/returns.pl | 9 +- > circ/selectbranchprinter.pl | 9 +- > circ/transferstoreceive.pl | 9 +- > circ/view_holdsqueue.pl | 9 +- > circ/waitingreserves.pl | 9 +- > .../bug_16530-add_circSidebar_syspref.sql | 1 + > installer/data/mysql/sysprefs.sql | 1 + > .../intranet-tmpl/prog/en/includes/circ-nav.inc | 54 ++++++++ > .../en/modules/admin/preferences/circulation.pref | 6 + > .../prog/en/modules/circ/branchoverdues.tt | 10 +- > .../prog/en/modules/circ/branchtransfers.tt | 13 +- > .../prog/en/modules/circ/circulation.tt | 20 ++- > .../intranet-tmpl/prog/en/modules/circ/renew.tt | 14 ++- > .../intranet-tmpl/prog/en/modules/circ/returns.tt | 131 ++++++++++---------- > .../prog/en/modules/circ/selectbranchprinter.tt | 10 +- > .../prog/en/modules/circ/transferstoreceive.tt | 10 +- > .../prog/en/modules/circ/view_holdsqueue.tt | 10 +- > .../prog/en/modules/circ/waitingreserves.tt | 10 +- > .../prog/en/modules/offline_circ/list.tt | 4 +- > .../prog/en/modules/offline_circ/process_koc.tt | 2 - > 23 files changed, 278 insertions(+), 92 deletions(-) > create mode 100644 installer/data/mysql/atomicupdate/bug_16530-add_circSidebar_syspref.sql > create mode 100644 koha-tmpl/intranet-tmpl/prog/en/includes/circ-nav.inc > >diff --git a/circ/branchoverdues.pl b/circ/branchoverdues.pl >index d04f5e4..ce815b2 100755 >--- a/circ/branchoverdues.pl >+++ b/circ/branchoverdues.pl >@@ -62,7 +62,7 @@ use Data::Dumper; > my $input = new CGI; > my $dbh = C4::Context->dbh; > >-my ( $template, $loggedinuser, $cookie ) = get_template_and_user({ >+my ( $template, $loggedinuser, $cookie, $flags ) = get_template_and_user({ > template_name => "circ/branchoverdues.tt", > query => $input, > type => "intranet", >@@ -156,4 +156,11 @@ $template->param( > location => $location, > ); > >+# Checking if there is a Fast Cataloging Framework >+my $fa = getframeworkinfo('FA'); >+$template->param( fast_cataloging => 1 ) if (defined $fa); >+ >+# Checking if the transfer page needs to be displayed >+$template->param( display_transfer => 1 ) if ( ($flags->{'superlibrarian'} == 1) || (C4::Context->preference("IndependentBranches") == 0) ); >+ > output_html_with_http_headers $input, $cookie, $template->output; >diff --git a/circ/branchtransfers.pl b/circ/branchtransfers.pl >index a826289..03f8963 100755 >--- a/circ/branchtransfers.pl >+++ b/circ/branchtransfers.pl >@@ -51,7 +51,7 @@ if (!C4::Context->userenv){ > > ####################################################################################### > # Make the page ..... >-my ($template, $user, $cookie) = get_template_and_user( >+my ($template, $user, $cookie, $flags ) = get_template_and_user( > { > template_name => "circ/branchtransfers.tt", > query => $query, >@@ -240,5 +240,13 @@ $template->param( > errmsgloop => \@errmsgloop, > CircAutocompl => C4::Context->preference("CircAutocompl") > ); >+ >+# Checking if there is a Fast Cataloging Framework >+my $fa = getframeworkinfo('FA'); >+$template->param( fast_cataloging => 1 ) if (defined $fa); >+ >+# Checking if the transfer page needs to be displayed >+$template->param( display_transfer => 1 ) if ( ($flags->{'superlibrarian'} == 1) || (C4::Context->preference("IndependentBranches") == 0) ); >+ > output_html_with_http_headers $query, $cookie, $template->output; > >diff --git a/circ/renew.pl b/circ/renew.pl >index a87ba76..122473b 100755 >--- a/circ/renew.pl >+++ b/circ/renew.pl >@@ -26,10 +26,11 @@ use C4::Output; > use C4::Circulation; > use Koha::DateUtils; > use Koha::Database; >+use C4::Koha; > > my $cgi = new CGI; > >-my ( $template, $librarian, $cookie ) = get_template_and_user( >+my ( $template, $librarian, $cookie, $flags ) = get_template_and_user( > { > template_name => "circ/renew.tt", > query => $cgi, >@@ -109,4 +110,11 @@ if ($barcode) { > ); > } > >+# Checking if there is a Fast Cataloging Framework >+my $fa = getframeworkinfo('FA'); >+$template->param( fast_cataloging => 1 ) if (defined $fa); >+ >+# Checking if the transfer page needs to be displayed >+$template->param( display_transfer => 1 ) if ( ($flags->{'superlibrarian'} == 1) || (C4::Context->preference("IndependentBranches") == 0) ); >+ > output_html_with_http_headers( $cgi, $cookie, $template->output ); >diff --git a/circ/returns.pl b/circ/returns.pl >index c91d784..3d3e469 100755 >--- a/circ/returns.pl >+++ b/circ/returns.pl >@@ -54,7 +54,7 @@ use Koha::Calendar; > my $query = new CGI; > > #getting the template >-my ( $template, $librarian, $cookie ) = get_template_and_user( >+my ( $template, $librarian, $cookie, $flags ) = get_template_and_user( > { > template_name => "circ/returns.tt", > query => $query, >@@ -657,5 +657,12 @@ if ( $itemnumber ) { > } > } > >+# Checking if there is a Fast Cataloging Framework >+my $fa = getframeworkinfo('FA'); >+$template->param( fast_cataloging => 1 ) if (defined $fa); >+ >+# Checking if the transfer page needs to be displayed >+$template->param( display_transfer => 1 ) if ( ($flags->{'superlibrarian'} == 1) || (C4::Context->preference("IndependentBranches") == 0) ); >+ > # actually print the page! > output_html_with_http_headers $query, $cookie, $template->output; >diff --git a/circ/selectbranchprinter.pl b/circ/selectbranchprinter.pl >index 36fd9a5..30009e2 100755 >--- a/circ/selectbranchprinter.pl >+++ b/circ/selectbranchprinter.pl >@@ -32,7 +32,7 @@ use C4::Branch; # GetBranches GetBranchesLoop > > my $query = CGI->new(); > >-my ( $template, $borrowernumber, $cookie ) = get_template_and_user({ >+my ( $template, $borrowernumber, $cookie, $flags ) = get_template_and_user({ > template_name => "circ/selectbranchprinter.tt", > query => $query, > type => "intranet", >@@ -138,4 +138,11 @@ $template->param( > recycle_loop=> \@recycle_loop, > ); > >+# Checking if there is a Fast Cataloging Framework >+my $fa = getframeworkinfo('FA'); >+$template->param( fast_cataloging => 1 ) if (defined $fa); >+ >+# Checking if the transfer page needs to be displayed >+$template->param( display_transfer => 1 ) if ( ($flags->{'superlibrarian'} == 1) || (C4::Context->preference("IndependentBranches") == 0) ); >+ > output_html_with_http_headers $query, $cookie, $template->output; >diff --git a/circ/transferstoreceive.pl b/circ/transferstoreceive.pl >index 4be2bcb..f8c41d3 100755 >--- a/circ/transferstoreceive.pl >+++ b/circ/transferstoreceive.pl >@@ -41,7 +41,7 @@ use C4::Reserves; > my $input = new CGI; > my $itemnumber = $input->param('itemnumber'); > >-my ( $template, $loggedinuser, $cookie ) = get_template_and_user( >+my ( $template, $loggedinuser, $cookie, $flags ) = get_template_and_user( > { > template_name => "circ/transferstoreceive.tt", > query => $input, >@@ -124,5 +124,12 @@ $template->param( > latetransfers => $latetransfers ? 1 : 0, > ); > >+# Checking if there is a Fast Cataloging Framework >+my $fa = getframeworkinfo('FA'); >+$template->param( fast_cataloging => 1 ) if (defined $fa); >+ >+# Checking if the transfer page needs to be displayed >+$template->param( display_transfer => 1 ) if ( ($flags->{'superlibrarian'} == 1) || (C4::Context->preference("IndependentBranches") == 0) ); >+ > output_html_with_http_headers $input, $cookie, $template->output; > >diff --git a/circ/view_holdsqueue.pl b/circ/view_holdsqueue.pl >index d21d3e7..5b75377 100755 >--- a/circ/view_holdsqueue.pl >+++ b/circ/view_holdsqueue.pl >@@ -34,7 +34,7 @@ use C4::Branch; # GetBranches > use C4::HoldsQueue qw(GetHoldsQueueItems); > > my $query = new CGI; >-my ( $template, $loggedinuser, $cookie ) = get_template_and_user( >+my ( $template, $loggedinuser, $cookie, $flags ) = get_template_and_user( > { > template_name => "circ/view_holdsqueue.tt", > query => $query, >@@ -76,5 +76,12 @@ $template->param( > itemtypeloop => \@itemtypesloop, > ); > >+# Checking if there is a Fast Cataloging Framework >+my $fa = getframeworkinfo('FA'); >+$template->param( fast_cataloging => 1 ) if (defined $fa); >+ >+# Checking if the transfer page needs to be displayed >+$template->param( display_transfer => 1 ) if ( ($flags->{'superlibrarian'} == 1) || (C4::Context->preference("IndependentBranches") == 0) ); >+ > # writing the template > output_html_with_http_headers $query, $cookie, $template->output; >diff --git a/circ/waitingreserves.pl b/circ/waitingreserves.pl >index 0050ce3..946f284 100755 >--- a/circ/waitingreserves.pl >+++ b/circ/waitingreserves.pl >@@ -48,7 +48,7 @@ my $all_branches = $input->param('allbranches') || ''; > my $cancelall = $input->param('cancelall'); > my $tab = $input->param('tab'); > >-my ( $template, $loggedinuser, $cookie ) = get_template_and_user( >+my ( $template, $loggedinuser, $cookie, $flags ) = get_template_and_user( > { > template_name => "circ/waitingreserves.tt", > query => $input, >@@ -164,6 +164,13 @@ $template->param( > ReservesMaxPickUpDelay => $max_pickup_delay, > ); > >+# Checking if there is a Fast Cataloging Framework >+my $fa = getframeworkinfo('FA'); >+$template->param( fast_cataloging => 1 ) if (defined $fa); >+ >+# Checking if the transfer page needs to be displayed >+$template->param( display_transfer => 1 ) if ( ($flags->{'superlibrarian'} == 1) || (C4::Context->preference("IndependentBranches") == 0) ); >+ > if ($item && $tab eq 'holdsover') { > print $input->redirect("/cgi-bin/koha/circ/waitingreserves.pl#holdsover"); > } elsif ($cancelall) { >diff --git a/installer/data/mysql/atomicupdate/bug_16530-add_circSidebar_syspref.sql b/installer/data/mysql/atomicupdate/bug_16530-add_circSidebar_syspref.sql >new file mode 100644 >index 0000000..b717dd4 >--- /dev/null >+++ b/installer/data/mysql/atomicupdate/bug_16530-add_circSidebar_syspref.sql >@@ -0,0 +1 @@ >+INSERT IGNORE INTO systempreferences (`variable`, `value`, `options`, `explanation`, `type`) VALUES ('circSidebar','0','','Activate or deactivate the navigation sidebar on all Circulation pages','YesNo'); >diff --git a/installer/data/mysql/sysprefs.sql b/installer/data/mysql/sysprefs.sql >index af42f66..9868065 100644 >--- a/installer/data/mysql/sysprefs.sql >+++ b/installer/data/mysql/sysprefs.sql >@@ -94,6 +94,7 @@ INSERT INTO systempreferences ( `variable`, `value`, `options`, `explanation`, ` > ('CircAutocompl','1',NULL,'If ON, autocompletion is enabled for the Circulation input','YesNo'), > ('CircAutoPrintQuickSlip','qslip',NULL,'Choose what should happen when an empty barcode field is submitted in circulation: Display a print quick slip window, Display a print slip window or Clear the screen.','Choice'), > ('CircControl','ItemHomeLibrary','PickupLibrary|PatronLibrary|ItemHomeLibrary','Specify the agency that controls the circulation and fines policy','Choice'), >+('circSidebar','0',NULL,'Activate or deactivate the navigation sidebar on all Circulation pages','YesNo'), > ('ClaimsBccCopy','0','','Bcc the ClaimAcquisition and ClaimIssues alerts','YesNo'), > ('Coce','0', NULL, 'If on, enables cover retrieval from the configured Coce server', 'YesNo'), > ('CoceHost', '', NULL, 'Coce server URL', 'Free'), >diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/circ-nav.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/circ-nav.inc >new file mode 100644 >index 0000000..7bcd94c >--- /dev/null >+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/circ-nav.inc >@@ -0,0 +1,54 @@ >+<script type="text/javascript">//<![CDATA[ >+ $(document).ready(function() { >+ var path = location.pathname.substring(1); >+ var url = window.location.toString(); >+ var params = ''; >+ if ( url.match(/\?(.+)$/) ) { >+ params = "?" + RegExp.$1; >+ } >+ $('#navmenulist a[href$="/' + path + params + '"]').css('font-weight','bold'); >+ }); >+//]]> >+</script> >+ >+<div id="navmenu"> >+ <div id="navmenulist"> >+ >+ <ul> >+ <li><a href="/cgi-bin/koha/circ/circulation-home.pl">Circulation home</a></li> >+ </ul> >+ >+ <h5>Circulation</h5> >+ <ul> >+ <li><a href="/cgi-bin/koha/circ/circulation.pl">Check out</a></li> >+ <li><a href="/cgi-bin/koha/circ/returns.pl">Check in</a></li> >+ <li><a href="/cgi-bin/koha/circ/renew.pl">Renew</a></li> >+ [% IF ( display_transfer ) %] >+ <li><a href="/cgi-bin/koha/circ/branchtransfers.pl">Transfer</a></li> >+ [% END %] >+ [% IF ( AutoLocation ) %][% ELSE %][% IF ( IndependentBranches ) %][% ELSE %] >+ <li><a href="/cgi-bin/koha/circ/selectbranchprinter.pl">Set library</a></li> >+ [% END %][% END %] >+ [% IF ( fast_cataloging ) %][% IF ( CAN_user_editcatalogue_fast_cataloging ) %] >+ <li><a href="/cgi-bin/koha/cataloguing/addbiblio.pl?frameworkcode=FA">Fast cataloging</a></li> >+ [% END %][% END %] >+ </ul> >+ >+ <h5>Circulation reports</h5> >+ <ul> >+ <li><a href="/cgi-bin/koha/circ/view_holdsqueue.pl">Holds queue</a></li> >+ <li><a href="/cgi-bin/koha/circ/pendingreserves.pl">Holds to pull</a></li> >+ <li><a href="/cgi-bin/koha/circ/waitingreserves.pl">Holds awaiting pickup</a></li> >+ <li><a href="/cgi-bin/koha/circ/reserveratios.pl">Hold ratios</a></li> >+ <li><a href="/cgi-bin/koha/circ/transferstoreceive.pl">Transfers to receive</a></li> >+ [% IF ( CAN_user_circulate_overdues_report ) %] >+ <li><a href="/cgi-bin/koha/circ/overdue.pl">Overdues</a></li> >+ [% END %] >+ <li><a href="/cgi-bin/koha/circ/branchoverdues.pl">Overdues with fines</a></li> >+ [% IF Koha.Preference('OnSiteCheckouts') %] >+ <li><a href="/cgi-bin/koha/circ/on-site_checkouts.pl">Pending on-site checkouts</a></li> >+ [% END %] >+ </ul> >+ >+ </div> >+</div> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref >index 9a37608..88c68b0 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/circulation.pref >@@ -2,6 +2,12 @@ Circulation: > # FIXME: printcirculationslips is also omitted. It _technically_ could work, but C4::Print is HLT specific and needs a little bit of refactoring. > Interface: > - >+ - pref: circSidebar >+ choices: >+ yes: Activate >+ no: Deactivate >+ - the navigation sidebar on all Circulation pages. >+ - > - pref: CircAutocompl > choices: > yes: Try >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/branchoverdues.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/branchoverdues.tt >index ff3424d..9e6d99f 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/branchoverdues.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/branchoverdues.tt >@@ -1,3 +1,4 @@ >+[% USE Koha %] > [% INCLUDE 'doc-head-open.inc' %] > <title>Koha › Circulation › Overdues at [% LoginBranchname %]</title> > [% INCLUDE 'doc-head-close.inc' %] >@@ -8,10 +9,11 @@ > > <div id="breadcrumbs"><a href="/cgi-bin/koha/mainpage.pl">Home</a> › <a href="/cgi-bin/koha/circ/circulation-home.pl">Circulation</a> › Overdues at [% LoginBranchname %]</div> > >-<div id="doc" class="yui-t7"> >+[% IF Koha.Preference('circSidebar') %]<div id="doc3" class="yui-t2">[% ELSE %]<div id="doc" class="yui-t7">[% END %] > > <div id="bd"> > <div id="yui-main"> >+ [% IF Koha.Preference('circSidebar') %]<div class="yui-b">[% END %] > <div class="yui-g"> > > >@@ -162,5 +164,11 @@ > > </div> > </div> >+[% IF Koha.Preference('circSidebar') %] >+</div> >+<div class="yui-b noprint"> >+ [% INCLUDE 'circ-nav.inc' %] >+</div> >+[% END %] > </div> > [% INCLUDE 'intranet-bottom.inc' %] >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/branchtransfers.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/branchtransfers.tt >index e2916d9..546d867 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/branchtransfers.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/branchtransfers.tt >@@ -11,9 +11,10 @@ > > <div id="breadcrumbs"><a href="/cgi-bin/koha/mainpage.pl">Home</a> › <a href="/cgi-bin/koha/circ/circulation-home.pl">Circulation</a> › Transfers</div> > >-<div id="doc" class="yui-t7"> >+[% IF Koha.Preference('circSidebar') %]<div id="doc3" class="yui-t2">[% ELSE %]<div id="doc" class="yui-t7">[% END %] > <div id="bd"> >- >+<div id="yui-main"> >+[% IF Koha.Preference('circSidebar') %]<div class="yui-b">[% END %] > [% IF ( found ) %] > <div class="yui-g"> <h3>Reserve found</h3> > <table> >@@ -203,6 +204,12 @@ > > > [% END %] >- >+</div> >+[% IF Koha.Preference('circSidebar') %] >+<div class="yui-b noprint"> >+[% INCLUDE 'circ-nav.inc' %] >+</div> >+[% END %] >+</div> > </div> > [% INCLUDE 'intranet-bottom.inc' %] >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt >index 0056dfc..a93f122 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt >@@ -962,12 +962,24 @@ No patron matched <span class="ex">[% message %]</span> > [% END %] <!-- borrowernumber and borrower--> > </div></div> > [% END %] >- >+[% IF Koha.Preference('circSidebar') %] >+[% UNLESS ( borrowers ) %] >+ [% IF not( borrowernumber and borrower ) %] >+ <div class="yui-b noprint"> >+ [% INCLUDE 'circ-nav.inc' %] >+ </div> >+ [% END %] >+[% END %] >+[% END %] > </div> > </div> >-[% UNLESS ( borrowers ) %][% IF borrowernumber and borrower %]<div class="yui-b"> >-[% INCLUDE 'circ-menu.inc' %] >-</div>[% END %][% END %] >+[% UNLESS ( borrowers ) %] >+ [% IF borrowernumber and borrower %] >+ <div class="yui-b"> >+ [% INCLUDE 'circ-menu.inc' %] >+ </div> >+ [% END %] >+[% END %] > </div> > <!-- Modal --> > <div id="barcodeSubmittedModal" class="modal hide fade" tabindex="-1" role="dialog" aria-labelledby="barcodeSubmittedModalLabel" aria-hidden="true"> >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/renew.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/renew.tt >index 7c481f4..d632036 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/renew.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/renew.tt >@@ -26,9 +26,10 @@ > > <div id="breadcrumbs"><a href="/cgi-bin/koha/mainpage.pl">Home</a> › <a href="/cgi-bin/koha/circ/circulation-home.pl">Circulation</a> › Renew</div> > >-<div id="doc" class="yui-t7"> >+[% IF Koha.Preference('circSidebar') %]<div id="doc3" class="yui-t2">[% ELSE %]<div id="doc" class="yui-t7">[% END %] > <div id="bd"> >- >+ <div id="yui-main"> >+ [% IF Koha.Preference('circSidebar') %]<div class="yui-b">[% END %] > [% IF error %] > <div class="dialog alert"> > <h3>Cannot renew:</h3> >@@ -128,7 +129,6 @@ > </p> > </div> > [% END %] >- </div> > > <div class="yui-g"> > >@@ -151,9 +151,15 @@ > [% END %] > > </div> >- > </div> > >+[% IF Koha.Preference('circSidebar') %] >+</div> >+ >+<div class="yui-b noprint"> >+ [% INCLUDE 'circ-nav.inc' %] >+</div> >+[% END %] > <div> > > [% INCLUDE 'intranet-bottom.inc' %] >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt >index 0c4180c..a062270 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt >@@ -140,11 +140,11 @@ $(document).ready(function () { > > <div id="breadcrumbs"><a href="/cgi-bin/koha/mainpage.pl">Home</a> › <a href="/cgi-bin/koha/circ/circulation-home.pl">Circulation</a> › Check in</div> > >-<div id="doc" class="yui-t7"> >- >- <div id="bd"> >- <div id="yui-main"> >+[% IF Koha.Preference('circSidebar') %]<div id="doc3" class="yui-t2">[% ELSE %]<div id="doc" class="yui-t7">[% END %] > >+<div id="bd"> >+<div id="yui-main"> >+[% IF Koha.Preference('circSidebar') %]<div class="yui-b">[% END %] > <div class="yui-g"> > > [% IF privacy == 2 AND NOT Koha.Preference('AnonymousPatron') %] >@@ -461,66 +461,64 @@ $(document).ready(function () { > <p><a href="/cgi-bin/koha/catalogue/detail.pl?type=intra&biblionumber=[% itembiblionumber %]">[% itembarcode |html %]: [% title |html %]</a></p> > [% END %] > [% FOREACH errmsgloo IN errmsgloop %] >- [% IF ( errmsgloo.NotForLoanStatusUpdated ) %] >- <p class="problem"> >- Not for loan status updated. >- <br />Old value: >- [% IF errmsgloo.NotForLoanStatusUpdated.from %] >- [% AuthorisedValues.GetByCode( 'NOT_LOAN', errmsgloo.NotForLoanStatusUpdated.from ) %]. >- [% ELSE %] >- Available for loan. >- [% END %] >- <br />New value: >- [% IF errmsgloo.NotForLoanStatusUpdated.to %] >- [% AuthorisedValues.GetByCode( 'NOT_LOAN', errmsgloo.NotForLoanStatusUpdated.to ) %]. >- [% ELSE %] >- Available for loan. >- [% END %] >- </p> >- [% END %] >- [% IF ( errmsgloo.badbarcode ) %] >- <p class="problem">No item with barcode: [% errmsgloo.msg %]</p> >- [% END %] >- [% IF ( errmsgloo.ispermanent ) %] >- <p class="problem">Please return item to: [% errmsgloo.msg %]</p> >- [% END %] >- [% IF ( errmsgloo.notissued ) %] >- <p class="problem">Not checked out.</p> >- [% END %] >- [% IF ( errmsgloo.localuse) %] >- <p class="problem">Local use recorded</p> >- [% END %] >- [% IF ( errmsgloo.waslost ) %] >- <p class="problem">Item was lost, now found.</p> >- [% IF ( LostItemFeeRefunded ) %] >- <p class="problem">A refund has been applied to the borrowing patron's account.</p> >- [% ELSE %] >- <p class="problem">Any lost item fees for this item will remain on the patron's account.</p> >- [% END %] >- [% END %] >- [% IF ( errmsgloo.withdrawn ) %] >- [% IF BlockReturnOfWithdrawnItems %] >- <h5>Cannot check in</h5> >- <p><strong>NOT CHECKED IN</strong></p> >- <p class="problem">Item is withdrawn.</p> >- [% ELSE %] >- <p class="problem">Item is withdrawn.</p> >- [% END %] >- [% END %] >- [% IF ( errmsgloo.debarred ) %] >- <p class="problem"><a href="/cgi-bin/koha/circ/circulation.pl?borrowernumber=[% errmsgloo.debarborrowernumber %]">[% errmsgloo.debarname %]([% errmsgloo.debarcardnumber %])</a> is now debarred until [% errmsgloo.debarred | $KohaDates %].</p> >- [% END %] >- [% IF ( errmsgloo.prevdebarred ) %] >- <p class="problem"><b>Reminder: </b>Patron was earlier restricted until [% errmsgloo.prevdebarred | $KohaDates %].</p> >+ [% IF ( errmsgloo.NotForLoanStatusUpdated ) %] >+ <p class="problem"> >+ Not for loan status updated. >+ <br />Old value: >+ [% IF errmsgloo.NotForLoanStatusUpdated.from %] >+ [% AuthorisedValues.GetByCode( 'NOT_LOAN', errmsgloo.NotForLoanStatusUpdated.from ) %]. >+ [% ELSE %] >+ Available for loan. > [% END %] >- [% IF ( errmsgloo.foreverdebarred ) %] >- <p class="problem"><b>Reminder: </b>Patron has an indefinite restriction.</p> >+ <br />New value: >+ [% IF errmsgloo.NotForLoanStatusUpdated.to %] >+ [% AuthorisedValues.GetByCode( 'NOT_LOAN', errmsgloo.NotForLoanStatusUpdated.to ) %]. >+ [% ELSE %] >+ Available for loan. > [% END %] >- >+ </p> >+ [% END %] >+ [% IF ( errmsgloo.badbarcode ) %] >+ <p class="problem">No item with barcode: [% errmsgloo.msg %]</p> >+ [% END %] >+ [% IF ( errmsgloo.ispermanent ) %] >+ <p class="problem">Please return item to: [% errmsgloo.msg %]</p> >+ [% END %] >+ [% IF ( errmsgloo.notissued ) %] >+ <p class="problem">Not checked out.</p> >+ [% END %] >+ [% IF ( errmsgloo.localuse) %] >+ <p class="problem">Local use recorded</p> >+ [% END %] >+ [% IF ( errmsgloo.waslost ) %] >+ <p class="problem">Item was lost, now found.</p> >+ [% IF ( LostItemFeeRefunded ) %] >+ <p class="problem">A refund has been applied to the borrowing patron's account.</p> >+ [% ELSE %] >+ <p class="problem">Any lost item fees for this item will remain on the patron's account.</p> >+ [% END %] >+ [% END %] >+ [% IF ( errmsgloo.withdrawn ) %] >+ [% IF BlockReturnOfWithdrawnItems %] >+ <h5>Cannot check in</h5> >+ <p><strong>NOT CHECKED IN</strong></p> >+ <p class="problem">Item is withdrawn.</p> >+ [% ELSE %] >+ <p class="problem">Item is withdrawn.</p> >+ [% END %] >+ [% END %] >+ [% IF ( errmsgloo.debarred ) %] >+ <p class="problem"><a href="/cgi-bin/koha/circ/circulation.pl?borrowernumber=[% errmsgloo.debarborrowernumber %]">[% errmsgloo.debarname %]([% errmsgloo.debarcardnumber %])</a> is now debarred until [% errmsgloo.debarred | $KohaDates %].</p> >+ [% END %] >+ [% IF ( errmsgloo.prevdebarred ) %] >+ <p class="problem"><b>Reminder: </b>Patron was earlier restricted until [% errmsgloo.prevdebarred | $KohaDates %].</p> >+ [% END %] >+ [% IF ( errmsgloo.foreverdebarred ) %] >+ <p class="problem"><b>Reminder: </b>Patron has an indefinite restriction.</p> > [% END %] >- [% ELSE %] > [% END %] > </div> >+[% END %] > > [% IF ( checkinmsg ) %] > [% IF ( checkinmsgtype == 'alert' ) %] >@@ -542,7 +540,7 @@ $(document).ready(function () { > <p>Book drop mode. (Effective checkin date is [% dropboxdate %] ).</p> > </div> > </div> >- <div class="yui-g"> >+<div class="yui-g"> > <form id="checkin-form" method="post" action="/cgi-bin/koha/circ/returns.pl" autocomplete="off" > > <div class="yui-u first"> > <fieldset> >@@ -655,9 +653,16 @@ $(document).ready(function () { > </td> > </tr> > [% END %] >- </table></div> >+ </table> > [% END %] >- >- >+</div> >+[% IF Koha.Preference('circSidebar') %] >+</div> >+<div class="yui-b noprint"> >+ [% INCLUDE 'circ-nav.inc' %] >+</div> >+[% END %] >+</div> >+</div> > </div> > [% INCLUDE 'intranet-bottom.inc' %] >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/selectbranchprinter.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/selectbranchprinter.tt >index faaa963..3d1fade 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/selectbranchprinter.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/selectbranchprinter.tt >@@ -1,3 +1,4 @@ >+[% USE Koha %] > [% INCLUDE 'doc-head-open.inc' %] > <title>Koha › Circulation › Set library</title> > [% INCLUDE 'doc-head-close.inc' %] >@@ -15,7 +16,7 @@ > › <a href="/cgi-bin/koha/circ/selectbranchprinter.pl">Set library</a> > </div> > >-<div id="doc" class="yui-t7"> >+[% IF Koha.Preference('circSidebar') %]<div id="doc3" class="yui-t2">[% ELSE %]<div id="doc" class="yui-t7">[% END %] > <div id="bd"> > <div id="yui-main"> > <div class="yui-b"> >@@ -94,5 +95,10 @@ Updated:<ul> > > </div> > </div> >- </div> >+[% IF Koha.Preference('circSidebar') %] >+<div class="yui-b noprint"> >+ [% INCLUDE 'circ-nav.inc' %] >+</div> >+[% END %] >+</div> > [% INCLUDE 'intranet-bottom.inc' %] >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/transferstoreceive.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/transferstoreceive.tt >index 6c2f533..05809ca 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/transferstoreceive.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/transferstoreceive.tt >@@ -1,3 +1,4 @@ >+[% USE Koha %] > [% USE KohaDates %] > [% USE Branches %] > [% INCLUDE 'doc-head-open.inc' %] >@@ -29,10 +30,11 @@ $(document).ready(function() { > > <div id="breadcrumbs"><a href="/cgi-bin/koha/mainpage.pl">Home</a> › <a href="/cgi-bin/koha/circ/circulation-home.pl">Circulation</a> › Transfers to your library</div> > >-<div id="doc" class="yui-t7"> >+[% IF Koha.Preference('circSidebar') %]<div id="doc3" class="yui-t2">[% ELSE %]<div id="doc" class="yui-t7">[% END %] > > <div id="bd"> > <div id="yui-main"> >+ [% IF Koha.Preference('circSidebar') %]<div class="yui-b">[% END %] > <div class="yui-g"> > > <h1>Transfers made to your library as of [% show_date %]</h1> >@@ -95,5 +97,11 @@ $(document).ready(function() { > > </div> > </div> >+[% IF Koha.Preference('circSidebar') %] >+</div> >+<div class="yui-b noprint"> >+ [% INCLUDE 'circ-nav.inc' %] >+</div> >+[% END %] > </div> > [% INCLUDE 'intranet-bottom.inc' %] >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/view_holdsqueue.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/view_holdsqueue.tt >index 541ddbd..cecb3de 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/view_holdsqueue.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/view_holdsqueue.tt >@@ -1,3 +1,4 @@ >+[% USE Koha %] > [% USE KohaDates %] > [% USE ItemTypes %] > [% USE Branches %] >@@ -52,10 +53,11 @@ $(document).ready(function() { > [% IF ( run_report ) %] › Results[% END %] > </div> > >-<div id="doc2" class="yui-t7"> >+[% IF Koha.Preference('circSidebar') %]<div id="doc3" class="yui-t2">[% ELSE %]<div id="doc2" class="yui-t7">[% END %] > > <div id="bd"> > <div id="yui-main"> >+ [% IF Koha.Preference('circSidebar') %]<div class="yui-b">[% END %] > <div class="yui-g"> > > <h1>Holds queue</h1> >@@ -220,5 +222,11 @@ $(document).ready(function() { > > </div> > </div> >+[% IF Koha.Preference('circSidebar') %] >+</div> >+<div class="yui-b noprint"> >+ [% INCLUDE 'circ-nav.inc' %] >+</div> >+[% END %] > </div> > [% INCLUDE 'intranet-bottom.inc' %] >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/waitingreserves.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/waitingreserves.tt >index 3183442..298398b 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/waitingreserves.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/waitingreserves.tt >@@ -1,3 +1,4 @@ >+[% USE Koha %] > [% USE KohaDates %] > [% USE Branches %] > [% INCLUDE 'doc-head-open.inc' %] >@@ -31,10 +32,11 @@ > › <a href="/cgi-bin/koha/circ/waitingreserves.pl">Holds awaiting pickup</a> > </div> > >-<div id="doc3" class="yui-t7"> >+[% IF Koha.Preference('circSidebar') %]<div id="doc3" class="yui-t2">[% ELSE %]<div id="doc" class="yui-t7">[% END %] > > <div id="bd"> > <div id="yui-main"> >+ [% IF Koha.Preference('circSidebar') %]<div class="yui-b">[% END %] > <div class="yui-g"> > > <h2>Holds awaiting pickup for your library on: [% show_date %] >@@ -193,5 +195,11 @@ > > </div> > </div> >+[% IF Koha.Preference('circSidebar') %] >+</div> >+<div class="yui-b noprint"> >+ [% INCLUDE 'circ-nav.inc' %] >+</div> >+[% END %] > </div> > [% INCLUDE 'intranet-bottom.inc' %] >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/offline_circ/list.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/offline_circ/list.tt >index 2500456..528c44c 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/offline_circ/list.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/offline_circ/list.tt >@@ -61,7 +61,7 @@ > > <div id="doc" class="yui-t7"> > >- <div id="bd"> >+ <div id="bd"> > > <h2>Offline circulation</h2> > >@@ -132,6 +132,6 @@ > > [% END %] > >- </div> >+</div> > > [% INCLUDE 'intranet-bottom.inc' %] >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/offline_circ/process_koc.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/offline_circ/process_koc.tt >index a126288..3a8bb1f 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/offline_circ/process_koc.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/offline_circ/process_koc.tt >@@ -127,7 +127,5 @@ function CheckForm(f) { > <div id="jobfailed" style="display:none"></div> > </form> > [% END %] >- >- > </div> > [% INCLUDE 'intranet-bottom.inc' %] >-- >1.7.10.4
<b>You cannot view the attachment while viewing its details because your browser does not support IFRAMEs. <a href="attachment.cgi?id=53072">View the attachment on a separate page</a>.</b>
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 16530
:
51555
|
51777
|
53008
|
53072
|
57869
|
57901
|
59807
|
59808