Minor - format code, fix query

main/exercice/question.class.php

@@ -146,13 +146,10 @@ abstract class Question
 
                 $tblQuiz = Database::get_course_table(TABLE_QUIZ_TEST);
 
-                /*$sql = "SELECT exercice_id FROM $TBL_EXERCISE_QUESTION
-                        WHERE c_id = $course_id AND question_id = $id";*/
-
                 $sql = "SELECT DISTINCT q.exercice_id
                         FROM $TBL_EXERCISE_QUESTION q
                         INNER JOIN $tblQuiz e
-                        ON e.c_id = q.c_id AND e.id = e.exercice_id
+                        ON e.c_id = q.c_id AND e.id = q.exercice_id
                         WHERE
                             q.c_id = $course_id AND
                             q.question_id = $id AND
@@ -250,7 +247,7 @@ abstract class Question
      * @author Nicolas Raynaud
      * @return integer - level of the question, 0 by default.
      */
-    function selectLevel()
+    public function selectLevel()
     {
         return $this->level;
     }
@@ -274,6 +271,7 @@ abstract class Question
         if (!empty($this->picture)) {
             return api_get_path(WEB_COURSE_PATH) . $this->course['path'] . '/document/images/' . $this->picture;
         }
+
         return false;
     }
 
@@ -294,7 +292,8 @@ abstract class Question
      * @author Olivier Brouckaert
      * @return integer - number of exercises
      */
-    public function selectNbrExercises() {
+    public function selectNbrExercises()
+    {
         return sizeof($this->exerciseList);
     }
 
@@ -302,6 +301,7 @@ abstract class Question
      * changes the question title
      *
      * @author Olivier Brouckaert
+     *
      * @param string $title - question title
      */
     public function updateTitle($title)
@@ -321,6 +321,7 @@ abstract class Question
      * changes the question description
      *
      * @author Olivier Brouckaert
+     *
      * @param string $description - question description
      */
     public function updateDescription($description)
@@ -742,7 +743,8 @@ abstract class Question
      * @author Olivier Brouckaert
      * @return boolean - true if moved, otherwise false
      */
-    function getTmpPicture() {
+    function getTmpPicture()
+    {
         global $picturePath;
 
         // if the question has got an ID and if the picture exists
@@ -1725,6 +1727,7 @@ abstract class Question
             array('class' => 'ribbon')
         );
         $header .= Display::div($this->description, array('id' => 'question_description'));
+
         return $header;
     }
 
@@ -1807,6 +1810,7 @@ abstract class Question
         $tabQuestionList = Question::get_question_type_list(); // [0]=file to include [1]=type name
 
         require_once $tabQuestionList[$type][0];
+
         $img = $explanation = null;
         eval('$img = ' . $tabQuestionList[$type][1] . '::$typePicture;');
         eval('$explanation = get_lang(' . $tabQuestionList[$type][1] . '::$explanationLangVar);');
@@ -1824,8 +1828,7 @@ abstract class Question
         $sidx = "question",
         $sord = "ASC",
         $where_condition = array()
-    )
+    ) {
-    {
         $table_question = Database::get_course_table(TABLE_QUIZ_QUESTION);
         $default_where = array('c_id = ? AND parent_id = 0 AND type = ?' => array($course_id, MEDIA_QUESTION));
         $result = Database::select(
@@ -1837,6 +1840,7 @@ abstract class Question
                 'order' => "$sidx $sord"
             )
         );
+
         return $result;
     }