Dev: Throw exception if question is curropt (has no type)

LimeSurvey · Mar 21, 2016 · f5a244d · f5a244d
1 parent 3b9df36
commit f5a244d
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/application/models/QuestionAttribute.php b/application/models/QuestionAttribute.php
@@ -127,6 +127,13 @@ public function getQuestionAttributes($iQuestionID)
 
             // Get all atribute set for this question
             $sType=$oQuestion->type;
+
+            // For some reason this happened in bug #10684
+            if ($sType == null)
+            {
+                throw new \CException("Question is curropt: no type defined for question " . $iQuestionID);
+            }
+
             $aAttributeNames = questionAttributes();
             $aAttributeNames = $aAttributeNames[$sType];
             $oAttributeValues = QuestionAttribute::model()->findAll("qid=:qid",array('qid'=>$iQuestionID));