[open-ils-commits] r17283 - trunk/Open-ILS/web/js/ui/default/acq/invoice (erickson)

[svn at svn.open-ils.org]

Author: erickson
Date: 2010-08-20 10:11:27 -0400 (Fri, 20 Aug 2010)
New Revision: 17283

Modified:
   trunk/Open-ILS/web/js/ui/default/acq/invoice/view.js
Log:
repair faulty lineitem detachment from invoices

Modified: trunk/Open-ILS/web/js/ui/default/acq/invoice/view.js
===================================================================
--- trunk/Open-ILS/web/js/ui/default/acq/invoice/view.js	2010-08-20 14:01:26 UTC (rev 17282)
+++ trunk/Open-ILS/web/js/ui/default/acq/invoice/view.js	2010-08-20 14:11:27 UTC (rev 17283)
@@ -472,11 +472,19 @@
 
     nodeByName('detach', row).onclick = function() {
         var cost = widgetRegistry.acqie[entry.id()].cost_billed.getFormattedValue();
+        var idents = [];
+        dojo.forEach(['isbn', 'upc', 'issn'], 
+            function(ident) { 
+                var val = liMarcAttr(entry.lineitem(), ident);
+                if(val) idents.push(val); 
+            }
+        );
+
         var msg = dojo.string.substitute(
             localeStrings.INVOICE_CONFIRM_ENTRY_DETACH, [
                 cost || 0,
-                liMarcAttr(lineitem, 'title'),
-                liMarcAttr(lineitem, 'author'),
+                liMarcAttr(entry.lineitem(), 'title'),
+                liMarcAttr(entry.lineitem(), 'author'),
                 idents.join(',')
             ]
         );