Changeset 461 for Dev/trunk/src/client/qed-client/widgets/_ComplexValueMixin.js

Timestamp:

06/10/13 01:07:16 (12 years ago)

Author:

hendrikvanantwerpen

Message:

Save answers, improved response page and question widgets.

• Warn if _ComplexValueMixin is not pout on a form element, to prevent 'name' collisions. Changed all depending widgets to <form>'s.
• Fixed names for question widgets to actually save answers.
• Fixed and improved response page.
• Fixed names in MultipleChoice? widget, which behaves different for radio and check.

File:

• Dev/trunk/src/client/qed-client/widgets/_ComplexValueMixin.js (modified) (3 diffs)

Dev/trunk/src/client/qed-client/widgets/_ComplexValueMixin.js

@@ -3 +3 @@
     "dijit/form/_FormMixin",
     "dojo/_base/array",
-    "dojo/_base/declare"
-], function(_Container, _FormMixin, array, declare) {
+    "dojo/_base/declare",
+    "dojo/_base/event"
+], function(_Container, _FormMixin, array, declare, event) {
     return declare([_Container,_FormMixin],{
         name: "",
@@ -10 +11 @@
         disabled: false,
         readOnly: false,
+        postCreate: function() {
+            this.inherited(arguments);
+            if ( this.domNode.tagName.toLowerCase() !== "form" ) {
+                console.warn("Not scoping a _ComplexValueMixin in a form element can cause name clashes. E.g. radio buttons might stop working correctly. It is recommended to use <form> as the root element in your template for "+this.declaredClass+".");
+            }
+        },
         _setDisabledAttr: function(value) {
             this._set("disabled", value);
@@ -27 +34 @@
                 children[0].focus();
             }
+        },
+        onSubmit: function(e) {
+            // since this widget is used to create more complex
+            // widgets within other forms, the onSubmit must either be
+            // ignored or propagated, but not handled here.
+            if ( e ) { event.stop(e); }
+            return false;
+        },
+        _onSubmit: function(e) {
+            // since this widget is used to create more complex
+            // widgets within other forms, the onSubmit must either be
+            // ignored or propagated, but not handled here.
+            if ( e ) { event.stop(e); }
+            return false;
         }
     });