Changeset 237 for Dev/branches/jos-branch/js/questionEditorScripts.js

Timestamp:

01/17/12 18:17:51 (13 years ago)

Author:

fpvanagthoven

Message:

• Faal met perongeluk de database committen-_- Maar is weer deleted, zou nu weer moeten werken.
• QuestionEditor? werkt nu grotendeels, paar kleine afwerkingsfoutjes (en data die nog niet in de DB passen)
• selectQuestion scherm gemaakt, is nog niet zo multifunctioneel als je zou hopen. Ideaal gezien zou dit ipv een statische PHP pagina, een JS driven widget worden die je bijvoorbeeld kan gebruiken voor "Add existing question" in de surveyEditor.
• Zelfde voor selectApplication.php en selectSurvey.php.
• objectSelectionWidget class maken voor dit doeleind? (Na demo!)

File:

• Dev/branches/jos-branch/js/questionEditorScripts.js (modified) (10 diffs)

Dev/branches/jos-branch/js/questionEditorScripts.js

@@ -21 +21 @@
 }
 
-function updateIdentifier() {
-    var identField = document.getElementById("questionIdentifierField");
-    if (identField.value == undefined && identField.value == "") {
-        return;
-    }
-    var headerField = document.getElementById("header_identifier");
-    headerField.innerHTML = identField.value;
-}
-
-
-//////////////////////////////////////////
-/* QUESTION EDITOR CLASS BASED APPROACH */
-//////////////////////////////////////////
-
 function QuestionEditor() {
     // Properties    
@@ -47 +33 @@
     // Basic functionality
     this.setValues = function(arguments) {
-        var question = JSON.parse(arguments);        
+        debugger;
+        var question = JSON.parse(arguments)[0];
+        var qeTypeField = ge("qeTypeField");
+        ge("qeCodeField").value = question.code;
+        ge("qeBodyTextField").value = question.description;
+        switch (question.type.toLowerCase()) {
+            case "int":
+                // First make sure all the necessary fields are present
+                me.type_Integer();
+                // Then fill them in using the available data
+                qeTypeField.value = "int";
+                ge("qeMinValueField").value = question.minValue;
+                ge("qeMaxValueField").value = question.maxValue;
+                break;
+            case "scale":
+                me.type_Scale();
+                qeTypeField.value = "scale";
+                ge("qeNumChoicesField").value = question.numChoices;
+                ge("qeLegendsEnabledField").checked = question.legendsEnabled;
+                ge("qeLowerLegendField").value = question.lowerLegend;
+                ge("qeUpperLegendField").value = question.upperLegend;
+                break;
+            case "choice":
+                me.type_Choice();
+                qeTypeField.value = "choice";
+                ge("qeNumOptionsFIeld").value = question.numOptions;
+                ge("qeMultipleAnswersField").value = question.multipleAnswers;
+                // then some more to add textboxes (qeParamAnswerGroup) for all the possble choices.
+                // Maybe a central version that appends these groups/textinputs? Maybe not, though. Don't want stuff too abstracted...
+                break;
+            case "text":
+                me.type_Text();
+                qeTypeField.value = "text";
+                ge("qeTextMaxLengthField").value = question.maxTextLength;
+                break;
+                
+        }
     }
     this.init = function() {
@@ -150 +172 @@
     this.save = function() {
         var request = {
-            "title": ge("qeTitleField").value,
-            "type": ge("qeTypeField").value
+            "title": ge("qeTitleField").innerHTML, 
+            "code": ge("qeCodeField").value, 
+            "description": ge("qeBodyTextField").value
         }
         
@@ -158 +181 @@
                 request.answerType = "int";
                 request.minValue = parseInt(ge("qeMinValueField").value);
+                if (request.minValue == "NaN") request.minValue = -1;   // Is this the correct way to do this?
                 request.maxValue = parseInt(ge("qeMaxValueField").value);
+                if (request.maxValue == "NaN") request.maxValue = -1;
+                // Include error checking and form validation!!
                 break;
             case "scale":
@@ -166 +192 @@
                 request.lowerLegend = ge("qeLowerLegendField").value;
                 request.upperLegend = ge("qeUpperLegendField").value;
+                // Include error checking and form validation!!
                 break;
             case "choice":
                 request.answerType = "choice";
+                request.multipleAnswers = ge("qeMultipleAnswersField").checked;
+                request.possibleAnswers = array();
+                var answerFieldset = ge("qeParamsAnswerFieldset");
+                var count = ge("qeNumOptionsField").value;
+                
+                for (var i = 0; i < count; i++) {
+                    var el = ge("qeAnswerField"+i);
+                    request.possibleAnswers.push(el.value);
+                }
+                // Include error checking and form validation!!
                 break;
             case "text":
                 request.answerType = "text";
-                break;
-        }
-        
-        newAjaxRequest(requestString, "createObject.php", function(result){
-            // Display a success message, or throw an error.
-            }, true);
+                request.maxTextLength = ge("qeTextMaxLength").value;
+                // Include error checking and form validation!!
+                break;
+        }
+        
+        requestString = "args="+JSON.stringify(request);
+        
+        newAjaxRequest(requestString, "setQuestion.php", function(result){
+            // Then add the returned uid, if existing, to the sequencer.survey.questions array and set it for update
+            debugger;
+            var response = JSON.parse(result.responseText);
+            console.log(response);
+            if (response.created == true) {
+                if (response.uid) {
+                    // Created a new question, add it to the sequencer content array and set it for an update
+                    sequencer.survey.questions.uids.push(response.uid);
+                    sequencer.survey.questions.upToDate.push(false);
+                }
+                else {
+                    alert("ERROR!");
+                }
+            } else {
+                if (response.uid){
+                    // Edited an existing question that was already in the sequencer content. Set it for an update
+                    sequencer.survey.questions.upToDate[sequencer.survey.questions.uids.indexOf(response.uid)] = false;
+                }
+                else {
+                    alert("ERROR!");
+                }
+            }
+            
+            
+            // Then remove the editor from the sequencer content, so that it can be replaced with the correct question view.
+            me.element.parentNode.removeChild(me.element);
+            sequencer.state.editing = false;
+            updateSequencer();
+        }, true);
+    }
+    this.discard = function() {
+        me.element.parentNode.removeChild(me.element);
+        me.init();
+        sequencer.state.loaded = true;
+        sequencer.state.editing = false;
     }
     this.reset = function() {
@@ -340 +414 @@
     }
     this.type_Choice_CheckAnswerFields = function() {
-        debugger;
         var container = ge("qeParamsAnswerFieldset");
         var numAnswers = parseInt(document.getElementById("qeNumOptionsField").value, 10);
@@ -358 +431 @@
             // Extra inputs need to be added
             var n = numAnswers - numAnswerFields;
-            for (var x = 0; x < n; x++) {
+            for (var x = 1; x < n+1; x++) {
                 var group = ce("div");
                 group.className = "qeChoiceAnswerGroup";
@@ -387 +460 @@
     // Editing
     this.editQuestion = function(uid) {
+        debugger;
         if (sequencer.state.editing == true) return;
         if (sequencer.state.loaded == false) return;
         sequencer.state.editing = true;
-        
-        var request = {
-            "type": "Question",
-            "uid": uid
-        }
-        
+        sequencer.state.loaded = false;
+        
+        var request = new Array({
+            type: "Question",
+            uid: uid
+        });
+        me.init();
+        var oldElement = ge(uid);
+        if (oldElement) {
+            // There really should be.... I don't know why I am doing this check...
+            oldElement.parentNode.replaceChild(me.element, oldElement);
+        }
         var requestString = "args="+JSON.stringify(request);
-        sequencer.state.loaded = false;
-        newAjaxRequest(requestString, getObject.php, function(result){
+        newAjaxRequest(requestString, "getObject.php", function(result){
             // Once results are in
-            questionEditor.setValues(result.responseText);
+            me.setValues(result.responseText);
             sequencer.state.loaded = true;
         }, true);
@@ -406 +485 @@
     this.createNewQuestion = function() {
         if (sequencer.state.editing == true) return;
-        if (sequencer.state.loading == true) return;
+        if (sequencer.state.loading == false) return;
         sequencer.state.editing = true;
     
@@ -414 +493 @@
     }
 }
+
+// IT LIIIIIIVESSSSS
+// TODO: Add database fields for all the necessary question parameters. Maybe only one question parameter property that holds all the settings, then read it out in javascript?
+// Why have a question Description? Is this even necessary?
+// Needed properties:
+// Also not exactly sure what "question->category" is for. Is this one of those questionSet things that Julia was talking about?
+//