Changeset 441 for Dev/trunk/client/qed/model/widgets/QuestionEditorToolkit.js

Timestamp:

04/29/13 19:35:10 (12 years ago)

Author:

hendrikvanantwerpen

Message:

Big cleanup of the question content.

• Replaced old list implementations with a new one that behaves like a form widget.
• All question content is now in separate widgets, not in the factory itself.
• Added form and widget validation for question editing.

File:

• Dev/trunk/client/qed/model/widgets/QuestionEditorToolkit.js (modified) (8 diffs)

Dev/trunk/client/qed/model/widgets/QuestionEditorToolkit.js

@@ -25 +25 @@
         _topicSelect: null,
         _topicStore: null,
-        _categories : null,
 
         _contentItems: [
@@ -37 +36 @@
             { type: "StringInput" },
             { type: "TextInput" },
-            { type: "IntegerInput" },
+            { type: "NumberInput" },
             { type: "ScaleInput" },
             { type: "MultipleChoiceInput" }
@@ -49 +48 @@
             "StringInput": "Text line",
             "TextInput": "Free text",
-            "IntegerInput": "Integer number",
+            "NumberInput": "Number",
             "ScaleInput": "Scale",
             "MultipleChoiceInput": "Multiple choice"
@@ -61 +60 @@
             "StringInput": "Text",
             "TextInput": "Text",
-            "IntegerInput": "Integer",
+            "NumberInput": "Number",
             "ScaleInput": "Scale",
             "MultipleChoiceInput": "MultipleChoice"
@@ -87 +86 @@
         },
         _setValueAttr: function(question) {
-            this.propertiesForm.set('value', question);
-            this._categories = question.categories || [];
             this._setupListView();
             this._setupCategories();
             this._setupTopic(question.topic);
+            this.propertiesForm.set('value', question);
+            this._list.set('value', question.categories);
         },
         _getValueAttr: function() {
             var value = this.propertiesForm.get('value');
-            value.categories = this._categories;
+            value.categories = this._list.get('value');
             return value;
         },
@@ -125 +124 @@
         },
         _setupListView: function() {
-            this._list = new CategoryListView( {
-                removeCallback: lang.hitch(this, this._removeCategory)
+            this._list = new CategoryListView({
             }).placeAt(this.listNode);
             this._list.startup();                
-            for (var category in this._categories) {
-                this._list.appendItem(this._categories[category]);
-            }    
         },
         _setupCategories: function() {
@@ -144 +139 @@
                 searchAttr: "id"
             }, this.categoriesBoxNode);
-            this._supportingWidgets.push(this._categorySelect);
-
         },
         _setupTopic: function(topic) {
@@ -159 +152 @@
                 value: topic
             }, this.topicBoxNode);
-            this._supportingWidgets.push(this._topicSelect);
         },
         _addCategory: function(item) {
-            this._categories.push(item);
             this._list.appendItem(item);
-        },
-        _removeCategory: function(item) {
-            this._categories.splice(this._categories.indexOf(item), 1);
         }