@@ -34,7 +34,7 @@ discard block |
||
| 34 | 34 | |
| 35 | 35 | // General parameters passed via POST/GET |
| 36 | 36 | if ($debug) { |
| 37 | - error_log('Entered exercise_result.php: ' . print_r($_POST, 1)); |
|
| 37 | + error_log('Entered exercise_result.php: '.print_r($_POST, 1)); |
|
| 38 | 38 | } |
| 39 | 39 | |
| 40 | 40 | if (empty($formSent)) { |
@@ -131,19 +131,19 @@ discard block |
||
| 131 | 131 | } |
| 132 | 132 | |
| 133 | 133 | if (!empty($gradebook) && $gradebook == 'view') { |
| 134 | - $interbreadcrumb[] = array('url' => '../gradebook/' . $_SESSION['gradebook_dest'], 'name' => get_lang('ToolGradebook')); |
|
| 134 | + $interbreadcrumb[] = array('url' => '../gradebook/'.$_SESSION['gradebook_dest'], 'name' => get_lang('ToolGradebook')); |
|
| 135 | 135 | } |
| 136 | 136 | |
| 137 | 137 | $fromlink = ''; |
| 138 | 138 | |
| 139 | -$interbreadcrumb[] = array("url" => "exercise.php?" . api_get_cidreq(), "name" => get_lang('Exercises')); |
|
| 140 | -$interbreadcrumb[] = array("url" => "overview.php?exerciseId=" . $exercise_id . '&' . api_get_cidreq(), "name" => $objExercise->name); |
|
| 139 | +$interbreadcrumb[] = array("url" => "exercise.php?".api_get_cidreq(), "name" => get_lang('Exercises')); |
|
| 140 | +$interbreadcrumb[] = array("url" => "overview.php?exerciseId=".$exercise_id.'&'.api_get_cidreq(), "name" => $objExercise->name); |
|
| 141 | 141 | $interbreadcrumb[] = array("url" => "#", "name" => get_lang('Result')); |
| 142 | 142 | |
| 143 | 143 | $this_section = SECTION_COURSES; |
| 144 | 144 | |
| 145 | -$htmlHeadXtra[] = '<link rel="stylesheet" href="' . api_get_path(WEB_LIBRARY_JS_PATH) . 'hotspot/css/hotspot.css">'; |
|
| 146 | -$htmlHeadXtra[] = '<script src="' . api_get_path(WEB_LIBRARY_JS_PATH) . 'hotspot/js/hotspot.js"></script>'; |
|
| 145 | +$htmlHeadXtra[] = '<link rel="stylesheet" href="'.api_get_path(WEB_LIBRARY_JS_PATH).'hotspot/css/hotspot.css">'; |
|
| 146 | +$htmlHeadXtra[] = '<script src="'.api_get_path(WEB_LIBRARY_JS_PATH).'hotspot/js/hotspot.js"></script>'; |
|
| 147 | 147 | |
| 148 | 148 | if ($origin != 'learnpath') { |
| 149 | 149 | Display::display_header(''); |
@@ -280,29 +280,29 @@ discard block |
||
| 280 | 280 | $i = $totalScore = $totalWeighting = 0; |
| 281 | 281 | |
| 282 | 282 | if ($debug > 0) { |
| 283 | - error_log("ExerciseResult: " . print_r($exerciseResult, 1)); |
|
| 284 | - error_log("QuestionList: " . print_r($questionList, 1)); |
|
| 283 | + error_log("ExerciseResult: ".print_r($exerciseResult, 1)); |
|
| 284 | + error_log("QuestionList: ".print_r($questionList, 1)); |
|
| 285 | 285 | } |
| 286 | 286 | |
| 287 | 287 | $arrques = array(); |
| 288 | 288 | $arrans = array(); |
| 289 | 289 | |
| 290 | -$user_restriction = $is_allowedToEdit ? '' : "AND user_id=" . intval($student_id) . " "; |
|
| 290 | +$user_restriction = $is_allowedToEdit ? '' : "AND user_id=".intval($student_id)." "; |
|
| 291 | 291 | $sql = "SELECT attempts.question_id, answer |
| 292 | 292 | FROM $TBL_TRACK_ATTEMPT as attempts |
| 293 | - INNER JOIN " . $TBL_TRACK_EXERCISES . " AS stats_exercises |
|
| 293 | + INNER JOIN ".$TBL_TRACK_EXERCISES." AS stats_exercises |
|
| 294 | 294 | ON stats_exercises.exe_id=attempts.exe_id |
| 295 | 295 | INNER JOIN $TBL_EXERCISE_QUESTION AS quizz_rel_questions |
| 296 | 296 | ON |
| 297 | 297 | quizz_rel_questions.exercice_id=stats_exercises.exe_exo_id AND |
| 298 | 298 | quizz_rel_questions.question_id = attempts.question_id AND |
| 299 | - quizz_rel_questions.c_id=" . api_get_course_int_id() . " |
|
| 300 | - INNER JOIN " . $TBL_QUESTIONS . " AS questions |
|
| 299 | + quizz_rel_questions.c_id=".api_get_course_int_id()." |
|
| 300 | + INNER JOIN " . $TBL_QUESTIONS." AS questions |
|
| 301 | 301 | ON |
| 302 | 302 | questions.id=quizz_rel_questions.question_id AND |
| 303 | - questions.c_id = " . api_get_course_int_id() . " |
|
| 303 | + questions.c_id = " . api_get_course_int_id()." |
|
| 304 | 304 | WHERE |
| 305 | - attempts.exe_id = " . intval($id) . " $user_restriction |
|
| 305 | + attempts.exe_id = " . intval($id)." $user_restriction |
|
| 306 | 306 | GROUP BY quizz_rel_questions.question_order, attempts.question_id"; |
| 307 | 307 | |
| 308 | 308 | $result = Database::query($sql); |
@@ -530,26 +530,26 @@ discard block |
||
| 530 | 530 | $table_resume = '<table class="data_table"> |
| 531 | 531 | <tr class="row_odd" > |
| 532 | 532 | <td></td> |
| 533 | - <td ><b>' . get_lang('Requirements') . '</b></td> |
|
| 534 | - <td><b>' . get_lang('YourAnswer') . '</b></td> |
|
| 533 | + <td ><b>' . get_lang('Requirements').'</b></td> |
|
| 534 | + <td><b>' . get_lang('YourAnswer').'</b></td> |
|
| 535 | 535 | </tr> |
| 536 | 536 | |
| 537 | 537 | <tr class="row_even"> |
| 538 | - <td><b>' . get_lang('Overlap') . '</b></td> |
|
| 539 | - <td>' . get_lang('Min') . ' ' . $threadhold1 . '</td> |
|
| 540 | - <td><div style="color:' . $overlap_color . '">' . (($final_overlap < 0) ? 0 : intval($final_overlap)) . '</div></td> |
|
| 538 | + <td><b>' . get_lang('Overlap').'</b></td> |
|
| 539 | + <td>' . get_lang('Min').' '.$threadhold1.'</td> |
|
| 540 | + <td><div style="color:' . $overlap_color.'">'.(($final_overlap < 0) ? 0 : intval($final_overlap)).'</div></td> |
|
| 541 | 541 | </tr> |
| 542 | 542 | |
| 543 | 543 | <tr> |
| 544 | - <td><b>' . get_lang('Excess') . '</b></td> |
|
| 545 | - <td>' . get_lang('Max') . ' ' . $threadhold2 . '</td> |
|
| 546 | - <td><div style="color:' . $excess_color . '">' . (($final_excess < 0) ? 0 : intval($final_excess)) . '</div></td> |
|
| 544 | + <td><b>' . get_lang('Excess').'</b></td> |
|
| 545 | + <td>' . get_lang('Max').' '.$threadhold2.'</td> |
|
| 546 | + <td><div style="color:' . $excess_color.'">'.(($final_excess < 0) ? 0 : intval($final_excess)).'</div></td> |
|
| 547 | 547 | </tr> |
| 548 | 548 | |
| 549 | 549 | <tr class="row_even"> |
| 550 | - <td><b>' . get_lang('Missing') . '</b></td> |
|
| 551 | - <td>' . get_lang('Max') . ' ' . $threadhold3 . '</td> |
|
| 552 | - <td><div style="color:' . $missing_color . '">' . (($final_missing < 0) ? 0 : intval($final_missing)) . '</div></td> |
|
| 550 | + <td><b>' . get_lang('Missing').'</b></td> |
|
| 551 | + <td>' . get_lang('Max').' '.$threadhold3.'</td> |
|
| 552 | + <td><div style="color:' . $missing_color.'">'.(($final_missing < 0) ? 0 : intval($final_missing)).'</div></td> |
|
| 553 | 553 | </tr></table>'; |
| 554 | 554 | |
| 555 | 555 | if ($answerType != HOT_SPOT_DELINEATION) { |
@@ -573,24 +573,24 @@ discard block |
||
| 573 | 573 | } |
| 574 | 574 | } |
| 575 | 575 | |
| 576 | - echo '<h1><div style="color:#333;">' . get_lang('Feedback') . '</div></h1>'; |
|
| 576 | + echo '<h1><div style="color:#333;">'.get_lang('Feedback').'</div></h1>'; |
|
| 577 | 577 | if ($answerType == HOT_SPOT_DELINEATION) { |
| 578 | 578 | if ($organs_at_risk_hit > 0) { |
| 579 | - $message = '<br />' . get_lang('ResultIs') . ' <b>' . $result_comment . '</b><br />'; |
|
| 580 | - $message .= '<p style="color:#DC0A0A;"><b>' . get_lang('OARHit') . '</b></p>'; |
|
| 579 | + $message = '<br />'.get_lang('ResultIs').' <b>'.$result_comment.'</b><br />'; |
|
| 580 | + $message .= '<p style="color:#DC0A0A;"><b>'.get_lang('OARHit').'</b></p>'; |
|
| 581 | 581 | } else { |
| 582 | - $message = '<p>' . get_lang('YourDelineation') . '</p>'; |
|
| 582 | + $message = '<p>'.get_lang('YourDelineation').'</p>'; |
|
| 583 | 583 | $message .= $table_resume; |
| 584 | - $message .= '<br />' . get_lang('ResultIs') . ' <b>' . $result_comment . '</b><br />'; |
|
| 584 | + $message .= '<br />'.get_lang('ResultIs').' <b>'.$result_comment.'</b><br />'; |
|
| 585 | 585 | } |
| 586 | - $message .= '<p>' . $comment . '</p>'; |
|
| 586 | + $message .= '<p>'.$comment.'</p>'; |
|
| 587 | 587 | echo $message; |
| 588 | 588 | } else { |
| 589 | - echo '<p>' . $comment . '</p>'; |
|
| 589 | + echo '<p>'.$comment.'</p>'; |
|
| 590 | 590 | } |
| 591 | 591 | |
| 592 | 592 | //showing the score |
| 593 | - $queryfree = "select marks from " . $TBL_TRACK_ATTEMPT . " WHERE exe_id = " . intval($id) . " and question_id= " . intval($questionId) . ""; |
|
| 593 | + $queryfree = "select marks from ".$TBL_TRACK_ATTEMPT." WHERE exe_id = ".intval($id)." and question_id= ".intval($questionId).""; |
|
| 594 | 594 | $resfree = Database::query($queryfree); |
| 595 | 595 | $questionScore = Database::result($resfree, 0, "marks"); |
| 596 | 596 | $totalScore += $questionScore; |
@@ -645,8 +645,8 @@ discard block |
||
| 645 | 645 | $marksname = ''; |
| 646 | 646 | |
| 647 | 647 | if ($isFeedbackAllowed) { |
| 648 | - $name = "fckdiv" . $questionId; |
|
| 649 | - $marksname = "marksName" . $questionId; |
|
| 648 | + $name = "fckdiv".$questionId; |
|
| 649 | + $marksname = "marksName".$questionId; |
|
| 650 | 650 | if (in_array($answerType, array(FREE_ANSWER, ORAL_EXPRESSION))) { |
| 651 | 651 | $url_name = get_lang('EditCommentsAndMarks'); |
| 652 | 652 | } else { |
@@ -657,32 +657,32 @@ discard block |
||
| 657 | 657 | } |
| 658 | 658 | } |
| 659 | 659 | echo '<br />'; |
| 660 | - echo Display::url($url_name, 'javascript://', array('class' => 'btn', 'onclick' => "showfck('" . $name . "', '" . $marksname . "');")); |
|
| 660 | + echo Display::url($url_name, 'javascript://', array('class' => 'btn', 'onclick' => "showfck('".$name."', '".$marksname."');")); |
|
| 661 | 661 | echo '<br />'; |
| 662 | 662 | |
| 663 | - echo '<div id="feedback_' . $name . '" style="width:100%">'; |
|
| 663 | + echo '<div id="feedback_'.$name.'" style="width:100%">'; |
|
| 664 | 664 | $comnt = trim(Event::get_comments($id, $questionId)); |
| 665 | 665 | if (empty($comnt)) { |
| 666 | 666 | echo '<br />'; |
| 667 | 667 | } else { |
| 668 | - echo '<div id="question_feedback">' . $comnt . '</div>'; |
|
| 668 | + echo '<div id="question_feedback">'.$comnt.'</div>'; |
|
| 669 | 669 | } |
| 670 | 670 | echo '</div>'; |
| 671 | 671 | |
| 672 | - echo '<div id="' . $name . '" style="display:none">'; |
|
| 672 | + echo '<div id="'.$name.'" style="display:none">'; |
|
| 673 | 673 | $arrid[] = $questionId; |
| 674 | - $feedback_form = new FormValidator('frmcomments' . $questionId, 'post', ''); |
|
| 674 | + $feedback_form = new FormValidator('frmcomments'.$questionId, 'post', ''); |
|
| 675 | 675 | $feedback_form->addElement('html', '<br>'); |
| 676 | - $renderer =& $feedback_form->defaultRenderer(); |
|
| 676 | + $renderer = & $feedback_form->defaultRenderer(); |
|
| 677 | 677 | $renderer->setFormTemplate('<form{attributes}><div align="left">{content}</div></form>'); |
| 678 | 678 | $renderer->setCustomElementTemplate('<div align="left">{element}</div>'); |
| 679 | 679 | $comnt = Event::get_comments($id, $questionId); |
| 680 | - $default = array('comments_' . $questionId => $comnt); |
|
| 680 | + $default = array('comments_'.$questionId => $comnt); |
|
| 681 | 681 | |
| 682 | 682 | if ($useAdvancedEditor) { |
| 683 | 683 | $feedback_form->addElement( |
| 684 | 684 | 'html_editor', |
| 685 | - 'comments_' . $questionId, |
|
| 685 | + 'comments_'.$questionId, |
|
| 686 | 686 | null, |
| 687 | 687 | null, |
| 688 | 688 | array( |
@@ -692,7 +692,7 @@ discard block |
||
| 692 | 692 | ) |
| 693 | 693 | ); |
| 694 | 694 | } else { |
| 695 | - $feedback_form->addElement('textarea', 'comments_' . $questionId); |
|
| 695 | + $feedback_form->addElement('textarea', 'comments_'.$questionId); |
|
| 696 | 696 | } |
| 697 | 697 | $feedback_form->addElement('html', '<br>'); |
| 698 | 698 | $feedback_form->setDefaults($default); |
@@ -703,21 +703,21 @@ discard block |
||
| 703 | 703 | $comnt = Event::get_comments($id, $questionId); |
| 704 | 704 | echo '<br />'; |
| 705 | 705 | if (!empty($comnt)) { |
| 706 | - echo '<b>' . get_lang('Feedback') . '</b>'; |
|
| 707 | - echo '<div id="question_feedback">' . $comnt . '</div>'; |
|
| 706 | + echo '<b>'.get_lang('Feedback').'</b>'; |
|
| 707 | + echo '<div id="question_feedback">'.$comnt.'</div>'; |
|
| 708 | 708 | } |
| 709 | 709 | } |
| 710 | 710 | |
| 711 | 711 | if ($is_allowedToEdit && $isFeedbackAllowed) { |
| 712 | 712 | if (in_array($answerType, array(FREE_ANSWER, ORAL_EXPRESSION))) { |
| 713 | - $marksname = "marksName" . $questionId; |
|
| 714 | - echo '<div id="' . $marksname . '" style="display:none">'; |
|
| 715 | - echo '<form name="marksform_' . $questionId . '" method="post" action="">'; |
|
| 713 | + $marksname = "marksName".$questionId; |
|
| 714 | + echo '<div id="'.$marksname.'" style="display:none">'; |
|
| 715 | + echo '<form name="marksform_'.$questionId.'" method="post" action="">'; |
|
| 716 | 716 | $arrmarks[] = $questionId; |
| 717 | 717 | echo get_lang("AssignMarks"); |
| 718 | 718 | echo " <select name='marks' id='marks'>"; |
| 719 | 719 | for ($i = 0; $i <= $questionWeighting; $i++) { |
| 720 | - echo '<option ' . (($i == $questionScore) ? "selected='selected'" : '') . '>' . $i . '</option>'; |
|
| 720 | + echo '<option '.(($i == $questionScore) ? "selected='selected'" : '').'>'.$i.'</option>'; |
|
| 721 | 721 | } |
| 722 | 722 | echo '</select>'; |
| 723 | 723 | echo '</form><br /></div>'; |
@@ -728,8 +728,8 @@ discard block |
||
| 728 | 728 | } |
| 729 | 729 | } else { |
| 730 | 730 | $arrmarks[] = $questionId; |
| 731 | - echo '<div id="' . $marksname . '" style="display:none"><form name="marksform_' . $questionId . '" method="post" action=""> |
|
| 732 | - <select name="marks" id="marks" style="display:none;"><option>' . $questionScore . '</option></select></form><br/ ></div>'; |
|
| 731 | + echo '<div id="'.$marksname.'" style="display:none"><form name="marksform_'.$questionId.'" method="post" action=""> |
|
| 732 | + <select name="marks" id="marks" style="display:none;"><option>' . $questionScore.'</option></select></form><br/ ></div>'; |
|
| 733 | 733 | } |
| 734 | 734 | } else { |
| 735 | 735 | if ($questionScore == -1) { |
@@ -785,7 +785,7 @@ discard block |
||
| 785 | 785 | |
| 786 | 786 | $score = array(); |
| 787 | 787 | if ($show_results) { |
| 788 | - $score['result'] = get_lang('Score') . " : " . ExerciseLib::show_score( |
|
| 788 | + $score['result'] = get_lang('Score')." : ".ExerciseLib::show_score( |
|
| 789 | 789 | $my_total_score, |
| 790 | 790 | $my_total_weight, |
| 791 | 791 | false, |
@@ -857,7 +857,7 @@ discard block |
||
| 857 | 857 | |
| 858 | 858 | if ($isFeedbackAllowed) { |
| 859 | 859 | if (in_array($origin, array('tracking_course', 'user_course', 'correct_exercise_in_lp'))) { |
| 860 | - $formUrl = api_get_path(WEB_CODE_PATH) . 'exercice/exercise_report.php?' . api_get_cidreq() . '&'; |
|
| 860 | + $formUrl = api_get_path(WEB_CODE_PATH).'exercice/exercise_report.php?'.api_get_cidreq().'&'; |
|
| 861 | 861 | $formUrl .= http_build_query([ |
| 862 | 862 | 'exerciseId' => $exercise_id, |
| 863 | 863 | 'filter' => 2, |
@@ -869,14 +869,14 @@ discard block |
||
| 869 | 869 | ]); |
| 870 | 870 | $formUrl .= $fromlink; |
| 871 | 871 | |
| 872 | - echo '<form name="myform" id="myform" action="' . $formUrl . '" method="post">'; |
|
| 873 | - echo '<input type = "hidden" name="lp_item_id" value="' . $learnpath_id . '">'; |
|
| 874 | - echo '<input type = "hidden" name="lp_item_view_id" value="' . $lp_item_view_id . '">'; |
|
| 875 | - echo '<input type = "hidden" name="student_id" value="' . $student_id . '">'; |
|
| 876 | - echo '<input type = "hidden" name="total_score" value="' . $totalScore . '"> '; |
|
| 877 | - echo '<input type = "hidden" name="my_exe_exo_id" value="' . $exercise_id . '"> '; |
|
| 872 | + echo '<form name="myform" id="myform" action="'.$formUrl.'" method="post">'; |
|
| 873 | + echo '<input type = "hidden" name="lp_item_id" value="'.$learnpath_id.'">'; |
|
| 874 | + echo '<input type = "hidden" name="lp_item_view_id" value="'.$lp_item_view_id.'">'; |
|
| 875 | + echo '<input type = "hidden" name="student_id" value="'.$student_id.'">'; |
|
| 876 | + echo '<input type = "hidden" name="total_score" value="'.$totalScore.'"> '; |
|
| 877 | + echo '<input type = "hidden" name="my_exe_exo_id" value="'.$exercise_id.'"> '; |
|
| 878 | 878 | } else { |
| 879 | - $formUrl = api_get_path(WEB_CODE_PATH) . 'exercice/exercise_report.php?' . api_get_cidreq() . '&'; |
|
| 879 | + $formUrl = api_get_path(WEB_CODE_PATH).'exercice/exercise_report.php?'.api_get_cidreq().'&'; |
|
| 880 | 880 | $formUrl .= http_build_query([ |
| 881 | 881 | 'exerciseId' => $exercise_id, |
| 882 | 882 | 'filter' => 1, |
@@ -884,11 +884,11 @@ discard block |
||
| 884 | 884 | 'exeid' => $id |
| 885 | 885 | ]); |
| 886 | 886 | |
| 887 | - echo ' <form name="myform" id="myform" action="' . $formUrl . '" method="post">'; |
|
| 887 | + echo ' <form name="myform" id="myform" action="'.$formUrl.'" method="post">'; |
|
| 888 | 888 | } |
| 889 | 889 | |
| 890 | 890 | if ($origin != 'learnpath' && $origin != 'student_progress') { |
| 891 | - echo '<label><input type= "checkbox" name="send_notification"> ' . get_lang('SendEmail') . '</label>'; |
|
| 891 | + echo '<label><input type= "checkbox" name="send_notification"> '.get_lang('SendEmail').'</label>'; |
|
| 892 | 892 | ?> |
| 893 | 893 | <br/> |
| 894 | 894 | <button type="submit" class="btn btn-primary" value="<?php echo get_lang('Ok'); ?>" |
@@ -921,7 +921,7 @@ discard block |
||
| 921 | 921 | } else { |
| 922 | 922 | if (!isset($_GET['fb_type'])) { |
| 923 | 923 | $lp_mode = $_SESSION['lp_mode']; |
| 924 | - $url = '../newscorm/lp_controller.php?' . api_get_cidreq() . '&'; |
|
| 924 | + $url = '../newscorm/lp_controller.php?'.api_get_cidreq().'&'; |
|
| 925 | 925 | $url .= http_build_url([ |
| 926 | 926 | 'action' => 'view', |
| 927 | 927 | 'lp_id' => $learnpath_id, |
@@ -929,13 +929,13 @@ discard block |
||
| 929 | 929 | 'exeId' => $exeId, |
| 930 | 930 | 'fb_type' => $feedback_type |
| 931 | 931 | ]); |
| 932 | - $href = ($lp_mode == 'fullscreen') ? ' window.opener.location.href="' . $url . '" ' : ' top.location.href="' . $url . '" '; |
|
| 933 | - echo '<script type="text/javascript">' . $href . '</script>'; |
|
| 932 | + $href = ($lp_mode == 'fullscreen') ? ' window.opener.location.href="'.$url.'" ' : ' top.location.href="'.$url.'" '; |
|
| 933 | + echo '<script type="text/javascript">'.$href.'</script>'; |
|
| 934 | 934 | // Record the results in the learning path, using the SCORM interface (API) |
| 935 | 935 | echo "<script>window.parent.API.void_save_asset('$totalScore', '$totalWeighting', 0, 'completed'); </script>"; |
| 936 | 936 | echo '</body></html>'; |
| 937 | 937 | } else { |
| 938 | - Display::display_normal_message(get_lang('ExerciseFinished') . ' ' . get_lang('ToContinueUseMenu')); |
|
| 938 | + Display::display_normal_message(get_lang('ExerciseFinished').' '.get_lang('ToContinueUseMenu')); |
|
| 939 | 939 | echo '<br />'; |
| 940 | 940 | } |
| 941 | 941 | } |