Author: sichen
Date: Tue Feb  6 16:44:54 2007
New Revision: 504389

URL: http://svn.apache.org/viewvc?view=rev&rev=504389
Log:
Safari doesn't honour the onchange attribute for radio buttons, so firing both 
onchange and onClick events when a payment method type is selected during 
ordermgr normal checkout, in the payment screen when no payment methods are 
available.

Modified:
    ofbiz/trunk/applications/order/webapp/ordermgr/entry/billsettings.ftl

Modified: ofbiz/trunk/applications/order/webapp/ordermgr/entry/billsettings.ftl
URL: 
http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/entry/billsettings.ftl?view=diff&rev=504389&r1=504388&r2=504389
==============================================================================
--- ofbiz/trunk/applications/order/webapp/ordermgr/entry/billsettings.ftl 
(original)
+++ ofbiz/trunk/applications/order/webapp/ordermgr/entry/billsettings.ftl Tue 
Feb  6 16:44:54 2007
@@ -521,23 +521,23 @@
             <table width="100%" border="0" cellpadding="1" cellspacing="0"> 
               <#if !requestParameters.createNew?exists>                        
            
               <tr>
-                <td width='1%' nowrap><input type="radio" 
name="paymentMethodTypeAndId" value="EXT_OFFLINE" <#if checkOutPaymentId?exists 
&& checkOutPaymentId == "EXT_OFFLINE">checked="checked"</#if> 
onchange="setCheckoutPaymentId(this.value)"/></td>
+                <td width='1%' nowrap><input type="radio" 
name="paymentMethodTypeAndId" value="EXT_OFFLINE" <#if checkOutPaymentId?exists 
&& checkOutPaymentId == "EXT_OFFLINE">checked="checked"</#if> 
onchange="setCheckoutPaymentId(this.value)" 
onClick="setCheckoutPaymentId(this.value)"/></td>
                 <td width='50%'nowrap><div 
class="tabletext">${uiLabelMap.OrderPaymentOfflineCheckMoney}</div></td>
               </tr>
               <tr><td colspan="2"><hr class="sepbar"/></td></tr>
               <tr>
-                <td width="1%" nowrap><input type="radio" 
name="paymentMethodTypeAndId" value="EXT_COD" <#if checkOutPaymentId?exists && 
checkOutPaymentId == "EXT_COD">checked="checked"</#if> 
onchange="setCheckoutPaymentId(this.value)"/></td>
+                <td width="1%" nowrap><input type="radio" 
name="paymentMethodTypeAndId" value="EXT_COD" <#if checkOutPaymentId?exists && 
checkOutPaymentId == "EXT_COD">checked="checked"</#if> 
onchange="setCheckoutPaymentId(this.value)" 
onClick="setCheckoutPaymentId(this.value)"/></td>
                 <td width="50%"nowrap><div 
class="tabletext">${uiLabelMap.OrderCOD}</div></td>
               </tr>
               <tr><td colspan="2"><hr class="sepbar"/></td></tr>
               </#if>
               <tr>
-                <td width='1%' nowrap><input type="radio" 
name="paymentMethodTypeAndId" value="CC" 
onchange="setCheckoutPaymentId(this.value)"/>
+                <td width='1%' nowrap><input type="radio" 
name="paymentMethodTypeAndId" value="CC" 
onchange="setCheckoutPaymentId(this.value)" 
onClick="setCheckoutPaymentId(this.value)"/>
                 <td width='50%' nowrap><div 
class="tabletext">${uiLabelMap.AccountingVisaMastercardAmexDiscover}</div></td>
               </tr>
               <tr><td colspan="2"><hr class="sepbar"/></td></tr>
               <tr>
-                <td width='1%' nowrap><input type="radio" 
name="paymentMethodTypeAndId" value="EFT" 
onchange="setCheckoutPaymentId(this.value)"/>
+                <td width='1%' nowrap><input type="radio" 
name="paymentMethodTypeAndId" value="EFT" 
onchange="setCheckoutPaymentId(this.value)" 
onClick="setCheckoutPaymentId(this.value)"/>
                 <td width='50%' nowrap><div 
class="tabletext">${uiLabelMap.AccountingAHCElectronicCheck}</div></td>
               </tr>
             </table>


Reply via email to