On Fri, Jun 5, 2009 at 2:23 PM, Mike Rylander <[email protected]> wrote:
> On Fri, Jun 5, 2009 at 12:19 PM, Grant Johnson <[email protected]> wrote:
>> Hey all,
>>
>> a Bookbag with one item displays fine using MOST Supercat syntax
>>
>> Sample:
>> http://islandpines.roblib.upei.ca/opac/extras/feed/bookbag/htmlholdings-full/66
>> http://islandpines.roblib.upei.ca/opac/extras/feed/bookbag/rss2/66
>> http://islandpines.roblib.upei.ca/opac/extras/feed/bookbag/rss2-full/66
>> http://islandpines.roblib.upei.ca/opac/extras/feed/bookbag/html/66
>> http://islandpines.roblib.upei.ca/opac/extras/feed/bookbag/html-full/66
>> Note that this is a bookbag with exactly one record.
>>
>> Now try:
>> http://islandpines.roblib.upei.ca/opac/extras/feed/bookbag/opac/66
>>
>> Strange... Any idea's?
>>
>
> It (the rlist display code in the OPAC) is splitting the number into
> digits. Instead of giving you record 373480, it's giving you records
> 3, 7, 3 (again), 4, 8, and 0 (which doesn't exist).
>
> I'll look at that now in trunk.
It was working under the assumption that rl would always have multiple
values. The code now checks for that, and you can apply the attached
patch to your installation to get the update.
--
Mike Rylander
| VP, Research and Design
| Equinox Software, Inc. / The Evergreen Experts
| phone: 1-877-OPEN-ILS (673-6457)
| email: [email protected]
| web: http://www.esilibrary.com
Index: Open-ILS/web/opac/skin/default/js/rresult.js
===================================================================
--- Open-ILS/web/opac/skin/default/js/rresult.js (revision 13332)
+++ Open-ILS/web/opac/skin/default/js/rresult.js (working copy)
@@ -200,7 +200,14 @@
function rresultHandleList() {
var ids = new CGI().param(PARAM_RLIST);
- if(ids) _rresultHandleIds(ids, ids.length);
+ var count;
+ if(!dojo.isArray(ids)) {
+ count = 1;
+ ids = [ids];
+ } else {
+ count = ids.length;
+ }
+ if(ids) _rresultHandleIds(ids, count);
}
var rresultTries = 0;
