Attachment 35540 Details for Bug 13623 – Bug 13623 - Enhance html for holds queue viewer

[patch] Bug 13623 - Enhance html for holds queue viewer

Bug-13623---Enhance-html-for-holds-queue-viewer.patch (text/plain), 4.28 KB, created by Kyle M Hall (khall) on 2015-01-26 16:30:54 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2015-01-26 16:30:54 UTC

Size: 4.28 KB

patch

obsolete

>From d43f5be9a17f2f2e778bf74b54deb12f0daf0f98 Mon Sep 17 00:00:00 2001
>From: ByWater Staff <staff@bywatersolutions.com>
>Date: Mon, 26 Jan 2015 10:16:42 -0600
>Subject: [PATCH] Bug 13623 - Enhance html for holds queue viewer
>
>It would be very helpful from a css/scripting point of view if the
>author field in the holds queue report had a selectable class. In
>edition, it would be great if the biblionumber were easy to grab from a
>hidden div.
>
>Test Plan:
>1) Apply this patch
>2) Load the holds queue report ( view_holdsqueue.pl )
>3) Inspect an author name with your web browser ( or view the html )
>   a) Note the hq-author class of the wrapping div
>   b) Note the hidden hq-biblionumber div
>---
> .../intranet-tmpl/prog/en/css/staff-global.css     |    4 ++
> .../prog/en/modules/circ/view_holdsqueue.tt        |   31 ++++++++++++++------
> 2 files changed, 26 insertions(+), 9 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css b/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css
>index 18bae46..6d6bbe2 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css
>+++ b/koha-tmpl/intranet-tmpl/prog/en/css/staff-global.css
>@@ -2764,3 +2764,7 @@ span.onsite_checkout {
>     border-radius: 4px;
>     border : 1px solid #FFF2CE;
> }
>+
>+.hq-author {
>+    font-weight: bold;
>+}
>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 b247706..181731a 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
>@@ -67,15 +67,28 @@ $(document).ready(function() {
> 	</thead>
>      <tbody>[% FOREACH itemsloo IN itemsloop %]
>         <tr>
>-            <td class="hq-title"><p><a href="/cgi-bin/koha/catalogue/detail.pl?biblionumber=[% itemsloo.biblionumber %]"><strong>[% itemsloo.title |html %]</strong> [% IF ( itemsloo.subtitle ) %][% itemsloo.subtitle %][% END %]</a></p>
>-                         <p><strong>[% itemsloo.author %]</strong>
>-                       <div class="hq-pubdata">  [% IF ( itemsloo.publishercode ) %][% itemsloo.publishercode %][% END %]
>-                                                                                 [% IF ( itemsloo.publicationyear ) %], [% itemsloo.publicationyear %]
>-                                                                                 [% ELSIF ( itemsloo.copyrightdate ) %], [% itemsloo.copyrightdate %][% END %]
>-                                         [% IF ( itemsloo.pages ) %]: [% itemsloo.pages %] [% END %]
>-                                         [% IF ( itemsloo.size ) %][% itemsloo.size %][% END %] [% IF ( itemsloo.isbn ) %]ISBN: [% itemsloo.isbn %][% END %]</p>
>-			</div>
>-			</td>
>+            <td class="hq-title">
>+                <p><a href="/cgi-bin/koha/catalogue/detail.pl?biblionumber=[% itemsloo.biblionumber %]"><strong>[% itemsloo.title | html %]</strong> [% IF ( itemsloo.subtitle ) %][% itemsloo.subtitle %][% END %]</a></p>
>+                <p>
>+                    <div class="hq-biblionumber content_hidden">[% itemsloo.biblionumber %]</div>
>+                    <div class="hq-author">[% itemsloo.author %]</div>
>+                    <div class="hq-pubdata">
>+                        [% IF ( itemsloo.publishercode ) %][% itemsloo.publishercode %][% END %]
>+
>+                        [% IF ( itemsloo.publicationyear ) %]
>+                            , [% itemsloo.publicationyear %]
>+                        [% ELSIF ( itemsloo.copyrightdate ) %]
>+                            , [% itemsloo.copyrightdate %]
>+                        [% END %]
>+                        
>+                        [% IF ( itemsloo.pages ) %]: [% itemsloo.pages %] [% END %]
>+
>+                        [% IF ( itemsloo.size ) %][% itemsloo.size %][% END %]
>+
>+                        [% IF ( itemsloo.isbn ) %]ISBN: [% itemsloo.isbn %][% END %]
>+                    </div>
>+                </p>
>+            </td>
>             <td class="hq-collection">[% AuthorisedValues.GetByCode( 'CCODE', itemsloo.ccode ) %]</td>
>             <td class="hq-itemtype">[% ItemTypes.GetDescription( itemsloo.itype ) %]</td>
>             <td class="hq-callnumber">[% IF ( itemsloo.location ) %]<em>[% AuthorisedValues.GetByCode( 'LOC', itemsloo.location ) %]</em> [% END %][% itemsloo.itemcallnumber %]</td>
>-- 
>1.7.2.5

Actions: View | Diff | Splinter Review

Attachments on bug 13623: 35540 | 35541 | 37845 | 37849 | 38143 | 38144