You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by as...@apache.org on 2009/11/30 13:51:50 UTC

svn commit: r885403 - in /ofbiz/trunk/applications/accounting: config/AccountingUiLabels.xml webapp/accounting/finaccounttrans/FinAccountTrans.ftl

Author: ashish
Date: Mon Nov 30 12:51:49 2009
New Revision: 885403

URL: http://svn.apache.org/viewvc?rev=885403&view=rev
Log:
Applied patch from jira issue: OFBIZ-3270 - Financial Account Transaction - Enhance Deposit Slip Popup.
Improve Deposit Slip Popup according to following points
1) Add a new column of deposit slip id (PaymentGroupId) with hyperlink.
2) Give hyperlink to paymentId
3) Show Party name instead of partyId and link it.
4) Show currency with payment amount.

Thanks Parimal & Sumit for the contribution.

Modified:
    ofbiz/trunk/applications/accounting/config/AccountingUiLabels.xml
    ofbiz/trunk/applications/accounting/webapp/accounting/finaccounttrans/FinAccountTrans.ftl

Modified: ofbiz/trunk/applications/accounting/config/AccountingUiLabels.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/accounting/config/AccountingUiLabels.xml?rev=885403&r1=885402&r2=885403&view=diff
==============================================================================
--- ofbiz/trunk/applications/accounting/config/AccountingUiLabels.xml (original)
+++ ofbiz/trunk/applications/accounting/config/AccountingUiLabels.xml Mon Nov 30 12:51:49 2009
@@ -2707,6 +2707,9 @@
         <value xml:lang="en">Deposit Slip</value>
         <value xml:lang="hi_IN">जमा पर्ची</value>
     </property>
+    <property key="AccountingDepositSlipId">
+        <value xml:lang="en">Deposit Slip Id</value>
+    </property>
     <property key="AccountingDepositSlips">
         <value xml:lang="en">Deposit Slips</value>
         <value xml:lang="hi_IN">जमा पर्चीयां</value>

Modified: ofbiz/trunk/applications/accounting/webapp/accounting/finaccounttrans/FinAccountTrans.ftl
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/accounting/webapp/accounting/finaccounttrans/FinAccountTrans.ftl?rev=885403&r1=885402&r2=885403&view=diff
==============================================================================
--- ofbiz/trunk/applications/accounting/webapp/accounting/finaccounttrans/FinAccountTrans.ftl (original)
+++ ofbiz/trunk/applications/accounting/webapp/accounting/finaccounttrans/FinAccountTrans.ftl Mon Nov 30 12:51:49 2009
@@ -178,12 +178,13 @@
             <td>
               <#if payments?has_content>
                 <a id="togglePayment_${finAccountTrans.finAccountTransId}" href="javascript:void(0)"><img src="<@o...@ofbizContentUrl>"/></a> ${finAccountTrans.finAccountTransId}
-                <div id="displayPayments_${finAccountTrans.finAccountTransId}" class="popup" style="display: none;width: 500px;">
+                <div id="displayPayments_${finAccountTrans.finAccountTransId}" class="popup" style="display: none;width: 650px;">
                   <div align="right">
                     <input class="popup_closebox buttontext" type="button" value="X"/>
                   </div>
                   <table class="basic-table hover-bar" cellspacing="0" style"width :">
                     <tr class="header-row-2">
+                      <th>${uiLabelMap.AccountingDepositSlipId}</th>
                       <th>${uiLabelMap.FormFieldTitle_paymentId}</th>
                       <th>${uiLabelMap.OrderPaymentType}</th>
                       <th>${uiLabelMap.FormFieldTitle_paymentMethodTypeId}</th>
@@ -198,13 +199,30 @@
                       <#if payment?has_content && payment.paymentMethodTypeId?has_content>
                         <#assign paymentMethodType = delegator.findOne("PaymentMethodType", {"paymentMethodTypeId" : payment.paymentMethodTypeId}, true)>
                       </#if>
+                      <#if payment?has_content>
+                        <#assign paymentGroupMembers = Static["org.ofbiz.entity.util.EntityUtil"].filterByDate(payment.getRelated("PaymentGroupMember")?if_exists) />
+                        <#assign fromParty = payment.getRelatedOne("FromParty")?if_exists />
+                        <#assign fromPartyName = fromParty.getRelatedOne("Person")?if_exists />
+                        <#if !(fromPartyName?has_content)>
+                          <#assign fromPartyName = fromParty.getRelatedOne("PartyGroup")?if_exists />
+                        </#if>
+                        <#assign toParty = payment.getRelatedOne("ToParty")?if_exists />
+                        <#assign toPartyName = toParty.getRelatedOne("Person")?if_exists />
+                        <#if !(toPartyName?has_content)>
+                          <#assign toPartyName = toParty.getRelatedOne("PartyGroup")?if_exists />
+                        </#if>
+                        <#if paymentGroupMembers?has_content>
+                          <#assign paymentGroupMember = Static["org.ofbiz.entity.util.EntityUtil"].getFirst(paymentGroupMembers) />
+                        </#if>
+                      </#if>
                       <tr valign="middle"<#if alt_row> class="alternate-row"</#if>>
-                        <td>${payment.paymentId?if_exists}</td>
+                        <td><#if paymentGroupMember?has_content><a href="<@o...@ofbizUrl>">${paymentGroupMember.paymentGroupId?if_exists}</a></#if></td>
+                        <td><#if payment?has_content><a href="<@o...@ofbizUrl>">${payment.paymentId?if_exists}</a></#if></td>
                         <td><#if paymentType?has_content>${paymentType.description?if_exists}</#if></td>
                         <td><#if paymentMethodType?has_content>${paymentMethodType.description?if_exists}</#if></td>
-                        <td>${payment.amount?if_exists}</td>
-                        <td>${payment.partyIdFrom?if_exists}</td>
-                        <td>${payment.partyIdTo?if_exists}</td>
+                        <td><@ofbizCurrency amount=payment.amount?if_exists/></td>
+                        <td><#if fromPartyName?has_content>${fromPartyName.groupName?if_exists}${fromPartyName.firstName?if_exists} ${fromPartyName.lastName?if_exists}<a href="/partymgr/control/viewprofile?partyId=${fromPartyName.partyId?if_exists}">[${fromPartyName.partyId?if_exists}]</a></#if></td>
+                        <td><#if toPartyName?has_content>${toPartyName.groupName?if_exists}${toPartyName.firstName?if_exists} ${toPartyName.lastName?if_exists}<a href="/partymgr/control/viewprofile?partyId=${toPartyName.partyId?if_exists}">[${toPartyName.partyId?if_exists}]</a></#if></td>
                       </tr>
                     </#list>
                   </table>