Changeset 378 for Dev/branches/rest-dojo-ui/client/rft/ui/QuestionEditorPreviewItem.js

Timestamp:

07/26/12 16:03:40 (13 years ago)

Author:

jkraaijeveld

Message:

Created a factory for the inner widgets on the Question edit screen - can be easily made more generic if necessary too.

File:

• Dev/branches/rest-dojo-ui/client/rft/ui/QuestionEditorPreviewItem.js (modified) (4 diffs)

Dev/branches/rest-dojo-ui/client/rft/ui/QuestionEditorPreviewItem.js

@@ -9 +9 @@
         'dijit/_WidgetsInTemplateMixin',
         'dijit/form/TextBox',
-        'rft/ui/PreviewWidgets/HeaderItem',
-        'rft/ui/PreviewWidgets/TextItem',
+        'rft/ui/InnerWidgetFactory',
         'dojo/text!./templates/QuestionEditorPreviewItem.html',
-        ], function(declare, fx, _WidgetBase, domClass, lang, on, _TemplatedMixin, _WidgetsInTemplateMixin, TextBox, HeaderItem, TextItem, templateFull) {
+        ], function(declare, fx, _WidgetBase, domClass, lang, on, _TemplatedMixin, _WidgetsInTemplateMixin, TextBox, InnerWidgetFactory, templateFull) {
                 return declare("rft.ui.QuestionEditorPreviewItem", [_WidgetBase, _TemplatedMixin, _WidgetsInTemplateMixin], {
 
@@ -42 +41 @@
                                         this.editButtonNode.iconNode.className = this.editButtonNode.iconNode.className.replace("rftIconAccept", "rftIconEdit");
                                         this.editButtonNode.set("label", "Edit");
-                                        this.innerWidget.save();
                                 }
                                 else {
                                         this.editButtonNode.iconNode.className = this.editButtonNode.iconNode.className.replace("rftIconEdit", "rftIconAccept");
                                         this.editButtonNode.set("label", "Save");
-                                        this.innerWidget.edit();
                                 }
                                 this._editing = !this._editing;
@@ -84 +81 @@
                         },
                         getContent: function() {
-                                return {        data: this.innerWidget.getContent(),
-                                                        type: this.item.type };
+                                return this.innerWidget.getObject();
                         },
                         removeObject: function(widget) {
@@ -93 +89 @@
                                 // This always creates a textbox as innerwidget pending creation of actual innerWidgets.
                                 // Introduce a better way to create these widgets than a switch statement, based on item.widgetType? Perhaps "new eval(item.widgetType)({});" ?
+
+                                this.innerWidget = new InnerWidgetFactory().createWidget( this.item );
+                                /*
                                 this.innerWidget = eval("new "+ this.item.type + "({ disabled: true });");
                                 this.innerWidget.setContent(this.item.data);