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>


Reply via email to