[Patches] [PATCH] bug_7140: Added item description to complement icon to search result and

koha-patchbot at kohaaloha.com koha-patchbot at kohaaloha.com
Fri Nov 4 19:56:15 NZDT 2011


From: Srdjan Jankovic <srdjan at catalyst.net.nz>
Date: Wed, 2 Nov 2011 19:30:03 +1300
Subject: [PATCH] bug_7140: Added item description to complement icon to search result and
 biblio detail pages

---
 C4/Search.pm                                       |    5 +-
 .../prog/en/modules/catalogue/detail.tt            |   14 ++---
 .../prog/en/modules/catalogue/results.tt           |   70 +++++++++++++-------
 3 files changed, 55 insertions(+), 34 deletions(-)

diff --git a/C4/Search.pm b/C4/Search.pm
index 98db887..2aba42e 100644
--- a/C4/Search.pm
+++ b/C4/Search.pm
@@ -1622,6 +1622,7 @@ sub searchResults {
             foreach my $code ( keys %subfieldstosearch ) {
                 $item->{$code} = $field->subfield( $subfieldstosearch{$code} );
             }
+            $item->{description} = $itemtypes{ $item->{itype} }{description};
 
 	        # Hidden items
 	        my @items = ($item);
@@ -1650,6 +1651,7 @@ sub searchResults {
 				$onloan_items->{$key}->{branchname} = $item->{branchname};
 				$onloan_items->{$key}->{location} = $shelflocations->{ $item->{location} };
 				$onloan_items->{$key}->{itemcallnumber} = $item->{itemcallnumber};
+				$onloan_items->{$key}->{description} = $item->{description};
 				$onloan_items->{$key}->{imageurl} = getitemtypeimagelocation( $search_context, $itemtypes{ $item->{itype} }->{imageurl} );
                 # if something's checked out and lost, mark it as 'long overdue'
                 if ( $item->{itemlost} ) {
@@ -1733,6 +1735,7 @@ sub searchResults {
 					$other_items->{$key}->{notforloan} = GetAuthorisedValueDesc('','',$item->{notforloan},'','',$notforloan_authorised_value) if $notforloan_authorised_value;
 					$other_items->{$key}->{count}++ if $item->{$hbranch};
 					$other_items->{$key}->{location} = $shelflocations->{ $item->{location} };
+					$other_items->{$key}->{description} = $item->{description};
 					$other_items->{$key}->{imageurl} = getitemtypeimagelocation( $search_context, $itemtypes{ $item->{itype} }->{imageurl} );
                 }
                 # item is available
@@ -1740,7 +1743,7 @@ sub searchResults {
                     $can_place_holds = 1;
                     $available_count++;
 					$available_items->{$prefix}->{count}++ if $item->{$hbranch};
-					foreach (qw(branchname itemcallnumber hideatopac)) {
+					foreach (qw(branchname itemcallnumber hideatopac description)) {
                     	$available_items->{$prefix}->{$_} = $item->{$_};
 					}
 					$available_items->{$prefix}->{location} = $shelflocations->{ $item->{location} };
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 b1e225f..53aa03d 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt
@@ -255,16 +255,12 @@ function verify_images() {
             [% FOREACH itemloo IN itemloop %]
                 <tr>
                     [% IF ( item_level_itypes ) %]
-                      <td class="itype">
-                        [% IF ( noItemTypeImages ) %]
-                          [% itemloo.description %]
-                        [% ELSE %]
-                          [% IF ( itemloo.imageurl ) %]
+                    <td class="itype">
+                        [% IF !noItemTypeImages && itemloo.imageurl %]
                             <img src="[% itemloo.imageurl %]" alt="[% itemloo.description %]" title="[% itemloo.description %]" />
-                          [% ELSE %]
-                            [% itemloo.description %]
-                          [% END %]
-                        [% END %]</td>
+                        [% END %]
+                        [% itemloo.description %]
+                    </td>
                     [% END %]
                     <td class="location">[% UNLESS ( singlebranchmode ) %][% itemloo.branchname %] [% END %]</td>
                     <td class="homebranch">[% itemloo.homebranch %]<span class="shelvingloc">[% itemloo.location %]</span> </td>
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/results.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/results.tt
index 2bed51c..cf283dc 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/results.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/results.tt
@@ -470,17 +470,19 @@ YAHOO.util.Event.onContentReady("searchheader", function () {
                                         [% END %]
                                     <p>
                                     [% UNLESS ( item_level_itypes ) %]
-                                    [% UNLESS ( noItemTypeImages ) %][% IF ( SEARCH_RESULT.imageurl ) %]
-                                    <img src="[% SEARCH_RESULT.imageurl %]" title="[% SEARCH_RESULT.description %]" style="float: left; margin: .1em;" alt="" />
-                                    [% END %][% END %]
+                                    [% IF !noItemTypeImages && SEARCH_RESULT.imageurl %]
+                                    <img src="[% SEARCH_RESULT.imageurl %]" title="[% SEARCH_RESULT.description %]" style="float: left; margin: .1em;" alt="[% SEARCH_RESULT.description %]" />
+                                    [% END %]
+                                    [% SEARCH_RESULT.description %]
                                     [% END %]
                                     [% SEARCH_RESULT.summary %]</p>
                                 [% ELSE %]
                                     <p>
                                     [% UNLESS ( item_level_itypes ) %]
-                                    [% UNLESS ( noItemTypeImages ) %][% IF ( SEARCH_RESULT.imageurl ) %]
+                                    [% IF !noItemTypeImages && SEARCH_RESULT.imageurl %]
                                     <img src="[% SEARCH_RESULT.imageurl %]" title="[% SEARCH_RESULT.description %]" style="float: left; margin: .1em;" alt="" />
-                                    [% END %][% END %]
+                                    [% END %]
+                                    [% SEARCH_RESULT.description %]
                                     [% END %]
 
                                         [% IF ( SEARCH_RESULT.author ) %]
@@ -532,41 +534,56 @@ YAHOO.util.Event.onContentReady("searchheader", function () {
                                     <ul>
                                     [% FOREACH available_items_loo IN SEARCH_RESULT.available_items_loop %]
 
-                                        [% IF ( noItemTypeImages ) %]<li>[% ELSE %][% IF ( item_level_itypes ) %][% IF ( available_items_loo.imageurl ) %]<li style="list-style: none; list-style-type: none;"><img src="[% available_items_loo.imageurl %]" title="[% available_items_loo.description %]" alt="[% available_items_loo.description %]" />[% ELSE %]<li>[% END %][% ELSE %]<li>[% END %][% END %]
+                                        [% IF item_level_itypes && !noItemTypeImages && available_items_loo.imageurl %]
+                                        <li style="list-style: none; list-style-type: none;">
+                                          <img src="[% available_items_loo.imageurl %]" title="[% available_items_loo.description %]" alt="[% available_items_loo.description %]" />
+                                        [% ELSE %]
+                                        <li>
+                                        [% END %]
                                         [% IF ( available_items_loo.branchname ) %][% available_items_loo.branchname %][% END %]
                                         [% IF ( available_items_loo.location ) %][% available_items_loo.location %][% END %]
                                         [% IF ( available_items_loo.itemcallnumber ) %][<a href="/cgi-bin/koha/catalogue/search.pl?idx=callnum&q=[% available_items_loo.itemcallnumber |url %]">[% available_items_loo.itemcallnumber %]</a>][% END %]
-                                        ([% available_items_loo.count %])</li>
-                                    [% END %]</ul>
+                                        ([% available_items_loo.count %])
+                                        [% IF item_level_itypes && available_items_loo.description %]
+                                        <br/>[% available_items_loo.description %]
+                                        [% END %]
+                                        </li>
+                                    [% END %]
+                                    </ul>
                                     [% END %]
 
                                    [% IF ( SEARCH_RESULT.onloancount ) %]
                                    <span class="status">[% SEARCH_RESULT.onloancount %] on loan:</span>
                                     <ul>
                                     [% FOREACH onloan_items_loo IN SEARCH_RESULT.onloan_items_loop %]
-                                       [% IF ( noItemTypeImages ) %]<li>[% ELSE %][% IF ( item_level_itypes ) %]
-                                        [% IF ( onloan_items_loo.imageurl ) %]
-                                        <li style="list-style: none; list-style-type: none;"><img src="[% onloan_items_loo.imageurl %]" title="[% onloan_items_loo.description %]" alt="[% onloan_items_loo.description %]" />
-                                        [% ELSE %]<li>[% END %]
-                                        [% ELSE %]<li>[% END %][% END %]
-
+                                        [% IF item_level_itypes && !noItemTypeImages && onloan_items_loo.imageurl %]
+                                        <li style="list-style: none; list-style-type: none;">
+                                          <img src="[% onloan_items_loo.imageurl %]" title="[% onloan_items_loo.description %]" alt="[% onloan_items_loo.description %]" />
+                                        [% ELSE %]
+                                        <li>
+                                        [% END %]
                                         [% IF ( onloan_items_loo.branchname ) %][% onloan_items_loo.branchname %][% END %]
                                         [% IF ( onloan_items_loo.location ) %][% onloan_items_loo.location %][% END %]
                                         [% IF ( onloan_items_loo.itemcallnumber ) %][<a href="/cgi-bin/koha/catalogue/search.pl?idx=callnum&q=[% onloan_items_loo.itemcallnumber |url %]">[% onloan_items_loo.itemcallnumber %]</a>][% END %]
-                                        ([% onloan_items_loo.count %][% IF ( onloan_items_loo.longoverdue ) %], [% onloan_items_loo.longoverdue %] long overdue[% END %]) date due: [% onloan_items_loo.due_date %]</li>
-                                    [% END %]</ul>
+                                        ([% onloan_items_loo.count %][% IF ( onloan_items_loo.longoverdue ) %], [% onloan_items_loo.longoverdue %] long overdue[% END %]) date due: [% onloan_items_loo.due_date %]
+                                        [% IF item_level_itypes && onloan_items_loo.description %]
+                                        <br/>[% onloan_items_loo.description %]
+                                        [% END %]
+                                        </li>
+                                    [% END %]
+                                    </ul>
                                     [% END %]
 
                                     [% IF ( SEARCH_RESULT.othercount ) %]
                                     <span class="unavailable">[% SEARCH_RESULT.othercount %] unavailable:</span>
                                     <ul>
                                     [% FOREACH other_items_loo IN SEARCH_RESULT.other_items_loop %]
-                                        [% IF ( noItemTypeImages ) %]<li>[% ELSE %][% IF ( item_level_itypes ) %]
-                                        [% IF ( other_items_loo.imageurl ) %]
-                                        <li style="list-style: none; list-style-type: none;"><img src="[% other_items_loo.imageurl %]" title="[% other_items_loo.description %]" alt="[% other_items_loo.description %]" />
-                                        [% ELSE %]<li>[% END %]
-                                        [% ELSE %]<li>[% END %][% END %]
-
+                                        [% IF item_level_itypes && !noItemTypeImages && other_items_loo.imageurl %]
+                                        <li style="list-style: none; list-style-type: none;">
+                                          <img src="[% other_items_loo.imageurl %]" title="[% other_items_loo.description %]" alt="[% other_items_loo.description %]" />
+                                        [% ELSE %]
+                                        <li>
+                                        [% END %]
                                         [% IF ( other_items_loo.branchname ) %][% other_items_loo.branchname %][% END %]
                                         [% IF ( other_items_loo.location ) %][% other_items_loo.location %][% END %]
                                         [% IF ( other_items_loo.itemcallnumber ) %][<a href="/cgi-bin/koha/catalogue/search.pl?idx=callnum&q=[% other_items_loo.itemcallnumber |url %]">[% other_items_loo.itemcallnumber %]</a>][% END %]
@@ -576,8 +593,13 @@ YAHOO.util.Event.onContentReady("searchheader", function () {
                                         [% IF ( other_items_loo.intransit ) %](In transit)[% END %]
                                         [% IF ( other_items_loo.onhold ) %](On hold)[% END %]
                                         [% IF ( other_items_loo.notforloan ) %][% other_items_loo.notforloan %][% END %]
-                                        ([% other_items_loo.count %])</li>
-                                    [% END %]</ul>
+                                        ([% other_items_loo.count %])
+                                        [% IF item_level_itypes && other_items_loo.description %]
+                                        <br/>[% other_items_loo.description %]
+                                        [% END %]
+                                        </li>
+                                    [% END %]
+                                    </ul>
                                     [% END %]
                                     [% ELSE %]
                                     [% IF ( SEARCH_RESULT.ALTERNATEHOLDINGS.count ) %]
-- 
1.6.5




More information about the Patches mailing list