You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by di...@apache.org on 2016/10/04 14:21:17 UTC
svn commit: r1763281 -
/ofbiz/trunk/applications/party/template/party/profileblocks/PaymentMethods.ftl
Author: diveshdutta
Date: Tue Oct 4 14:21:17 2016
New Revision: 1763281
URL: http://svn.apache.org/viewvc?rev=1763281&view=rev
Log:
Fixed: Payment section was breaking up when in Party profile page when Party does not have Check account.
(OFBIZ-6899)
Thanks: Japreet for the fix.
Modified:
ofbiz/trunk/applications/party/template/party/profileblocks/PaymentMethods.ftl
Modified: ofbiz/trunk/applications/party/template/party/profileblocks/PaymentMethods.ftl
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/party/template/party/profileblocks/PaymentMethods.ftl?rev=1763281&r1=1763280&r2=1763281&view=diff
==============================================================================
--- ofbiz/trunk/applications/party/template/party/profileblocks/PaymentMethods.ftl (original)
+++ ofbiz/trunk/applications/party/template/party/profileblocks/PaymentMethods.ftl Tue Oct 4 14:21:17 2016
@@ -124,56 +124,68 @@ under the License.
<a href="<@o...@ofbizUrl>">${uiLabelMap.CommonUpdate}</a>
</#if>
<#elseif "COMPANY_CHECK" == paymentMethod.paymentMethodTypeId>
- <#assign checkAccount = paymentMethodValueMap.companyCheckAccount>
- <td class="label" valign="top">
- <#-- TODO: Convert hard-coded text to UI label properties -->
- Company Check
- </td>
- <td>
- ${checkAccount.nameOnAccount} - <#if checkAccount.bankName?has_content>${uiLabelMap.PartyBank}: ${checkAccount.bankName}</#if>
- <#if checkAccount.accountNumber?has_content>${uiLabelMap.PartyAccount} #: ${checkAccount.accountNumber}</#if>
- <#if paymentMethod.description?has_content>(${paymentMethod.description})</#if>
- <#if paymentMethod.glAccountId?has_content>(for GL Account ${paymentMethod.glAccountId})</#if>
- <#if paymentMethod.fromDate?has_content>(${uiLabelMap.CommonUpdated}: ${paymentMethod.fromDate!})</#if>
- <#if paymentMethod.thruDate?has_content>(${uiLabelMap.PartyContactEffectiveThru}: ${paymentMethod.thruDate.toString()}</#if>
- </td>
- <td class="button-col">
- <#if security.hasEntityPermission("PAY_INFO", "_UPDATE", session) || security.hasEntityPermission("ACCOUNTING", "_UPDATE", session)>
- <a href="<@o...@ofbizUrl>">${uiLabelMap.CommonUpdate}</a>
+ <#if paymentMethodValueMap.companyCheckAccount?has_content>
+ <#assign checkAccount = paymentMethodValueMap.companyCheckAccount>
+ </#if>
+ <#if checkAccount?has_content>
+ <td class="label" valign="top">
+ <#-- TODO: Convert hard-coded text to UI label properties -->
+ Company Check
+ </td>
+ <td>
+ ${checkAccount.nameOnAccount} - <#if checkAccount.bankName?has_content>${uiLabelMap.PartyBank}: ${checkAccount.bankName}</#if>
+ <#if checkAccount.accountNumber?has_content>${uiLabelMap.PartyAccount} #: ${checkAccount.accountNumber}</#if>
+ <#if paymentMethod.description?has_content>(${paymentMethod.description})</#if>
+ <#if paymentMethod.glAccountId?has_content>(for GL Account ${paymentMethod.glAccountId})</#if>
+ <#if paymentMethod.fromDate?has_content>(${uiLabelMap.CommonUpdated}: ${paymentMethod.fromDate!})</#if>
+ <#if paymentMethod.thruDate?has_content>(${uiLabelMap.PartyContactEffectiveThru}: ${paymentMethod.thruDate.toString()}</#if>
+ </td>
+ <td class="button-col">
+ <#if security.hasEntityPermission("PAY_INFO", "_UPDATE", session) || security.hasEntityPermission("ACCOUNTING", "_UPDATE", session)>
+ <a href="<@o...@ofbizUrl>">${uiLabelMap.CommonUpdate}</a>
+ </#if>
</#if>
<#elseif "PERSONAL_CHECK" == paymentMethod.paymentMethodTypeId>
- <#assign checkAccount = paymentMethodValueMap.personalCheckAccount>
- <td class="label" valign="top">
- Personal Check
- </td>
- <td>
- ${checkAccount.nameOnAccount} - <#if checkAccount.bankName?has_content>${uiLabelMap.PartyBank}: ${checkAccount.bankName}</#if>
- <#if checkAccount.accountNumber?has_content>${uiLabelMap.PartyAccount} #: ${checkAccount.accountNumber}</#if>
- <#if paymentMethod.description?has_content>(${paymentMethod.description})</#if>
- <#if paymentMethod.glAccountId?has_content>(for GL Account ${paymentMethod.glAccountId})</#if>
- <#if paymentMethod.fromDate?has_content>(${uiLabelMap.CommonUpdated}: ${paymentMethod.fromDate!})</#if>
- <#if paymentMethod.thruDate?has_content>(${uiLabelMap.PartyContactEffectiveThru}: ${paymentMethod.thruDate.toString()}</#if>
- </td>
- <td class="button-col">
- <#if security.hasEntityPermission("PAY_INFO", "_UPDATE", session) || security.hasEntityPermission("ACCOUNTING", "_UPDATE", session)>
- <a href="<@o...@ofbizUrl>">${uiLabelMap.CommonUpdate}</a>
+ <#if paymentMethodValueMap.personalCheckAccount?has_content>
+ <#assign checkAccount = paymentMethodValueMap.personalCheckAccount>
+ </#if>
+ <#if checkAccount?has_content>
+ <td class="label" valign="top">
+ Personal Check
+ </td>
+ <td>
+ ${checkAccount.nameOnAccount} - <#if checkAccount.bankName?has_content>${uiLabelMap.PartyBank}: ${checkAccount.bankName}</#if>
+ <#if checkAccount.accountNumber?has_content>${uiLabelMap.PartyAccount} #: ${checkAccount.accountNumber}</#if>
+ <#if paymentMethod.description?has_content>(${paymentMethod.description})</#if>
+ <#if paymentMethod.glAccountId?has_content>(for GL Account ${paymentMethod.glAccountId})</#if>
+ <#if paymentMethod.fromDate?has_content>(${uiLabelMap.CommonUpdated}: ${paymentMethod.fromDate!})</#if>
+ <#if paymentMethod.thruDate?has_content>(${uiLabelMap.PartyContactEffectiveThru}: ${paymentMethod.thruDate.toString()}</#if>
+ </td>
+ <td class="button-col">
+ <#if security.hasEntityPermission("PAY_INFO", "_UPDATE", session) || security.hasEntityPermission("ACCOUNTING", "_UPDATE", session)>
+ <a href="<@o...@ofbizUrl>">${uiLabelMap.CommonUpdate}</a>
+ </#if>
</#if>
<#elseif "CERTIFIED_CHECK" == paymentMethod.paymentMethodTypeId>
- <#assign checkAccount = paymentMethodValueMap.certifiedCheckAccount>
- <td class="label" valign="top">
- Certified Check
- </td>
- <td>
- ${checkAccount.nameOnAccount} - <#if checkAccount.bankName?has_content>${uiLabelMap.PartyBank}: ${checkAccount.bankName}</#if>
- <#if checkAccount.accountNumber?has_content>${uiLabelMap.PartyAccount} #: ${checkAccount.accountNumber}</#if>
- <#if paymentMethod.description?has_content>(${paymentMethod.description})</#if>
- <#if paymentMethod.glAccountId?has_content>(for GL Account ${paymentMethod.glAccountId})</#if>
- <#if paymentMethod.fromDate?has_content>(${uiLabelMap.CommonUpdated}: ${paymentMethod.fromDate!})</#if>
- <#if paymentMethod.thruDate?has_content>(${uiLabelMap.PartyContactEffectiveThru}: ${paymentMethod.thruDate.toString()}</#if>
- </td>
- <td class="button-col">
- <#if security.hasEntityPermission("PAY_INFO", "_UPDATE", session) || security.hasEntityPermission("ACCOUNTING", "_UPDATE", session)>
- <a href="<@o...@ofbizUrl>">${uiLabelMap.CommonUpdate}</a>
+ <#if paymentMethodValueMap.certifiedCheckAccount?has_content>
+ <#assign checkAccount = paymentMethodValueMap.certifiedCheckAccount>
+ </#if>
+ <#if checkAccount?has_content>
+ <td class="label" valign="top">
+ Certified Check
+ </td>
+ <td>
+ ${checkAccount.nameOnAccount} - <#if checkAccount.bankName?has_content>${uiLabelMap.PartyBank}: ${checkAccount.bankName}</#if>
+ <#if checkAccount.accountNumber?has_content>${uiLabelMap.PartyAccount} #: ${checkAccount.accountNumber}</#if>
+ <#if paymentMethod.description?has_content>(${paymentMethod.description})</#if>
+ <#if paymentMethod.glAccountId?has_content>(for GL Account ${paymentMethod.glAccountId})</#if>
+ <#if paymentMethod.fromDate?has_content>(${uiLabelMap.CommonUpdated}: ${paymentMethod.fromDate!})</#if>
+ <#if paymentMethod.thruDate?has_content>(${uiLabelMap.PartyContactEffectiveThru}: ${paymentMethod.thruDate.toString()}</#if>
+ </td>
+ <td class="button-col">
+ <#if security.hasEntityPermission("PAY_INFO", "_UPDATE", session) || security.hasEntityPermission("ACCOUNTING", "_UPDATE", session)>
+ <a href="<@o...@ofbizUrl>">${uiLabelMap.CommonUpdate}</a>
+ </#if>
</#if>
<#else>
<td class="button-col">