Attachment 42781 Details for Bug 13918 – Bug 13918 [QA Followup] - Remove reserves_count

[patch] Bug 13918 [QA Followup] - Remove reserves_count

Bug-13918-QA-Followup---Remove-reservescount.patch (text/plain), 2.94 KB, created by Kyle M Hall (khall) on 2015-09-22 13:03:32 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2015-09-22 13:03:32 UTC

Size: 2.94 KB

patch

obsolete

>From c1e95fd6cf2493d02c568ee1694b9ae68d7f1118 Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Mon, 24 Aug 2015 10:55:28 -0400
>Subject: [PATCH] Bug 13918 [QA Followup] - Remove reserves_count
>
>---
> .../opac-tmpl/bootstrap/en/modules/opac-user.tt    |    8 ++++----
> opac/opac-user.pl                                  |    1 -
> 2 files changed, 4 insertions(+), 5 deletions(-)
>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt
>index bb0d148..3a2cc76 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt
>@@ -111,7 +111,7 @@
>                                 [% IF ( BORROWER_INF.amountlessthanzero ) %]<li><a href="#opac-user-fines">Credits ([% BORROWER_INF.amountoutstanding %])</a></li>[% END %]
>                             [% END %]
>                             [% IF ( waiting_count ) %][% IF ( BORROWER_INF.atdestination ) %]<li><a href="#opac-user-waiting">Waiting ([% waiting_count %])</a></li>[% END %][% END %]
>-                            [% IF ( reserves_count ) %]<li><a href="#opac-user-holds">Holds ([% reserves_count %])</a></li>[% END %]
>+                            [% IF ( RESERVES.count ) %]<li><a href="#opac-user-holds">Holds ([% RESERVES.count %])</a></li>[% END %]
>                         </ul>
> 
>                         <div id="opac-user-checkouts">
>@@ -488,10 +488,10 @@
>                         [% END # /overdues_count %]
> 
> 
>-                        [% IF ( reserves_count ) %]
>+                        [% IF ( RESERVES.count ) %]
>                             <div id="opac-user-holds">
>                                 <table id="holdst" class="table table-bordered table-striped">
>-                                    <caption>Holds <span class="count">([% reserves_count %] total)</span></caption>
>+                                    <caption>Holds <span class="count">([% RESERVES.count %] total)</span></caption>
>                                     
>                                     <thead>
>                                         <tr>
>@@ -676,7 +676,7 @@
>                                 </div>
>                             [% END %]
>                         </div> 
>-                        [% END # / #reserves_count %]
>+                        [% END # / #RESERVES.count %]
>                     </div> 
>                 </div> 
>             </div> 
>diff --git a/opac/opac-user.pl b/opac/opac-user.pl
>index 37325a5..c91d57a 100755
>--- a/opac/opac-user.pl
>+++ b/opac/opac-user.pl
>@@ -281,7 +281,6 @@ my $reserves = Koha::Holds->search( { borrowernumber => $borrowernumber } );
> 
> $template->param(
>     RESERVES       => $reserves,
>-    reserves_count => $reserves->count(),
>     showpriority   => $show_priority,
>     WAITING        => $reserves->waiting()
> );
>-- 
>1.7.2.5

Actions: View | Diff | Splinter Review

Attachments on bug 13918: 37271 | 37272 | 40033 | 40035 | 41862 | 41863 | 41864 | 41865 | 41945 | 41946 | 41947 | 41948 | 42652 | 42653 | 42654 | 42655 | 42656 | 42745 | 42746 | 42747 | 42748 | 42749 | 42779 | 42780 | 42781 | 42782 | 42783 | 43429 | 43430 | 43431 | 43432 | 43433 | 43438 | 43502 | 43503 | 43504 | 43505 | 43506 | 43507 | 46130 | 46131 | 46132 | 46133 | 46134 | 46135