[OFBiz] SVN: r6883 - trunk/applications/accounting/webapp/accounting/payment

sichen@svn.ofbiz.org sichen at svn.ofbiz.org
Wed Mar 1 19:18:38 CST 2006


Author: sichen
Date: 2006-03-01 19:18:34 -0600 (Wed, 01 Mar 2006)
New Revision: 6883

Modified:
   trunk/applications/accounting/webapp/accounting/payment/PaymentTabBar.ftl
Log:
Fixed bug where the payment tab bar crashes for payments with no statusId (those which are only for internal recordkeeping and are not disbursements or receipts)

Modified: trunk/applications/accounting/webapp/accounting/payment/PaymentTabBar.ftl
===================================================================
--- trunk/applications/accounting/webapp/accounting/payment/PaymentTabBar.ftl	2006-03-02 00:54:19 UTC (rev 6882)
+++ trunk/applications/accounting/webapp/accounting/payment/PaymentTabBar.ftl	2006-03-02 01:18:34 UTC (rev 6883)
@@ -21,6 +21,8 @@
 
 <div class="tabContainer">
       <a href="<@ofbizUrl>paymentOverview?paymentId=${payment.paymentId}</@ofbizUrl>" class="${selectedClassMap.paymentOverview?default(unselectedClassName)}">Overview</a>
+      <#-- Payments that are a matter of internal record keeping do not have statusId, so we check that first -->
+      <#if payment.statusId?exists>
       <#if payment.statusId == "PMNT_NOT_PAID">
           <a href="<@ofbizUrl>editPayment?paymentId=${payment.paymentId}</@ofbizUrl>" class="${selectedClassMap.editPayment?default(unselectedClassName)}">Header</a>
       </#if>
@@ -28,9 +30,12 @@
           <a href="<@ofbizUrl>editPaymentApplications?paymentId=${payment.paymentId}</@ofbizUrl>" class="${selectedClassMap.editPaymentApplications?default(unselectedClassName)}">Applications</a>
       </#if>
       <!--a href="<@ofbizUrl>payment.pdf?paymentId=${payment.paymentId}</@ofbizUrl>" class="${unselectedClassName}">${uiLabelMap.AccountingpaymentPDF}</a-->
+      </#if>
 </div>
 <div>
 <a href="<@ofbizUrl>editPayment</@ofbizUrl>" class="buttontext">Create new Payment</a>
+<#-- Payments that are a matter of internal record keeping do not have statusId, so we check that first -->
+<#if payment.statusId?exists>
 <#if payment.statusId == "PMNT_NOT_PAID">
 <!-- the SENT status is only for disbursements -->
   <#if Static['org.ofbiz.accounting.util.UtilAccounting'].isDisbursement(payment)>
@@ -46,5 +51,6 @@
 <#if payment.statusId == "PMNT_NOT_PAID">
     <a href="javascript:confirmActionLink('You want to cancel this payment number ${payment.paymentId}?','setPaymentStatus?paymentId=${payment.paymentId}&statusId=PMNT_CANCELLED')" class="buttontext">Status to 'Cancelled'</a>
 </#if>
+</#if>
 </div>
 <br/>



More information about the Svn mailing list