[COCOON-1846] [PATCH] BooleanField and radio do not send on-value-changed at the rigth time with IE

• Type: Bug
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Affects Version/s: 2.1.9
• Component/s: Blocks: Forms
• Other Info: Patch available
• Assignee: Simone Gianni
• Reporter: Vincent Demay
• Created: 09/May/06 20:17
• Updated: 10/May/06 15:18

Description

BooleanField and radio list do not send on-value-changed event when the value changed but when the widget loose focus with IE.
So with ajax mode and submit-on-change='true', the form is submitted only when the widget lost the focus.
Here is a patch to use event onclick rather than on value change :


--- forms-field-styling.xsl 2006-05-09 14:53:40.014409000 +0200
+++ forms-field-styling.xsl 2006-05-09 14:45:01.499752000 +0200
@@ -91,11 +91,20 @@
       +-->
   <xsl:template match="fi:*" mode="styling">
     <xsl:apply-templates select="fi:styling/@*" mode="styling"/>
-
     <!-- Auto submit on fields which are listening -->
- <xsl:if test="@listening = 'true' and not(fi:styling/@submit-on-change ='false') and not(fi:styling/@onchange) and not(fi:styling/@list-type = 'double-listbox')">
- <xsl:attribute name="onchange">forms_submitForm(this)</xsl:attribute>
- </xsl:if>
+ <xsl:choose>
+ <xsl:when test="name() = 'fi:booleanfield' or fi:styling/@list-type = 'radio' ">
+ <xsl:if test="@listening = 'true' and not(fi:styling/@submit-on-change = 'false') and not(fi:styling/@onchange) and not(fi:styling/@list-type = 'double-listbox')">
+ <xsl:attribute name="onclick">forms_submitForm(this)</xsl:attribute>
+ </xsl:if>
+ </xsl:when>
+ <xsl:otherwise>
+ <xsl:if test="@listening = 'true' and not(fi:styling/@submit-on-change = 'false') and not(fi:styling/@onchange) and not(fi:styling/@list-type ='double-listbox')">
+ <xsl:attribute name="onchange">forms_submitForm(this)</xsl:attribute>
+ </xsl:if>
+ </xsl:otherwise>
+ </xsl:choose>
+

     <xsl:if test="@state = 'disabled'">
       <xsl:attribute name="disabled">disabled</xsl:attribute>