[Koha-patches] [PATCH 1/1] Bug 8199 : Replace table in opac-results with div

Jonathan Druart jonathan.druart at biblibre.com
Wed Jun 6 16:37:01 CEST 2012


Test plan:
launch a query from your opac search form and compare results before and
after applying this patch.

Normally, You don't show a lot of differences.
---
 koha-tmpl/opac-tmpl/prog/en/css/opac.css           |   44 ++
 .../opac-tmpl/prog/en/modules/opac-results.tt      |  520 +++++++++++--------
 2 files changed, 345 insertions(+), 219 deletions(-)

diff --git a/koha-tmpl/opac-tmpl/prog/en/css/opac.css b/koha-tmpl/opac-tmpl/prog/en/css/opac.css
index ea538aa..0bf2421 100644
--- a/koha-tmpl/opac-tmpl/prog/en/css/opac.css
+++ b/koha-tmpl/opac-tmpl/prog/en/css/opac.css
@@ -1169,6 +1169,50 @@ a.cancel {
 	margin : 0 0 0 0;
 }
 
+.searchresults .controls {
+    background-color : #EEE;
+    border : 1px solid #E8E8E8;
+    vertical-align : middle;
+    padding : 3px 3px 5px 5px;
+}
+
+.searchresults .controls {
+    font-size: 90%;
+}
+
+.searchresults .controls img {
+    vertical-align: middle;
+}
+
+.searchresults .controls select {
+    font-size: 90%;
+}
+
+.searchresults .controls label {
+    font-weight : bold;
+    margin-left : .5em;
+}
+
+.result {
+    overflow: hidden;
+    border-bottom: 1px solid #DDDDDD;
+    padding: 5px 10px 0;
+}
+.all_results>div.odd {
+    background-color: #F3F3F3;
+}
+
+.result > input{
+    float: left;
+}
+.result > span {
+    float: left;
+    margin-left: 5px;
+}
+.result > span.image {
+    float: right;
+}
+
 #CheckAll, #CheckNone {
 	font-weight : normal;
 	margin : 0 .5em;
diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-results.tt b/koha-tmpl/opac-tmpl/prog/en/modules/opac-results.tt
index 67b1686..cd1b873 100644
--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-results.tt
+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-results.tt
@@ -285,7 +285,6 @@ $(document).ready(function(){
 </div>
 [% END %]
 
-<!-- Search Results Table -->
 [% IF ( total ) %]
 [% IF ( scan ) %]
     <h1>Scan index:</h1>
@@ -345,7 +344,7 @@ $(document).ready(function(){
 [% ELSE %]
 
     <div class="searchresults">
-    <form action="/cgi-bin/koha/opac-search.pl" method="get" name="bookbag_form" id="bookbag_form">
+      <form action="/cgi-bin/koha/opac-search.pl" method="get" name="bookbag_form" id="bookbag_form">
         [% IF ( searchdesc ) %]
         [% FOREACH QUERY_INPUT IN QUERY_INPUTS %]
         <input type="hidden" name="[% QUERY_INPUT.input_name |html %]" value="[% QUERY_INPUT.input_value |html %]"/>
@@ -355,79 +354,92 @@ $(document).ready(function(){
         [% END %]
         [% END %]
 
-        <!-- TABLE RESULTS START -->
-        <table>
-        <tr><td colspan="5" class="resultscontrol">
-        <div class="resort"> <select id="sort_by" name="sort_by"> [% INCLUDE 'resort_form.inc' %] </select> <input type="submit" class="submit clearfix" id="sortsubmit" value="Go" /></div>
-        <div class="cartlist">
-            <!-- checkall, clearall are now needed for placehold -->
-            <span class="checkall"></span>
-            <span class="clearall"></span>
-            [% IF ( OpacHighlightedWords ) %]
-                <a href="#" class="highlight_toggle" id="highlight_toggle_off">Unhighlight</a>
-                <a href="#" class="highlight_toggle" id="highlight_toggle_on">Highlight</a>
-            [% END %]
-            [% IF ( opacbookbag || virtualshelves ) %]
-                <span class="addto"></span>
-            [% END %]
-            <span id="placehold"><!-- input class="submit" type="submit" value="Place Hold"/ --></span>
-            <div id="holdDetails"></div>
-
-            [% IF ( opacuserlogin ) %][% IF ( loggedinusername ) %][% IF ( TagsEnabled ) %]
-                        [% IF ( TagsInputOnList ) %]
-                            <span id="tagsel_span"></span>
-                          [% IF ( loggedinusername ) %]
-                            <span id="tagsel_form" style="display:none">
-                              <label for="tagsel_new">New tag:</label>
-                              <input name="tagsel_new" id="tagsel_new" maxlength="100" />
-                              <input id="tagsel_button" name="tagsel_button" class="input tagsel_button" title="tagsel_button" type="submit" value="Add" />
-                              <a href="#" id="tagsel_cancel">Cancel</a>
-                            </span>
-                            <span id="tagsel_status" class="tagsel_tatus" style="display:none;">
-                              Tag status here.
-                            </span>
+        <div class="controls">
+          <div class="resort">
+            <select id="sort_by" name="sort_by"> [% INCLUDE 'resort_form.inc' %] </select> <input type="submit" class="submit clearfix" id="sortsubmit" value="Go" />
+          </div>
+          <div class="cartlist">
+              <!-- checkall, clearall are now needed for placehold -->
+              <span class="checkall"></span>
+              <span class="clearall"></span>
+              [% IF ( OpacHighlightedWords ) %]
+                  <a href="#" class="highlight_toggle" id="highlight_toggle_off">Unhighlight</a>
+                  <a href="#" class="highlight_toggle" id="highlight_toggle_on">Highlight</a>
+              [% END %]
+              [% IF ( opacbookbag || virtualshelves ) %]
+                  <span class="addto"></span>
+              [% END %]
+              <span id="placehold"><!-- input class="submit" type="submit" value="Place Hold"/ --></span>
+              <div id="holdDetails"></div>
+
+              [% IF ( opacuserlogin ) %][% IF ( loggedinusername ) %][% IF ( TagsEnabled ) %]
+                          [% IF ( TagsInputOnList ) %]
+                              <span id="tagsel_span"></span>
+                            [% IF ( loggedinusername ) %]
+                              <span id="tagsel_form" style="display:none">
+                                <label for="tagsel_new">New tag:</label>
+                                <input name="tagsel_new" id="tagsel_new" maxlength="100" />
+                                <input id="tagsel_button" name="tagsel_button" class="input tagsel_button" title="tagsel_button" type="submit" value="Add" />
+                                <a href="#" id="tagsel_cancel">Cancel</a>
+                              </span>
+                              <span id="tagsel_status" class="tagsel_tatus" style="display:none;">
+                                Tag status here.
+                              </span>
+                            [% END %]
                           [% END %]
-                        [% END %]
-                [% END %][% END %][% END %]
+              [% END %][% END %][% END %]
+          </div>
         </div>
 
-        </td></tr>
-            <!-- Actual Search Results -->
-            [% FOREACH SEARCH_RESULT IN SEARCH_RESULTS %]
+        <div class="all_results">
+          <!-- Actual Search Results -->
+          [% FOREACH SEARCH_RESULT IN SEARCH_RESULTS %]
             [% UNLESS ( loop.odd ) %]
-                <tr class="highlight">
+              <div class="result odd">
             [% ELSE %]
-                <tr>
+              <div class="result">
             [% END %]
+              [% IF ( opacbookbag ) %]
+                <input type="checkbox" id="bib[% SEARCH_RESULT.biblionumber %]" name="biblionumber" value="[% SEARCH_RESULT.biblionumber %]" /> <label for="bib[% SEARCH_RESULT.biblionumber %]"></label>
+              [% ELSE %]
+                [% IF ( virtualshelves ) %]
+                  <input type="checkbox" id="bib[% SEARCH_RESULT.biblionumber %]" name="biblionumber" value="[% SEARCH_RESULT.biblionumber %]" /> <label for="bib[% SEARCH_RESULT.biblionumber %]"></label>
+                [% ELSE %]
+                  [% IF ( RequestOnOpac ) %]
+                    [% UNLESS ( SEARCH_RESULT.norequests ) %]
+                      [% IF ( opacuserlogin ) %]
+                        <input type="checkbox" id="bib[% SEARCH_RESULT.biblionumber %]" name="biblionumber" value="[% SEARCH_RESULT.biblionumber %]" /> <label for="bib[% SEARCH_RESULT.biblionumber %]"></label>
+                      [% END %]
+                    [% END %]
+                  [% END %]
+                [% END %]
+              [% END %]</td>
+              [% UNLESS suppress_result_number %]
+                <span class="select selectcol">[% SEARCH_RESULT.result_number %].</span>
+              [% END %]
 
-                <td class="select selectcol">[% IF ( opacbookbag ) %]<input type="checkbox" id="bib[% SEARCH_RESULT.biblionumber %]" name="biblionumber" value="[% SEARCH_RESULT.biblionumber %]" /> <label for="bib[% SEARCH_RESULT.biblionumber %]"></label>[% ELSE %]
-[% IF ( virtualshelves ) %]<input type="checkbox" id="bib[% SEARCH_RESULT.biblionumber %]" name="biblionumber" value="[% SEARCH_RESULT.biblionumber %]" /> <label for="bib[% SEARCH_RESULT.biblionumber %]"></label>[% ELSE %]
-[% IF ( RequestOnOpac ) %][% UNLESS ( SEARCH_RESULT.norequests ) %][% IF ( opacuserlogin ) %]<input type="checkbox" id="bib[% SEARCH_RESULT.biblionumber %]" name="biblionumber" value="[% SEARCH_RESULT.biblionumber %]" /> <label for="bib[% SEARCH_RESULT.biblionumber %]"></label>[% END %][% END %][% END %][% END %][% END %]</td>
-                <td class="select selectcol">[% UNLESS suppress_result_number %][% SEARCH_RESULT.result_number %].[% END %]</td>
-
-                [% UNLESS ( item_level_itypes ) %]
+              [% UNLESS ( item_level_itypes ) %]
                 [% UNLESS ( noItemTypeImages ) %]
-
-                <td class="itypecol">
-                [% IF ( SEARCH_RESULT.imageurl ) %]
-                <img src="[% SEARCH_RESULT.imageurl %]" title="[% SEARCH_RESULT.description %]" alt="[% SEARCH_RESULT.description %]" />
-                [% END %]
-                [% IF ( SEARCH_RESULT.score_avg ) %]
-                    <img src="[% themelang %]/../images/bonus.png" title="bonus" style="max-height: 35px;"/>
-                [% END %]
-                </td>
-                [% END %]
+                  <span class="itypecol">
+                    [% IF ( SEARCH_RESULT.imageurl ) %]
+                      <img src="[% SEARCH_RESULT.imageurl %]" title="[% SEARCH_RESULT.description %]" alt="[% SEARCH_RESULT.description %]" />
+                    [% END %]
+                    [% IF ( SEARCH_RESULT.score_avg ) %]
+                      <img src="[% themelang %]/../images/bonus.png" title="bonus" style="max-height: 35px;"/>
+                    [% END %]
+                  </span>
                 [% END %]
+              [% END %]
 
-                [% IF ( SEARCH_RESULT.AuthorisedValueImages ) %]
-                <td class="itypecol">
-                [% FOREACH authorised_value_image IN SEARCH_RESULT.authorised_value_images %]
-                <img src="[% authorised_value_image.imageurl %]" alt="[% authorised_value_image.label %]" title="[% authorised_value_image.label %]">
-                [% END %]
-                </td>
-                [% END %]
+              [% IF ( SEARCH_RESULT.AuthorisedValueImages ) %]
+                <span class="itypecol">
+                  [% FOREACH authorised_value_image IN SEARCH_RESULT.authorised_value_images %]
+                    <img src="[% authorised_value_image.imageurl %]" alt="[% authorised_value_image.label %]" title="[% authorised_value_image.label %]">
+                  [% END %]
+                </span>
+              [% END %]
 
-                <td>
+              <span class="td">
                 [% IF ( COinSinOPACResults ) %]
                     [% IF ( SEARCH_RESULT.coins ) %]
                         <!-- COinS / Openurl --><span class="Z3988" title="[% SEARCH_RESULT.coins %]"></span>
@@ -435,204 +447,274 @@ $(document).ready(function(){
                 [% END %]
 
                 [% IF ( OPACXSLTResultsDisplay ) %]
-                [% SEARCH_RESULT.XSLTResultsRecord %]
+                  [% SEARCH_RESULT.XSLTResultsRecord %]
                 [% ELSE %]
-                [% IF ( BiblioDefaultViewmarc ) %]<a class="title" href="/cgi-bin/koha/opac-MARCdetail.pl?biblionumber=[% SEARCH_RESULT.biblionumber |url %]" title="View details for this title">
-                                [% ELSE %]
-                                [% IF ( BiblioDefaultViewisbd ) %]<a class="title" href="/cgi-bin/koha/opac-ISBDdetail.pl?biblionumber=[% SEARCH_RESULT.biblionumber |url %]" title="View details for this title">
-                                [% ELSE %]<a class="title" href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% SEARCH_RESULT.biblionumber |url %]" title="View details for this title">
-                                [% END %]
-                                [% END %]
-                                [% IF ( SEARCH_RESULT.title ) %][% SEARCH_RESULT.title |html %][% ELSE %]No title[% END %] [% FOREACH subtitl IN SEARCH_RESULT.subtitle %], [% subtitl.subfield|html %][% END %]</a>
-                                [% IF ( SEARCH_RESULT.author ) %]by <a href="/cgi-bin/koha/opac-search.pl?q=au:[% SEARCH_RESULT.author |url %]" title="Search for works by this author" class="author">[% SEARCH_RESULT.author %]</a>
-                                [% ELSE %]&nbsp;
-                                [% END %]
-                <span class="results_summary"><span class="label">Publication:</span>
-                        [% IF ( SEARCH_RESULT.place ) %][% SEARCH_RESULT.place %] [% END %][% IF ( SEARCH_RESULT.publishercode ) %][% SEARCH_RESULT.publishercode|html %][% END %][% IF ( SEARCH_RESULT.publicationyear ) %] [% SEARCH_RESULT.publicationyear %]
-                    [% ELSE %][% IF ( SEARCH_RESULT.copyrightdate ) %] [% SEARCH_RESULT.copyrightdate %][% END %][% END %]
-                        [% IF ( SEARCH_RESULT.pages ) %]. [% SEARCH_RESULT.pages %][% END %]
-                        [% IF ( SEARCH_RESULT.notes ) %], [% SEARCH_RESULT.notes|html %][% END %]
-                        [% IF ( SEARCH_RESULT.size ) %] [% SEARCH_RESULT.size %][% END %]
-                        [% IF ( SEARCH_RESULT.timestamp ) %] <i>(modified on [% SEARCH_RESULT.timestamp %])</i>[% END %]
-                </span>
-                [% IF ( SEARCH_RESULT.summary ) %]
-                <span class="result_summary">
-                    [% SEARCH_RESULT.summary %]
-                </span>
-                [% END %]
-                [% IF ( SEARCH_RESULT.copyrightdate ) %]<span class="results_summary"><span class="label">Date:</span>[% SEARCH_RESULT.copyrightdate %]</span>[% END %]
-
-                <span class="results_summary">
-                <span class="label">Availability:</span>
-                    [% IF ( SEARCH_RESULT.available_items_loop.0 ) %]
-                    <span class="available"><strong>Copies available:</strong>
-                    [% FOREACH available_items_loo IN SEARCH_RESULT.available_items_loop %]
-                    [% UNLESS ( available_items_loo.hideatopac ) %]
-                    [% IF ( singleBranchMode ) %]
-                        [% available_items_loo.location %]
+                  [% IF ( BiblioDefaultViewmarc ) %]
+                    <a class="title" href="/cgi-bin/koha/opac-MARCdetail.pl?biblionumber=[% SEARCH_RESULT.biblionumber |url %]" title="View details for this title">
+                  [% ELSE %]
+                    [% IF ( BiblioDefaultViewisbd ) %]
+                      <a class="title" href="/cgi-bin/koha/opac-ISBDdetail.pl?biblionumber=[% SEARCH_RESULT.biblionumber |url %]" title="View details for this title">
                     [% ELSE %]
-                        [% available_items_loo.branchname %]
+                      <a class="title" href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% SEARCH_RESULT.biblionumber |url %]" title="View details for this title">
                     [% END %]
-
-                    [% IF ( OPACItemsResultsDisplay ) %]
-                        [% UNLESS ( singleBranchMode ) %][% available_items_loo.location %][% END %]
-                        [% IF ( available_items_loo.itemcallnumber ) %][<a href="/cgi-bin/koha/opac-search.pl?q=callnum:[% available_items_loo.itemcallnumber |url %]">[% available_items_loo.itemcallnumber %]</a>][% END %]
-                    [% END %]
-                    ([% available_items_loo.count %]),
+                  [% END %]
+                  [% IF ( SEARCH_RESULT.title ) %]
+                    [% SEARCH_RESULT.title |html %]
+                  [% ELSE %]
+                    No title
+                  [% END %]
+                  [% FOREACH subtitl IN SEARCH_RESULT.subtitle %], [% subtitl.subfield|html %][% END %]</a>
+                  [% IF ( SEARCH_RESULT.author ) %]
+                    by <a href="/cgi-bin/koha/opac-search.pl?q=au:[% SEARCH_RESULT.author |url %]" title="Search for works by this author" class="author">[% SEARCH_RESULT.author %]</a>
+                  [% ELSE %]
+                    &nbsp;
+                  [% END %]
+                  <span class="results_summary">
+                    <span class="label">Publication:</span>
+                    [% IF ( SEARCH_RESULT.place ) %]
+                      [% SEARCH_RESULT.place %]
                     [% END %]
+                    [% IF ( SEARCH_RESULT.publishercode ) %]
+                      [% SEARCH_RESULT.publishercode|html %]
                     [% END %]
-                    </span>
+                    [% IF ( SEARCH_RESULT.publicationyear ) %]
+                      [% SEARCH_RESULT.publicationyear %]
                     [% ELSE %]
-                    [% IF ( SEARCH_RESULT.ALTERNATEHOLDINGS ) %]
-                    [% FOREACH ALTERNATEHOLDING IN SEARCH_RESULT.ALTERNATEHOLDINGS %]
-                        &nbsp;<span id="alternateholdings">[% ALTERNATEHOLDING.holding %]</span>,
+                      [% IF ( SEARCH_RESULT.copyrightdate ) %]
+                        [% SEARCH_RESULT.copyrightdate %]
+                      [% END %]
                     [% END %]
+                    [% IF ( SEARCH_RESULT.pages ) %]. [% SEARCH_RESULT.pages %][% END %]
+                    [% IF ( SEARCH_RESULT.notes ) %], [% SEARCH_RESULT.notes|html %][% END %]
+                    [% IF ( SEARCH_RESULT.size ) %] [% SEARCH_RESULT.size %][% END %]
+                    [% IF ( SEARCH_RESULT.timestamp ) %] <i>(modified on [% SEARCH_RESULT.timestamp %])</i>[% END %]
+                  </span>
+                  [% IF ( SEARCH_RESULT.summary ) %]
+                    <span class="result_summary">
+                      [% SEARCH_RESULT.summary %]
+                    </span>
+                  [% END %]
+                  [% IF ( SEARCH_RESULT.copyrightdate ) %]
+                    <span class="results_summary"><span class="label">Date:</span>[% SEARCH_RESULT.copyrightdate %]</span>
+                  [% END %]
+
+                  <span class="results_summary">
+                    <span class="label">Availability:</span>
+                    [% IF ( SEARCH_RESULT.available_items_loop.0 ) %]
+                      <span class="available">
+                        <strong>Copies available:</strong>
+                        [% FOREACH available_items_loo IN SEARCH_RESULT.available_items_loop %]
+                          [% UNLESS ( available_items_loo.hideatopac ) %]
+                            [% IF ( singleBranchMode ) %]
+                              [% available_items_loo.location %]
+                            [% ELSE %]
+                              [% available_items_loo.branchname %]
+                            [% END %]
+
+                            [% IF ( OPACItemsResultsDisplay ) %]
+                              [% UNLESS ( singleBranchMode ) %][% available_items_loo.location %][% END %]
+                              [% IF ( available_items_loo.itemcallnumber ) %]
+                                [<a href="/cgi-bin/koha/opac-search.pl?q=callnum:[% available_items_loo.itemcallnumber |url %]">[% available_items_loo.itemcallnumber %]</a>]
+                              [% END %]
+                            [% END %]
+                            ([% available_items_loo.count %]),
+                          [% END %]
+                        [% END %]
+                      </span>
                     [% ELSE %]
+                      [% IF ( SEARCH_RESULT.ALTERNATEHOLDINGS ) %]
+                        [% FOREACH ALTERNATEHOLDING IN SEARCH_RESULT.ALTERNATEHOLDINGS %]
+                          &nbsp;<span id="alternateholdings">[% ALTERNATEHOLDING.holding %]</span>,
+                        [% END %]
+                      [% ELSE %]
                         <span class="unavailable">No items available:</span>
-                    [% END %]
+                      [% END %]
                     [% END %]
                     <span class="unavailable">
-                    [% IF ( SEARCH_RESULT.onloancount ) %] Checked out ([% SEARCH_RESULT.onloancount %]), [% END %]
-                    [% IF ( SEARCH_RESULT.wthdrawncount ) %] Withdrawn ([% SEARCH_RESULT.wthdrawncount %]), [% END %]
-                    [% UNLESS ( SEARCH_RESULT.hidelostitems ) %][% IF ( SEARCH_RESULT.itemlostcount ) %] Lost ([% SEARCH_RESULT.itemlostcount %]),[% END %][% END %]
-                    [% IF ( SEARCH_RESULT.damagedcount ) %] Damaged ([% SEARCH_RESULT.damagedcount %]),[% END %]
-                    [% IF ( SEARCH_RESULT.orderedcount ) %] On order ([% SEARCH_RESULT.orderedcount %]),[% END %]
-                    [% IF ( SEARCH_RESULT.onholdcount ) %] On hold ([% SEARCH_RESULT.onholdcount %]),[% END %]
-                    [% IF ( SEARCH_RESULT.intransitcount ) %] In transit ([% SEARCH_RESULT.intransitcount %]),[% END %]
+                      [% IF ( SEARCH_RESULT.onloancount ) %] Checked out ([% SEARCH_RESULT.onloancount %]), [% END %]
+                      [% IF ( SEARCH_RESULT.wthdrawncount ) %] Withdrawn ([% SEARCH_RESULT.wthdrawncount %]), [% END %]
+                      [% UNLESS ( SEARCH_RESULT.hidelostitems ) %][% IF ( SEARCH_RESULT.itemlostcount ) %] Lost ([% SEARCH_RESULT.itemlostcount %]),[% END %][% END %]
+                      [% IF ( SEARCH_RESULT.damagedcount ) %] Damaged ([% SEARCH_RESULT.damagedcount %]),[% END %]
+                      [% IF ( SEARCH_RESULT.orderedcount ) %] On order ([% SEARCH_RESULT.orderedcount %]),[% END %]
+                      [% IF ( SEARCH_RESULT.onholdcount ) %] On hold ([% SEARCH_RESULT.onholdcount %]),[% END %]
+                      [% IF ( SEARCH_RESULT.intransitcount ) %] In transit ([% SEARCH_RESULT.intransitcount %]),[% END %]
                     </span>
-                </span>
+                  </span>
                 [% END %]
 
                 [% IF ( SEARCH_RESULT.score_avg ) %]
-                    <span class="result_summary">
-                        <img src="[% themelang %]/../images/Star[% SEARCH_RESULT.score_int %].gif" title="" style="max-height: 15px;"/> <span style="font-size: 85%;">[% SEARCH_RESULT.score_avg %] / 5 (on [% SEARCH_RESULT.num_scores %] rates)</span>
-                        [% IF ( SEARCH_RESULT.num_critics ) %]
-                            <span class="social_data">[% SEARCH_RESULT.num_critics %] Internet user critics</span>
-                        [% END %]
-                        [% IF ( SEARCH_RESULT.num_critics_pro ) %]
-                            <span class="social_data">[% SEARCH_RESULT.num_critics_pro %] Professional critics</span>
-                        [% END %]
-                        [% IF ( SEARCH_RESULT.num_videos ) %]
-                            <span class="social_data">[% SEARCH_RESULT.num_videos %] Video extracts</span>
-                        [% END %]
-                        [% IF ( SEARCH_RESULT.num_quotations ) %]
-                            <span class="social_data">[% SEARCH_RESULT.num_quotations %] Quotations</span>
-                        [% END %]
-                    </span>
+                  <span class="result_summary">
+                    <img src="[% themelang %]/../images/Star[% SEARCH_RESULT.score_int %].gif" title="" style="max-height: 15px;"/>
+                    <span style="font-size: 85%;">[% SEARCH_RESULT.score_avg %] / 5 (on [% SEARCH_RESULT.num_scores %] rates)</span>
+                    [% IF ( SEARCH_RESULT.num_critics ) %]
+                      <span class="social_data">[% SEARCH_RESULT.num_critics %] Internet user critics</span>
+                    [% END %]
+                    [% IF ( SEARCH_RESULT.num_critics_pro ) %]
+                      <span class="social_data">[% SEARCH_RESULT.num_critics_pro %] Professional critics</span>
+                    [% END %]
+                    [% IF ( SEARCH_RESULT.num_videos ) %]
+                      <span class="social_data">[% SEARCH_RESULT.num_videos %] Video extracts</span>
+                    [% END %]
+                    [% IF ( SEARCH_RESULT.num_quotations ) %]
+                      <span class="social_data">[% SEARCH_RESULT.num_quotations %] Quotations</span>
+                    [% END %]
+                  </span>
                 [% END %]
 
                 [% IF ( LibraryThingForLibrariesID ) %]<div class="ltfl_reviews"></div>[% END %]
-                [% IF ( opacuserlogin ) %][% IF ( TagsEnabled ) %]
-                                [% IF ( TagsShowOnList ) %]
-                                [% IF ( SEARCH_RESULT.TagLoop.size ) %]
-                                        <div class="results_summary"><span class="label">Tags:</span>
-                                        <ul style="display: inline; list-style: none;">[% FOREACH TagLoo IN SEARCH_RESULT.TagLoop %]<li style="display: inline; list-style: none;"><a href="/cgi-bin/koha/opac-search.pl?tag=[% TagLoo.term |url %]&amp;q=[% TagLoo.term |url %]">[% TagLoo.term %]</a> <span class="weight">([% TagLoo.weight_total %])</span></li>
-                                            [% END %]
-                                        </ul>
-                                   [% END %]
-                                    [% IF ( SEARCH_RESULT.TagLoop.size ) %]
-                                    </div>[% END %]
-                                [% END %]
-                                [% END %][% END %]
-                [% IF ( SEARCH_RESULT.searchhighlightblob ) %]<span class="results_summary"><span class="label">Match:</span> [% SEARCH_RESULT.searchhighlightblob %]</span>[% END %]
-
-
-
-
-                [% IF ( OpacStarRatings == 'all' ) %]
-                <div class="results_summary">
-
-[% FOREACH i  IN [ 1 2 3 4 5  ] %]
-
-[% IF ( SEARCH_RESULT.rating_avg == i ) %]
-    <input class="star" type="radio"  name="rating-[% SEARCH_RESULT.biblionumber %]" value="[% i %]" checked="checked" disabled="disabled"   />
-[% ELSE   %]
-    <input class="star" type="radio"  name="rating-[% SEARCH_RESULT.biblionumber %]" value="[% i %]" disabled="disabled"   />
-[% END %]
-
-[% END %]
-                <input type="hidden" name='biblionumber'  value="[% SEARCH_RESULT.biblionumber %]" />
-                <input type="hidden" name='loggedinuser'  value="[% loggedinuser %]" />
-
-                [% IF (  SEARCH_RESULT.rating_total ) > 0  %]
-                    <span id="rating_total_[% SEARCH_RESULT.biblionumber %]">&nbsp;&nbsp;([% SEARCH_RESULT.rating_total %] votes)</span>
-                [% ELSE %]
-                    <br />
+                [% IF ( opacuserlogin ) %]
+                  [% IF ( TagsEnabled ) %]
+                    [% IF ( TagsShowOnList ) %]
+                      [% IF ( SEARCH_RESULT.TagLoop.size ) %]
+                        <div class="results_summary">
+                          <span class="label">Tags:</span>
+                          <ul style="display: inline; list-style: none;">
+                            [% FOREACH TagLoo IN SEARCH_RESULT.TagLoop %]
+                              <li style="display: inline; list-style: none;">
+                                <a href="/cgi-bin/koha/opac-search.pl?tag=[% TagLoo.term |url %]&amp;q=[% TagLoo.term |url %]">[% TagLoo.term %]</a>
+                                <span class="weight">([% TagLoo.weight_total %])</span>
+                              </li>
+                            [% END %]
+                          </ul>
+                        </div>
+                      [% END %]
+                    [% END %]
+                  [% END %]
                 [% END %]
+                [% IF ( SEARCH_RESULT.searchhighlightblob ) %]
+                  <span class="results_summary"><span class="label">Match:</span> [% SEARCH_RESULT.searchhighlightblob %]</span>
+                [% END %]
+                [% IF ( OpacStarRatings == 'all' ) %]
+                  <div class="results_summary">
+
+                    [% FOREACH i  IN [ 1 2 3 4 5  ] %]
+                      [% IF ( SEARCH_RESULT.rating_avg == i ) %]
+                        <input class="star" type="radio"  name="rating-[% SEARCH_RESULT.biblionumber %]" value="[% i %]" checked="checked" disabled="disabled"   />
+                      [% ELSE   %]
+                        <input class="star" type="radio"  name="rating-[% SEARCH_RESULT.biblionumber %]" value="[% i %]" disabled="disabled"   />
+                      [% END %]
+                    [% END %]
+                    <input type="hidden" name='biblionumber'  value="[% SEARCH_RESULT.biblionumber %]" />
+                    <input type="hidden" name='loggedinuser'  value="[% loggedinuser %]" />
 
-                </div>
+                    [% IF (  SEARCH_RESULT.rating_total ) > 0  %]
+                      <span id="rating_total_[% SEARCH_RESULT.biblionumber %]">&nbsp;&nbsp;([% SEARCH_RESULT.rating_total %] votes)</span>
+                    [% ELSE %]
+                      <br />
+                    [% END %]
+                  </div>
                 [% END %]
 
-<span class="results_summary actions"><span class="label">Actions:</span>
-                [% IF ( RequestOnOpac ) %]
+                <span class="results_summary actions">
+                  <span class="label">Actions:</span>
+                  [% IF ( RequestOnOpac ) %]
                     [% UNLESS ( SEARCH_RESULT.norequests ) %]
-                        [% IF ( opacuserlogin ) %]
-                            [% IF ( AllowOnShelfHolds ) %]
-                                <a class="hold" href="/cgi-bin/koha/opac-reserve.pl?biblionumber=[% SEARCH_RESULT.biblionumber %]">Place hold</a><!-- add back when available 0 holds in queue-->
-                            [% ELSE %]
-                                [% IF ( SEARCH_RESULT.itemsissued ) %]
-                                    <a class="hold" href="/cgi-bin/koha/opac-reserve.pl?biblionumber=[% SEARCH_RESULT.biblionumber %]">Place hold</a><!-- add back when available 0 holds in queue-->
-                                [% END %]
-                            [% END %]
+                      [% IF ( opacuserlogin ) %]
+                        [% IF ( AllowOnShelfHolds ) %]
+                          <a class="hold" href="/cgi-bin/koha/opac-reserve.pl?biblionumber=[% SEARCH_RESULT.biblionumber %]">Place hold</a><!-- add back when available 0 holds in queue-->
+                        [% ELSE %]
+                          [% IF ( SEARCH_RESULT.itemsissued ) %]
+                             <a class="hold" href="/cgi-bin/koha/opac-reserve.pl?biblionumber=[% SEARCH_RESULT.biblionumber %]">Place hold</a><!-- add back when available 0 holds in queue-->
+                          [% END %]
                         [% END %]
+                      [% END %]
                     [% END %]
-                [% END %]
+                  [% END %]
 
-                [% IF ( TagsInputOnList ) %]
+                  [% IF ( TagsInputOnList ) %]
                     [% IF ( loggedinusername ) %]
-                        <a class="tag_results_add" id="tag_add[% SEARCH_RESULT.biblionumber %]" href="#">Add tag</a>
-                        <span id="tagform[% SEARCH_RESULT.biblionumber %]" class="tag_results_input"><label for="newtag[% SEARCH_RESULT.biblionumber %]">New tag(s):</label><input name="newtag[% SEARCH_RESULT.biblionumber %]" id="newtag[% SEARCH_RESULT.biblionumber %]" maxlength="100" />
-                        <input name="tagbutton" class="tagbutton" title="[% SEARCH_RESULT.biblionumber %]" type="submit" value="Add" /> <a class="cancel_tag_add" id="cancel[% SEARCH_RESULT.biblionumber %]" href="#">Cancel</a>
+                      <a class="tag_results_add" id="tag_add[% SEARCH_RESULT.biblionumber %]" href="#">Add tag</a>
+                      <span id="tagform[% SEARCH_RESULT.biblionumber %]" class="tag_results_input">
+                        <label for="newtag[% SEARCH_RESULT.biblionumber %]">New tag(s):</label>
+                        <input name="newtag[% SEARCH_RESULT.biblionumber %]" id="newtag[% SEARCH_RESULT.biblionumber %]" maxlength="100" />
+                        <input name="tagbutton" class="tagbutton" title="[% SEARCH_RESULT.biblionumber %]" type="submit" value="Add" />
+                        <a class="cancel_tag_add" id="cancel[% SEARCH_RESULT.biblionumber %]" href="#">Cancel</a>
                         <span id="newtag[% SEARCH_RESULT.biblionumber %]_status" class="tagstatus" style="display:none;">
-                        Tag status here.
-                        </span></span>
-                    [% ELSIF ( loop.first ) %]<span id="login4tags">Log in to add tags.</span>
+                          Tag status here.
+                        </span>
+                      </span>
+                    [% ELSIF ( loop.first ) %]
+                      <span id="login4tags">Log in to add tags.</span>
                     [% END %]
-                [% END %]
+                  [% END %]
 
-                [% IF ( opacuserlogin ) %][% IF ( loggedinusername ) %][% IF ( virtualshelves ) %]<a class="addtoshelf" href="/cgi-bin/koha/opac-addbybiblionumber.pl?biblionumber=[% SEARCH_RESULT.biblionumber %]" onclick="Dopop('opac-addbybiblionumber.pl?biblionumber=[% SEARCH_RESULT.biblionumber %]'); return false;">Save to Lists</a>
-                [% END %][% END %][% END %]
-                [% IF ( opacbookbag ) %]
+                  [% IF ( opacuserlogin ) %]
+                    [% IF ( loggedinusername ) %]
+                      [% IF ( virtualshelves ) %]
+                        <a class="addtoshelf" href="/cgi-bin/koha/opac-addbybiblionumber.pl?biblionumber=[% SEARCH_RESULT.biblionumber %]" onclick="Dopop('opac-addbybiblionumber.pl?biblionumber=[% SEARCH_RESULT.biblionumber %]'); return false;">Save to Lists</a>
+                      [% END %]
+                    [% END %]
+                  [% END %]
+                  [% IF ( opacbookbag ) %]
                     [% IF ( SEARCH_RESULT.incart ) %]
-                        <a class="addtocart cart[% SEARCH_RESULT.biblionumber %]" href="#" onclick="addRecord('[% SEARCH_RESULT.biblionumber %]'); return false;">In your cart</a> <a class="cartRemove cartR[% SEARCH_RESULT.biblionumber %]" href="#" onclick="delSingleRecord('[% SEARCH_RESULT.biblionumber %]'); return false;">(remove)</a>
+                      <a class="addtocart cart[% SEARCH_RESULT.biblionumber %]" href="#" onclick="addRecord('[% SEARCH_RESULT.biblionumber %]'); return false;">In your cart</a> <a class="cartRemove cartR[% SEARCH_RESULT.biblionumber %]" href="#" onclick="delSingleRecord('[% SEARCH_RESULT.biblionumber %]'); return false;">(remove)</a>
                     [% ELSE %]
-                        <a class="addtocart cart[% SEARCH_RESULT.biblionumber %]" href="#" onclick="addRecord('[% SEARCH_RESULT.biblionumber %]'); return false;">Add to cart</a> <a style="display:none;" class="cartRemove cartR[% SEARCH_RESULT.biblionumber %]" href="#" onclick="delSingleRecord('[% SEARCH_RESULT.biblionumber %]'); return false;">(remove)</a>
+                      <a class="addtocart cart[% SEARCH_RESULT.biblionumber %]" href="#" onclick="addRecord('[% SEARCH_RESULT.biblionumber %]'); return false;">Add to cart</a> <a style="display:none;" class="cartRemove cartR[% SEARCH_RESULT.biblionumber %]" href="#" onclick="delSingleRecord('[% SEARCH_RESULT.biblionumber %]'); return false;">(remove)</a>
                     [% END %]
-                [% END %]
+                  [% END %]
                 </span>
-                </td><td>
-                    <a class="p1" href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% SEARCH_RESULT.biblionumber %]">
-            [% IF ( OPACLocalCoverImages ) %]<span title="[% SEARCH_RESULT.biblionumber |url %]" class="[% SEARCH_RESULT.biblionumber %]" id="local-thumbnail[% loop.count %]"></span>[% END %]
-                    [% IF ( OPACAmazonEnabled ) %][% IF ( OPACAmazonCoverImages ) %][% IF ( SEARCH_RESULT.normalized_isbn ) %]<img src="http://images.amazon.com/images/P/[% SEARCH_RESULT.normalized_isbn %].01.TZZZZZZZ.jpg" alt="" class="thumbnail" />[% ELSE %]<span class="no-image">No cover image available</span>[% END %][% END %][% END %]
-
-                [% IF ( SyndeticsEnabled ) %]
+              </span>
+
+              <span class="image">
+                <a class="p1" href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% SEARCH_RESULT.biblionumber %]">
+                  [% IF ( OPACLocalCoverImages ) %]
+                    <span title="[% SEARCH_RESULT.biblionumber |url %]" class="[% SEARCH_RESULT.biblionumber %]" id="local-thumbnail[% loop.count %]"></span>
+                  [% END %]
+                  [% IF ( OPACAmazonEnabled ) %]
+                    [% IF ( OPACAmazonCoverImages ) %]
+                      [% IF ( SEARCH_RESULT.normalized_isbn ) %]
+                        <img src="http://images.amazon.com/images/P/[% SEARCH_RESULT.normalized_isbn %].01.TZZZZZZZ.jpg" alt="" class="thumbnail" />
+                      [% ELSE %]
+                        <span class="no-image">No cover image available</span>
+                      [% END %]
+                    [% END %]
+                  [% END %]
+                  [% IF ( SyndeticsEnabled ) %]
                     [% IF ( SyndeticsCoverImages ) %]
-                        [% IF SEARCH_RESULT.normalized_isbn %]
-                                [% IF ( using_https ) %]
-                            <img src="https://secure.syndetics.com/index.aspx?isbn=[% SEARCH_RESULT.normalized_isbn %]/SC.GIF&amp;client=[% SyndeticsClientCode %]&amp;type=xw10&amp;upc=[% SEARCH_RESULT.normalized_upc %]&amp;oclc=[% SEARCH_RESULT.normalized_oclc %]" alt="" class="thumbnail" />
+                      [% IF SEARCH_RESULT.normalized_isbn %]
+                        [% IF ( using_https ) %]
+                          <img src="https://secure.syndetics.com/index.aspx?isbn=[% SEARCH_RESULT.normalized_isbn %]/SC.GIF&amp;client=[% SyndeticsClientCode %]&amp;type=xw10&amp;upc=[% SEARCH_RESULT.normalized_upc %]&amp;oclc=[% SEARCH_RESULT.normalized_oclc %]" alt="" class="thumbnail" />
                         [% ELSE %]
-                            <img src="http://www.syndetics.com/index.aspx?isbn=[% SEARCH_RESULT.normalized_isbn %]/SC.GIF&amp;client=[% SyndeticsClientCode %]&amp;type=xw10&amp;upc=[% SEARCH_RESULT.normalized_upc %]&amp;oclc=[% SEARCH_RESULT.normalized_oclc %]" alt="" class="thumbnail" />
-                    [% END %]
-                    [% ELSE %]
+                          <img src="http://www.syndetics.com/index.aspx?isbn=[% SEARCH_RESULT.normalized_isbn %]/SC.GIF&amp;client=[% SyndeticsClientCode %]&amp;type=xw10&amp;upc=[% SEARCH_RESULT.normalized_upc %]&amp;oclc=[% SEARCH_RESULT.normalized_oclc %]" alt="" class="thumbnail" />
+                        [% END %]
+                      [% ELSE %]
                         <span class="no-image">No cover image available</span>
+                      [% END %]
                     [% END %]
-            [% END %]
-                [% END %]
+                  [% END %]
 
-                [% IF ( GoogleJackets ) %][% IF ( SEARCH_RESULT.normalized_isbn ) %]<span style="block" title="[% SEARCH_RESULT.biblionumber |url %]" class="[% SEARCH_RESULT.normalized_isbn %]" id="gbs-thumbnail[% loop.count %]"></span>[% ELSE %]<span class="no-image">No cover image available</span>[% END %][% END %]
-        [% IF OpenLibraryCovers %][% IF SEARCH_RESULT.normalized_isbn %]<span style="block" title="[% SEARCH_RESULT.biblionumber %]" class="[% SEARCH_RESULT.normalized_isbn %]" id="openlibrary-thumbnail[% loop.count %]"></span>[% ELSE %]<span class="no-image">No cover image available</span>[% END %][% END %]
+                  [% IF ( GoogleJackets ) %]
+                    [% IF ( SEARCH_RESULT.normalized_isbn ) %]
+                      <span style="block" title="[% SEARCH_RESULT.biblionumber |url %]" class="[% SEARCH_RESULT.normalized_isbn %]" id="gbs-thumbnail[% loop.count %]"></span>
+                    [% ELSE %]
+                      <span class="no-image">No cover image available</span>
+                    [% END %]
+                  [% END %]
+                  [% IF OpenLibraryCovers %]
+                    [% IF SEARCH_RESULT.normalized_isbn %]
+                      <span style="block" title="[% SEARCH_RESULT.biblionumber %]" class="[% SEARCH_RESULT.normalized_isbn %]" id="openlibrary-thumbnail[% loop.count %]"></span>
+                    [% ELSE %]
+                      <span class="no-image">No cover image available</span>
+                    [% END %]
+                  [% END %]
                 </a>
-                [% IF ( BakerTaylorEnabled ) %][% IF ( SEARCH_RESULT.normalized_isbn ) %]<a href="https://[% BakerTaylorBookstoreURL |html %][% SEARCH_RESULT.normalized_isbn %]"><img alt="See Baker &amp; Taylor" src="[% BakerTaylorImageURL |html %][% SEARCH_RESULT.normalized_isbn %]" /></a>[% ELSE %]<span class="no-image">No cover image available</span>[% END %][% END %]
-                </td>
-                </tr>
+                [% IF ( BakerTaylorEnabled ) %]
+                  [% IF ( SEARCH_RESULT.normalized_isbn ) %]
+                    <a href="https://[% BakerTaylorBookstoreURL |html %][% SEARCH_RESULT.normalized_isbn %]"><img alt="See Baker &amp; Taylor" src="[% BakerTaylorImageURL |html %][% SEARCH_RESULT.normalized_isbn %]" /></a>
+                  [% ELSE %]
+                    <span class="no-image">No cover image available</span>
+                  [% END %]
                 [% END %]
-            </table>
-    </form>
-    <form id="hold_form" name="hold_form" method="get" action="/cgi-bin/koha/opac-reserve.pl">
+              </span>
+            </div>
+          [% END %]
+        </div>
+      </form>
+      <form id="hold_form" name="hold_form" method="get" action="/cgi-bin/koha/opac-reserve.pl">
         <!-- The value will be set by holdBiblioNums() in basket.js -->
         <input id="hold_form_biblios" type="hidden" name="biblionumbers" value=""/>
-    </form>
-        </div>
+      </form>
+    </div>
     [% END %]
     [% INCLUDE 'page-numbers.inc' %]
 
-- 
1.7.7.3



More information about the Koha-patches mailing list