Skip to content

Commit b8261e5

Browse files
committed
Minor - format code.
1 parent 8da0d49 commit b8261e5

File tree

1 file changed

+24
-30
lines changed

1 file changed

+24
-30
lines changed

main/tracking/question_course_report.php

Lines changed: 24 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,17 @@
11
<?php
2+
/* For licensing terms, see /license.txt */
3+
24
/**
35
* Report
46
* @package chamilo.tracking
57
*/
6-
/**
7-
* Code
8-
*/
9-
$language_file = array ('registration', 'index', 'tracking', 'exercice','survey');
8+
9+
$language_file = array('registration', 'index', 'tracking', 'exercice','survey');
1010
$cidReset = true;
1111
require_once '../inc/global.inc.php';
1212

1313
require_once api_get_path(SYS_CODE_PATH).'exercice/exercise.class.php';
1414
require_once api_get_path(SYS_CODE_PATH).'exercice/question.class.php';
15-
require_once api_get_path(LIBRARY_PATH).'formvalidator/FormValidator.class.php';
1615
require_once api_get_path(LIBRARY_PATH).'pear/Spreadsheet_Excel_Writer/Writer.php';
1716

1817
require_once api_get_path(SYS_CODE_PATH).'newscorm/learnpathList.class.php';
@@ -59,9 +58,14 @@ function load_courses() {
5958
$temp_course_list = SessionManager::get_course_list_by_session_id($session_id);
6059
}
6160

62-
foreach($temp_course_list as $temp_course_item) {
61+
foreach ($temp_course_list as $temp_course_item) {
6362
$course_item = CourseManager ::get_course_information($temp_course_item['code']);
64-
$course_list[]= array('db_name' =>$course_item['db_name'],'code'=>$course_item['code'], 'title'=>$course_item['title'], 'visual_code'=>$course_item['visual_code']);
63+
$course_list[] = array(
64+
'db_name' => $course_item['db_name'],
65+
'code' => $course_item['code'],
66+
'title' => $course_item['title'],
67+
'visual_code' => $course_item['visual_code']
68+
);
6569
$course_select_list[$temp_course_item['code']] = $course_item['title'];
6670
}
6771

@@ -94,41 +98,35 @@ function load_courses() {
9498
$main_question_list = array();
9599
foreach ($lp_list as $lp_id =>$lp) {
96100
$exercise_list = get_all_exercises_from_lp($lp_id, $course_info['real_id']);
97-
//var_dump($exercise_list);
98101
foreach ($exercise_list as $exercise) {
99102
$my_exercise = new Exercise();
100-
//$my_exercise->read($exercise['ref']);
101103
$my_exercise->read($exercise['path']);
102104
$question_list = $my_exercise->selectQuestionList();
103105

104-
$exercise_stats = get_all_exercise_event_from_lp($exercise['path'],$course_info['id'], $session_id);
105-
//echo '<pre>'; print_r($exercise_stats);
106+
$exercise_stats = get_all_exercise_event_from_lp(
107+
$exercise['path'],
108+
$course_info['id'],
109+
$session_id
110+
);
106111

107-
foreach($question_list as $question_id) {
112+
foreach ($question_list as $question_id) {
108113
$question_data = Question::read($question_id);
109-
///var_dump($question_data);
110114
$main_question_list[$question_id] = $question_data;
111115
$quantity_exercises = 0;
112116
$question_result = 0;
113-
//echo '<pre>';
114-
//print_r($exercise_stats);
115117
foreach($exercise_stats as $stats) {
116118
if (!empty($stats['question_list'])) {
117119
foreach($stats['question_list'] as $my_question_stat) {
118-
// var_dump($my_question_stat);
119120
if ($question_id == $my_question_stat['question_id']) {
120-
//var_dump($my_question_stat);
121121
$question_result = $question_result + $my_question_stat['marks'];
122-
// var_dump($my_question_stat['marks']);
123122
$quantity_exercises++;
124123
}
125124
}
126125
}
127126
}
128-
//echo $question_id;
129-
//var_dump($question_result.' - '.$quantity_exercises.$main_question_list[$question_id]->weighting);
130-
if(!empty($quantity_exercises)) {
131-
$main_question_list[$question_id]->results =(($question_result / ($quantity_exercises)) ) ; // Score % average
127+
if (!empty($quantity_exercises)) {
128+
// Score % average
129+
$main_question_list[$question_id]->results =(($question_result / ($quantity_exercises)) ) ;
132130
} else {
133131
$main_question_list[$question_id]->results = 0;
134132
}
@@ -139,11 +137,6 @@ function load_courses() {
139137
}
140138
}
141139

142-
//var_dump($main_question_list);
143-
144-
//var_dump($main_question_list);
145-
//$course_list = SessionManager::get_course_list_by_session_id($session_id);
146-
147140
if (!$export_to_xls) {
148141

149142
Display :: display_header(get_lang("MySpace"));
@@ -173,7 +166,6 @@ function load_courses() {
173166
}
174167

175168
$course_average = array();
176-
177169
$counter = 0;
178170

179171
if (!empty($main_question_list) && is_array($main_question_list)) {
@@ -228,7 +220,8 @@ function load_courses() {
228220
exit;
229221
}
230222

231-
function sort_user($a, $b) {
223+
function sort_user($a, $b)
224+
{
232225
if (is_numeric($a['score']) && is_numeric($b['score'])) {
233226
echo $a['score'].' : '.$b['score'];
234227
echo '<br />';
@@ -240,7 +233,8 @@ function sort_user($a, $b) {
240233
return 1;
241234
}
242235

243-
function export_complete_report_xls($filename, $array) {
236+
function export_complete_report_xls($filename, $array)
237+
{
244238
global $charset;
245239
$workbook = new Spreadsheet_Excel_Writer();
246240
$workbook ->setTempDir(api_get_path(SYS_ARCHIVE_PATH));

0 commit comments

Comments
 (0)