chamilo /
chamilo-lms
@@ -29,7 +29,7 @@ discard block |
||
| 29 | 29 | public $feedback_type; |
| 30 | 30 | public $end_time; |
| 31 | 31 | public $start_time; |
| 32 | - public $questionList; // array with the list of this exercise's questions |
|
| 32 | + public $questionList; // array with the list of this exercise's questions |
|
| 33 | 33 | /* including question list of the media */ |
| 34 | 34 | public $questionListUncompressed; |
| 35 | 35 | public $results_disabled; |
@@ -132,7 +132,7 @@ discard block |
||
| 132 | 132 | $TBL_EXERCISES = Database::get_course_table(TABLE_QUIZ_TEST); |
| 133 | 133 | $table_lp_item = Database::get_course_table(TABLE_LP_ITEM); |
| 134 | 134 | |
| 135 | - $id = intval($id); |
|
| 135 | + $id = intval($id); |
|
| 136 | 136 | if (empty($this->course_id)) { |
| 137 | 137 | |
| 138 | 138 | return false; |
@@ -478,7 +478,7 @@ discard block |
||
| 478 | 478 | */ |
| 479 | 479 | public function isRandom() |
| 480 | 480 | { |
| 481 | - if($this->random > 0 || $this->random == -1) { |
|
| 481 | + if ($this->random > 0 || $this->random == -1) { |
|
| 482 | 482 | return true; |
| 483 | 483 | } else { |
| 484 | 484 | return false; |
@@ -639,7 +639,7 @@ discard block |
||
| 639 | 639 | $question_media = null; |
| 640 | 640 | if (!empty($objQuestionTmp->parent_id)) { |
| 641 | 641 | $objQuestionMedia = Question::read($objQuestionTmp->parent_id); |
| 642 | - $question_media = Question::getMediaLabel($objQuestionMedia->question); |
|
| 642 | + $question_media = Question::getMediaLabel($objQuestionMedia->question); |
|
| 643 | 643 | } |
| 644 | 644 | |
| 645 | 645 | $questionType = Display::tag('div', Display::return_icon($typeImg, $typeExpl, array(), ICON_SIZE_MEDIUM).$question_media); |
@@ -1032,7 +1032,7 @@ discard block |
||
| 1032 | 1032 | |
| 1033 | 1033 | foreach ($questions_by_category as $categoryId => $questionList) { |
| 1034 | 1034 | $cat = new TestCategory($categoryId); |
| 1035 | - $cat = (array)$cat; |
|
| 1035 | + $cat = (array) $cat; |
|
| 1036 | 1036 | $cat['iid'] = $cat['id']; |
| 1037 | 1037 | //*$cat['name'] = $cat['name']; |
| 1038 | 1038 | |
@@ -1097,7 +1097,7 @@ discard block |
||
| 1097 | 1097 | { |
| 1098 | 1098 | if ($this->specialCategoryOrders == false) { |
| 1099 | 1099 | if ($from_db && !empty($this->id)) { |
| 1100 | - $TBL_EXERCISE_QUESTION = Database::get_course_table(TABLE_QUIZ_TEST_QUESTION); |
|
| 1100 | + $TBL_EXERCISE_QUESTION = Database::get_course_table(TABLE_QUIZ_TEST_QUESTION); |
|
| 1101 | 1101 | $TBL_QUESTIONS = Database::get_course_table(TABLE_QUIZ_QUESTION); |
| 1102 | 1102 | |
| 1103 | 1103 | $sql = "SELECT DISTINCT e.question_order |
@@ -1124,7 +1124,7 @@ discard block |
||
| 1124 | 1124 | $question_list = array(); |
| 1125 | 1125 | |
| 1126 | 1126 | while ($new_object = Database::fetch_object($result)) { |
| 1127 | - $question_list[$new_object->question_order]= $new_object->question_id; |
|
| 1127 | + $question_list[$new_object->question_order] = $new_object->question_id; |
|
| 1128 | 1128 | $temp_question_list[$counter] = $new_object->question_id; |
| 1129 | 1129 | $counter++; |
| 1130 | 1130 | } |
@@ -1268,7 +1268,7 @@ discard block |
||
| 1268 | 1268 | public function isInList($questionId) |
| 1269 | 1269 | { |
| 1270 | 1270 | if (is_array($this->questionList)) |
| 1271 | - return in_array($questionId,$this->questionList); |
|
| 1271 | + return in_array($questionId, $this->questionList); |
|
| 1272 | 1272 | else |
| 1273 | 1273 | return false; |
| 1274 | 1274 | } |
@@ -1281,7 +1281,7 @@ discard block |
||
| 1281 | 1281 | */ |
| 1282 | 1282 | public function updateTitle($title) |
| 1283 | 1283 | { |
| 1284 | - $this->exercise=$title; |
|
| 1284 | + $this->exercise = $title; |
|
| 1285 | 1285 | } |
| 1286 | 1286 | |
| 1287 | 1287 | /** |
@@ -1291,7 +1291,7 @@ discard block |
||
| 1291 | 1291 | */ |
| 1292 | 1292 | public function updateAttempts($attempts) |
| 1293 | 1293 | { |
| 1294 | - $this->attempts=$attempts; |
|
| 1294 | + $this->attempts = $attempts; |
|
| 1295 | 1295 | } |
| 1296 | 1296 | |
| 1297 | 1297 | /** |
@@ -1301,7 +1301,7 @@ discard block |
||
| 1301 | 1301 | */ |
| 1302 | 1302 | public function updateFeedbackType($feedback_type) |
| 1303 | 1303 | { |
| 1304 | - $this->feedback_type=$feedback_type; |
|
| 1304 | + $this->feedback_type = $feedback_type; |
|
| 1305 | 1305 | } |
| 1306 | 1306 | |
| 1307 | 1307 | /** |
@@ -1312,7 +1312,7 @@ discard block |
||
| 1312 | 1312 | */ |
| 1313 | 1313 | public function updateDescription($description) |
| 1314 | 1314 | { |
| 1315 | - $this->description=$description; |
|
| 1315 | + $this->description = $description; |
|
| 1316 | 1316 | } |
| 1317 | 1317 | |
| 1318 | 1318 | /** |
@@ -1442,23 +1442,23 @@ discard block |
||
| 1442 | 1442 | * @param string $sound - exercise sound file |
| 1443 | 1443 | * @param string $delete - ask to delete the file |
| 1444 | 1444 | */ |
| 1445 | - public function updateSound($sound,$delete) |
|
| 1445 | + public function updateSound($sound, $delete) |
|
| 1446 | 1446 | { |
| 1447 | 1447 | global $audioPath, $documentPath; |
| 1448 | 1448 | $TBL_DOCUMENT = Database::get_course_table(TABLE_DOCUMENT); |
| 1449 | 1449 | |
| 1450 | - if ($sound['size'] && (strstr($sound['type'],'audio') || strstr($sound['type'],'video'))) { |
|
| 1451 | - $this->sound=$sound['name']; |
|
| 1450 | + if ($sound['size'] && (strstr($sound['type'], 'audio') || strstr($sound['type'], 'video'))) { |
|
| 1451 | + $this->sound = $sound['name']; |
|
| 1452 | 1452 | |
| 1453 | - if (@move_uploaded_file($sound['tmp_name'],$audioPath.'/'.$this->sound)) { |
|
| 1453 | + if (@move_uploaded_file($sound['tmp_name'], $audioPath.'/'.$this->sound)) { |
|
| 1454 | 1454 | $query = "SELECT 1 FROM $TBL_DOCUMENT |
| 1455 | - WHERE c_id = ".$this->course_id." AND path='".str_replace($documentPath,'',$audioPath).'/'.$this->sound."'"; |
|
| 1456 | - $result=Database::query($query); |
|
| 1455 | + WHERE c_id = ".$this->course_id." AND path='".str_replace($documentPath, '', $audioPath).'/'.$this->sound."'"; |
|
| 1456 | + $result = Database::query($query); |
|
| 1457 | 1457 | |
| 1458 | 1458 | if (!Database::num_rows($result)) { |
| 1459 | 1459 | $id = add_document( |
| 1460 | 1460 | $this->course, |
| 1461 | - str_replace($documentPath,'',$audioPath).'/'.$this->sound, |
|
| 1461 | + str_replace($documentPath, '', $audioPath).'/'.$this->sound, |
|
| 1462 | 1462 | 'file', |
| 1463 | 1463 | $sound['size'], |
| 1464 | 1464 | $sound['name'] |
@@ -1477,8 +1477,8 @@ discard block |
||
| 1477 | 1477 | ); |
| 1478 | 1478 | } |
| 1479 | 1479 | } |
| 1480 | - } elseif($delete && is_file($audioPath.'/'.$this->sound)) { |
|
| 1481 | - $this->sound=''; |
|
| 1480 | + } elseif ($delete && is_file($audioPath.'/'.$this->sound)) { |
|
| 1481 | + $this->sound = ''; |
|
| 1482 | 1482 | } |
| 1483 | 1483 | } |
| 1484 | 1484 | |
@@ -1490,7 +1490,7 @@ discard block |
||
| 1490 | 1490 | */ |
| 1491 | 1491 | public function updateType($type) |
| 1492 | 1492 | { |
| 1493 | - $this->type=$type; |
|
| 1493 | + $this->type = $type; |
|
| 1494 | 1494 | } |
| 1495 | 1495 | |
| 1496 | 1496 | /** |
@@ -1526,7 +1526,7 @@ discard block |
||
| 1526 | 1526 | */ |
| 1527 | 1527 | public function enable() |
| 1528 | 1528 | { |
| 1529 | - $this->active=1; |
|
| 1529 | + $this->active = 1; |
|
| 1530 | 1530 | } |
| 1531 | 1531 | |
| 1532 | 1532 | /** |
@@ -1536,7 +1536,7 @@ discard block |
||
| 1536 | 1536 | */ |
| 1537 | 1537 | public function disable() |
| 1538 | 1538 | { |
| 1539 | - $this->active=0; |
|
| 1539 | + $this->active = 0; |
|
| 1540 | 1540 | } |
| 1541 | 1541 | |
| 1542 | 1542 | /** |
@@ -1664,7 +1664,7 @@ discard block |
||
| 1664 | 1664 | api_get_user_id() |
| 1665 | 1665 | ); |
| 1666 | 1666 | |
| 1667 | - if (api_get_setting('search_enabled')=='true') { |
|
| 1667 | + if (api_get_setting('search_enabled') == 'true') { |
|
| 1668 | 1668 | $this->search_engine_edit(); |
| 1669 | 1669 | } |
| 1670 | 1670 | } else { |
@@ -1812,7 +1812,7 @@ discard block |
||
| 1812 | 1812 | public function removeFromList($questionId) |
| 1813 | 1813 | { |
| 1814 | 1814 | // searches the position of the question ID in the list |
| 1815 | - $pos = array_search($questionId,$this->questionList); |
|
| 1815 | + $pos = array_search($questionId, $this->questionList); |
|
| 1816 | 1816 | |
| 1817 | 1817 | // question not found |
| 1818 | 1818 | if ($pos === false) { |
@@ -1847,7 +1847,7 @@ discard block |
||
| 1847 | 1847 | api_item_property_update($this->course, TOOL_QUIZ, $this->id, 'QuizDeleted', api_get_user_id()); |
| 1848 | 1848 | api_item_property_update($this->course, TOOL_QUIZ, $this->id, 'delete', api_get_user_id()); |
| 1849 | 1849 | |
| 1850 | - if (api_get_setting('search_enabled')=='true' && extension_loaded('xapian') ) { |
|
| 1850 | + if (api_get_setting('search_enabled') == 'true' && extension_loaded('xapian')) { |
|
| 1851 | 1851 | $this->search_engine_delete(); |
| 1852 | 1852 | } |
| 1853 | 1853 | } |
@@ -1856,7 +1856,7 @@ discard block |
||
| 1856 | 1856 | * Creates the form to create / edit an exercise |
| 1857 | 1857 | * @param FormValidator $form |
| 1858 | 1858 | */ |
| 1859 | - public function createForm($form, $type='full') |
|
| 1859 | + public function createForm($form, $type = 'full') |
|
| 1860 | 1860 | { |
| 1861 | 1861 | if (empty($type)) { |
| 1862 | 1862 | $type = 'full'; |
@@ -1887,7 +1887,7 @@ discard block |
||
| 1887 | 1887 | 'Width' => '100%', |
| 1888 | 1888 | 'Height' => '150', |
| 1889 | 1889 | ); |
| 1890 | - if (is_array($type)){ |
|
| 1890 | + if (is_array($type)) { |
|
| 1891 | 1891 | $editor_config = array_merge($editor_config, $type); |
| 1892 | 1892 | } |
| 1893 | 1893 | |
@@ -1941,7 +1941,7 @@ discard block |
||
| 1941 | 1941 | '2', |
| 1942 | 1942 | array('id' => 'exerciseType_2') |
| 1943 | 1943 | ); |
| 1944 | - $form->addGroup($radios_feedback, null, array(get_lang('FeedbackType'),get_lang('FeedbackDisplayOptions')), ''); |
|
| 1944 | + $form->addGroup($radios_feedback, null, array(get_lang('FeedbackType'), get_lang('FeedbackDisplayOptions')), ''); |
|
| 1945 | 1945 | |
| 1946 | 1946 | // Type of results display on the final page |
| 1947 | 1947 | $radios_results_disabled = array(); |
@@ -1985,8 +1985,8 @@ discard block |
||
| 1985 | 1985 | // Type of questions disposition on page |
| 1986 | 1986 | $radios = array(); |
| 1987 | 1987 | |
| 1988 | - $radios[] = $form->createElement('radio', 'exerciseType', null, get_lang('SimpleExercise'), '1', array('onclick' => 'check_per_page_all()', 'id'=>'option_page_all')); |
|
| 1989 | - $radios[] = $form->createElement('radio', 'exerciseType', null, get_lang('SequentialExercise'),'2', array('onclick' => 'check_per_page_one()', 'id'=>'option_page_one')); |
|
| 1988 | + $radios[] = $form->createElement('radio', 'exerciseType', null, get_lang('SimpleExercise'), '1', array('onclick' => 'check_per_page_all()', 'id'=>'option_page_all')); |
|
| 1989 | + $radios[] = $form->createElement('radio', 'exerciseType', null, get_lang('SequentialExercise'), '2', array('onclick' => 'check_per_page_one()', 'id'=>'option_page_one')); |
|
| 1990 | 1990 | |
| 1991 | 1991 | $form->addGroup($radios, null, get_lang('QuestionsPerPage'), ''); |
| 1992 | 1992 | |
@@ -1996,33 +1996,33 @@ discard block |
||
| 1996 | 1996 | |
| 1997 | 1997 | // feedback type |
| 1998 | 1998 | $radios_feedback = array(); |
| 1999 | - $radios_feedback[] = $form->createElement('radio', 'exerciseFeedbackType', null, get_lang('ExerciseAtTheEndOfTheTest'),'0',array('id' =>'exerciseType_0', 'onclick' => 'check_feedback()')); |
|
| 1999 | + $radios_feedback[] = $form->createElement('radio', 'exerciseFeedbackType', null, get_lang('ExerciseAtTheEndOfTheTest'), '0', array('id' =>'exerciseType_0', 'onclick' => 'check_feedback()')); |
|
| 2000 | 2000 | |
| 2001 | 2001 | if (api_get_setting('enable_quiz_scenario') == 'true') { |
| 2002 | - $radios_feedback[] = $form->createElement('radio', 'exerciseFeedbackType', null, get_lang('DirectFeedback'), '1', array('id' =>'exerciseType_1' , 'onclick' => 'check_direct_feedback()')); |
|
| 2002 | + $radios_feedback[] = $form->createElement('radio', 'exerciseFeedbackType', null, get_lang('DirectFeedback'), '1', array('id' =>'exerciseType_1', 'onclick' => 'check_direct_feedback()')); |
|
| 2003 | 2003 | } |
| 2004 | - $radios_feedback[] = $form->createElement('radio', 'exerciseFeedbackType', null, get_lang('NoFeedback'),'2',array('id' =>'exerciseType_2')); |
|
| 2005 | - $form->addGroup($radios_feedback, null, array(get_lang('FeedbackType'),get_lang('FeedbackDisplayOptions'))); |
|
| 2004 | + $radios_feedback[] = $form->createElement('radio', 'exerciseFeedbackType', null, get_lang('NoFeedback'), '2', array('id' =>'exerciseType_2')); |
|
| 2005 | + $form->addGroup($radios_feedback, null, array(get_lang('FeedbackType'), get_lang('FeedbackDisplayOptions'))); |
|
| 2006 | 2006 | |
| 2007 | 2007 | //$form->addElement('select', 'exerciseFeedbackType',get_lang('FeedbackType'),$feedback_option,'onchange="javascript:feedbackselection()"'); |
| 2008 | 2008 | $radios_results_disabled = array(); |
| 2009 | 2009 | $radios_results_disabled[] = $form->createElement('radio', 'results_disabled', null, get_lang('ShowScoreAndRightAnswer'), '0', array('id'=>'result_disabled_0')); |
| 2010 | - $radios_results_disabled[] = $form->createElement('radio', 'results_disabled', null, get_lang('DoNotShowScoreNorRightAnswer'), '1',array('id'=>'result_disabled_1','onclick' => 'check_results_disabled()')); |
|
| 2011 | - $radios_results_disabled[] = $form->createElement('radio', 'results_disabled', null, get_lang('OnlyShowScore'), '2',array('id'=>'result_disabled_2','onclick' => 'check_results_disabled()')); |
|
| 2012 | - $form->addGroup($radios_results_disabled, null, get_lang('ShowResultsToStudents'),''); |
|
| 2010 | + $radios_results_disabled[] = $form->createElement('radio', 'results_disabled', null, get_lang('DoNotShowScoreNorRightAnswer'), '1', array('id'=>'result_disabled_1', 'onclick' => 'check_results_disabled()')); |
|
| 2011 | + $radios_results_disabled[] = $form->createElement('radio', 'results_disabled', null, get_lang('OnlyShowScore'), '2', array('id'=>'result_disabled_2', 'onclick' => 'check_results_disabled()')); |
|
| 2012 | + $form->addGroup($radios_results_disabled, null, get_lang('ShowResultsToStudents'), ''); |
|
| 2013 | 2013 | |
| 2014 | 2014 | // Type of questions disposition on page |
| 2015 | 2015 | $radios = array(); |
| 2016 | - $radios[] = $form->createElement('radio', 'exerciseType', null, get_lang('SimpleExercise'), '1'); |
|
| 2017 | - $radios[] = $form->createElement('radio', 'exerciseType', null, get_lang('SequentialExercise'),'2'); |
|
| 2016 | + $radios[] = $form->createElement('radio', 'exerciseType', null, get_lang('SimpleExercise'), '1'); |
|
| 2017 | + $radios[] = $form->createElement('radio', 'exerciseType', null, get_lang('SequentialExercise'), '2'); |
|
| 2018 | 2018 | $form->addGroup($radios, null, get_lang('ExerciseType')); |
| 2019 | 2019 | |
| 2020 | 2020 | } else { |
| 2021 | 2021 | //Show options freeze |
| 2022 | 2022 | $radios_results_disabled[] = $form->createElement('radio', 'results_disabled', null, get_lang('ShowScoreAndRightAnswer'), '0', array('id'=>'result_disabled_0')); |
| 2023 | - $radios_results_disabled[] = $form->createElement('radio', 'results_disabled', null, get_lang('DoNotShowScoreNorRightAnswer'), '1',array('id'=>'result_disabled_1','onclick' => 'check_results_disabled()')); |
|
| 2024 | - $radios_results_disabled[] = $form->createElement('radio', 'results_disabled', null, get_lang('OnlyShowScore'), '2',array('id'=>'result_disabled_2','onclick' => 'check_results_disabled()')); |
|
| 2025 | - $result_disable_group = $form->addGroup($radios_results_disabled, null, get_lang('ShowResultsToStudents'),''); |
|
| 2023 | + $radios_results_disabled[] = $form->createElement('radio', 'results_disabled', null, get_lang('DoNotShowScoreNorRightAnswer'), '1', array('id'=>'result_disabled_1', 'onclick' => 'check_results_disabled()')); |
|
| 2024 | + $radios_results_disabled[] = $form->createElement('radio', 'results_disabled', null, get_lang('OnlyShowScore'), '2', array('id'=>'result_disabled_2', 'onclick' => 'check_results_disabled()')); |
|
| 2025 | + $result_disable_group = $form->addGroup($radios_results_disabled, null, get_lang('ShowResultsToStudents'), ''); |
|
| 2026 | 2026 | $result_disable_group->freeze(); |
| 2027 | 2027 | |
| 2028 | 2028 | //we force the options to the DirectFeedback exercisetype |
@@ -2030,8 +2030,8 @@ discard block |
||
| 2030 | 2030 | $form->addElement('hidden', 'exerciseType', ONE_PER_PAGE); |
| 2031 | 2031 | |
| 2032 | 2032 | // Type of questions disposition on page |
| 2033 | - $radios[] = $form->createElement('radio', 'exerciseType', null, get_lang('SimpleExercise'), '1', array('onclick' => 'check_per_page_all()', 'id'=>'option_page_all')); |
|
| 2034 | - $radios[] = $form->createElement('radio', 'exerciseType', null, get_lang('SequentialExercise'),'2', array('onclick' => 'check_per_page_one()', 'id'=>'option_page_one')); |
|
| 2033 | + $radios[] = $form->createElement('radio', 'exerciseType', null, get_lang('SimpleExercise'), '1', array('onclick' => 'check_per_page_all()', 'id'=>'option_page_all')); |
|
| 2034 | + $radios[] = $form->createElement('radio', 'exerciseType', null, get_lang('SequentialExercise'), '2', array('onclick' => 'check_per_page_one()', 'id'=>'option_page_one')); |
|
| 2035 | 2035 | |
| 2036 | 2036 | $type_group = $form->addGroup($radios, null, get_lang('QuestionsPerPage'), ''); |
| 2037 | 2037 | $type_group->freeze(); |
@@ -2163,26 +2163,26 @@ discard block |
||
| 2163 | 2163 | |
| 2164 | 2164 | // number of random question |
| 2165 | 2165 | |
| 2166 | - $max = ($this->id > 0) ? $this->selectNbrQuestions() : 10 ; |
|
| 2166 | + $max = ($this->id > 0) ? $this->selectNbrQuestions() : 10; |
|
| 2167 | 2167 | $option = range(0, $max); |
| 2168 | 2168 | $option[0] = get_lang('No'); |
| 2169 | 2169 | $option[-1] = get_lang('AllQuestionsShort'); |
| 2170 | - $form->addElement('select', 'randomQuestions',array(get_lang('RandomQuestions'), get_lang('RandomQuestionsHelp')), $option, array('id'=>'randomQuestions')); |
|
| 2170 | + $form->addElement('select', 'randomQuestions', array(get_lang('RandomQuestions'), get_lang('RandomQuestionsHelp')), $option, array('id'=>'randomQuestions')); |
|
| 2171 | 2171 | |
| 2172 | 2172 | // Random answers |
| 2173 | 2173 | $radios_random_answers = array(); |
| 2174 | - $radios_random_answers[] = $form->createElement('radio', 'randomAnswers', null, get_lang('Yes'),'1'); |
|
| 2175 | - $radios_random_answers[] = $form->createElement('radio', 'randomAnswers', null, get_lang('No'),'0'); |
|
| 2174 | + $radios_random_answers[] = $form->createElement('radio', 'randomAnswers', null, get_lang('Yes'), '1'); |
|
| 2175 | + $radios_random_answers[] = $form->createElement('radio', 'randomAnswers', null, get_lang('No'), '0'); |
|
| 2176 | 2176 | $form->addGroup($radios_random_answers, null, get_lang('RandomAnswers'), ''); |
| 2177 | 2177 | |
| 2178 | 2178 | // Random by category |
| 2179 | - $form->addElement('html','<div class="clear"> </div>'); |
|
| 2179 | + $form->addElement('html', '<div class="clear"> </div>'); |
|
| 2180 | 2180 | $radiocat = array(); |
| 2181 | - $radiocat[] = $form->createElement('radio', 'randomByCat', null, get_lang('YesWithCategoriesShuffled'),'1'); |
|
| 2182 | - $radiocat[] = $form->createElement('radio', 'randomByCat', null, get_lang('YesWithCategoriesSorted'),'2'); |
|
| 2183 | - $radiocat[] = $form->createElement('radio', 'randomByCat', null, get_lang('No'),'0'); |
|
| 2181 | + $radiocat[] = $form->createElement('radio', 'randomByCat', null, get_lang('YesWithCategoriesShuffled'), '1'); |
|
| 2182 | + $radiocat[] = $form->createElement('radio', 'randomByCat', null, get_lang('YesWithCategoriesSorted'), '2'); |
|
| 2183 | + $radiocat[] = $form->createElement('radio', 'randomByCat', null, get_lang('No'), '0'); |
|
| 2184 | 2184 | $radioCatGroup = $form->addGroup($radiocat, null, get_lang('RandomQuestionByCategory'), ''); |
| 2185 | - $form->addElement('html','<div class="clear"> </div>'); |
|
| 2185 | + $form->addElement('html', '<div class="clear"> </div>'); |
|
| 2186 | 2186 | |
| 2187 | 2187 | // add the radio display the category name for student |
| 2188 | 2188 | $radio_display_cat_name = array(); |
@@ -2203,36 +2203,36 @@ discard block |
||
| 2203 | 2203 | ); |
| 2204 | 2204 | |
| 2205 | 2205 | // Exercise time limit |
| 2206 | - $form->addElement('checkbox', 'activate_start_date_check',null, get_lang('EnableStartTime'), array('onclick' => 'activate_start_date()')); |
|
| 2206 | + $form->addElement('checkbox', 'activate_start_date_check', null, get_lang('EnableStartTime'), array('onclick' => 'activate_start_date()')); |
|
| 2207 | 2207 | |
| 2208 | 2208 | $var = Exercise::selectTimeLimit(); |
| 2209 | 2209 | |
| 2210 | 2210 | if (($this->start_time != '0000-00-00 00:00:00')) |
| 2211 | - $form->addElement('html','<div id="start_date_div" style="display:block;">'); |
|
| 2211 | + $form->addElement('html', '<div id="start_date_div" style="display:block;">'); |
|
| 2212 | 2212 | else |
| 2213 | - $form->addElement('html','<div id="start_date_div" style="display:none;">'); |
|
| 2213 | + $form->addElement('html', '<div id="start_date_div" style="display:none;">'); |
|
| 2214 | 2214 | |
| 2215 | 2215 | $form->addElement('date_time_picker', 'start_time'); |
| 2216 | 2216 | |
| 2217 | - $form->addElement('html','</div>'); |
|
| 2217 | + $form->addElement('html', '</div>'); |
|
| 2218 | 2218 | |
| 2219 | - $form->addElement('checkbox', 'activate_end_date_check', null , get_lang('EnableEndTime'), array('onclick' => 'activate_end_date()')); |
|
| 2219 | + $form->addElement('checkbox', 'activate_end_date_check', null, get_lang('EnableEndTime'), array('onclick' => 'activate_end_date()')); |
|
| 2220 | 2220 | |
| 2221 | 2221 | if (($this->end_time != '0000-00-00 00:00:00')) |
| 2222 | - $form->addElement('html','<div id="end_date_div" style="display:block;">'); |
|
| 2222 | + $form->addElement('html', '<div id="end_date_div" style="display:block;">'); |
|
| 2223 | 2223 | else |
| 2224 | - $form->addElement('html','<div id="end_date_div" style="display:none;">'); |
|
| 2224 | + $form->addElement('html', '<div id="end_date_div" style="display:none;">'); |
|
| 2225 | 2225 | |
| 2226 | 2226 | $form->addElement('date_time_picker', 'end_time'); |
| 2227 | - $form->addElement('html','</div>'); |
|
| 2227 | + $form->addElement('html', '</div>'); |
|
| 2228 | 2228 | |
| 2229 | 2229 | //$check_option=$this->selectType(); |
| 2230 | 2230 | $diplay = 'block'; |
| 2231 | 2231 | $form->addElement('checkbox', 'propagate_neg', null, get_lang('PropagateNegativeResults')); |
| 2232 | - $form->addElement('html','<div class="clear"> </div>'); |
|
| 2232 | + $form->addElement('html', '<div class="clear"> </div>'); |
|
| 2233 | 2233 | $form->addElement('checkbox', 'review_answers', null, get_lang('ReviewAnswers')); |
| 2234 | 2234 | |
| 2235 | - $form->addElement('html','<div id="divtimecontrol" style="display:'.$diplay.';">'); |
|
| 2235 | + $form->addElement('html', '<div id="divtimecontrol" style="display:'.$diplay.';">'); |
|
| 2236 | 2236 | |
| 2237 | 2237 | //Timer control |
| 2238 | 2238 | //$time_hours_option = range(0,12); |
@@ -2248,12 +2248,12 @@ discard block |
||
| 2248 | 2248 | 'onload' => 'check_load_time()', |
| 2249 | 2249 | ) |
| 2250 | 2250 | ); |
| 2251 | - $expired_date = (int)$this->selectExpiredTime(); |
|
| 2251 | + $expired_date = (int) $this->selectExpiredTime(); |
|
| 2252 | 2252 | |
| 2253 | - if (($expired_date!='0')) { |
|
| 2254 | - $form->addElement('html','<div id="timercontrol" style="display:block;">'); |
|
| 2253 | + if (($expired_date != '0')) { |
|
| 2254 | + $form->addElement('html', '<div id="timercontrol" style="display:block;">'); |
|
| 2255 | 2255 | } else { |
| 2256 | - $form->addElement('html','<div id="timercontrol" style="display:none;">'); |
|
| 2256 | + $form->addElement('html', '<div id="timercontrol" style="display:none;">'); |
|
| 2257 | 2257 | } |
| 2258 | 2258 | $form->addText( |
| 2259 | 2259 | 'enabletimercontroltotalminutes', |
@@ -2264,7 +2264,7 @@ discard block |
||
| 2264 | 2264 | 'cols-size' => [2, 2, 8] |
| 2265 | 2265 | ] |
| 2266 | 2266 | ); |
| 2267 | - $form->addElement('html','</div>'); |
|
| 2267 | + $form->addElement('html', '</div>'); |
|
| 2268 | 2268 | |
| 2269 | 2269 | $form->addElement( |
| 2270 | 2270 | 'text', |
@@ -2288,23 +2288,23 @@ discard block |
||
| 2288 | 2288 | $defaults = array(); |
| 2289 | 2289 | |
| 2290 | 2290 | if (api_get_setting('search_enabled') === 'true') { |
| 2291 | - require_once api_get_path(LIBRARY_PATH) . 'specific_fields_manager.lib.php'; |
|
| 2291 | + require_once api_get_path(LIBRARY_PATH).'specific_fields_manager.lib.php'; |
|
| 2292 | 2292 | |
| 2293 | - $form->addElement ('checkbox', 'index_document','', get_lang('SearchFeatureDoIndexDocument')); |
|
| 2294 | - $form->addElement ('select_language', 'language', get_lang('SearchFeatureDocumentLanguage')); |
|
| 2293 | + $form->addElement('checkbox', 'index_document', '', get_lang('SearchFeatureDoIndexDocument')); |
|
| 2294 | + $form->addElement('select_language', 'language', get_lang('SearchFeatureDocumentLanguage')); |
|
| 2295 | 2295 | |
| 2296 | 2296 | $specific_fields = get_specific_field_list(); |
| 2297 | 2297 | |
| 2298 | 2298 | foreach ($specific_fields as $specific_field) { |
| 2299 | - $form->addElement ('text', $specific_field['code'], $specific_field['name']); |
|
| 2299 | + $form->addElement('text', $specific_field['code'], $specific_field['name']); |
|
| 2300 | 2300 | $filter = array( |
| 2301 | 2301 | 'c_id' => api_get_course_int_id(), |
| 2302 | 2302 | 'field_id' => $specific_field['id'], |
| 2303 | 2303 | 'ref_id' => $this->id, |
| 2304 | - 'tool_id' => "'" . TOOL_QUIZ . "'" |
|
| 2304 | + 'tool_id' => "'".TOOL_QUIZ."'" |
|
| 2305 | 2305 | ); |
| 2306 | 2306 | $values = get_specific_field_values_list($filter, array('value')); |
| 2307 | - if ( !empty($values) ) { |
|
| 2307 | + if (!empty($values)) { |
|
| 2308 | 2308 | $arr_str_values = array(); |
| 2309 | 2309 | foreach ($values as $value) { |
| 2310 | 2310 | $arr_str_values[] = $value['value']; |
@@ -2315,8 +2315,8 @@ discard block |
||
| 2315 | 2315 | //$form->addElement ('html','</div>'); |
| 2316 | 2316 | } |
| 2317 | 2317 | |
| 2318 | - $form->addElement('html','</div>'); //End advanced setting |
|
| 2319 | - $form->addElement('html','</div>'); |
|
| 2318 | + $form->addElement('html', '</div>'); //End advanced setting |
|
| 2319 | + $form->addElement('html', '</div>'); |
|
| 2320 | 2320 | } |
| 2321 | 2321 | |
| 2322 | 2322 | // submit |
@@ -2336,10 +2336,10 @@ discard block |
||
| 2336 | 2336 | } |
| 2337 | 2337 | |
| 2338 | 2338 | // defaults |
| 2339 | - if ($type=='full') { |
|
| 2339 | + if ($type == 'full') { |
|
| 2340 | 2340 | if ($this->id > 0) { |
| 2341 | 2341 | if ($this->random > $this->selectNbrQuestions()) { |
| 2342 | - $defaults['randomQuestions'] = $this->selectNbrQuestions(); |
|
| 2342 | + $defaults['randomQuestions'] = $this->selectNbrQuestions(); |
|
| 2343 | 2343 | } else { |
| 2344 | 2344 | $defaults['randomQuestions'] = $this->random; |
| 2345 | 2345 | } |
@@ -2366,8 +2366,8 @@ discard block |
||
| 2366 | 2366 | $defaults['activate_end_date_check'] = 1; |
| 2367 | 2367 | } |
| 2368 | 2368 | |
| 2369 | - $defaults['start_time'] = ($this->start_time!='0000-00-00 00:00:00') ? api_get_local_time($this->start_time) : date('Y-m-d 12:00:00'); |
|
| 2370 | - $defaults['end_time'] = ($this->end_time!='0000-00-00 00:00:00') ? api_get_local_time($this->end_time) : date('Y-m-d 12:00:00', time()+84600); |
|
| 2369 | + $defaults['start_time'] = ($this->start_time != '0000-00-00 00:00:00') ? api_get_local_time($this->start_time) : date('Y-m-d 12:00:00'); |
|
| 2370 | + $defaults['end_time'] = ($this->end_time != '0000-00-00 00:00:00') ? api_get_local_time($this->end_time) : date('Y-m-d 12:00:00', time() + 84600); |
|
| 2371 | 2371 | |
| 2372 | 2372 | // Get expired time |
| 2373 | 2373 | if ($this->expired_time != '0') { |
@@ -2388,7 +2388,7 @@ discard block |
||
| 2388 | 2388 | $defaults['text_when_finished'] = ""; |
| 2389 | 2389 | $defaults['start_time'] = date('Y-m-d 12:00:00'); |
| 2390 | 2390 | $defaults['display_category_name'] = 1; |
| 2391 | - $defaults['end_time'] = date('Y-m-d 12:00:00', time()+84600); |
|
| 2391 | + $defaults['end_time'] = date('Y-m-d 12:00:00', time() + 84600); |
|
| 2392 | 2392 | $defaults['pass_percentage'] = ''; |
| 2393 | 2393 | $defaults['end_button'] = $this->selectEndButton(); |
| 2394 | 2394 | $defaults['question_selection_type'] = 1; |
@@ -2474,7 +2474,7 @@ discard block |
||
| 2474 | 2474 | $end_time = $form->getSubmitValue('end_time'); |
| 2475 | 2475 | $this->end_time = api_get_utc_datetime($end_time); |
| 2476 | 2476 | } else { |
| 2477 | - $this->end_time = '0000-00-00 00:00:00'; |
|
| 2477 | + $this->end_time = '0000-00-00 00:00:00'; |
|
| 2478 | 2478 | } |
| 2479 | 2479 | |
| 2480 | 2480 | if ($form->getSubmitValue('enabletimercontrol') == 1) { |
@@ -2487,9 +2487,9 @@ discard block |
||
| 2487 | 2487 | } |
| 2488 | 2488 | |
| 2489 | 2489 | if ($form->getSubmitValue('randomAnswers') == 1) { |
| 2490 | - $this->random_answers=1; |
|
| 2490 | + $this->random_answers = 1; |
|
| 2491 | 2491 | } else { |
| 2492 | - $this->random_answers=0; |
|
| 2492 | + $this->random_answers = 0; |
|
| 2493 | 2493 | } |
| 2494 | 2494 | $this->save($type); |
| 2495 | 2495 | } |
@@ -2501,9 +2501,9 @@ discard block |
||
| 2501 | 2501 | } |
| 2502 | 2502 | $course_id = api_get_course_id(); |
| 2503 | 2503 | |
| 2504 | - require_once api_get_path(LIBRARY_PATH) . 'search/ChamiloIndexer.class.php'; |
|
| 2505 | - require_once api_get_path(LIBRARY_PATH) . 'search/IndexableChunk.class.php'; |
|
| 2506 | - require_once api_get_path(LIBRARY_PATH) . 'specific_fields_manager.lib.php'; |
|
| 2504 | + require_once api_get_path(LIBRARY_PATH).'search/ChamiloIndexer.class.php'; |
|
| 2505 | + require_once api_get_path(LIBRARY_PATH).'search/IndexableChunk.class.php'; |
|
| 2506 | + require_once api_get_path(LIBRARY_PATH).'specific_fields_manager.lib.php'; |
|
| 2507 | 2507 | |
| 2508 | 2508 | $specific_fields = get_specific_field_list(); |
| 2509 | 2509 | $ic_slide = new IndexableChunk(); |
@@ -2513,7 +2513,7 @@ discard block |
||
| 2513 | 2513 | if (isset($_REQUEST[$specific_field['code']])) { |
| 2514 | 2514 | $sterms = trim($_REQUEST[$specific_field['code']]); |
| 2515 | 2515 | if (!empty($sterms)) { |
| 2516 | - $all_specific_terms .= ' '. $sterms; |
|
| 2516 | + $all_specific_terms .= ' '.$sterms; |
|
| 2517 | 2517 | $sterms = explode(',', $sterms); |
| 2518 | 2518 | foreach ($sterms as $sterm) { |
| 2519 | 2519 | $ic_slide->addTerm(trim($sterm), $specific_field['code']); |
@@ -2530,15 +2530,15 @@ discard block |
||
| 2530 | 2530 | $xapian_data = array( |
| 2531 | 2531 | SE_COURSE_ID => $course_id, |
| 2532 | 2532 | SE_TOOL_ID => TOOL_QUIZ, |
| 2533 | - SE_DATA => array('type' => SE_DOCTYPE_EXERCISE_EXERCISE, 'exercise_id' => (int)$this->id), |
|
| 2534 | - SE_USER => (int)api_get_user_id(), |
|
| 2533 | + SE_DATA => array('type' => SE_DOCTYPE_EXERCISE_EXERCISE, 'exercise_id' => (int) $this->id), |
|
| 2534 | + SE_USER => (int) api_get_user_id(), |
|
| 2535 | 2535 | ); |
| 2536 | 2536 | $ic_slide->xapian_data = serialize($xapian_data); |
| 2537 | - $exercise_description = $all_specific_terms .' '. $this->description; |
|
| 2537 | + $exercise_description = $all_specific_terms.' '.$this->description; |
|
| 2538 | 2538 | $ic_slide->addValue("content", $exercise_description); |
| 2539 | 2539 | |
| 2540 | 2540 | $di = new ChamiloIndexer(); |
| 2541 | - isset($_POST['language'])? $lang=Database::escape_string($_POST['language']): $lang = 'english'; |
|
| 2541 | + isset($_POST['language']) ? $lang = Database::escape_string($_POST['language']) : $lang = 'english'; |
|
| 2542 | 2542 | $di->connectDb(NULL, NULL, $lang); |
| 2543 | 2543 | $di->addChunk($ic_slide); |
| 2544 | 2544 | |
@@ -2557,7 +2557,7 @@ discard block |
||
| 2557 | 2557 | function search_engine_edit() |
| 2558 | 2558 | { |
| 2559 | 2559 | // update search enchine and its values table if enabled |
| 2560 | - if (api_get_setting('search_enabled')=='true' && extension_loaded('xapian')) { |
|
| 2560 | + if (api_get_setting('search_enabled') == 'true' && extension_loaded('xapian')) { |
|
| 2561 | 2561 | $course_id = api_get_course_id(); |
| 2562 | 2562 | |
| 2563 | 2563 | // actually, it consists on delete terms from db, |
@@ -2569,9 +2569,9 @@ discard block |
||
| 2569 | 2569 | $res = Database::query($sql); |
| 2570 | 2570 | |
| 2571 | 2571 | if (Database::num_rows($res) > 0) { |
| 2572 | - require_once(api_get_path(LIBRARY_PATH) . 'search/ChamiloIndexer.class.php'); |
|
| 2573 | - require_once(api_get_path(LIBRARY_PATH) . 'search/IndexableChunk.class.php'); |
|
| 2574 | - require_once(api_get_path(LIBRARY_PATH) . 'specific_fields_manager.lib.php'); |
|
| 2572 | + require_once(api_get_path(LIBRARY_PATH).'search/ChamiloIndexer.class.php'); |
|
| 2573 | + require_once(api_get_path(LIBRARY_PATH).'search/IndexableChunk.class.php'); |
|
| 2574 | + require_once(api_get_path(LIBRARY_PATH).'specific_fields_manager.lib.php'); |
|
| 2575 | 2575 | |
| 2576 | 2576 | $se_ref = Database::fetch_array($res); |
| 2577 | 2577 | $specific_fields = get_specific_field_list(); |
@@ -2582,7 +2582,7 @@ discard block |
||
| 2582 | 2582 | delete_all_specific_field_value($course_id, $specific_field['id'], TOOL_QUIZ, $this->id); |
| 2583 | 2583 | if (isset($_REQUEST[$specific_field['code']])) { |
| 2584 | 2584 | $sterms = trim($_REQUEST[$specific_field['code']]); |
| 2585 | - $all_specific_terms .= ' '. $sterms; |
|
| 2585 | + $all_specific_terms .= ' '.$sterms; |
|
| 2586 | 2586 | $sterms = explode(',', $sterms); |
| 2587 | 2587 | foreach ($sterms as $sterm) { |
| 2588 | 2588 | $ic_slide->addTerm(trim($sterm), $specific_field['code']); |
@@ -2598,17 +2598,17 @@ discard block |
||
| 2598 | 2598 | $xapian_data = array( |
| 2599 | 2599 | SE_COURSE_ID => $course_id, |
| 2600 | 2600 | SE_TOOL_ID => TOOL_QUIZ, |
| 2601 | - SE_DATA => array('type' => SE_DOCTYPE_EXERCISE_EXERCISE, 'exercise_id' => (int)$this->id), |
|
| 2602 | - SE_USER => (int)api_get_user_id(), |
|
| 2601 | + SE_DATA => array('type' => SE_DOCTYPE_EXERCISE_EXERCISE, 'exercise_id' => (int) $this->id), |
|
| 2602 | + SE_USER => (int) api_get_user_id(), |
|
| 2603 | 2603 | ); |
| 2604 | 2604 | $ic_slide->xapian_data = serialize($xapian_data); |
| 2605 | - $exercise_description = $all_specific_terms .' '. $this->description; |
|
| 2605 | + $exercise_description = $all_specific_terms.' '.$this->description; |
|
| 2606 | 2606 | $ic_slide->addValue("content", $exercise_description); |
| 2607 | 2607 | |
| 2608 | 2608 | $di = new ChamiloIndexer(); |
| 2609 | - isset($_POST['language'])? $lang=Database::escape_string($_POST['language']): $lang = 'english'; |
|
| 2609 | + isset($_POST['language']) ? $lang = Database::escape_string($_POST['language']) : $lang = 'english'; |
|
| 2610 | 2610 | $di->connectDb(NULL, NULL, $lang); |
| 2611 | - $di->remove_document((int)$se_ref['search_did']); |
|
| 2611 | + $di->remove_document((int) $se_ref['search_did']); |
|
| 2612 | 2612 | $di->addChunk($ic_slide); |
| 2613 | 2613 | |
| 2614 | 2614 | //index and return search engine document id |
@@ -2633,7 +2633,7 @@ discard block |
||
| 2633 | 2633 | function search_engine_delete() |
| 2634 | 2634 | { |
| 2635 | 2635 | // remove from search engine if enabled |
| 2636 | - if (api_get_setting('search_enabled') == 'true' && extension_loaded('xapian') ) { |
|
| 2636 | + if (api_get_setting('search_enabled') == 'true' && extension_loaded('xapian')) { |
|
| 2637 | 2637 | $course_id = api_get_course_id(); |
| 2638 | 2638 | $tbl_se_ref = Database::get_main_table(TABLE_MAIN_SEARCH_ENGINE_REF); |
| 2639 | 2639 | $sql = 'SELECT * FROM %s WHERE course_code=\'%s\' AND tool_id=\'%s\' AND ref_id_high_level=%s AND ref_id_second_level IS NULL LIMIT 1'; |
@@ -2641,19 +2641,19 @@ discard block |
||
| 2641 | 2641 | $res = Database::query($sql); |
| 2642 | 2642 | if (Database::num_rows($res) > 0) { |
| 2643 | 2643 | $row = Database::fetch_array($res); |
| 2644 | - require_once(api_get_path(LIBRARY_PATH) .'search/ChamiloIndexer.class.php'); |
|
| 2644 | + require_once(api_get_path(LIBRARY_PATH).'search/ChamiloIndexer.class.php'); |
|
| 2645 | 2645 | $di = new ChamiloIndexer(); |
| 2646 | - $di->remove_document((int)$row['search_did']); |
|
| 2646 | + $di->remove_document((int) $row['search_did']); |
|
| 2647 | 2647 | unset($di); |
| 2648 | 2648 | $tbl_quiz_question = Database::get_course_table(TABLE_QUIZ_QUESTION); |
| 2649 | - foreach ( $this->questionList as $question_i) { |
|
| 2649 | + foreach ($this->questionList as $question_i) { |
|
| 2650 | 2650 | $sql = 'SELECT type FROM %s WHERE id=%s'; |
| 2651 | 2651 | $sql = sprintf($sql, $tbl_quiz_question, $question_i); |
| 2652 | 2652 | $qres = Database::query($sql); |
| 2653 | 2653 | if (Database::num_rows($qres) > 0) { |
| 2654 | 2654 | $qrow = Database::fetch_array($qres); |
| 2655 | 2655 | $objQuestion = Question::getInstance($qrow['type']); |
| 2656 | - $objQuestion = Question::read((int)$question_i); |
|
| 2656 | + $objQuestion = Question::read((int) $question_i); |
|
| 2657 | 2657 | $objQuestion->search_engine_edit($this->id, FALSE, TRUE); |
| 2658 | 2658 | unset($objQuestion); |
| 2659 | 2659 | } |
@@ -2664,7 +2664,7 @@ discard block |
||
| 2664 | 2664 | Database::query($sql); |
| 2665 | 2665 | |
| 2666 | 2666 | // remove terms from db |
| 2667 | - require_once api_get_path(LIBRARY_PATH) .'specific_fields_manager.lib.php'; |
|
| 2667 | + require_once api_get_path(LIBRARY_PATH).'specific_fields_manager.lib.php'; |
|
| 2668 | 2668 | delete_all_values_for_item($course_id, TOOL_QUIZ, $this->id); |
| 2669 | 2669 | } |
| 2670 | 2670 | } |
@@ -2759,7 +2759,7 @@ discard block |
||
| 2759 | 2759 | */ |
| 2760 | 2760 | public function copy_exercise() |
| 2761 | 2761 | { |
| 2762 | - $exercise_obj= new Exercise(); |
|
| 2762 | + $exercise_obj = new Exercise(); |
|
| 2763 | 2763 | $exercise_obj = $this; |
| 2764 | 2764 | |
| 2765 | 2765 | // force the creation of a new exercise |
@@ -2831,25 +2831,25 @@ discard block |
||
| 2831 | 2831 | $lp_id = 0; |
| 2832 | 2832 | } |
| 2833 | 2833 | if (empty($lp_item_id)) { |
| 2834 | - $lp_item_id = 0; |
|
| 2834 | + $lp_item_id = 0; |
|
| 2835 | 2835 | } |
| 2836 | 2836 | if (empty($lp_item_view_id)) { |
| 2837 | 2837 | $lp_item_view_id = 0; |
| 2838 | 2838 | } |
| 2839 | - $condition = ' WHERE exe_exo_id = ' . "'" . $this->id . "'" .' AND |
|
| 2840 | - exe_user_id = ' . "'" . api_get_user_id() . "'" . ' AND |
|
| 2841 | - c_id = ' . api_get_course_int_id() . ' AND |
|
| 2842 | - status = ' . "'" . Database::escape_string($status). "'" . ' AND |
|
| 2843 | - orig_lp_id = ' . "'" . $lp_id . "'" . ' AND |
|
| 2844 | - orig_lp_item_id = ' . "'" . $lp_item_id . "'" . ' AND |
|
| 2845 | - orig_lp_item_view_id = ' . "'" . $lp_item_view_id . "'" . ' AND |
|
| 2846 | - session_id = ' . "'" . api_get_session_id() . "' LIMIT 1"; //Adding limit 1 just in case |
|
| 2839 | + $condition = ' WHERE exe_exo_id = '."'".$this->id."'".' AND |
|
| 2840 | + exe_user_id = ' . "'".api_get_user_id()."'".' AND |
|
| 2841 | + c_id = ' . api_get_course_int_id().' AND |
|
| 2842 | + status = ' . "'".Database::escape_string($status)."'".' AND |
|
| 2843 | + orig_lp_id = ' . "'".$lp_id."'".' AND |
|
| 2844 | + orig_lp_item_id = ' . "'".$lp_item_id."'".' AND |
|
| 2845 | + orig_lp_item_view_id = ' . "'".$lp_item_view_id."'".' AND |
|
| 2846 | + session_id = ' . "'".api_get_session_id()."' LIMIT 1"; //Adding limit 1 just in case |
|
| 2847 | 2847 | |
| 2848 | 2848 | $sql_track = 'SELECT * FROM '.$track_exercises.$condition; |
| 2849 | 2849 | |
| 2850 | 2850 | $result = Database::query($sql_track); |
| 2851 | 2851 | $new_array = array(); |
| 2852 | - if (Database::num_rows($result) > 0 ) { |
|
| 2852 | + if (Database::num_rows($result) > 0) { |
|
| 2853 | 2853 | $new_array = Database::fetch_array($result, 'ASSOC'); |
| 2854 | 2854 | $new_array['num_exe'] = Database::num_rows($result); |
| 2855 | 2855 | } |
@@ -2892,12 +2892,12 @@ discard block |
||
| 2892 | 2892 | $questionList = array_map('intval', $questionList); |
| 2893 | 2893 | |
| 2894 | 2894 | $params = array( |
| 2895 | - 'exe_exo_id' => $this->id , |
|
| 2895 | + 'exe_exo_id' => $this->id, |
|
| 2896 | 2896 | 'exe_user_id' => api_get_user_id(), |
| 2897 | 2897 | 'c_id' => api_get_course_int_id(), |
| 2898 | 2898 | 'status' => 'incomplete', |
| 2899 | 2899 | 'session_id' => api_get_session_id(), |
| 2900 | - 'data_tracking' => implode(',', $questionList) , |
|
| 2900 | + 'data_tracking' => implode(',', $questionList), |
|
| 2901 | 2901 | 'start_date' => api_get_utc_datetime(), |
| 2902 | 2902 | 'orig_lp_id' => $safe_lp_id, |
| 2903 | 2903 | 'orig_lp_item_id' => $safe_lp_item_id, |
@@ -2929,7 +2929,7 @@ discard block |
||
| 2929 | 2929 | $nbrQuestions = $this->get_count_question_list(); |
| 2930 | 2930 | |
| 2931 | 2931 | $all_button = $html = $label = ''; |
| 2932 | - $hotspot_get = isset($_POST['hotspot']) ? Security::remove_XSS($_POST['hotspot']):null; |
|
| 2932 | + $hotspot_get = isset($_POST['hotspot']) ? Security::remove_XSS($_POST['hotspot']) : null; |
|
| 2933 | 2933 | |
| 2934 | 2934 | if ($this->selectFeedbackType() == EXERCISE_FEEDBACK_TYPE_DIRECT && $this->type == ONE_PER_PAGE) { |
| 2935 | 2935 | $urlTitle = get_lang('ContinueTest'); |
@@ -2940,7 +2940,7 @@ discard block |
||
| 2940 | 2940 | |
| 2941 | 2941 | $html .= Display::url( |
| 2942 | 2942 | $urlTitle, |
| 2943 | - 'exercise_submit_modal.php?' . http_build_query([ |
|
| 2943 | + 'exercise_submit_modal.php?'.http_build_query([ |
|
| 2944 | 2944 | 'learnpath_id' => $safe_lp_id, |
| 2945 | 2945 | 'learnpath_item_id' => $safe_lp_item_id, |
| 2946 | 2946 | 'learnpath_item_view_id' => $safe_lp_item_view_id, |
@@ -2957,7 +2957,7 @@ discard block |
||
| 2957 | 2957 | 'data-size' => 'md' |
| 2958 | 2958 | ] |
| 2959 | 2959 | ); |
| 2960 | - $html .='<br />'; |
|
| 2960 | + $html .= '<br />'; |
|
| 2961 | 2961 | } else { |
| 2962 | 2962 | // User |
| 2963 | 2963 | if (api_is_allowed_to_session_edit()) { |
@@ -2982,7 +2982,7 @@ discard block |
||
| 2982 | 2982 | |
| 2983 | 2983 | //Next question |
| 2984 | 2984 | if (!empty($questions_in_media)) { |
| 2985 | - $questions_in_media = "['".implode("','",$questions_in_media)."']"; |
|
| 2985 | + $questions_in_media = "['".implode("','", $questions_in_media)."']"; |
|
| 2986 | 2986 | $all_button .= ' <a href="javascript://" class="'.$class.'" onclick="save_question_list('.$questions_in_media.'); ">'.$label.'</a>'; |
| 2987 | 2987 | } else { |
| 2988 | 2988 | $all_button .= ' <a href="javascript://" class="'.$class.'" onclick="save_now('.$question_id.', \'\', \''.$currentAnswer.'\'); ">'.$label.'</a>'; |
@@ -3000,7 +3000,7 @@ discard block |
||
| 3000 | 3000 | } |
| 3001 | 3001 | $class .= ' question-validate-btn'; // used to select it with jquery |
| 3002 | 3002 | $all_button = ' <a href="javascript://" class="'.$class.'" onclick="validate_all(); ">'.$all_label.'</a>'; |
| 3003 | - $all_button .= ' ' . Display::span(null, ['id' => 'save_all_reponse']); |
|
| 3003 | + $all_button .= ' '.Display::span(null, ['id' => 'save_all_reponse']); |
|
| 3004 | 3004 | $html .= $all_button; |
| 3005 | 3005 | } |
| 3006 | 3006 | } |
@@ -3243,8 +3243,8 @@ discard block |
||
| 3243 | 3243 | $params = array(); |
| 3244 | 3244 | $params['course_id'] = $course_id; |
| 3245 | 3245 | $params['session_id'] = api_get_session_id(); |
| 3246 | - $params['user_id'] = isset($exe_info['exe_user_id'])? $exe_info['exe_user_id'] : api_get_user_id(); |
|
| 3247 | - $params['exercise_id'] = isset($exe_info['exe_exo_id'])? $exe_info['exe_exo_id'] : $this->id; |
|
| 3246 | + $params['user_id'] = isset($exe_info['exe_user_id']) ? $exe_info['exe_user_id'] : api_get_user_id(); |
|
| 3247 | + $params['exercise_id'] = isset($exe_info['exe_exo_id']) ? $exe_info['exe_exo_id'] : $this->id; |
|
| 3248 | 3248 | $params['question_id'] = $questionId; |
| 3249 | 3249 | $params['exe_id'] = isset($exe_info['exe_id']) ? $exe_info['exe_id'] : $exeId; |
| 3250 | 3250 | |
@@ -3297,7 +3297,7 @@ discard block |
||
| 3297 | 3297 | $answer = $objAnswerTmp->selectAnswer($answerId); |
| 3298 | 3298 | $answerComment = $objAnswerTmp->selectComment($answerId); |
| 3299 | 3299 | $answerCorrect = $objAnswerTmp->isCorrect($answerId); |
| 3300 | - $answerWeighting = (float)$objAnswerTmp->selectWeighting($answerId); |
|
| 3300 | + $answerWeighting = (float) $objAnswerTmp->selectWeighting($answerId); |
|
| 3301 | 3301 | $answerAutoId = $objAnswerTmp->selectAutoId($answerId); |
| 3302 | 3302 | |
| 3303 | 3303 | if ($debug) { |
@@ -3307,7 +3307,7 @@ discard block |
||
| 3307 | 3307 | |
| 3308 | 3308 | // Delineation |
| 3309 | 3309 | $delineation_cord = $objAnswerTmp->selectHotspotCoordinates(1); |
| 3310 | - $answer_delineation_destination=$objAnswerTmp->selectDestination(1); |
|
| 3310 | + $answer_delineation_destination = $objAnswerTmp->selectDestination(1); |
|
| 3311 | 3311 | |
| 3312 | 3312 | switch ($answerType) { |
| 3313 | 3313 | // for unique answer |
@@ -3320,7 +3320,7 @@ discard block |
||
| 3320 | 3320 | exe_id = '".$exeId."' AND |
| 3321 | 3321 | question_id= '".$questionId."'"; |
| 3322 | 3322 | $result = Database::query($sql); |
| 3323 | - $choice = Database::result($result,0,"answer"); |
|
| 3323 | + $choice = Database::result($result, 0, "answer"); |
|
| 3324 | 3324 | |
| 3325 | 3325 | $studentChoice = $choice == $answerAutoId ? 1 : 0; |
| 3326 | 3326 | if ($studentChoice) { |
@@ -3371,7 +3371,7 @@ discard block |
||
| 3371 | 3371 | } else { |
| 3372 | 3372 | // If no result then the user just hit don't know |
| 3373 | 3373 | $studentChoice = 3; |
| 3374 | - $questionScore += $doubt_score; |
|
| 3374 | + $questionScore += $doubt_score; |
|
| 3375 | 3375 | } |
| 3376 | 3376 | $totalScore = $questionScore; |
| 3377 | 3377 | break; |
@@ -3387,17 +3387,17 @@ discard block |
||
| 3387 | 3387 | } |
| 3388 | 3388 | |
| 3389 | 3389 | $studentChoice = isset($choice[$answerAutoId]) ? $choice[$answerAutoId] : null; |
| 3390 | - $real_answers[$answerId] = (bool)$studentChoice; |
|
| 3390 | + $real_answers[$answerId] = (bool) $studentChoice; |
|
| 3391 | 3391 | |
| 3392 | 3392 | if ($studentChoice) { |
| 3393 | - $questionScore +=$answerWeighting; |
|
| 3393 | + $questionScore += $answerWeighting; |
|
| 3394 | 3394 | } |
| 3395 | 3395 | } else { |
| 3396 | 3396 | $studentChoice = isset($choice[$answerAutoId]) ? $choice[$answerAutoId] : null; |
| 3397 | - $real_answers[$answerId] = (bool)$studentChoice; |
|
| 3397 | + $real_answers[$answerId] = (bool) $studentChoice; |
|
| 3398 | 3398 | |
| 3399 | 3399 | if (isset($studentChoice)) { |
| 3400 | - $questionScore += $answerWeighting; |
|
| 3400 | + $questionScore += $answerWeighting; |
|
| 3401 | 3401 | } |
| 3402 | 3402 | } |
| 3403 | 3403 | $totalScore += $answerWeighting; |
@@ -3415,16 +3415,16 @@ discard block |
||
| 3415 | 3415 | $choice[$ind] = 1; |
| 3416 | 3416 | } |
| 3417 | 3417 | $studentChoice = isset($choice[$answerAutoId]) ? $choice[$answerAutoId] : null; |
| 3418 | - $real_answers[$answerId] = (bool)$studentChoice; |
|
| 3418 | + $real_answers[$answerId] = (bool) $studentChoice; |
|
| 3419 | 3419 | if ($studentChoice) { |
| 3420 | - $questionScore +=$answerWeighting; |
|
| 3420 | + $questionScore += $answerWeighting; |
|
| 3421 | 3421 | } |
| 3422 | 3422 | } else { |
| 3423 | 3423 | $studentChoice = isset($choice[$answerAutoId]) ? $choice[$answerAutoId] : null; |
| 3424 | 3424 | if (isset($studentChoice)) { |
| 3425 | 3425 | $questionScore += $answerWeighting; |
| 3426 | 3426 | } |
| 3427 | - $real_answers[$answerId] = (bool)$studentChoice; |
|
| 3427 | + $real_answers[$answerId] = (bool) $studentChoice; |
|
| 3428 | 3428 | } |
| 3429 | 3429 | $totalScore += $answerWeighting; |
| 3430 | 3430 | if ($debug) error_log("studentChoice: $studentChoice"); |
@@ -3436,7 +3436,7 @@ discard block |
||
| 3436 | 3436 | $resultans = Database::query($sql); |
| 3437 | 3437 | while ($row = Database::fetch_array($resultans)) { |
| 3438 | 3438 | $ind = $row['answer']; |
| 3439 | - $result = explode(':',$ind); |
|
| 3439 | + $result = explode(':', $ind); |
|
| 3440 | 3440 | if (isset($result[0])) { |
| 3441 | 3441 | $my_answer_id = isset($result[0]) ? $result[0] : ''; |
| 3442 | 3442 | $option = isset($result[1]) ? $result[1] : ''; |
@@ -3556,10 +3556,10 @@ discard block |
||
| 3556 | 3556 | } |
| 3557 | 3557 | // adds the piece of text that is before the blank |
| 3558 | 3558 | //and ends with '[' into a general storage array |
| 3559 | - $real_text[] = api_substr($temp, 0, $pos +1); |
|
| 3560 | - $answer .= api_substr($temp, 0, $pos +1); |
|
| 3559 | + $real_text[] = api_substr($temp, 0, $pos + 1); |
|
| 3560 | + $answer .= api_substr($temp, 0, $pos + 1); |
|
| 3561 | 3561 | //take the string remaining (after the last "[" we found) |
| 3562 | - $temp = api_substr($temp, $pos +1); |
|
| 3562 | + $temp = api_substr($temp, $pos + 1); |
|
| 3563 | 3563 | // quit the loop if there are no more blanks, and update $pos to the position of next ']' |
| 3564 | 3564 | if (($pos = api_strpos($temp, ']')) === false) { |
| 3565 | 3565 | // adds the end of the text |
@@ -3595,7 +3595,7 @@ discard block |
||
| 3595 | 3595 | //put the contents of the [] answer tag into correct_tags[] |
| 3596 | 3596 | $correct_tags[] = api_substr($temp, 0, $pos); |
| 3597 | 3597 | $j++; |
| 3598 | - $temp = api_substr($temp, $pos +1); |
|
| 3598 | + $temp = api_substr($temp, $pos + 1); |
|
| 3599 | 3599 | } |
| 3600 | 3600 | $answer = ''; |
| 3601 | 3601 | $real_correct_tags = $correct_tags; |
@@ -3618,7 +3618,7 @@ discard block |
||
| 3618 | 3618 | } elseif (!empty($user_tags[$i])) { |
| 3619 | 3619 | // else if the word entered by the student IS NOT the same as the one defined by the professor |
| 3620 | 3620 | // adds the word in red at the end of the string, and strikes it |
| 3621 | - $answer .= '<font color="red"><s>' . $user_tags[$i] . '</s></font>'; |
|
| 3621 | + $answer .= '<font color="red"><s>'.$user_tags[$i].'</s></font>'; |
|
| 3622 | 3622 | } else { |
| 3623 | 3623 | // adds a tabulation if no word has been typed by the student |
| 3624 | 3624 | $answer .= ''; // remove that causes issue |
@@ -3637,17 +3637,17 @@ discard block |
||
| 3637 | 3637 | } elseif (!empty ($user_tags[$i])) { |
| 3638 | 3638 | // else if the word entered by the student IS NOT the same as the one defined by the professor |
| 3639 | 3639 | // adds the word in red at the end of the string, and strikes it |
| 3640 | - $answer .= '<font color="red"><s>' . $user_tags[$i] . '</s></font>'; |
|
| 3640 | + $answer .= '<font color="red"><s>'.$user_tags[$i].'</s></font>'; |
|
| 3641 | 3641 | } else { |
| 3642 | 3642 | // adds a tabulation if no word has been typed by the student |
| 3643 | - $answer .= ''; // remove that causes issue |
|
| 3643 | + $answer .= ''; // remove that causes issue |
|
| 3644 | 3644 | } |
| 3645 | 3645 | } |
| 3646 | 3646 | |
| 3647 | 3647 | // adds the correct word, followed by ] to close the blank |
| 3648 | - $answer .= ' / <font color="green"><b>' . $real_correct_tags[$i] . '</b></font>]'; |
|
| 3649 | - if (isset($real_text[$i +1])) { |
|
| 3650 | - $answer .= $real_text[$i +1]; |
|
| 3648 | + $answer .= ' / <font color="green"><b>'.$real_correct_tags[$i].'</b></font>]'; |
|
| 3649 | + if (isset($real_text[$i + 1])) { |
|
| 3650 | + $answer .= $real_text[$i + 1]; |
|
| 3651 | 3651 | } |
| 3652 | 3652 | } |
| 3653 | 3653 | } else { |
@@ -3780,10 +3780,10 @@ discard block |
||
| 3780 | 3780 | } |
| 3781 | 3781 | // adds the piece of text that is before the blank |
| 3782 | 3782 | //and ends with '[' into a general storage array |
| 3783 | - $realText[] = api_substr($temp, 0, $pos +1); |
|
| 3784 | - $answer .= api_substr($temp, 0, $pos +1); |
|
| 3783 | + $realText[] = api_substr($temp, 0, $pos + 1); |
|
| 3784 | + $answer .= api_substr($temp, 0, $pos + 1); |
|
| 3785 | 3785 | //take the string remaining (after the last "[" we found) |
| 3786 | - $temp = api_substr($temp, $pos +1); |
|
| 3786 | + $temp = api_substr($temp, $pos + 1); |
|
| 3787 | 3787 | // quit the loop if there are no more blanks, and update $pos to the position of next ']' |
| 3788 | 3788 | if (($pos = api_strpos($temp, ']')) === false) { |
| 3789 | 3789 | // adds the end of the text |
@@ -3817,7 +3817,7 @@ discard block |
||
| 3817 | 3817 | //put the contents of the [] answer tag into correct_tags[] |
| 3818 | 3818 | $correctTags[] = api_substr($temp, 0, $pos); |
| 3819 | 3819 | $j++; |
| 3820 | - $temp = api_substr($temp, $pos +1); |
|
| 3820 | + $temp = api_substr($temp, $pos + 1); |
|
| 3821 | 3821 | } |
| 3822 | 3822 | $answer = ''; |
| 3823 | 3823 | $realCorrectTags = $correctTags; |
@@ -3826,7 +3826,7 @@ discard block |
||
| 3826 | 3826 | $queryfill = "SELECT answer, marks FROM ".$TBL_TRACK_ATTEMPT." |
| 3827 | 3827 | WHERE |
| 3828 | 3828 | exe_id = '".$exeId."' AND |
| 3829 | - question_id= ".intval($questionId) ; |
|
| 3829 | + question_id= ".intval($questionId); |
|
| 3830 | 3830 | $resfill = Database::query($queryfill); |
| 3831 | 3831 | $rowFill = Database::fetch_assoc($resfill); |
| 3832 | 3832 | $answer = $rowFill['answer']; |
@@ -3849,7 +3849,7 @@ discard block |
||
| 3849 | 3849 | } elseif (!empty($userTags[$i])) { |
| 3850 | 3850 | // else if the word entered by the student IS NOT the same as the one defined by the professor |
| 3851 | 3851 | // adds the word in red at the end of the string, and strikes it |
| 3852 | - $answer .= '<font color="red"><s>' . $userTags[$i] . '</s></font>'; |
|
| 3852 | + $answer .= '<font color="red"><s>'.$userTags[$i].'</s></font>'; |
|
| 3853 | 3853 | } else { |
| 3854 | 3854 | // adds a tabulation if no word has been typed by the student |
| 3855 | 3855 | $answer .= ''; // remove that causes issue |
@@ -3866,19 +3866,19 @@ discard block |
||
| 3866 | 3866 | |
| 3867 | 3867 | if ($addCorrecWord) { |
| 3868 | 3868 | // adds the correct word, followed by ] to close the blank |
| 3869 | - $answer .= ' / <font color="green"><b>' . $realCorrectTags[$i] . '</b></font>'; |
|
| 3869 | + $answer .= ' / <font color="green"><b>'.$realCorrectTags[$i].'</b></font>'; |
|
| 3870 | 3870 | } |
| 3871 | 3871 | |
| 3872 | 3872 | $answer .= ']'; |
| 3873 | 3873 | |
| 3874 | - if (isset($realText[$i +1])) { |
|
| 3875 | - $answer .= $realText[$i +1]; |
|
| 3874 | + if (isset($realText[$i + 1])) { |
|
| 3875 | + $answer .= $realText[$i + 1]; |
|
| 3876 | 3876 | } |
| 3877 | 3877 | } |
| 3878 | 3878 | break; |
| 3879 | 3879 | case FREE_ANSWER: |
| 3880 | 3880 | if ($from_database) { |
| 3881 | - $query = "SELECT answer, marks FROM ".$TBL_TRACK_ATTEMPT." |
|
| 3881 | + $query = "SELECT answer, marks FROM ".$TBL_TRACK_ATTEMPT." |
|
| 3882 | 3882 | WHERE exe_id = '".$exeId."' AND question_id= '".$questionId."'"; |
| 3883 | 3883 | $resq = Database::query($query); |
| 3884 | 3884 | $data = Database::fetch_array($resq); |
@@ -3889,9 +3889,9 @@ discard block |
||
| 3889 | 3889 | $questionScore = $data['marks']; |
| 3890 | 3890 | |
| 3891 | 3891 | if ($questionScore == -1) { |
| 3892 | - $totalScore+= 0; |
|
| 3892 | + $totalScore += 0; |
|
| 3893 | 3893 | } else { |
| 3894 | - $totalScore+= $questionScore; |
|
| 3894 | + $totalScore += $questionScore; |
|
| 3895 | 3895 | } |
| 3896 | 3896 | if ($questionScore == '') { |
| 3897 | 3897 | $questionScore = 0; |
@@ -3912,14 +3912,14 @@ discard block |
||
| 3912 | 3912 | $query = "SELECT answer, marks FROM ".$TBL_TRACK_ATTEMPT." |
| 3913 | 3913 | WHERE exe_id = '".$exeId."' AND question_id= '".$questionId."'"; |
| 3914 | 3914 | $resq = Database::query($query); |
| 3915 | - $choice = Database::result($resq,0,'answer'); |
|
| 3915 | + $choice = Database::result($resq, 0, 'answer'); |
|
| 3916 | 3916 | $choice = str_replace('\r\n', '', $choice); |
| 3917 | 3917 | $choice = stripslashes($choice); |
| 3918 | - $questionScore = Database::result($resq,0,"marks"); |
|
| 3918 | + $questionScore = Database::result($resq, 0, "marks"); |
|
| 3919 | 3919 | if ($questionScore == -1) { |
| 3920 | - $totalScore+=0; |
|
| 3920 | + $totalScore += 0; |
|
| 3921 | 3921 | } else { |
| 3922 | - $totalScore+=$questionScore; |
|
| 3922 | + $totalScore += $questionScore; |
|
| 3923 | 3923 | } |
| 3924 | 3924 | $arrques = $questionName; |
| 3925 | 3925 | $arrans = $choice; |
@@ -3964,7 +3964,7 @@ discard block |
||
| 3964 | 3964 | |
| 3965 | 3965 | while ($a_answers = Database::fetch_array($res_answers)) { |
| 3966 | 3966 | $i_answer_id = $a_answers['id']; //3 |
| 3967 | - $s_answer_label = $a_answers['answer']; // your daddy - your mother |
|
| 3967 | + $s_answer_label = $a_answers['answer']; // your daddy - your mother |
|
| 3968 | 3968 | $i_answer_correct_answer = $a_answers['correct']; //1 - 2 |
| 3969 | 3969 | $i_answer_id_auto = $a_answers['id_auto']; // 3 - 4 |
| 3970 | 3970 | |
@@ -4019,8 +4019,8 @@ discard block |
||
| 4019 | 4019 | $user_answer = ''; |
| 4020 | 4020 | } |
| 4021 | 4021 | echo '<tr>'; |
| 4022 | - echo '<td>' . $s_answer_label . '</td>'; |
|
| 4023 | - echo '<td>' . $user_answer; |
|
| 4022 | + echo '<td>'.$s_answer_label.'</td>'; |
|
| 4023 | + echo '<td>'.$user_answer; |
|
| 4024 | 4024 | |
| 4025 | 4025 | if (in_array($answerType, [MATCHING, MATCHING_DRAGGABLE])) { |
| 4026 | 4026 | if (isset($real_list[$i_answer_correct_answer]) && $showTotalScoreAndUserChoices == false) { |
@@ -4128,7 +4128,7 @@ discard block |
||
| 4128 | 4128 | if ($from_database) { |
| 4129 | 4129 | // getting the user answer |
| 4130 | 4130 | $TBL_TRACK_HOTSPOT = Database::get_main_table(TABLE_STATISTIC_TRACK_E_HOTSPOT); |
| 4131 | - $query = "SELECT hotspot_correct, hotspot_coordinate |
|
| 4131 | + $query = "SELECT hotspot_correct, hotspot_coordinate |
|
| 4132 | 4132 | FROM $TBL_TRACK_HOTSPOT |
| 4133 | 4133 | WHERE |
| 4134 | 4134 | hotspot_exe_id = '".$exeId."' AND |
@@ -4136,20 +4136,20 @@ discard block |
||
| 4136 | 4136 | hotspot_answer_id='1'"; |
| 4137 | 4137 | //by default we take 1 because it's a delineation |
| 4138 | 4138 | $resq = Database::query($query); |
| 4139 | - $row = Database::fetch_array($resq,'ASSOC'); |
|
| 4139 | + $row = Database::fetch_array($resq, 'ASSOC'); |
|
| 4140 | 4140 | |
| 4141 | 4141 | $choice = $row['hotspot_correct']; |
| 4142 | 4142 | $user_answer = $row['hotspot_coordinate']; |
| 4143 | 4143 | |
| 4144 | 4144 | // THIS is very important otherwise the poly_compile will throw an error!! |
| 4145 | 4145 | // round-up the coordinates |
| 4146 | - $coords = explode('/',$user_answer); |
|
| 4146 | + $coords = explode('/', $user_answer); |
|
| 4147 | 4147 | $user_array = ''; |
| 4148 | 4148 | foreach ($coords as $coord) { |
| 4149 | - list($x,$y) = explode(';',$coord); |
|
| 4149 | + list($x, $y) = explode(';', $coord); |
|
| 4150 | 4150 | $user_array .= round($x).';'.round($y).'/'; |
| 4151 | 4151 | } |
| 4152 | - $user_array = substr($user_array,0,-1); |
|
| 4152 | + $user_array = substr($user_array, 0, -1); |
|
| 4153 | 4153 | } else { |
| 4154 | 4154 | if (!empty($studentChoice)) { |
| 4155 | 4155 | $newquestionList[] = $questionId; |
@@ -4159,7 +4159,7 @@ discard block |
||
| 4159 | 4159 | $studentChoice = $choice[$answerId]; |
| 4160 | 4160 | $questionScore += $answerWeighting; |
| 4161 | 4161 | |
| 4162 | - if ($hotspot_delineation_result[1]==1) { |
|
| 4162 | + if ($hotspot_delineation_result[1] == 1) { |
|
| 4163 | 4163 | $totalScore += $answerWeighting; //adding the total |
| 4164 | 4164 | } |
| 4165 | 4165 | } |
@@ -4298,13 +4298,13 @@ discard block |
||
| 4298 | 4298 | $user_answer = $_SESSION['exerciseResultCoordinates'][$questionId]; |
| 4299 | 4299 | |
| 4300 | 4300 | //round-up the coordinates |
| 4301 | - $coords = explode('/',$user_answer); |
|
| 4301 | + $coords = explode('/', $user_answer); |
|
| 4302 | 4302 | $user_array = ''; |
| 4303 | 4303 | foreach ($coords as $coord) { |
| 4304 | - list($x,$y) = explode(';',$coord); |
|
| 4304 | + list($x, $y) = explode(';', $coord); |
|
| 4305 | 4305 | $user_array .= round($x).';'.round($y).'/'; |
| 4306 | 4306 | } |
| 4307 | - $user_array = substr($user_array,0,-1); |
|
| 4307 | + $user_array = substr($user_array, 0, -1); |
|
| 4308 | 4308 | |
| 4309 | 4309 | if ($next) { |
| 4310 | 4310 | |
@@ -4331,7 +4331,7 @@ discard block |
||
| 4331 | 4331 | //$overlap = round(polygons_overlap($poly_answer,$poly_user)); |
| 4332 | 4332 | // //this is an area in pixels |
| 4333 | 4333 | if ($debug > 0) { |
| 4334 | - error_log(__LINE__ . ' - Polygons results are ' . print_r($poly_results, 1), 0); |
|
| 4334 | + error_log(__LINE__.' - Polygons results are '.print_r($poly_results, 1), 0); |
|
| 4335 | 4335 | } |
| 4336 | 4336 | |
| 4337 | 4337 | if ($overlap < 1) { |
@@ -4344,43 +4344,43 @@ discard block |
||
| 4344 | 4344 | // that is overlapped by the user's polygon |
| 4345 | 4345 | $final_overlap = round(((float) $overlap / (float) $poly_answer_area) * 100); |
| 4346 | 4346 | if ($debug > 1) { |
| 4347 | - error_log(__LINE__ . ' - Final overlap is ' . $final_overlap, 0); |
|
| 4347 | + error_log(__LINE__.' - Final overlap is '.$final_overlap, 0); |
|
| 4348 | 4348 | } |
| 4349 | 4349 | // the final missing area is the percentage of the initial polygon |
| 4350 | 4350 | // that is not overlapped by the user's polygon |
| 4351 | 4351 | $final_missing = 100 - $final_overlap; |
| 4352 | 4352 | if ($debug > 1) { |
| 4353 | - error_log(__LINE__ . ' - Final missing is ' . $final_missing, 0); |
|
| 4353 | + error_log(__LINE__.' - Final missing is '.$final_missing, 0); |
|
| 4354 | 4354 | } |
| 4355 | 4355 | // the final excess area is the percentage of the initial polygon's size |
| 4356 | 4356 | // that is covered by the user's polygon outside of the initial polygon |
| 4357 | 4357 | $final_excess = round((((float) $poly_user_area - (float) $overlap) / (float) $poly_answer_area) * 100); |
| 4358 | 4358 | if ($debug > 1) { |
| 4359 | - error_log(__LINE__ . ' - Final excess is ' . $final_excess, 0); |
|
| 4359 | + error_log(__LINE__.' - Final excess is '.$final_excess, 0); |
|
| 4360 | 4360 | } |
| 4361 | 4361 | } |
| 4362 | 4362 | |
| 4363 | 4363 | //checking the destination parameters parsing the "@@" |
| 4364 | - $destination_items= explode('@@', $answerDestination); |
|
| 4364 | + $destination_items = explode('@@', $answerDestination); |
|
| 4365 | 4365 | $threadhold_total = $destination_items[0]; |
| 4366 | - $threadhold_items=explode(';',$threadhold_total); |
|
| 4366 | + $threadhold_items = explode(';', $threadhold_total); |
|
| 4367 | 4367 | $threadhold1 = $threadhold_items[0]; // overlap |
| 4368 | 4368 | $threadhold2 = $threadhold_items[1]; // excess |
| 4369 | - $threadhold3 = $threadhold_items[2]; //missing |
|
| 4369 | + $threadhold3 = $threadhold_items[2]; //missing |
|
| 4370 | 4370 | |
| 4371 | 4371 | // if is delineation |
| 4372 | - if ($answerId===1) { |
|
| 4372 | + if ($answerId === 1) { |
|
| 4373 | 4373 | //setting colors |
| 4374 | - if ($final_overlap>=$threadhold1) { |
|
| 4375 | - $overlap_color=true; //echo 'a'; |
|
| 4374 | + if ($final_overlap >= $threadhold1) { |
|
| 4375 | + $overlap_color = true; //echo 'a'; |
|
| 4376 | 4376 | } |
| 4377 | 4377 | //echo $excess.'-'.$threadhold2; |
| 4378 | - if ($final_excess<=$threadhold2) { |
|
| 4379 | - $excess_color=true; //echo 'b'; |
|
| 4378 | + if ($final_excess <= $threadhold2) { |
|
| 4379 | + $excess_color = true; //echo 'b'; |
|
| 4380 | 4380 | } |
| 4381 | 4381 | //echo '--------'.$missing.'-'.$threadhold3; |
| 4382 | - if ($final_missing<=$threadhold3) { |
|
| 4383 | - $missing_color=true; //echo 'c'; |
|
| 4382 | + if ($final_missing <= $threadhold3) { |
|
| 4383 | + $missing_color = true; //echo 'c'; |
|
| 4384 | 4384 | } |
| 4385 | 4385 | |
| 4386 | 4386 | // if pass |
@@ -4389,67 +4389,67 @@ discard block |
||
| 4389 | 4389 | $final_missing <= $threadhold3 && |
| 4390 | 4390 | $final_excess <= $threadhold2 |
| 4391 | 4391 | ) { |
| 4392 | - $next=1; //go to the oars |
|
| 4393 | - $result_comment=get_lang('Acceptable'); |
|
| 4394 | - $final_answer = 1; // do not update with update_exercise_attempt |
|
| 4392 | + $next = 1; //go to the oars |
|
| 4393 | + $result_comment = get_lang('Acceptable'); |
|
| 4394 | + $final_answer = 1; // do not update with update_exercise_attempt |
|
| 4395 | 4395 | } else { |
| 4396 | - $next=0; |
|
| 4397 | - $result_comment=get_lang('Unacceptable'); |
|
| 4398 | - $comment=$answerDestination=$objAnswerTmp->selectComment(1); |
|
| 4399 | - $answerDestination=$objAnswerTmp->selectDestination(1); |
|
| 4396 | + $next = 0; |
|
| 4397 | + $result_comment = get_lang('Unacceptable'); |
|
| 4398 | + $comment = $answerDestination = $objAnswerTmp->selectComment(1); |
|
| 4399 | + $answerDestination = $objAnswerTmp->selectDestination(1); |
|
| 4400 | 4400 | //checking the destination parameters parsing the "@@" |
| 4401 | - $destination_items= explode('@@', $answerDestination); |
|
| 4401 | + $destination_items = explode('@@', $answerDestination); |
|
| 4402 | 4402 | } |
| 4403 | - } elseif($answerId>1) { |
|
| 4403 | + } elseif ($answerId > 1) { |
|
| 4404 | 4404 | if ($objAnswerTmp->selectHotspotType($answerId) == 'noerror') { |
| 4405 | - if ($debug>0) { |
|
| 4406 | - error_log(__LINE__.' - answerId is of type noerror',0); |
|
| 4405 | + if ($debug > 0) { |
|
| 4406 | + error_log(__LINE__.' - answerId is of type noerror', 0); |
|
| 4407 | 4407 | } |
| 4408 | 4408 | //type no error shouldn't be treated |
| 4409 | 4409 | $next = 1; |
| 4410 | 4410 | continue; |
| 4411 | 4411 | } |
| 4412 | - if ($debug>0) { |
|
| 4413 | - error_log(__LINE__.' - answerId is >1 so we\'re probably in OAR',0); |
|
| 4412 | + if ($debug > 0) { |
|
| 4413 | + error_log(__LINE__.' - answerId is >1 so we\'re probably in OAR', 0); |
|
| 4414 | 4414 | } |
| 4415 | 4415 | //check the intersection between the oar and the user |
| 4416 | 4416 | //echo 'user'; print_r($x_user_list); print_r($y_user_list); |
| 4417 | 4417 | //echo 'official';print_r($x_list);print_r($y_list); |
| 4418 | 4418 | //$result = get_intersection_data($x_list,$y_list,$x_user_list,$y_user_list); |
| 4419 | - $inter= $result['success']; |
|
| 4419 | + $inter = $result['success']; |
|
| 4420 | 4420 | |
| 4421 | 4421 | //$delineation_cord=$objAnswerTmp->selectHotspotCoordinates($answerId); |
| 4422 | - $delineation_cord=$objAnswerTmp->selectHotspotCoordinates($answerId); |
|
| 4422 | + $delineation_cord = $objAnswerTmp->selectHotspotCoordinates($answerId); |
|
| 4423 | 4423 | |
| 4424 | - $poly_answer = convert_coordinates($delineation_cord,'|'); |
|
| 4425 | - $max_coord = poly_get_max($poly_user,$poly_answer); |
|
| 4426 | - $poly_answer_compiled = poly_compile($poly_answer,$max_coord); |
|
| 4427 | - $overlap = poly_touch($poly_user_compiled, $poly_answer_compiled,$max_coord); |
|
| 4424 | + $poly_answer = convert_coordinates($delineation_cord, '|'); |
|
| 4425 | + $max_coord = poly_get_max($poly_user, $poly_answer); |
|
| 4426 | + $poly_answer_compiled = poly_compile($poly_answer, $max_coord); |
|
| 4427 | + $overlap = poly_touch($poly_user_compiled, $poly_answer_compiled, $max_coord); |
|
| 4428 | 4428 | |
| 4429 | 4429 | if ($overlap == false) { |
| 4430 | 4430 | //all good, no overlap |
| 4431 | 4431 | $next = 1; |
| 4432 | 4432 | continue; |
| 4433 | 4433 | } else { |
| 4434 | - if ($debug>0) { |
|
| 4435 | - error_log(__LINE__.' - Overlap is '.$overlap.': OAR hit',0); |
|
| 4434 | + if ($debug > 0) { |
|
| 4435 | + error_log(__LINE__.' - Overlap is '.$overlap.': OAR hit', 0); |
|
| 4436 | 4436 | } |
| 4437 | 4437 | $organs_at_risk_hit++; |
| 4438 | 4438 | //show the feedback |
| 4439 | - $next=0; |
|
| 4440 | - $comment=$answerDestination=$objAnswerTmp->selectComment($answerId); |
|
| 4441 | - $answerDestination=$objAnswerTmp->selectDestination($answerId); |
|
| 4442 | - |
|
| 4443 | - $destination_items= explode('@@', $answerDestination); |
|
| 4444 | - $try_hotspot=$destination_items[1]; |
|
| 4445 | - $lp_hotspot=$destination_items[2]; |
|
| 4446 | - $select_question_hotspot=$destination_items[3]; |
|
| 4447 | - $url_hotspot=$destination_items[4]; |
|
| 4439 | + $next = 0; |
|
| 4440 | + $comment = $answerDestination = $objAnswerTmp->selectComment($answerId); |
|
| 4441 | + $answerDestination = $objAnswerTmp->selectDestination($answerId); |
|
| 4442 | + |
|
| 4443 | + $destination_items = explode('@@', $answerDestination); |
|
| 4444 | + $try_hotspot = $destination_items[1]; |
|
| 4445 | + $lp_hotspot = $destination_items[2]; |
|
| 4446 | + $select_question_hotspot = $destination_items[3]; |
|
| 4447 | + $url_hotspot = $destination_items[4]; |
|
| 4448 | 4448 | } |
| 4449 | 4449 | } |
| 4450 | 4450 | } else { // the first delineation feedback |
| 4451 | - if ($debug>0) { |
|
| 4452 | - error_log(__LINE__.' first',0); |
|
| 4451 | + if ($debug > 0) { |
|
| 4452 | + error_log(__LINE__.' first', 0); |
|
| 4453 | 4453 | } |
| 4454 | 4454 | } |
| 4455 | 4455 | } elseif (in_array($answerType, [MATCHING, MATCHING_DRAGGABLE])) { |
@@ -4457,7 +4457,7 @@ discard block |
||
| 4457 | 4457 | echo Display::tag('td', $answerMatching[$answerId]); |
| 4458 | 4458 | echo Display::tag( |
| 4459 | 4459 | 'td', |
| 4460 | - "$user_answer / " . Display::tag( |
|
| 4460 | + "$user_answer / ".Display::tag( |
|
| 4461 | 4461 | 'strong', |
| 4462 | 4462 | $answerMatching[$answerCorrect], |
| 4463 | 4463 | ['style' => 'color: #008000; font-weight: bold;'] |
@@ -4609,7 +4609,7 @@ discard block |
||
| 4609 | 4609 | $questionId, |
| 4610 | 4610 | $nano, |
| 4611 | 4611 | $results_disabled |
| 4612 | - ) . '</td> |
|
| 4612 | + ).'</td> |
|
| 4613 | 4613 | </tr> |
| 4614 | 4614 | </table>'; |
| 4615 | 4615 | break; |
@@ -4670,7 +4670,7 @@ discard block |
||
| 4670 | 4670 | |
| 4671 | 4671 | //$overlap = round(polygons_overlap($poly_answer,$poly_user)); //this is an area in pixels |
| 4672 | 4672 | if ($debug > 0) { |
| 4673 | - error_log(__LINE__ . ' - Polygons results are ' . print_r($poly_results, 1), 0); |
|
| 4673 | + error_log(__LINE__.' - Polygons results are '.print_r($poly_results, 1), 0); |
|
| 4674 | 4674 | } |
| 4675 | 4675 | if ($overlap < 1) { |
| 4676 | 4676 | //shortcut to avoid complicated calculations |
@@ -4681,17 +4681,17 @@ discard block |
||
| 4681 | 4681 | // the final overlap is the percentage of the initial polygon that is overlapped by the user's polygon |
| 4682 | 4682 | $final_overlap = round(((float) $overlap / (float) $poly_answer_area) * 100); |
| 4683 | 4683 | if ($debug > 1) { |
| 4684 | - error_log(__LINE__ . ' - Final overlap is ' . $final_overlap, 0); |
|
| 4684 | + error_log(__LINE__.' - Final overlap is '.$final_overlap, 0); |
|
| 4685 | 4685 | } |
| 4686 | 4686 | // the final missing area is the percentage of the initial polygon that is not overlapped by the user's polygon |
| 4687 | 4687 | $final_missing = 100 - $final_overlap; |
| 4688 | 4688 | if ($debug > 1) { |
| 4689 | - error_log(__LINE__ . ' - Final missing is ' . $final_missing, 0); |
|
| 4689 | + error_log(__LINE__.' - Final missing is '.$final_missing, 0); |
|
| 4690 | 4690 | } |
| 4691 | 4691 | // the final excess area is the percentage of the initial polygon's size that is covered by the user's polygon outside of the initial polygon |
| 4692 | 4692 | $final_excess = round((((float) $poly_user_area - (float) $overlap) / (float) $poly_answer_area) * 100); |
| 4693 | 4693 | if ($debug > 1) { |
| 4694 | - error_log(__LINE__ . ' - Final excess is ' . $final_excess, 0); |
|
| 4694 | + error_log(__LINE__.' - Final excess is '.$final_excess, 0); |
|
| 4695 | 4695 | } |
| 4696 | 4696 | } |
| 4697 | 4697 | |
@@ -4701,7 +4701,7 @@ discard block |
||
| 4701 | 4701 | $threadhold_items = explode(';', $threadhold_total); |
| 4702 | 4702 | $threadhold1 = $threadhold_items[0]; // overlap |
| 4703 | 4703 | $threadhold2 = $threadhold_items[1]; // excess |
| 4704 | - $threadhold3 = $threadhold_items[2]; //missing |
|
| 4704 | + $threadhold3 = $threadhold_items[2]; //missing |
|
| 4705 | 4705 | // if is delineation |
| 4706 | 4706 | if ($answerId === 1) { |
| 4707 | 4707 | //setting colors |
@@ -4733,14 +4733,14 @@ discard block |
||
| 4733 | 4733 | } elseif ($answerId > 1) { |
| 4734 | 4734 | if ($objAnswerTmp->selectHotspotType($answerId) == 'noerror') { |
| 4735 | 4735 | if ($debug > 0) { |
| 4736 | - error_log(__LINE__ . ' - answerId is of type noerror', 0); |
|
| 4736 | + error_log(__LINE__.' - answerId is of type noerror', 0); |
|
| 4737 | 4737 | } |
| 4738 | 4738 | //type no error shouldn't be treated |
| 4739 | 4739 | $next = 1; |
| 4740 | 4740 | continue; |
| 4741 | 4741 | } |
| 4742 | 4742 | if ($debug > 0) { |
| 4743 | - error_log(__LINE__ . ' - answerId is >1 so we\'re probably in OAR', 0); |
|
| 4743 | + error_log(__LINE__.' - answerId is >1 so we\'re probably in OAR', 0); |
|
| 4744 | 4744 | } |
| 4745 | 4745 | //check the intersection between the oar and the user |
| 4746 | 4746 | //echo 'user'; print_r($x_user_list); print_r($y_user_list); |
@@ -4754,7 +4754,7 @@ discard block |
||
| 4754 | 4754 | $poly_answer = convert_coordinates($delineation_cord, '|'); |
| 4755 | 4755 | $max_coord = poly_get_max($poly_user, $poly_answer); |
| 4756 | 4756 | $poly_answer_compiled = poly_compile($poly_answer, $max_coord); |
| 4757 | - $overlap = poly_touch($poly_user_compiled, $poly_answer_compiled,$max_coord); |
|
| 4757 | + $overlap = poly_touch($poly_user_compiled, $poly_answer_compiled, $max_coord); |
|
| 4758 | 4758 | |
| 4759 | 4759 | if ($overlap == false) { |
| 4760 | 4760 | //all good, no overlap |
@@ -4762,7 +4762,7 @@ discard block |
||
| 4762 | 4762 | continue; |
| 4763 | 4763 | } else { |
| 4764 | 4764 | if ($debug > 0) { |
| 4765 | - error_log(__LINE__ . ' - Overlap is ' . $overlap . ': OAR hit', 0); |
|
| 4765 | + error_log(__LINE__.' - Overlap is '.$overlap.': OAR hit', 0); |
|
| 4766 | 4766 | } |
| 4767 | 4767 | $organs_at_risk_hit++; |
| 4768 | 4768 | //show the feedback |
@@ -4774,12 +4774,12 @@ discard block |
||
| 4774 | 4774 | $try_hotspot = $destination_items[1]; |
| 4775 | 4775 | $lp_hotspot = $destination_items[2]; |
| 4776 | 4776 | $select_question_hotspot = $destination_items[3]; |
| 4777 | - $url_hotspot=$destination_items[4]; |
|
| 4777 | + $url_hotspot = $destination_items[4]; |
|
| 4778 | 4778 | } |
| 4779 | 4779 | } |
| 4780 | 4780 | } else { // the first delineation feedback |
| 4781 | 4781 | if ($debug > 0) { |
| 4782 | - error_log(__LINE__ . ' first', 0); |
|
| 4782 | + error_log(__LINE__.' first', 0); |
|
| 4783 | 4783 | } |
| 4784 | 4784 | } |
| 4785 | 4785 | break; |
@@ -4801,7 +4801,7 @@ discard block |
||
| 4801 | 4801 | echo Display::tag('td', $answerMatching[$answerId]); |
| 4802 | 4802 | echo Display::tag( |
| 4803 | 4803 | 'td', |
| 4804 | - "$user_answer / " . Display::tag( |
|
| 4804 | + "$user_answer / ".Display::tag( |
|
| 4805 | 4805 | 'strong', |
| 4806 | 4806 | $answerMatching[$answerCorrect], |
| 4807 | 4807 | ['style' => 'color: #008000; font-weight: bold;'] |
@@ -4887,7 +4887,7 @@ discard block |
||
| 4887 | 4887 | // we use the results from the session (from_db=0) |
| 4888 | 4888 | // TODO Change this, because it is wrong to show the user |
| 4889 | 4889 | // some results that haven't been stored in the database yet |
| 4890 | - if ($answerType == HOT_SPOT || $answerType == HOT_SPOT_ORDER || $answerType == HOT_SPOT_DELINEATION ) { |
|
| 4890 | + if ($answerType == HOT_SPOT || $answerType == HOT_SPOT_ORDER || $answerType == HOT_SPOT_DELINEATION) { |
|
| 4891 | 4891 | |
| 4892 | 4892 | if ($debug) error_log('$from AND this is a hotspot kind of question '); |
| 4893 | 4893 | |
@@ -4896,19 +4896,19 @@ discard block |
||
| 4896 | 4896 | if ($answerType == HOT_SPOT_DELINEATION) { |
| 4897 | 4897 | if (0) { |
| 4898 | 4898 | if ($overlap_color) { |
| 4899 | - $overlap_color='green'; |
|
| 4899 | + $overlap_color = 'green'; |
|
| 4900 | 4900 | } else { |
| 4901 | - $overlap_color='red'; |
|
| 4901 | + $overlap_color = 'red'; |
|
| 4902 | 4902 | } |
| 4903 | 4903 | if ($missing_color) { |
| 4904 | - $missing_color='green'; |
|
| 4904 | + $missing_color = 'green'; |
|
| 4905 | 4905 | } else { |
| 4906 | - $missing_color='red'; |
|
| 4906 | + $missing_color = 'red'; |
|
| 4907 | 4907 | } |
| 4908 | 4908 | if ($excess_color) { |
| 4909 | - $excess_color='green'; |
|
| 4909 | + $excess_color = 'green'; |
|
| 4910 | 4910 | } else { |
| 4911 | - $excess_color='red'; |
|
| 4911 | + $excess_color = 'red'; |
|
| 4912 | 4912 | } |
| 4913 | 4913 | if (!is_numeric($final_overlap)) { |
| 4914 | 4914 | $final_overlap = 0; |
@@ -4920,33 +4920,33 @@ discard block |
||
| 4920 | 4920 | $final_excess = 0; |
| 4921 | 4921 | } |
| 4922 | 4922 | |
| 4923 | - if ($final_overlap>100) { |
|
| 4923 | + if ($final_overlap > 100) { |
|
| 4924 | 4924 | $final_overlap = 100; |
| 4925 | 4925 | } |
| 4926 | 4926 | |
| 4927 | - $table_resume='<table class="data_table"> |
|
| 4927 | + $table_resume = '<table class="data_table"> |
|
| 4928 | 4928 | <tr class="row_odd" > |
| 4929 | 4929 | <td></td> |
| 4930 | - <td ><b>' . get_lang('Requirements') . '</b></td> |
|
| 4931 | - <td><b>' . get_lang('YourAnswer') . '</b></td> |
|
| 4930 | + <td ><b>' . get_lang('Requirements').'</b></td> |
|
| 4931 | + <td><b>' . get_lang('YourAnswer').'</b></td> |
|
| 4932 | 4932 | </tr> |
| 4933 | 4933 | <tr class="row_even"> |
| 4934 | - <td><b>' . get_lang('Overlap') . '</b></td> |
|
| 4935 | - <td>' . get_lang('Min') . ' ' . $threadhold1 . '</td> |
|
| 4936 | - <td><div style="color:' . $overlap_color . '">' |
|
| 4937 | - . (($final_overlap < 0) ? 0 : intval($final_overlap)) . '</div></td> |
|
| 4934 | + <td><b>' . get_lang('Overlap').'</b></td> |
|
| 4935 | + <td>' . get_lang('Min').' '.$threadhold1.'</td> |
|
| 4936 | + <td><div style="color:' . $overlap_color.'">' |
|
| 4937 | + . (($final_overlap < 0) ? 0 : intval($final_overlap)).'</div></td> |
|
| 4938 | 4938 | </tr> |
| 4939 | 4939 | <tr> |
| 4940 | - <td><b>' . get_lang('Excess') . '</b></td> |
|
| 4941 | - <td>' . get_lang('Max') . ' ' . $threadhold2 . '</td> |
|
| 4942 | - <td><div style="color:' . $excess_color . '">' |
|
| 4943 | - . (($final_excess < 0) ? 0 : intval($final_excess)) . '</div></td> |
|
| 4940 | + <td><b>' . get_lang('Excess').'</b></td> |
|
| 4941 | + <td>' . get_lang('Max').' '.$threadhold2.'</td> |
|
| 4942 | + <td><div style="color:' . $excess_color.'">' |
|
| 4943 | + . (($final_excess < 0) ? 0 : intval($final_excess)).'</div></td> |
|
| 4944 | 4944 | </tr> |
| 4945 | 4945 | <tr class="row_even"> |
| 4946 | - <td><b>' . get_lang('Missing') . '</b></td> |
|
| 4947 | - <td>' . get_lang('Max') . ' ' . $threadhold3 . '</td> |
|
| 4948 | - <td><div style="color:' . $missing_color . '">' |
|
| 4949 | - . (($final_missing < 0) ? 0 : intval($final_missing)) . '</div></td> |
|
| 4946 | + <td><b>' . get_lang('Missing').'</b></td> |
|
| 4947 | + <td>' . get_lang('Max').' '.$threadhold3.'</td> |
|
| 4948 | + <td><div style="color:' . $missing_color.'">' |
|
| 4949 | + . (($final_missing < 0) ? 0 : intval($final_missing)).'</div></td> |
|
| 4950 | 4950 | </tr> |
| 4951 | 4951 | </table>'; |
| 4952 | 4952 | if ($next == 0) { |
@@ -4961,20 +4961,20 @@ discard block |
||
| 4961 | 4961 | $answerDestination = $objAnswerTmp->selectDestination($nbrAnswers); |
| 4962 | 4962 | } |
| 4963 | 4963 | |
| 4964 | - echo '<h1><div style="color:#333;">' . get_lang('Feedback') . '</div></h1> |
|
| 4964 | + echo '<h1><div style="color:#333;">'.get_lang('Feedback').'</div></h1> |
|
| 4965 | 4965 | <p style="text-align:center">'; |
| 4966 | 4966 | |
| 4967 | - $message = '<p>' . get_lang('YourDelineation') . '</p>'; |
|
| 4967 | + $message = '<p>'.get_lang('YourDelineation').'</p>'; |
|
| 4968 | 4968 | $message .= $table_resume; |
| 4969 | - $message .= '<br />' . get_lang('ResultIs') . ' ' . $result_comment . '<br />'; |
|
| 4969 | + $message .= '<br />'.get_lang('ResultIs').' '.$result_comment.'<br />'; |
|
| 4970 | 4970 | if ($organs_at_risk_hit > 0) { |
| 4971 | - $message .= '<p><b>' . get_lang('OARHit') . '</b></p>'; |
|
| 4971 | + $message .= '<p><b>'.get_lang('OARHit').'</b></p>'; |
|
| 4972 | 4972 | } |
| 4973 | - $message .='<p>' . $comment . '</p>'; |
|
| 4973 | + $message .= '<p>'.$comment.'</p>'; |
|
| 4974 | 4974 | echo $message; |
| 4975 | 4975 | } else { |
| 4976 | 4976 | echo $hotspot_delineation_result[0]; //prints message |
| 4977 | - $from_database = 1; // the hotspot_solution.swf needs this variable |
|
| 4977 | + $from_database = 1; // the hotspot_solution.swf needs this variable |
|
| 4978 | 4978 | } |
| 4979 | 4979 | |
| 4980 | 4980 | //save the score attempts |
@@ -4996,20 +4996,20 @@ discard block |
||
| 4996 | 4996 | $exerciseResultCoordinates[$quesId] |
| 4997 | 4997 | ); |
| 4998 | 4998 | } else { |
| 4999 | - if ($final_answer==0) { |
|
| 4999 | + if ($final_answer == 0) { |
|
| 5000 | 5000 | $questionScore = 0; |
| 5001 | - $answer=0; |
|
| 5001 | + $answer = 0; |
|
| 5002 | 5002 | Event::saveQuestionAttempt($questionScore, $answer, $quesId, $exeId, 0); |
| 5003 | 5003 | if (is_array($exerciseResultCoordinates[$quesId])) { |
| 5004 | - foreach($exerciseResultCoordinates[$quesId] as $idx => $val) { |
|
| 5005 | - Event::saveExerciseAttemptHotspot($exeId,$quesId,$idx,0,$val); |
|
| 5004 | + foreach ($exerciseResultCoordinates[$quesId] as $idx => $val) { |
|
| 5005 | + Event::saveExerciseAttemptHotspot($exeId, $quesId, $idx, 0, $val); |
|
| 5006 | 5006 | } |
| 5007 | 5007 | } |
| 5008 | 5008 | } else { |
| 5009 | 5009 | Event::saveQuestionAttempt($questionScore, $answer, $quesId, $exeId, 0); |
| 5010 | 5010 | if (is_array($exerciseResultCoordinates[$quesId])) { |
| 5011 | - foreach($exerciseResultCoordinates[$quesId] as $idx => $val) { |
|
| 5012 | - Event::saveExerciseAttemptHotspot($exeId,$quesId,$idx,$choice[$idx],$val); |
|
| 5011 | + foreach ($exerciseResultCoordinates[$quesId] as $idx => $val) { |
|
| 5012 | + Event::saveExerciseAttemptHotspot($exeId, $quesId, $idx, $choice[$idx], $val); |
|
| 5013 | 5013 | } |
| 5014 | 5014 | } |
| 5015 | 5015 | } |
@@ -5028,7 +5028,7 @@ discard block |
||
| 5028 | 5028 | echo " |
| 5029 | 5029 | <tr> |
| 5030 | 5030 | <td colspan=\"2\"> |
| 5031 | - <p><em>" . get_lang('HotSpot') . "</em></p> |
|
| 5031 | + <p><em>" . get_lang('HotSpot')."</em></p> |
|
| 5032 | 5032 | |
| 5033 | 5033 | <div id=\"hotspot-solution-$questionId\"></div> |
| 5034 | 5034 | |
@@ -5066,7 +5066,7 @@ discard block |
||
| 5066 | 5066 | |
| 5067 | 5067 | if ($saved_results) { |
| 5068 | 5068 | if ($debug) error_log("Save question results $saved_results"); |
| 5069 | - if ($debug) error_log(print_r($choice ,1 )); |
|
| 5069 | + if ($debug) error_log(print_r($choice, 1)); |
|
| 5070 | 5070 | |
| 5071 | 5071 | if (empty($choice)) { |
| 5072 | 5072 | $choice = 0; |
@@ -5078,14 +5078,14 @@ discard block |
||
| 5078 | 5078 | $ans = $reply[$i]; |
| 5079 | 5079 | Event::saveQuestionAttempt( |
| 5080 | 5080 | $questionScore, |
| 5081 | - $ans . ':' . $choice[$ans], |
|
| 5081 | + $ans.':'.$choice[$ans], |
|
| 5082 | 5082 | $quesId, |
| 5083 | 5083 | $exeId, |
| 5084 | 5084 | $i, |
| 5085 | 5085 | $this->id |
| 5086 | 5086 | ); |
| 5087 | 5087 | if ($debug) { |
| 5088 | - error_log('result =>' . $questionScore . ' ' . $ans . ':' . $choice[$ans]); |
|
| 5088 | + error_log('result =>'.$questionScore.' '.$ans.':'.$choice[$ans]); |
|
| 5089 | 5089 | } |
| 5090 | 5090 | } |
| 5091 | 5091 | } else { |
@@ -5096,7 +5096,7 @@ discard block |
||
| 5096 | 5096 | $reply = array_keys($choice); |
| 5097 | 5097 | |
| 5098 | 5098 | if ($debug) { |
| 5099 | - error_log("reply " . print_r($reply, 1) . ""); |
|
| 5099 | + error_log("reply ".print_r($reply, 1).""); |
|
| 5100 | 5100 | } |
| 5101 | 5101 | for ($i = 0; $i < sizeof($reply); $i++) { |
| 5102 | 5102 | $ans = $reply[$i]; |
@@ -5155,7 +5155,7 @@ discard block |
||
| 5155 | 5155 | |
| 5156 | 5156 | Event::saveQuestionAttempt($questionScore, implode('|', $answer), $quesId, $exeId, 0, $this->id); |
| 5157 | 5157 | } else { |
| 5158 | - Event::saveQuestionAttempt($questionScore, $answer, $quesId, $exeId, 0,$this->id); |
|
| 5158 | + Event::saveQuestionAttempt($questionScore, $answer, $quesId, $exeId, 0, $this->id); |
|
| 5159 | 5159 | } |
| 5160 | 5160 | } |
| 5161 | 5161 | |
@@ -5165,8 +5165,8 @@ discard block |
||
| 5165 | 5165 | |
| 5166 | 5166 | if ($saved_results) { |
| 5167 | 5167 | $stat_table = Database :: get_main_table(TABLE_STATISTIC_TRACK_E_EXERCISES); |
| 5168 | - $sql = 'UPDATE ' . $stat_table . ' SET |
|
| 5169 | - exe_result = exe_result + ' . floatval($questionScore) . ' |
|
| 5168 | + $sql = 'UPDATE '.$stat_table.' SET |
|
| 5169 | + exe_result = exe_result + ' . floatval($questionScore).' |
|
| 5170 | 5170 | WHERE exe_id = ' . $exeId; |
| 5171 | 5171 | if ($debug) error_log($sql); |
| 5172 | 5172 | Database::query($sql); |
@@ -5190,7 +5190,7 @@ discard block |
||
| 5190 | 5190 | */ |
| 5191 | 5191 | public function send_mail_notification_for_exam($question_list_answers, $origin, $exe_id) |
| 5192 | 5192 | { |
| 5193 | - if (api_get_course_setting('email_alert_manager_on_new_quiz') != 1 ) { |
|
| 5193 | + if (api_get_course_setting('email_alert_manager_on_new_quiz') != 1) { |
|
| 5194 | 5194 | return null; |
| 5195 | 5195 | } |
| 5196 | 5196 | // Email configuration settings |
@@ -5241,7 +5241,7 @@ discard block |
||
| 5241 | 5241 | $msg = str_replace("#course#", $courseInfo['name'], $msg1); |
| 5242 | 5242 | |
| 5243 | 5243 | if ($origin != 'learnpath') { |
| 5244 | - $msg.= '<br /><a href="#url#">'.get_lang('ClickToCommentAndGiveFeedback').'</a>'; |
|
| 5244 | + $msg .= '<br /><a href="#url#">'.get_lang('ClickToCommentAndGiveFeedback').'</a>'; |
|
| 5245 | 5245 | } |
| 5246 | 5246 | $msg1 = str_replace("#url#", $url_email, $msg); |
| 5247 | 5247 | $mail_content = $msg1; |
@@ -5270,7 +5270,7 @@ discard block |
||
| 5270 | 5270 | */ |
| 5271 | 5271 | function send_notification_for_open_questions($question_list_answers, $origin, $exe_id) |
| 5272 | 5272 | { |
| 5273 | - if (api_get_course_setting('email_alert_manager_on_new_quiz') != 1 ) { |
|
| 5273 | + if (api_get_course_setting('email_alert_manager_on_new_quiz') != 1) { |
|
| 5274 | 5274 | return null; |
| 5275 | 5275 | } |
| 5276 | 5276 | // Email configuration settings |
@@ -5333,11 +5333,11 @@ discard block |
||
| 5333 | 5333 | $msg .= '</table><br />'; |
| 5334 | 5334 | |
| 5335 | 5335 | |
| 5336 | - $msg1 = str_replace("#exercise#", $this->exercise, $msg); |
|
| 5337 | - $msg = str_replace("#firstName#", $user_info['firstname'],$msg1); |
|
| 5338 | - $msg1 = str_replace("#lastName#", $user_info['lastname'],$msg); |
|
| 5339 | - $msg = str_replace("#mail#", $user_info['email'],$msg1); |
|
| 5340 | - $msg = str_replace("#course#", $course_info['name'],$msg1); |
|
| 5336 | + $msg1 = str_replace("#exercise#", $this->exercise, $msg); |
|
| 5337 | + $msg = str_replace("#firstName#", $user_info['firstname'], $msg1); |
|
| 5338 | + $msg1 = str_replace("#lastName#", $user_info['lastname'], $msg); |
|
| 5339 | + $msg = str_replace("#mail#", $user_info['email'], $msg1); |
|
| 5340 | + $msg = str_replace("#course#", $course_info['name'], $msg1); |
|
| 5341 | 5341 | |
| 5342 | 5342 | if ($origin != 'learnpath') { |
| 5343 | 5343 | $msg .= '<br /><a href="#url#">'.get_lang('ClickToCommentAndGiveFeedback').'</a>'; |
@@ -5366,7 +5366,7 @@ discard block |
||
| 5366 | 5366 | |
| 5367 | 5367 | function send_notification_for_oral_questions($question_list_answers, $origin, $exe_id) |
| 5368 | 5368 | { |
| 5369 | - if (api_get_course_setting('email_alert_manager_on_new_quiz') != 1 ) { |
|
| 5369 | + if (api_get_course_setting('email_alert_manager_on_new_quiz') != 1) { |
|
| 5370 | 5370 | return null; |
| 5371 | 5371 | } |
| 5372 | 5372 | // Email configuration settings |
@@ -5390,7 +5390,7 @@ discard block |
||
| 5390 | 5390 | $answer_type = $item['answer_type']; |
| 5391 | 5391 | |
| 5392 | 5392 | if (!empty($question) && !empty($answer) && $answer_type == ORAL_EXPRESSION) { |
| 5393 | - $oral_question_list.='<br /><table width="730" height="136" border="0" cellpadding="3" cellspacing="3">' |
|
| 5393 | + $oral_question_list .= '<br /><table width="730" height="136" border="0" cellpadding="3" cellspacing="3">' |
|
| 5394 | 5394 | .'<tr>' |
| 5395 | 5395 | .'<td width="220" valign="top" bgcolor="#E5EDF8"> '.get_lang('Question').'</td>' |
| 5396 | 5396 | .'<td width="473" valign="top" bgcolor="#F3F3F3">'.$question.'</td>' |
@@ -5423,7 +5423,7 @@ discard block |
||
| 5423 | 5423 | .'<td> #mail#</td>' |
| 5424 | 5424 | .'</tr>' |
| 5425 | 5425 | .'</table>'; |
| 5426 | - $msg .= '<br />'.sprintf(get_lang('OralQuestionsAttemptedAreX'),$oral_question_list).'<br />'; |
|
| 5426 | + $msg .= '<br />'.sprintf(get_lang('OralQuestionsAttemptedAreX'), $oral_question_list).'<br />'; |
|
| 5427 | 5427 | $msg1 = str_replace("#exercise#", $this->exercise, $msg); |
| 5428 | 5428 | $msg = str_replace("#firstName#", $user_info['firstname'], $msg1); |
| 5429 | 5429 | $msg1 = str_replace("#lastName#", $user_info['lastname'], $msg); |
@@ -5431,7 +5431,7 @@ discard block |
||
| 5431 | 5431 | $msg = str_replace("#course#", $course_info['name'], $msg1); |
| 5432 | 5432 | |
| 5433 | 5433 | if ($origin != 'learnpath') { |
| 5434 | - $msg.= '<br /><a href="#url#">'.get_lang('ClickToCommentAndGiveFeedback').'</a>'; |
|
| 5434 | + $msg .= '<br /><a href="#url#">'.get_lang('ClickToCommentAndGiveFeedback').'</a>'; |
|
| 5435 | 5435 | } |
| 5436 | 5436 | $msg1 = str_replace("#url#", $url_email, $msg); |
| 5437 | 5437 | $mail_content = $msg1; |
@@ -5497,10 +5497,10 @@ discard block |
||
| 5497 | 5497 | } |
| 5498 | 5498 | $html = '<div class="question-result">'; |
| 5499 | 5499 | $html .= Display::page_header( |
| 5500 | - Display::return_icon('test-quiz.png', get_lang('Result'),null, ICON_SIZE_MEDIUM).' '.$this->exercise.' : '.get_lang('Result') |
|
| 5500 | + Display::return_icon('test-quiz.png', get_lang('Result'), null, ICON_SIZE_MEDIUM).' '.$this->exercise.' : '.get_lang('Result') |
|
| 5501 | 5501 | ); |
| 5502 | 5502 | $html .= Display::description($array); |
| 5503 | - $html .="</div>"; |
|
| 5503 | + $html .= "</div>"; |
|
| 5504 | 5504 | return $html; |
| 5505 | 5505 | } |
| 5506 | 5506 | |
@@ -5617,7 +5617,7 @@ discard block |
||
| 5617 | 5617 | false, |
| 5618 | 5618 | $objExercise->selectPropagateNeg() |
| 5619 | 5619 | ); |
| 5620 | - $totalScore += $question_result['score']; |
|
| 5620 | + $totalScore += $question_result['score']; |
|
| 5621 | 5621 | } |
| 5622 | 5622 | |
| 5623 | 5623 | if ($objExercise->selectPropagateNeg() == 0 && $totalScore < 0) { |
@@ -5823,7 +5823,7 @@ discard block |
||
| 5823 | 5823 | } |
| 5824 | 5824 | |
| 5825 | 5825 | $rawMessage = ""; |
| 5826 | - if (!empty($message)){ |
|
| 5826 | + if (!empty($message)) { |
|
| 5827 | 5827 | $rawMessage = $message; |
| 5828 | 5828 | $message = Display::return_message($message, 'warning', false); |
| 5829 | 5829 | } |
@@ -5839,7 +5839,7 @@ discard block |
||
| 5839 | 5839 | { |
| 5840 | 5840 | $TBL_LP_ITEM = Database::get_course_table(TABLE_LP_ITEM); |
| 5841 | 5841 | $sql = "SELECT max_score FROM $TBL_LP_ITEM |
| 5842 | - WHERE c_id = {$this->course_id} AND item_type = '" . TOOL_QUIZ . "' AND path = '{$this->id}'"; |
|
| 5842 | + WHERE c_id = {$this->course_id} AND item_type = '".TOOL_QUIZ."' AND path = '{$this->id}'"; |
|
| 5843 | 5843 | $result = Database::query($sql); |
| 5844 | 5844 | if (Database::num_rows($result) > 0) { |
| 5845 | 5845 | return true; |
@@ -6167,7 +6167,7 @@ discard block |
||
| 6167 | 6167 | $sql_track = "SELECT * FROM $track_exercises WHERE exe_id = $exe_id "; |
| 6168 | 6168 | $result = Database::query($sql_track); |
| 6169 | 6169 | $new_array = array(); |
| 6170 | - if (Database::num_rows($result) > 0 ) { |
|
| 6170 | + if (Database::num_rows($result) > 0) { |
|
| 6171 | 6171 | $new_array = Database::fetch_array($result, 'ASSOC'); |
| 6172 | 6172 | |
| 6173 | 6173 | $new_array['duration'] = null; |
@@ -6179,11 +6179,11 @@ discard block |
||
| 6179 | 6179 | $start_date = api_strtotime($start_date, 'UTC'); |
| 6180 | 6180 | $end_date = api_strtotime($end_date, 'UTC'); |
| 6181 | 6181 | if ($start_date && $end_date) { |
| 6182 | - $mytime = $end_date- $start_date; |
|
| 6182 | + $mytime = $end_date - $start_date; |
|
| 6183 | 6183 | $new_learnpath_item = new learnpathItem(null); |
| 6184 | 6184 | $time_attemp = $new_learnpath_item->get_scorm_time('js', $mytime); |
| 6185 | 6185 | $h = get_lang('h'); |
| 6186 | - $time_attemp = str_replace('NaN', '00' . $h . '00\'00"', $time_attemp); |
|
| 6186 | + $time_attemp = str_replace('NaN', '00'.$h.'00\'00"', $time_attemp); |
|
| 6187 | 6187 | $new_array['duration'] = $time_attemp; |
| 6188 | 6188 | } |
| 6189 | 6189 | } |
@@ -6205,7 +6205,7 @@ discard block |
||
| 6205 | 6205 | $result = Database::query($sql); |
| 6206 | 6206 | } |
| 6207 | 6207 | } else { |
| 6208 | - $remind_list = explode(',',$exercise_info['questions_to_check']); |
|
| 6208 | + $remind_list = explode(',', $exercise_info['questions_to_check']); |
|
| 6209 | 6209 | |
| 6210 | 6210 | $remind_list_string = ''; |
| 6211 | 6211 | if ($action == 'add') { |
@@ -6217,7 +6217,7 @@ discard block |
||
| 6217 | 6217 | } |
| 6218 | 6218 | $remind_list_string = implode(',', $remind_list); |
| 6219 | 6219 | } |
| 6220 | - } elseif ($action == 'delete') { |
|
| 6220 | + } elseif ($action == 'delete') { |
|
| 6221 | 6221 | if (!empty($remind_list)) { |
| 6222 | 6222 | if (in_array($question_id, $remind_list)) { |
| 6223 | 6223 | $remind_list = array_flip($remind_list); |
@@ -6459,7 +6459,7 @@ discard block |
||
| 6459 | 6459 | true |
| 6460 | 6460 | ); |
| 6461 | 6461 | |
| 6462 | - $counter += count($mediaQuestions[$questionId]) - 1 ; |
|
| 6462 | + $counter += count($mediaQuestions[$questionId]) - 1; |
|
| 6463 | 6463 | $before = count($questionList); |
| 6464 | 6464 | $wasMedia = true; |
| 6465 | 6465 | $nextValue += count($questionList); |
@@ -6538,7 +6538,7 @@ discard block |
||
| 6538 | 6538 | // If it was already seen, then merge the previous with |
| 6539 | 6539 | // the current category |
| 6540 | 6540 | $oldQuestionList = $newCategoryList[$rootElement]['question_list']; |
| 6541 | - $category['question_list'] = array_merge($oldQuestionList , $category['question_list']); |
|
| 6541 | + $category['question_list'] = array_merge($oldQuestionList, $category['question_list']); |
|
| 6542 | 6542 | $newCategoryList[$rootElement] = $category; |
| 6543 | 6543 | } |
| 6544 | 6544 | } |
@@ -6564,7 +6564,7 @@ discard block |
||
| 6564 | 6564 | |
| 6565 | 6565 | if ($useRootAsCategoryTitle) { |
| 6566 | 6566 | if (isset($category['parent_info'])) { |
| 6567 | - $categoryName = $category['parent_info']['title']; |
|
| 6567 | + $categoryName = $category['parent_info']['title']; |
|
| 6568 | 6568 | } |
| 6569 | 6569 | } |
| 6570 | 6570 | $html .= '<div class="row">'; |
@@ -6706,7 +6706,7 @@ discard block |
||
| 6706 | 6706 | // end foreach() |
| 6707 | 6707 | |
| 6708 | 6708 | if ($this->type == ALL_ON_ONE_PAGE) { |
| 6709 | - $exercise_actions = $this->show_button($questionId, $currentQuestion); |
|
| 6709 | + $exercise_actions = $this->show_button($questionId, $currentQuestion); |
|
| 6710 | 6710 | echo Display::div($exercise_actions, array('class'=>'exercise_actions')); |
| 6711 | 6711 | } |
| 6712 | 6712 | } |
@@ -6792,7 +6792,7 @@ discard block |
||
| 6792 | 6792 | |
| 6793 | 6793 | // Showing the question |
| 6794 | 6794 | |
| 6795 | - $exercise_actions = null; |
|
| 6795 | + $exercise_actions = null; |
|
| 6796 | 6796 | |
| 6797 | 6797 | echo '<a id="questionanchor'.$questionId.'"></a><br />'; |
| 6798 | 6798 | echo '<div id="question_div_'.$questionId.'" class="main_question '.$remind_highlight.'" >'; |
@@ -7060,7 +7060,7 @@ discard block |
||
| 7060 | 7060 | $header .= Display::tag('th', get_lang('Feedback')); |
| 7061 | 7061 | } |
| 7062 | 7062 | $s .= '<table class="data_table">'; |
| 7063 | - $s.= Display::tag('tr', $header, array('style' => 'text-align:left;')); |
|
| 7063 | + $s .= Display::tag('tr', $header, array('style' => 'text-align:left;')); |
|
| 7064 | 7064 | } |
| 7065 | 7065 | } |
| 7066 | 7066 | |
@@ -7160,7 +7160,7 @@ discard block |
||
| 7160 | 7160 | $s .= '<td>'; |
| 7161 | 7161 | $s .= $comment; |
| 7162 | 7162 | $s .= '</td>'; |
| 7163 | - $s .='</tr>'; |
|
| 7163 | + $s .= '</tr>'; |
|
| 7164 | 7164 | } else { |
| 7165 | 7165 | $s .= $answer_input; |
| 7166 | 7166 | } |
@@ -7174,7 +7174,7 @@ discard block |
||
| 7174 | 7174 | } |
| 7175 | 7175 | } |
| 7176 | 7176 | |
| 7177 | - $s .='<tr>'; |
|
| 7177 | + $s .= '<tr>'; |
|
| 7178 | 7178 | $s .= Display::tag('td', $answer); |
| 7179 | 7179 | |
| 7180 | 7180 | if (!empty($quiz_question_options)) { |
@@ -7201,7 +7201,7 @@ discard block |
||
| 7201 | 7201 | $s .= $comment; |
| 7202 | 7202 | $s .= '</td>'; |
| 7203 | 7203 | } |
| 7204 | - $s.='</tr>'; |
|
| 7204 | + $s .= '</tr>'; |
|
| 7205 | 7205 | } |
| 7206 | 7206 | |
| 7207 | 7207 | } elseif ($answerType == MULTIPLE_ANSWER_COMBINATION) { |
@@ -7230,16 +7230,16 @@ discard block |
||
| 7230 | 7230 | $answer_input .= '</label>'; |
| 7231 | 7231 | |
| 7232 | 7232 | if ($show_comment) { |
| 7233 | - $s.= '<tr>'; |
|
| 7233 | + $s .= '<tr>'; |
|
| 7234 | 7234 | $s .= '<td>'; |
| 7235 | - $s.= $answer_input; |
|
| 7235 | + $s .= $answer_input; |
|
| 7236 | 7236 | $s .= '</td>'; |
| 7237 | 7237 | $s .= '<td>'; |
| 7238 | 7238 | $s .= $comment; |
| 7239 | 7239 | $s .= '</td>'; |
| 7240 | - $s.= '</tr>'; |
|
| 7240 | + $s .= '</tr>'; |
|
| 7241 | 7241 | } else { |
| 7242 | - $s.= $answer_input; |
|
| 7242 | + $s .= $answer_input; |
|
| 7243 | 7243 | } |
| 7244 | 7244 | } elseif ($answerType == MULTIPLE_ANSWER_COMBINATION_TRUE_FALSE) { |
| 7245 | 7245 | $s .= '<input type="hidden" name="choice2['.$questionId.']" value="0" />'; |
@@ -7252,7 +7252,7 @@ discard block |
||
| 7252 | 7252 | } |
| 7253 | 7253 | } |
| 7254 | 7254 | $answer = Security::remove_XSS($answer); |
| 7255 | - $s .='<tr>'; |
|
| 7255 | + $s .= '<tr>'; |
|
| 7256 | 7256 | $s .= Display::tag('td', $answer); |
| 7257 | 7257 | |
| 7258 | 7258 | foreach ($objQuestionTmp->options as $key => $item) { |
@@ -7276,7 +7276,7 @@ discard block |
||
| 7276 | 7276 | $s .= $comment; |
| 7277 | 7277 | $s .= '</td>'; |
| 7278 | 7278 | } |
| 7279 | - $s.='</tr>'; |
|
| 7279 | + $s .= '</tr>'; |
|
| 7280 | 7280 | } elseif ($answerType == FILL_IN_BLANKS) { |
| 7281 | 7281 | list($answer) = explode('::', $answer); |
| 7282 | 7282 | |
@@ -7310,7 +7310,7 @@ discard block |
||
| 7310 | 7310 | $value = str_replace(' ', '', trim($value[0])); |
| 7311 | 7311 | } |
| 7312 | 7312 | $correct_item = preg_quote($correct_item); |
| 7313 | - $correct_item = api_preg_replace('|/|', '\/', $correct_item); // to prevent error if there is a / in the text to find |
|
| 7313 | + $correct_item = api_preg_replace('|/|', '\/', $correct_item); // to prevent error if there is a / in the text to find |
|
| 7314 | 7314 | $answer = api_preg_replace('/'.$correct_item.'/', Display::input('text', "choice[$questionId][]", $value), $answer, 1); |
| 7315 | 7315 | } |
| 7316 | 7316 | $i++; |
@@ -7367,7 +7367,7 @@ discard block |
||
| 7367 | 7367 | } |
| 7368 | 7368 | |
| 7369 | 7369 | if (!empty($answerCorrect) && !empty($selectedValue)) { |
| 7370 | - $s.= '<script> |
|
| 7370 | + $s .= '<script> |
|
| 7371 | 7371 | jsPlumb.ready(function() { |
| 7372 | 7372 | jsPlumb.connect({ |
| 7373 | 7373 | source: "window_'.$windowId.'", |
@@ -7382,14 +7382,14 @@ discard block |
||
| 7382 | 7382 | } |
| 7383 | 7383 | $s .= '</select></div></td>'; |
| 7384 | 7384 | |
| 7385 | - $s.='<td width="45%" valign="top" >'; |
|
| 7385 | + $s .= '<td width="45%" valign="top" >'; |
|
| 7386 | 7386 | |
| 7387 | 7387 | if (isset($select_items[$lines_count])) { |
| 7388 | - $s.= '<div id="window_'.$windowId.'_answer" class="window window_right_question"> |
|
| 7388 | + $s .= '<div id="window_'.$windowId.'_answer" class="window window_right_question"> |
|
| 7389 | 7389 | <b>'.$select_items[$lines_count]['letter'].'.</b> '.$select_items[$lines_count]['answer'].' |
| 7390 | 7390 | </div>'; |
| 7391 | 7391 | } else { |
| 7392 | - $s.=' '; |
|
| 7392 | + $s .= ' '; |
|
| 7393 | 7393 | } |
| 7394 | 7394 | |
| 7395 | 7395 | $s .= '</td>'; |
@@ -7403,16 +7403,16 @@ discard block |
||
| 7403 | 7403 | $s .= '<tr> |
| 7404 | 7404 | <td colspan="2"></td> |
| 7405 | 7405 | <td valign="top">'; |
| 7406 | - $s.='<b>'.$select_items[$lines_count]['letter'].'.</b>'; |
|
| 7406 | + $s .= '<b>'.$select_items[$lines_count]['letter'].'.</b>'; |
|
| 7407 | 7407 | $s .= $select_items[$lines_count]['answer']; |
| 7408 | - $s.="</td> |
|
| 7408 | + $s .= "</td> |
|
| 7409 | 7409 | </tr>"; |
| 7410 | 7410 | $lines_count++; |
| 7411 | 7411 | } // end while() |
| 7412 | 7412 | } // end if() |
| 7413 | 7413 | $matching_correct_answer++; |
| 7414 | 7414 | } |
| 7415 | - } elseif ($answerType == DRAGGABLE) { |
|
| 7415 | + } elseif ($answerType == DRAGGABLE) { |
|
| 7416 | 7416 | // matching type, showing suggestions and answers |
| 7417 | 7417 | // TODO: replace $answerId by $numAnswer |
| 7418 | 7418 | |
@@ -7454,7 +7454,7 @@ discard block |
||
| 7454 | 7454 | $s .= '</select>'; |
| 7455 | 7455 | |
| 7456 | 7456 | if (!empty($answerCorrect) && !empty($selectedValue)) { |
| 7457 | - $s.= '<script> |
|
| 7457 | + $s .= '<script> |
|
| 7458 | 7458 | $(function() { |
| 7459 | 7459 | deleteItem($("#'.$questionId.'_'.$selectedValue.'"), $("#drop_'.$windowId.'")); |
| 7460 | 7460 | }); |
@@ -7462,11 +7462,11 @@ discard block |
||
| 7462 | 7462 | } |
| 7463 | 7463 | |
| 7464 | 7464 | if (isset($select_items[$lines_count])) { |
| 7465 | - $s.= '<div id="window_'.$windowId.'_answer" class=""> |
|
| 7465 | + $s .= '<div id="window_'.$windowId.'_answer" class=""> |
|
| 7466 | 7466 | <b>'.$select_items[$lines_count]['letter'].'.</b> '.$select_items[$lines_count]['answer'].' |
| 7467 | 7467 | </div>'; |
| 7468 | 7468 | } else { |
| 7469 | - $s.=' '; |
|
| 7469 | + $s .= ' '; |
|
| 7470 | 7470 | } |
| 7471 | 7471 | $lines_count++; |
| 7472 | 7472 | //if the left side of the "matching" has been completely |
@@ -7475,7 +7475,7 @@ discard block |
||
| 7475 | 7475 | if (($lines_count - 1) == $num_suggestions) { |
| 7476 | 7476 | // if it remains answers to shown at the right side |
| 7477 | 7477 | while (isset($select_items[$lines_count])) { |
| 7478 | - $s.='<b>'.$select_items[$lines_count]['letter'].'.</b>'; |
|
| 7478 | + $s .= '<b>'.$select_items[$lines_count]['letter'].'.</b>'; |
|
| 7479 | 7479 | $s .= $select_items[$lines_count]['answer']; |
| 7480 | 7480 | $lines_count++; |
| 7481 | 7481 | } |
@@ -7578,9 +7578,9 @@ discard block |
||
| 7578 | 7578 | |
| 7579 | 7579 | if (!$only_questions) { |
| 7580 | 7580 | if ($show_title) { |
| 7581 | - $html .= TestCategory::getCategoryNamesForQuestion($objQuestionTmp->id); |
|
| 7582 | - $html .= '<div class="question_title">'.$current_item.'. '.$questionName.'</div>'; |
|
| 7583 | - $html .= $questionDescription; |
|
| 7581 | + $html .= TestCategory::getCategoryNamesForQuestion($objQuestionTmp->id); |
|
| 7582 | + $html .= '<div class="question_title">'.$current_item.'. '.$questionName.'</div>'; |
|
| 7583 | + $html .= $questionDescription; |
|
| 7584 | 7584 | } else { |
| 7585 | 7585 | $html .= '<div class="media">'; |
| 7586 | 7586 | $html .= '<div class="pull-left">'; |
@@ -7596,11 +7596,11 @@ discard block |
||
| 7596 | 7596 | $html .= '</div>'; |
| 7597 | 7597 | } |
| 7598 | 7598 | //@todo I need to the get the feedback type |
| 7599 | - $html .= '<input type="hidden" name="hidden_hotspot_id" value="'.$questionId.'" />'; |
|
| 7600 | - $html .= '<table class="exercise_questions"> |
|
| 7599 | + $html .= '<input type="hidden" name="hidden_hotspot_id" value="'.$questionId.'" />'; |
|
| 7600 | + $html .= '<table class="exercise_questions"> |
|
| 7601 | 7601 | <tr> |
| 7602 | 7602 | <td valign="top" colspan="2">'; |
| 7603 | - $html .= '</td></tr>'; |
|
| 7603 | + $html .= '</td></tr>'; |
|
| 7604 | 7604 | } |
| 7605 | 7605 | |
| 7606 | 7606 | $canClick = isset($_GET['editQuestion']) ? '0' : (isset($_GET['modifyAnswers']) ? '0' : '1'); |
