[open-ils-commits] r16196 - in trunk/Open-ILS/web: js/ui/default/acq/invoice templates/default/acq/po (erickson)

[svn at svn.open-ils.org]

Author: erickson
Date: 2010-04-09 14:35:04 -0400 (Fri, 09 Apr 2010)
New Revision: 16196

Modified:
   trunk/Open-ILS/web/js/ui/default/acq/invoice/view.js
   trunk/Open-ILS/web/templates/default/acq/po/view.tt2
Log:
fixed bug in save+prorate action (passing null ID)
changed default currency textbox width
removed stub number from view invoices link

Modified: trunk/Open-ILS/web/js/ui/default/acq/invoice/view.js
===================================================================
--- trunk/Open-ILS/web/js/ui/default/acq/invoice/view.js	2010-04-09 18:35:03 UTC (rev 16195)
+++ trunk/Open-ILS/web/js/ui/default/acq/invoice/view.js	2010-04-09 18:35:04 UTC (rev 16196)
@@ -168,7 +168,7 @@
     var totalCost = 0;    
     if(!totalInvoicedBox) {
         totalInvoicedBox = new dijit.form.CurrencyTextBox(
-            {style : 'width: 5em'}, dojo.byId('acq-invoice-total-invoiced'));
+            {style : 'width: 6em'}, dojo.byId('acq-invoice-total-invoiced'));
     }
     for(var id in widgetRegistry.acqii) 
         if(!widgetRegistry.acqii[id]._object.isdeleted())
@@ -181,7 +181,7 @@
     totalPaid = 0;    
     if(!totalPaidBox) {
         totalPaidBox = new dijit.form.CurrencyTextBox(
-            {style : 'width: 5em'}, dojo.byId('acq-invoice-total-paid'));
+            {style : 'width: 6em'}, dojo.byId('acq-invoice-total-paid'));
     }
     for(var id in widgetRegistry.acqii) 
         if(!widgetRegistry.acqii[id]._object.isdeleted())
@@ -255,7 +255,7 @@
                     fmClass : 'acqii',
                     fmObject : item,
                     fmField : field,
-                    dijitArgs : (field == 'cost_billed' || field == 'amount_paid') ? {required : true, style : 'width: 5em'} : null,
+                    dijitArgs : (field == 'cost_billed' || field == 'amount_paid') ? {required : true, style : 'width: 6em'} : null,
                     parentNode : nodeByName(field, row)
                 })
             )
@@ -389,7 +389,7 @@
     dojo.forEach(
         ['inv_item_count', 'phys_item_count', 'cost_billed', 'amount_paid'],
         function(field) {
-            var dijitArgs = {required : true, constraints : {min: 0}, style : 'width:5em'};
+            var dijitArgs = {required : true, constraints : {min: 0}, style : 'width:6em'};
             if(entry.isnew() && field == 'phys_item_count') dijitArgs.value = numReceived;
             registerWidget(
                 entry, 
@@ -508,7 +508,7 @@
                 var invoice = openils.Util.readResponse(r);
                 if(invoice) {
                     if(doProrate)
-                        return prorateInvoice();
+                        return prorateInvoice(invoice);
                     location.href = oilsBasePath + '/acq/invoice/view/' + invoice.id();
                 }
             }
@@ -516,7 +516,7 @@
     );
 }
 
-function prorateInvoice() {
+function prorateInvoice(invoice) {
     if(!confirm(localeStrings.INVOICE_CONFIRM_PRORATE)) return;
     progressDialog.show(true);
 

Modified: trunk/Open-ILS/web/templates/default/acq/po/view.tt2
===================================================================
--- trunk/Open-ILS/web/templates/default/acq/po/view.tt2	2010-04-09 18:35:03 UTC (rev 16195)
+++ trunk/Open-ILS/web/templates/default/acq/po/view.tt2	2010-04-09 18:35:04 UTC (rev 16196)
@@ -54,7 +54,7 @@
                 <tr id='acq-po-view-invoices'> <!-- TODO: hidden until applicable -->
                     <td>Invoicing</td>
                     <td>
-                        <a href="javascript:void(0);" id="acq-po-view-invoice-link">View Invoices(0)</a> / 
+                        <a href="javascript:void(0);" id="acq-po-view-invoice-link">View Invoices</a> / 
                         <a href="javascript:void(0);" id="acq-po-create-invoice-link">Create Invoice</a> / 
                         <a href="javascript:void(0);" id="acq-po-link-invoice-link">Link Invoice</a>
                     </td>