You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by jl...@apache.org on 2013/12/04 08:24:26 UTC
svn commit: r1547719 -
/ofbiz/trunk/applications/accounting/widget/InvoiceForms.xml
Author: jleroux
Date: Wed Dec 4 07:24:26 2013
New Revision: 1547719
URL: http://svn.apache.org/r1547719
Log:
Fixes an issue when no amount have been specified while creating an invoice item
Better groovy scriplets (elvis operator)
I believe using BigDecimal for both fields quantity and total can't hurt
Increases size of both fields quantity and total
Modified:
ofbiz/trunk/applications/accounting/widget/InvoiceForms.xml
Modified: ofbiz/trunk/applications/accounting/widget/InvoiceForms.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/accounting/widget/InvoiceForms.xml?rev=1547719&r1=1547718&r2=1547719&view=diff
==============================================================================
--- ofbiz/trunk/applications/accounting/widget/InvoiceForms.xml (original)
+++ ofbiz/trunk/applications/accounting/widget/InvoiceForms.xml Wed Dec 4 07:24:26 2013
@@ -142,8 +142,8 @@ under the License.
<form name="invoiceItems" list-name="invItemAndOrdItems" target="" title="" type="list" separate-columns="true" paginate-target="invoiceOverview"
odd-row-style="alternate-row" default-table-style="basic-table hover-bar">
<row-actions>
- <set field="quantity" value="${groovy: quantity==null?1:quantity}"/>
- <set field="total" value="${quantity * amount}"/>
+ <set field="quantity" value="${groovy: quantity ?: 1}" type="BigDecimal"/>
+ <set field="total" value="${quantity * amount ?: 0}" type="BigDecimal"/>
</row-actions>
<auto-fields-entity entity-name="InvoiceItem" default-field-type="display"/>
<field name="invoiceId"><hidden/></field>
@@ -348,10 +348,7 @@ under the License.
<form name="EditInvoiceItems" type="multi" list-name="invoiceItems" default-entity-name="InvoiceItem" target="updateInvoiceItem?invoiceId=${invoiceId}&viewSize=${viewSize}&viewIndex=${viewIndex}" title="Edit Invoice Items" paginate-target="listInvoiceItems"
odd-row-style="alternate-row" default-table-style="basic-table hover-bar">
<row-actions>
- <set field="total" value="${groovy:
- import java.math.BigDecimal;
- if(quantity == null) quantity = BigDecimal.ONE;
- return(quantity.multiply(amount));}"/>
+ <set field="total" value="${groovy: (quantity ?: 1) * (amount ?: 0)}" type="BigDecimal"/>
</row-actions>
<field name="viewSize"><hidden value="${viewSize}"/></field>
<field name="viewIndex"><hidden value="${viewIndex}"/></field>
@@ -364,7 +361,7 @@ under the License.
<parameter param-name="invoiceItemSeqId"/>
</hyperlink>
</field>
- <field name="quantity"><text size="3"/></field>
+ <field name="quantity"><text size="10"/></field>
<field name="invoiceItemTypeId">
<drop-down allow-empty="false">
<entity-options entity-name="InvoiceItemType" description="${description}" key-field-name="invoiceItemTypeId"/>
@@ -377,7 +374,7 @@ under the License.
<list-options list-name="glAccountOrganizationAndClassList" key-name="glAccountId" description="${glAccountId} ${accountName}"/>
</drop-down>
</field>
- <field name="amount" title="${uiLabelMap.AccountingUnitPrice}"><text size="7"/></field>
+ <field name="amount" title="${uiLabelMap.AccountingUnitPrice}"><text size="10"/></field>
<field name="total" widget-area-style="align-text"><display type="currency" currency="${invoice.currencyUomId}"/></field> <!-- this should not look like something you can modify, hence display -->
<field name="updateButton" title="${uiLabelMap.CommonUpdate}" widget-style="buttontext" ><submit button-type="text-link"/></field>
<field name="removeButton" title=" " widget-style="buttontext">