Attachment 23541 Details for Bug 10858 – Bug 10858: Browse selected biblios

[patch] Bug 10858: Browse selected biblios

Bug-10858-Browse-selected-biblios.patch (text/plain), 23.51 KB, created by Kyle M Hall (khall) on 2013-12-13 19:38:08 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2013-12-13 19:38:08 UTC

Size: 23.51 KB

patch

obsolete

>From 2a12dd8509c9a7db496538373a2dbf47fb8f84dd Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@biblibre.com>
>Date: Wed, 14 Nov 2012 17:05:45 +0100
>Subject: [PATCH] Bug 10858: Browse selected biblios
>
>After launching a search, some people would like to browse some results
>from several pages.
>
>This patch adds, at the OPAC, a link "Browse selected biblios >>" when
>at least one biblio is checked. You can choose biblios from several
>page, not the current one. In this way, checkboxes already checked
>become again checked when you come back on this results page.
>After clicking the link, the "Browse results" block is filled with
>selected biblios, not (like as the current behavior) with the classic
>results.
>
>Test plan:
>- Launch a search at the OPAC
>- Check some biblios
>- Go on another page
>- Check some biblios
>- Come back to the first page and check that already checked biblio is
>  automatically checked.
>- Click on the "Browse selected biblios >>" link
>- Check that you are able to browse results you had checked.
>
>Signed-off-by: Cedric Vita <cedric.vita@dracenie.com>
>
>Signed-off-by: Koha Team Amu <koha.aixmarseille@gmail.com>
>Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
>---
> .../opac-tmpl/prog/en/includes/doc-head-close.inc  |    3 +
> koha-tmpl/opac-tmpl/prog/en/js/commons.js          |   71 +++++++
> .../en/lib/jquery/plugins/jquery.cookie.min.js     |    1 +
> .../prog/en/lib/jquery/plugins/jquery.json.js      |  199 ++++++++++++++++++++
> koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt |   17 ++-
> .../opac-tmpl/prog/en/modules/opac-results.tt      |   48 +++++
> opac/opac-detail.pl                                |   21 ++-
> 7 files changed, 352 insertions(+), 8 deletions(-)
> create mode 100644 koha-tmpl/opac-tmpl/prog/en/js/commons.js
> create mode 100644 koha-tmpl/opac-tmpl/prog/en/lib/jquery/plugins/jquery.cookie.min.js
> create mode 100644 koha-tmpl/opac-tmpl/prog/en/lib/jquery/plugins/jquery.json.js
>
>diff --git a/koha-tmpl/opac-tmpl/prog/en/includes/doc-head-close.inc b/koha-tmpl/opac-tmpl/prog/en/includes/doc-head-close.inc
>index 66666f4..6002472 100644
>--- a/koha-tmpl/opac-tmpl/prog/en/includes/doc-head-close.inc
>+++ b/koha-tmpl/opac-tmpl/prog/en/includes/doc-head-close.inc
>@@ -35,6 +35,9 @@
> <script type="text/javascript" src="[% themelang %]/lib/jquery/jquery.js"></script>
> <script type="text/javascript" src="[% themelang %]/lib/jquery/jquery-ui.js"></script>
> <script type="text/javascript" src="[% themelang %]/lib/jquery/plugins/jquery.hoverIntent.minified.js"></script>
>+<script type="text/javascript" src="[% themelang %]/lib/jquery/plugins/jquery.json.js"></script>
>+<script type="text/javascript" src="[% themelang %]/lib/jquery/plugins/jquery.cookie.min.js"></script>
>+<script type="text/javascript" src="[% themelang %]/js/commons.js"></script>
> <script type="text/javascript" src="[% themelang %]/js/script.js"></script>
> [% IF ( OPACAmazonCoverImages ) %]
> <script type="text/javascript" language="javascript">//<![CDATA[
>diff --git a/koha-tmpl/opac-tmpl/prog/en/js/commons.js b/koha-tmpl/opac-tmpl/prog/en/js/commons.js
>new file mode 100644
>index 0000000..6661a61
>--- /dev/null
>+++ b/koha-tmpl/opac-tmpl/prog/en/js/commons.js
>@@ -0,0 +1,71 @@
>+// Extends jQuery API
>+// http://www.wskidmore.com/downloads/jquery-uniqueArray.min.js
>+jQuery.extend({uniqueArray:function(e){if($.isArray(e)){var c={};var a,b;for(b=0,a=e.length;b<a;b++){var d=e[b].toString();if(c[d]){e.splice(b,1);a--;b--}else{c[d]=true}}}return(e)}});
>+
>+function removeByValue(arr, val) {
>+    for(var i=0; i<arr.length; i++) {
>+        if(arr[i] == val) {
>+            arr.splice(i, 1);
>+            break;
>+        }
>+    }
>+}
>+
>+function paramOfUrl( url, param ) {
>+    param = param.replace(/[\[]/,"\\\[").replace(/[\]]/,"\\\]");
>+    var regexS = "[\\?&]"+param+"=([^&#]*)";
>+    var regex = new RegExp( regexS );
>+    var results = regex.exec( url );
>+    if( results == null ) {
>+        return "";
>+    } else {
>+        return results[1];
>+    }
>+}
>+
>+function addBibToContext( bibnum ) {
>+    var bibnums = getContextBiblioNumbers();
>+    bibnums.push(bibnum);
>+    setContextBiblioNumbers( bibnums );
>+    //var store = sessionStorage.getItem( "bibs_selected" );
>+}
>+
>+function delBibToContext( bibnum ) {
>+    var bibnums = getContextBiblioNumbers();
>+    removeByValue( bibnums, bibnum );
>+    setContextBiblioNumbers( $.uniqueArray( bibnums ) );
>+}
>+
>+function setContextBiblioNumbers( bibnums ) {
>+    //sessionStorage.setItem("bibs_selected", JSON.stringify( bibnums ) );
>+    $.cookie('bibs_selected', JSON.stringify( bibnums ));
>+}
>+
>+function getContextBiblioNumbers() {
>+    //var r = sessionStorage.getItem("bibs_selected");
>+    var r = $.cookie('bibs_selected');
>+    if ( r ) {
>+        return JSON.parse(r);
>+    }
>+    r = new Array();
>+    return r;
>+}
>+
>+function resetSearchContext() {
>+    setContextBiblioNumbers( new Array() );
>+}
>+
>+$(document).ready(function(){
>+    // forms with action leading to search
>+    $("form[action*='opac-search.pl']").submit(function(){
>+        resetSearchContext();
>+    });
>+    // any link to launch a search except navigation links
>+    $("[href*='opac-search.pl?']").not(".nav").not('.searchwithcontext').click(function(){
>+        resetSearchContext();
>+    });
>+    // any link to a detail page from the results page.
>+    $("#bookbag_form a[href*='opac-detail.pl?']").click(function(){
>+        resetSearchContext();
>+    });
>+});
>diff --git a/koha-tmpl/opac-tmpl/prog/en/lib/jquery/plugins/jquery.cookie.min.js b/koha-tmpl/opac-tmpl/prog/en/lib/jquery/plugins/jquery.cookie.min.js
>new file mode 100644
>index 0000000..eb129db
>--- /dev/null
>+++ b/koha-tmpl/opac-tmpl/prog/en/lib/jquery/plugins/jquery.cookie.min.js
>@@ -0,0 +1 @@
>+jQuery.cookie=function(b,j,m){if(typeof j!="undefined"){m=m||{};if(j===null){j="";m.expires=-1}var e="";if(m.expires&&(typeof m.expires=="number"||m.expires.toUTCString)){var f;if(typeof m.expires=="number"){f=new Date();f.setTime(f.getTime()+(m.expires*24*60*60*1000))}else{f=m.expires}e="; expires="+f.toUTCString()}var l=m.path?"; path="+(m.path):"";var g=m.domain?"; domain="+(m.domain):"";var a=m.secure?"; secure":"";document.cookie=[b,"=",encodeURIComponent(j),e,l,g,a].join("")}else{var d=null;if(document.cookie&&document.cookie!=""){var k=document.cookie.split(";");for(var h=0;h<k.length;h++){var c=jQuery.trim(k[h]);if(c.substring(0,b.length+1)==(b+"=")){d=decodeURIComponent(c.substring(b.length+1));break}}}return d}};
>\ No newline at end of file
>diff --git a/koha-tmpl/opac-tmpl/prog/en/lib/jquery/plugins/jquery.json.js b/koha-tmpl/opac-tmpl/prog/en/lib/jquery/plugins/jquery.json.js
>new file mode 100644
>index 0000000..d01e310
>--- /dev/null
>+++ b/koha-tmpl/opac-tmpl/prog/en/lib/jquery/plugins/jquery.json.js
>@@ -0,0 +1,199 @@
>+/**
>+ * jQuery JSON plugin 2.4-alpha
>+ *
>+ * @author Brantley Harris, 2009-2011
>+ * @author Timo Tijhof, 2011-2012
>+ * @source This plugin is heavily influenced by MochiKit's serializeJSON, which is
>+ *         copyrighted 2005 by Bob Ippolito.
>+ * @source Brantley Harris wrote this plugin. It is based somewhat on the JSON.org
>+ *         website's http://www.json.org/json2.js, which proclaims:
>+ *         "NO WARRANTY EXPRESSED OR IMPLIED. USE AT YOUR OWN RISK.", a sentiment that
>+ *         I uphold.
>+ * @license MIT License <http://www.opensource.org/licenses/mit-license.php>
>+ */
>+(function ($) {
>+     'use strict';
>+
>+ var escape = /["\\\x00-\x1f\x7f-\x9f]/g,
>+               meta = {
>+                       '\b': '\\b',
>+                   '\t': '\\t',
>+                   '\n': '\\n',
>+                   '\f': '\\f',
>+                   '\r': '\\r',
>+                   '"' : '\\"',
>+                   '\\': '\\\\'
>+           },
>+             hasOwn = Object.prototype.hasOwnProperty;
>+
>+     /**
>+     * jQuery.toJSON
>+        * Converts the given argument into a JSON representation.
>+      *
>+      * @param o {Mixed} The json-serializable *thing* to be converted
>+       *
>+      * If an object has a toJSON prototype, that will be used to get the representation.
>+    * Non-integer/string keys are skipped in the object, as are keys that point to a
>+       * function.
>+    *
>+      */
>+    $.toJSON = typeof JSON === 'object' && JSON.stringify ? JSON.stringify : function (o) {
>+                if (o === null) {
>+                      return 'null';
>+         }
>+
>+             var pairs, k, name, val,
>+                       type = $.type(o);
>+
>+             if (type === 'undefined') {
>+                    return undefined;
>+              }
>+
>+             // Also covers instantiated Number and Boolean objects,
>+                // which are typeof 'object' but thanks to $.type, we
>+          // catch them here. I don't know whether it is right
>+           // or wrong that instantiated primitives are not
>+               // exported to JSON as an {"object":..}.
>+               // We choose this path because that's what the browsers did.
>+           if (type === 'number' || type === 'boolean') {
>+                 return String(o);
>+              }
>+              if (type === 'string') {
>+                       return $.quoteString(o);
>+               }
>+              if (typeof o.toJSON === 'function') {
>+                  return $.toJSON(o.toJSON());
>+           }
>+              if (type === 'date') {
>+                 var month = o.getUTCMonth() + 1,
>+                               day = o.getUTCDate(),
>+                          year = o.getUTCFullYear(),
>+                             hours = o.getUTCHours(),
>+                               minutes = o.getUTCMinutes(),
>+                           seconds = o.getUTCSeconds(),
>+                           milli = o.getUTCMilliseconds();
>+
>+                       if (month < 10) {
>+                              month = '0' + month;
>+                   }
>+                      if (day < 10) {
>+                                day = '0' + day;
>+                       }
>+                      if (hours < 10) {
>+                              hours = '0' + hours;
>+                   }
>+                      if (minutes < 10) {
>+                            minutes = '0' + minutes;
>+                       }
>+                      if (seconds < 10) {
>+                            seconds = '0' + seconds;
>+                       }
>+                      if (milli < 100) {
>+                             milli = '0' + milli;
>+                   }
>+                      if (milli < 10) {
>+                              milli = '0' + milli;
>+                   }
>+                      return '"' + year + '-' + month + '-' + day + 'T' +
>+                            hours + ':' + minutes + ':' + seconds +
>+                                '.' + milli + 'Z"';
>+            }
>+
>+             pairs = [];
>+
>+           if ($.isArray(o)) {
>+                    for (k = 0; k < o.length; k++) {
>+                               pairs.push($.toJSON(o[k]) || 'null');
>+                  }
>+                      return '[' + pairs.join(',') + ']';
>+            }
>+
>+             // Any other object (plain object, RegExp, ..)
>+         // Need to do typeof instead of $.type, because we also
>+                // want to catch non-plain objects.
>+            if (typeof o === 'object') {
>+                   for (k in o) {
>+                         // Only include own properties,
>+                                // Filter out inherited prototypes
>+                             if (hasOwn.call(o, k)) {
>+                                       // Keys must be numerical or string. Skip others
>+                                       type = typeof k;
>+                                       if (type === 'number') {
>+                                               name = '"' + k + '"';
>+                                  } else if (type === 'string') {
>+                                                name = $.quoteString(k);
>+                                       } else {
>+                                               continue;
>+                                      }
>+                                      type = typeof o[k];
>+
>+                                   // Invalid values like these return undefined
>+                                  // from toJSON, however those object members
>+                                   // shouldn't be included in the JSON string at all.
>+                                    if (type !== 'function' && type !== 'undefined') {
>+                                             val = $.toJSON(o[k]);
>+                                          pairs.push(name + ':' + val);
>+                                  }
>+                              }
>+                      }
>+                      return '{' + pairs.join(',') + '}';
>+            }
>+      };
>+
>+    /**
>+     * jQuery.evalJSON
>+      * Evaluates a given json string.
>+       *
>+      * @param str {String}
>+  */
>+    $.evalJSON = typeof JSON === 'object' && JSON.parse ? JSON.parse : function (str) {
>+            /*jshint evil: true */
>+         return eval('(' + str + ')');
>+  };
>+
>+    /**
>+     * jQuery.secureEvalJSON
>+        * Evals JSON in a way that is *more* secure.
>+   *
>+      * @param str {String}
>+  */
>+    $.secureEvalJSON = typeof JSON === 'object' && JSON.parse ? JSON.parse : function (str) {
>+              var filtered =
>+                 str
>+                    .replace(/\\["\\\/bfnrtu]/g, '@')
>+                      .replace(/"[^"\\\n\r]*"|true|false|null|-?\d+(?:\.\d*)?(?:[eE][+\-]?\d+)?/g, ']')
>+                      .replace(/(?:^|:|,)(?:\s*\[)+/g, '');
>+
>+         if (/^[\],:{}\s]*$/.test(filtered)) {
>+                  /*jshint evil: true */
>+                 return eval('(' + str + ')');
>+          }
>+              throw new SyntaxError('Error parsing JSON, source is not valid.');
>+     };
>+
>+    /**
>+     * jQuery.quoteString
>+   * Returns a string-repr of a string, escaping quotes intelligently.
>+    * Mostly a support function for toJSON.
>+        * Examples:
>+    * >>> jQuery.quoteString('apple')
>+      * "apple"
>+      *
>+      * >>> jQuery.quoteString('"Where are we going?", she asked.')
>+  * "\"Where are we going?\", she asked."
>+        */
>+    $.quoteString = function (str) {
>+               if (str.match(escape)) {
>+                       return '"' + str.replace(escape, function (a) {
>+                                var c = meta[a];
>+                               if (typeof c === 'string') {
>+                                   return c;
>+                              }
>+                              c = a.charCodeAt();
>+                            return '\\u00' + Math.floor(c / 16).toString(16) + (c % 16).toString(16);
>+                      }) + '"';
>+              }
>+              return '"' + str + '"';
>+        };
>+
>+}(jQuery));
>diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt b/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt
>index 05c4010..51aad0e 100644
>--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt
>+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt
>@@ -154,6 +154,7 @@
>             renderPagIndexList(pag_index_ini, $("#listResults"));
>         }
>         $("#a_listResults").click(function(e) {
>+            $("#sidebar").toggle();
>             if (arrPagination.length > 0) {
>                 e.preventDefault();
>                 var navigation = $(".pagination");
>@@ -173,6 +174,7 @@
>             e.preventDefault();
>             var navigation = $(".pagination");
>             navigation.hide();
>+            $("#sidebar").toggle();
>         });
>     [% END %]
> 
>@@ -285,8 +287,9 @@ function parseIDBJSON( json ) {
> var timeoutRFW;
> var totalPagItemList = 10;
> 
>-function rewindList()
>+function rewindList(e)
> {
>+    e.preventDefault();
>     var ul = $("#listResults");
>     var li_ini = ul.children(':first').next();
>     var index_ini = pag_index_ini;
>@@ -304,8 +307,9 @@ function rewindList()
>     }
> }//rewindList
> 
>-function forwardList()
>+function forwardList(e)
> {
>+    e.preventDefault();
>     var ul = $("#listResults");
>     var li_ini = ul.children(':first').next();
>     var index_ini = pag_index_ini;
>@@ -336,7 +340,7 @@ function renderPagIndexList(index, ul)
>     var ini = index - 1;
>     var end = ini + totalPagItemList - 1;
>     li = $("<li />");
>-    html = (index > pag_index_ini)?"<a href='#' id='rew_list_index' onclick='rewindList()' title='" + _("Click to rewind the list to") + " " + ini + " - " + end + "'>&laquo;</a>":"&laquo;";
>+    html = (index > pag_index_ini)?"<a href='#' id='rew_list_index' onclick='rewindList(event)' title='" + _("Click to rewind the list to") + " " + ini + " - " + end + "'>&laquo;</a>":"&laquo;";
>     li.html(html);
>     ul.append(li);
>     var title = "";
>@@ -358,7 +362,7 @@ function renderPagIndexList(index, ul)
>     li = $("<li />");
>     ini = index + 1;
>     end = (arrPagination.length > index + totalPagItemList)?index + totalPagItemList:arrPagination.length - 1;
>-    html = (end <= arrPagination.length - 1 && (end - index) >= totalPagItemList)?"<a href='#' id='fw_list_index' onclick='forwardList()' title='" + _("Click to forward the list to") + " " + ini + " - " + end + "'>&raquo;</a>":"&raquo;";
>+    html = (end <= arrPagination.length - 1 && (end - index) >= totalPagItemList)?"<a href='#' id='fw_list_index' onclick='forwardList(event)' title='" + _("Click to forward the list to") + " " + ini + " - " + end + "'>&raquo;</a>":"&raquo;";
>     li.html(html);
>     ul.append(li);
> }//renderPagIndexList
>@@ -1424,7 +1428,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () {
>         <div class="l_Results">[% IF ( listResults ) %]<a href="#" id="a_listResults" title="Show pagination list ([% indexPag %]-[% indexPagEnd %] / [% totalPag %])">Browse results</a>[% ELSE %]<span>Browse results</span>[% END %]</div>
>         <ul class="pg_menu clearfix">
>             <li class="left_results">[% IF ( previous ) %]<a href="[% previous %]" title="See: [% IF ( previousTitle ) %][% previousTitle |html %][% ELSE %]previous biblio[% END %]">&laquo; Previous</a>[% ELSE %]<span>Previous</span>[% END %]</li>
>-            <li class="back_results"><a href="opac-search.pl?[% busc %]" title="Back to the results search list">Back to results</a></li>
>+            <li class="back_results"><a href="opac-search.pl?[% busc %]" title="Back to the results search list" class="searchwithcontext">Back to results</a></li>
>             <li class="right_results">[% IF ( next ) %]<a href="[% next %]" title="See: [% IF ( nextTitle ) %][% nextTitle |html %][% ELSE %]next biblio[% END %]">Next &raquo;</a>[% ELSE %]<span>Next</span>[% END %]</li>
>         </ul>
>         [% IF ( listResults ) %]
>@@ -1441,6 +1445,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () {
>     </div>
> [% END %]
> 
>+<div id="sidebar">
> [% INCLUDE 'opac-detail-sidebar.inc' %]
>         [% IF ( NovelistSelectProfile ) %] [% IF ( NovelistSelectView == 'right') %]
>          <div id="NovelistSelect">
>@@ -1469,6 +1474,8 @@ YAHOO.util.Event.onContentReady("furtherm", function () {
> [% END %]
> 
> </div>
>+
>+</div>
> </div>
> </div>
> 
>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 9235b2b..4ccc009 100644
>--- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-results.tt
>+++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-results.tt
>@@ -109,6 +109,19 @@ function highlightOn() {
>     $(".highlight_toggle").toggle();
> }
> [% END %]
>+
>+// Hide or show details link
>+// Update with first biblionumber in queue
>+function updateDetailsLink() {
>+    var bibnums = getContextBiblioNumbers();
>+    if ( bibnums.length > 0 ) {
>+        $(".details_link a").attr("href", "/cgi-bin/koha/opac-detail.pl?biblionumber=" + bibnums[0]);
>+        $(".details_link").show();
>+    } else {
>+        $(".details_link").hide();
>+    }
>+}
>+
> $(document).ready(function(){
>     [% IF ( OpacHighlightedWords ) %]
>         $('a.title').each(function() {
>@@ -233,10 +246,17 @@ $(document).ready(function(){
>     }
>     $("#CheckAll").click(function(){
>         $("#bookbag_form").checkCheckboxes();
>+        $("#bookbag_form").find("input[type='checkbox'][name='biblionumber']").each(function(){
>+            $(this).change();
>+        } );
>         return false;
>     });
>     $("#CheckNone").click(function(){
>         $("#bookbag_form").unCheckCheckboxes();
>+
>+        $("#bookbag_form").find("input[type='checkbox'][name='biblionumber']").each(function(){
>+            $(this).change();
>+        } );
>         return false;
>     });
> [% IF ( RequestOnOpac ) %]
>@@ -343,6 +363,33 @@ $(document).ready(function(){
>         });
> [% END %]
> 
>+    $("#bookbag_form").ready(function(){
>+        $("#bookbag_form").unCheckCheckboxes();
>+        var bibnums = getContextBiblioNumbers();
>+        if (bibnums) {
>+            for (var i=0; i < bibnums.length; i++) {
>+                var id = ('#bib' + bibnums[i]);
>+                if ($(id)) {
>+                    $(id).attr('checked', true);
>+                }
>+            }
>+        }
>+        updateDetailsLink();
>+    });
>+
>+    $("#bookbag_form :checkbox").change(function(){
>+        if ( $(this).is(':checked') == true ) {
>+            addBibToContext( $(this).val() );
>+        } else {
>+            delBibToContext( $(this).val() );
>+        }
>+        updateDetailsLink();
>+     });
>+
>+    $(".details_link a").click(function(){
>+        var bibnums = getContextBiblioNumbers();
>+
>+    });
> });
> 
> //]]>
>@@ -482,6 +529,7 @@ $(document).ready(function(){
>                               Tag status here.
>                             </span>
>             [% END %]
>+            <span class="details_link" style="display:none" ><a href="#">Browse selected biblios &gt;&gt;</a></span>
>         </div>
> 
>         </td></tr>
>diff --git a/opac/opac-detail.pl b/opac/opac-detail.pl
>index 77de12f..75d5dbe 100755
>--- a/opac/opac-detail.pl
>+++ b/opac/opac-detail.pl
>@@ -24,6 +24,7 @@ use strict;
> use warnings;
> 
> use CGI;
>+use JSON;
> use C4::Auth qw(:DEFAULT get_session);
> use C4::Branch;
> use C4::Koha;
>@@ -209,11 +210,22 @@ if ($session->param('busc')) {
>             push @{$arrParamsBusc{$key}}, $value;
>         }
>     }
>+
>     my $searchAgain = 0;
>+    my ( $offset, $results_per_page );
>+
>+    my $bibnums = $query->cookie('bibs_selected');
>+    $bibnums = JSON::from_json($bibnums);
>+    if ( scalar( @$bibnums ) ) {
>+        $arrParamsBusc{count} = scalar( @$bibnums );
>+        $arrParamsBusc{listBiblios} = join ',', @$bibnums;
>+        $results_per_page = $arrParamsBusc{count};
>+        $offset = 0;
>+    } else {
>+    $offset = ($arrParamsBusc{'offset'} && $arrParamsBusc{'offset'} =~ /^[0-9]+?/)?$arrParamsBusc{'offset'}:0;
>     my $count = C4::Context->preference('OPACnumSearchResults') || 20;
>-    my $results_per_page = ($arrParamsBusc{'count'} && $arrParamsBusc{'count'} =~ /^[0-9]+?/)?$arrParamsBusc{'count'}:$count;
>+    $results_per_page = ($arrParamsBusc{'count'} && $arrParamsBusc{'count'} =~ /^[0-9]+?/)?$arrParamsBusc{'count'}:$count;
>     $arrParamsBusc{'count'} = $results_per_page;
>-    my $offset = ($arrParamsBusc{'offset'} && $arrParamsBusc{'offset'} =~ /^[0-9]+?/)?$arrParamsBusc{'offset'}:0;
>     # The value OPACnumSearchResults has changed and the search has to be rebuild
>     if ($count != $results_per_page) {
>         if (exists($arrParamsBusc{'listBiblios'}) && $arrParamsBusc{'listBiblios'} =~ /^[0-9]+(?:,[0-9]+)*$/) {
>@@ -273,6 +285,9 @@ if ($session->param('busc')) {
>             @arrBusc = split(/\&(?:amp;)?/, $newbusc);
>         }
>     }
>+
>+    } # end if cookie bibs_selected exists
>+
>     my $buscParam = '';
>     my $j = 0;
>     # Rebuild the query for the button "back to results"
>@@ -339,7 +354,7 @@ if ($session->param('busc')) {
>         }
>         $offsetSearch = 0 if (defined($offsetSearch) && $offsetSearch < 0);
>     }
>-    if ($searchAgain) {
>+    if ($searchAgain and not scalar( @$bibnums )) {
>         my $newresultsRef = searchAgain(\%arrParamsBusc, $offsetSearch, $results_per_page);
>         my @newresults = @$newresultsRef;
>         # build the new listBiblios
>-- 
>1.7.2.5

Actions: View | Diff | Splinter Review

Attachments on bug 10858: 20947 | 21476 | 21483 | 21646 | 21647 | 21966 | 21967 | 23541 | 23542 | 23543 | 23633 | 23781 | 23782 | 25427 | 25428 | 25429 | 25430 | 25431 | 25432 | 25433 | 25434 | 25435 | 25436 | 25450 | 25451 | 25452 | 25453 | 25454 | 25455 | 25456 | 25457 | 25512 | 25513 | 25514 | 25643 | 25644 | 25645 | 25646 | 25647 | 25648 | 25649 | 25650 | 25651 | 25652 | 25653 | 36757 | 38033 | 38405 | 39502 | 39938 | 41838 | 41839 | 41840 | 41841 | 41842 | 44903 | 56479 | 56480 | 56481 | 56482 | 67073 | 67074 | 67075 | 67076 | 67086 | 67089 | 67090 | 67092 | 67093 | 67974 | 67975 | 67976 | 67977 | 67978 | 71218 | 71219 | 71220 | 71221 | 71222 | 71223 | 91584 | 91585 | 91586 | 91587 | 91588 | 91589 | 94833