From b88e34a654922d1efc58973c2d25adf850e68af5 Mon Sep 17 00:00:00 2001 From: erickson Date: Thu, 29 Dec 2005 15:27:41 +0000 Subject: [PATCH] fixed bug in MYOPAC where holds with the same target item were causing the display to bunch items together into a single row. git-svn-id: svn://svn.open-ils.org/ILS/trunk@2546 dcc99617-32d9-48b4-a31d-7c20da2025e4 --- Open-ILS/web/opac/skin/default/js/myopac.js | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/Open-ILS/web/opac/skin/default/js/myopac.js b/Open-ILS/web/opac/skin/default/js/myopac.js index dc6f0f1d33..60e7bd7a34 100644 --- a/Open-ILS/web/opac/skin/default/js/myopac.js +++ b/Open-ILS/web/opac/skin/default/js/myopac.js @@ -188,11 +188,11 @@ function myOPACDrawHolds(r) { var h = holds[i]; var row = holdsTemplateRow.cloneNode(true); - row.id = "myopac_holds_row_" + h.target(); + row.id = "myopac_holds_row_" + h.id() + '_' + h.target(); var formats = (h.holdable_formats()) ? h.holdable_formats() : null; var form = $n(row, "myopac_holds_formats"); - form.id = "myopac_holds_form_" + h.target(); + form.id = "myopac_holds_form_" + h.id() + '_' + h.target(); if(formats) form.appendChild(text(formats)); $n(row, "myopac_holds_location"). @@ -227,20 +227,21 @@ function myOPACDrawHoldTitle(hold) { if(hold.hold_type() == "T") method = FETCH_RMODS; var req = new Request(method, hold.target()); req.callback(myOPACFleshHoldTitle); + req.request.hold = hold.id(); req.send(); } function myOPACFleshHoldTitle(r) { var record = r.getResultObject(); - var row = $("myopac_holds_row_" + record.doc_id()); + var row = $("myopac_holds_row_" + r.hold + '_' + record.doc_id()); var title_link = $n(row, "myopac_holds_title_link"); var author_link = $n(row, "myopac_holds_author_link"); buildTitleDetailLink(record, title_link); buildSearchLink(STYPE_AUTHOR, record.author(), author_link); - var form = $("myopac_holds_form_" + record.doc_id()); + var form = $("myopac_holds_form_" + r.hold + '_' + record.doc_id()); if(form) { var img = elem("img"); -- 2.43.2