Jacques,
is there a way to implement this in a cleaner way?
I really don't know anything about this screen, but if we really need to
provide a non null string (but are we sure that it is not just a
configuration issue?) maybe we should adopt a more standard name: for
example "_NA_" instead of "Select one".
Jacopo
[EMAIL PROTECTED] wrote:
Author: jleroux
Date: Tue Dec 18 12:58:04 2007
New Revision: 605326
URL: http://svn.apache.org/viewvc?rev=605326&view=rev
Log:
A patch from BJ Freeman "Can not activate the Payment Authorization Service in
accounting ManualTransaction" (https://issues.apache.org/jira/browse/OFBIZ-1350) -
OFBIZ-1350
Modified:
ofbiz/trunk/applications/accounting/webapp/accounting/payment/manualTx.ftl
Modified:
ofbiz/trunk/applications/accounting/webapp/accounting/payment/manualTx.ftl
URL:
http://svn.apache.org/viewvc/ofbiz/trunk/applications/accounting/webapp/accounting/payment/manualTx.ftl?rev=605326&r1=605325&r2=605326&view=diff
==============================================================================
--- ofbiz/trunk/applications/accounting/webapp/accounting/payment/manualTx.ftl
(original)
+++ ofbiz/trunk/applications/accounting/webapp/accounting/payment/manualTx.ftl
Tue Dec 18 12:58:04 2007
@@ -67,6 +67,8 @@
<input type="hidden" name="transactionType"
value="${currentTx.enumId}">
<#else>
<select name="transactionType"
onchange="javascript:document.manualTxForm.submit();">
+ <#-- the select one option is so the list will fire on any seletion
-->
+ <option value="Select one">${uiLabelMap.CommonSelectOne}</option>
<#list paymentSettings as setting>
<option
value="${setting.enumId}">${setting.get("description",locale)}</option>
</#list>