Changeset 493 for Dev/trunk/src/client/qed-client/pages/question.js

Timestamp:

03/09/14 19:25:40 (11 years ago)

Author:

hendrikvanantwerpen

Message:

• _ComplexValueMixin propagates priorityChange to children.
• Deserialize updated docs after save too.
• Validate to false if definitions are missing.

File:

• Dev/trunk/src/client/qed-client/pages/question.js (modified) (3 diffs)

Dev/trunk/src/client/qed-client/pages/question.js

@@ -25 +25 @@
         _refresh: function() {
             this.titleNode.innerHTML = this.object.title || "";
-            this.propertiesForm.set('value',this.object);
-            this.contentList.set('value',this.object.content);
+            this.propertiesForm.set('value',this.object,null);
+            this.contentList.set('value',this.object.content,null);
         },
         _handlePropertiesChange: function() {
-            lang.mixin(this.object,this.propertiesForm.get('value'));
+            this._updateObject();
             this.markDirty();
             this._refresh();
         },
         _handleContentChange: function() {
-            this.object.content = this.contentList.get('value');
+            this._updateObject();
             this.markDirty();
             this._refresh();
         },
+        _updateObject: function() {
+            var pValid = this.propertiesForm.validate();
+            if ( pValid ) {
+                lang.mixin(this.object,this.propertiesForm.get('value'));
+            }
+            var cValid = this.contentList.validate();
+            if ( cValid ) {
+                this.object.content = this.contentList.get('value');
+            }
+            this._isValid = pValid && cValid;
+            return this._isValid;
+        },
         _save: function() {
-            if ( this._isValid ) {
+            if ( this._updateObject() ) {
                 return this.inherited(arguments);
             } else {
@@ -65 +77 @@
         },
         markDirty: function() {
-            this._isValid = this.propertiesForm.validate() &&
-                            this.contentList.validate();
             this.saveBtn.set('disabled',!this._isValid);
             this.saveAndCloseBtn.set('disabled',!this._isValid);
@@ -73 +83 @@
         },
         markClean: function() {
-            this._isValid = true;
             this.saveBtn.set('disabled',true);
             this.saveAndCloseBtn.set('disabled',true);