chamilo / chamilo-lms

main/reports/index.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
 $this_section = SECTION_REPORTS;
 
 // setting the name of the tool
-$tool_name=get_lang('Reports');
+$tool_name = get_lang('Reports');
 
 // Displaying the header
 foreach (array('jquery.ba-bbq.min.js', 'jquery.validate.js', 'jquery.form.js', 'jquery.form.wizard.js', 'jquery.dataTables.min.js') as $js)

main/reports/multiquery.lib.php

Spacing +6 added lines, -6 removed lines

@@ -4,20 +4,20 @@ 
 function multiquery_query($array) {
 	$result = array();
 	$field = 0;
-	for ($i = 0; $i <sizeof($array); $i++) {
+	for ($i = 0; $i < sizeof($array); $i++) {
 		// mysql handler
 		$result[$i]['mysql'] = mysql_query($array[$i]);
 
-		if (! $result[$i]['mysql'])
+		if (!$result[$i]['mysql'])
 			die("error in query $i : ".$array[$i]);
 
 		// fields
 		$result[$i]['num_fields'] = mysql_num_fields($result[$i]['mysql']);
 		for ($j = 0; $j < $result[$i]['num_fields']; $j++) {
 			$name = mysql_field_name($result[$i]['mysql'], $j);
-			$result['field'][$field]['query']=$i;
-			$result['field'][$field]['name']=$name;
-			$result['field'][$field]['id']=$j;
+			$result['field'][$field]['query'] = $i;
+			$result['field'][$field]['name'] = $name;
+			$result['field'][$field]['id'] = $j;
 			$result['field_assoc'][$name][$field];
 			$field++;
 		}
@@ -25,7 +25,7 @@ 
 	$result['num_queries'] = sizeof($array);
 	// rows
 	$numberOfResult = mysql_num_rows($result[0]['mysql']);
-	for ($i = 1; $i <$result['num_queries']; $i++) 
+	for ($i = 1; $i < $result['num_queries']; $i++) 
 		if ($numberOfResult != mysql_num_rows($result[$i]['mysql']))
 			die("wrong number of row: $numberOfResult vs ".
 				mysql_num_rows($result[$i]['mysql'])." on query $i");

main/reports/modules/quiz.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php 
 
-$reports_modules['quiz'] = array ();
+$reports_modules['quiz'] = array();
 
 
 function reports_modules_quiz_init() {
@@ -28,6 +28,6 @@ 
 			REPORTS_PROGRESS_COMPLETED.' as progress, '.
 			'exe_duration as time, exe_date as ts from '.
 			Database::get_main_table(TABLE_STATISTIC_TRACK_E_EXERCISES).
-			' where c_id = ' . $courseId .
+			' where c_id = '.$courseId.
 			' and exe_exo_id='.$quiz['child_id']);
 }

main/reports/modules/scorm.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php 
 
-$reports_modules['scorm'] = array ();
+$reports_modules['scorm'] = array();
 
 
 function reports_modules_scorm_init() {

main/reports/modules/course.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-$reports_modules['course'] = array ();
+$reports_modules['course'] = array();
 
 
 function reports_modules_course_init() {
@@ -24,6 +24,6 @@ 
 			'NULL as progress, '.
 			'(sum(logout_course_date) - sum(login_course_date)) as time, null as ts from '.
 			Database::get_main_table(TABLE_STATISTIC_TRACK_E_COURSE_ACCESS).
-			' where c_id = ' . $course['real_id'] .
+			' where c_id = '.$course['real_id'].
 			' group by user_id');
 }

main/reports/reports.php

Spacing +13 added lines, -13 removed lines

@@ -24,7 +24,7 @@ 
 $this_section = SECTION_REPORTS;
 
 // setting the name of the tool
-$tool_name=get_lang('Reports');
+$tool_name = get_lang('Reports');
 
 // loading templates
 reports_loadTemplates();
@@ -68,7 +68,7 @@ 
 }
 
 // outputing a link to csv file instead of outputing csv data directly
-if ($_REQUEST['format'] == 'csv')  {
+if ($_REQUEST['format'] == 'csv') {
 	// converting post vars to get uri
 	$params = '';
 	$kv = array();
@@ -96,7 +96,7 @@ 
 
 if (is_array($reports_template[$_REQUEST['type']])) {
 	$query = $reports_template[$_REQUEST['type']]['getSQL']();
-	if (! is_array($query))
+	if (!is_array($query))
 		$query = array($query);
 	if ($_REQUEST['format'] == 'sql')
 		die(var_export($query, true));
@@ -150,7 +150,7 @@ 
 	$columns = array();
 	$columns_islink = array();
 	echo '<thead><tr>';
-	for ($i=0; $i < $nfields; $i++)	{
+	for ($i = 0; $i < $nfields; $i++) {
 		$columns[$i] = multiquery_field_name($result, $i);
 		if (substr($columns[$i], -5, 5) != '_link') {
 			$column_islink[$i] = false;
@@ -162,7 +162,7 @@ 
 	// checking resolving link column id
 	$columns_flip = array_flip($columns);
 	$columns_link = array();
-	for ($i=0; $i < $nfields; $i++)
+	for ($i = 0; $i < $nfields; $i++)
 		if ($column_islink[$i] == false && array_key_exists($columns[$i].'_link', $columns_flip))
 			$columns_link[$i] = $columns_flip[$columns[$i].'_link'];
 		else
@@ -170,10 +170,10 @@ 
 	echo '</tr></thead><tbody>';
 	while ($row = multiquery_fetch_row($result)) {
 		echo '<tr>';
-		for ($i = 0; $i<$nfields; $i++)
-			if (!$columns_islink[$i]){ // ignore links
+		for ($i = 0; $i < $nfields; $i++)
+			if (!$columns_islink[$i]) { // ignore links
 				if ($columns_link[$i] != '') // link is defined
-					if (substr($columns_link[$i],0,10) == 'javascript') {
+					if (substr($columns_link[$i], 0, 10) == 'javascript') {
 						echo '<td><a href="#" onclick="'.$row[$columns_link[$i]].'">'.$row[$i].'</a></td>';
 					}
 					else {
@@ -201,7 +201,7 @@ 
 	$nfields = multiquery_num_fields($result);
 	$columns = array();
 	$columns_islink = array();
-	for ($i=0; $i < $nfields; $i++)	{
+	for ($i = 0; $i < $nfields; $i++) {
 		$columns[$i] = multiquery_field_name($result, $i);
 		if (substr($columns[$i], -5, 5) != '_link') {
 			$column_islink[$i] = false;
@@ -212,16 +212,16 @@ 
 
 	echo "\n";
 	while ($row = multiquery_fetch_row($result)) {
-		for ($i = 0; $i<$nfields; $i++)
+		for ($i = 0; $i < $nfields; $i++)
 			if (!$columns_islink[$i]) // ignore links
-				echo csv_escaping($row[$i]).',';  // fixme
+				echo csv_escaping($row[$i]).','; // fixme
 		echo "\n";
 	}
 } else die(get_lang('UnknownFormat'));
 
 function csv_escaping($value, $csv_separator = ',') {
-	$value = str_replace('"','""',$value);
-	if (strpos($value, '""') or strpos($value, $csv_separator) or $value != trim($value) ) {
+	$value = str_replace('"', '""', $value);
+	if (strpos($value, '""') or strpos($value, $csv_separator) or $value != trim($value)) {
 		$value = '"'.$value.'"';
 	}
 	return $value;

main/mySpace/lp_tracking.php

Spacing +14 added lines, -14 removed lines

@@ -17,7 +17,7 @@ 
 	$this_section = SECTION_COURSES;
 }
 
-$session_id  = isset($_REQUEST['id_session']) && !empty($_REQUEST['id_session']) ? intval($_REQUEST['id_session']) : api_get_session_id();
+$session_id = isset($_REQUEST['id_session']) && !empty($_REQUEST['id_session']) ? intval($_REQUEST['id_session']) : api_get_session_id();
 $export_csv = isset($_GET['export']) && $_GET['export'] == 'csv' ? true : false;
 $user_id = isset($_GET['student_id']) ? intval($_GET['student_id']) : api_get_user_id();
 $courseCode = isset($_GET['course']) ? Security::remove_XSS($_GET['course']) : api_get_course_id();
@@ -39,18 +39,18 @@ 
 }
 
 if ($origin == 'user_course') {
-	$interbreadcrumb[] = array ("url" => api_get_path(WEB_COURSE_PATH).$course_info['directory'], 'name' => $course_info['name']);
-	$interbreadcrumb[] = array ("url" => "../user/user.php?cidReq=".$courseCode, "name" => get_lang("Users"));
-} else if($origin == 'tracking_course') {
-	$interbreadcrumb[] = array ("url" => "../tracking/courseLog.php?cidReq=".$courseCode.'&id_session='.$session_id, "name" => get_lang("Tracking"));
+	$interbreadcrumb[] = array("url" => api_get_path(WEB_COURSE_PATH).$course_info['directory'], 'name' => $course_info['name']);
+	$interbreadcrumb[] = array("url" => "../user/user.php?cidReq=".$courseCode, "name" => get_lang("Users"));
+} else if ($origin == 'tracking_course') {
+	$interbreadcrumb[] = array("url" => "../tracking/courseLog.php?cidReq=".$courseCode.'&id_session='.$session_id, "name" => get_lang("Tracking"));
 } else {
-	$interbreadcrumb[] = array ("url" => "index.php", "name" => get_lang('MySpace'));
-	$interbreadcrumb[] = array ("url" => "student.php", "name" => get_lang("MyStudents"));
- 	$interbreadcrumb[] = array ("url" => "myStudents.php?student=".$user_id, "name" => get_lang("StudentDetails"));
- 	$nameTools=get_lang("DetailsStudentInCourse");
+	$interbreadcrumb[] = array("url" => "index.php", "name" => get_lang('MySpace'));
+	$interbreadcrumb[] = array("url" => "student.php", "name" => get_lang("MyStudents"));
+ 	$interbreadcrumb[] = array("url" => "myStudents.php?student=".$user_id, "name" => get_lang("StudentDetails"));
+ 	$nameTools = get_lang("DetailsStudentInCourse");
 }
 
-$interbreadcrumb[] = array("url" => "myStudents.php?student=".$user_id."&course=".$courseCode."&details=true&origin=".$origin , "name" => get_lang("DetailsStudentInCourse"));
+$interbreadcrumb[] = array("url" => "myStudents.php?student=".$user_id."&course=".$courseCode."&details=true&origin=".$origin, "name" => get_lang("DetailsStudentInCourse"));
 $nameTools = get_lang('LearningPathDetails');
 $sql = 'SELECT name	FROM '.Database::get_course_table(TABLE_LP_MAIN).' WHERE c_id = '.$course_info['real_id'].' AND id='.$lp_id;
 $rs  = Database::query($sql);
@@ -62,15 +62,15 @@ 
 
 Display :: display_header($nameTools);
 echo '<div class ="actions">';
-echo '<a href="javascript:history.back();">'.Display::return_icon('back.png',get_lang('Back'),'',ICON_SIZE_MEDIUM).'</a>';
+echo '<a href="javascript:history.back();">'.Display::return_icon('back.png', get_lang('Back'), '', ICON_SIZE_MEDIUM).'</a>';
 echo '<a href="javascript: void(0);" onclick="javascript: window.print();">
-        '.Display::return_icon('printer.png',get_lang('Print'),'',ICON_SIZE_MEDIUM).'</a>';
+        '.Display::return_icon('printer.png', get_lang('Print'), '', ICON_SIZE_MEDIUM).'</a>';
 echo '<a href="'.api_get_self().'?export=csv&'.Security::remove_XSS($_SERVER['QUERY_STRING']).'">
-        '.Display::return_icon('export_csv.png',get_lang('ExportAsCSV'),'',ICON_SIZE_MEDIUM).'</a>';
+        '.Display::return_icon('export_csv.png', get_lang('ExportAsCSV'), '', ICON_SIZE_MEDIUM).'</a>';
 echo '</div>';
 echo '<div class="clear"></div>';
 $session_name = api_get_session_name($session_id);
-$table_title = ($session_name? Display::return_icon('session.png', get_lang('Session'), array(), ICON_SIZE_SMALL).' '.$session_name.' ':' ').
+$table_title = ($session_name ? Display::return_icon('session.png', get_lang('Session'), array(), ICON_SIZE_SMALL).' '.$session_name.' ' : ' ').
     Display::return_icon('course.png', get_lang('Course'), array(), ICON_SIZE_SMALL).' '.$course_info['name'].' '.
     Display::return_icon('user.png', get_lang('User'), array(), ICON_SIZE_SMALL).' '.$name;
 echo Display::page_header($table_title);

main/mySpace/current_courses.php

Spacing +13 added lines, -13 removed lines

@@ -34,11 +34,11 @@ 
 		}
 
 		$teachers = CourseManager::get_teacher_list_from_course_code($course_code);
-		$teacher_list =  array();
+		$teacher_list = array();
 
 		if (!empty($teachers)) {
-			foreach($teachers as $teacher) {
-				$teacher_list[]= $teacher['firstname'].' '.$teacher['lastname'];
+			foreach ($teachers as $teacher) {
+				$teacher_list[] = $teacher['firstname'].' '.$teacher['lastname'];
 			}
 		}
 
@@ -54,7 +54,7 @@ 
 			$students[] = $student['user_id'];
 		}
 
-		$t_lp 	= Database :: get_course_table(TABLE_LP_MAIN);
+		$t_lp = Database :: get_course_table(TABLE_LP_MAIN);
 		$sql_lp = "SELECT lp.name, lp.id FROM $t_lp lp
 				   WHERE c_id = $course_id AND lp.session_id = 0";
 		$rs_lp 	= Database::query($sql_lp);
@@ -67,7 +67,7 @@ 
 		);
 
 		$total_tools = 0;
-		foreach($total_tools_list as $tool) {
+		foreach ($total_tools_list as $tool) {
 			$total_tools += $tool['count_access_tool'];
 		}
 
@@ -93,16 +93,16 @@ 
 				if (!empty($students)) {
 					foreach ($students  as $student_id) {
 						$avg_student_progress   = Tracking::get_avg_student_progress($student_id, $course_code, array($lp_id), $session_id);
-						$myavg_temp 			= Tracking::get_avg_student_score($student_id, $course_code, array($lp_id), $session_id);
+						$myavg_temp = Tracking::get_avg_student_score($student_id, $course_code, array($lp_id), $session_id);
 						$avg_progress_in_course = Tracking::get_avg_student_progress($student_id, $course_code, array($lp_id), $session_id);
 
 						if (intval($avg_progress_in_course) == 100) {
 							$count_students_complete_all_activities++;
 						}
 						if (intval($avg_progress_in_course) > 0 && intval($avg_progress_in_course) <= 50) {
-							$count_students_complete_all_activities_at_50 ++;
+							$count_students_complete_all_activities_at_50++;
 						}
-						$total_average_progress +=$avg_progress_in_course;
+						$total_average_progress += $avg_progress_in_course;
 
 						$time_spent = Tracking::get_time_spent_on_the_course($student_id, $course_id, $session_id);
 						$total_time_spent += $time_spent;
@@ -139,7 +139,7 @@ 
 						    ORDER BY item_type";
 				$res_lpi = Database::query($sql_lpi);
 				while ($row_lpi = Database::fetch_array($res_lpi)) {
-					switch($row_lpi['item_type']) {
+					switch ($row_lpi['item_type']) {
 						case 'document':
 							$array[$i]['learnpath_docs']++;
 							break;
@@ -171,10 +171,10 @@ 
 
 				if (!empty($student_count)) {
 					$array[$i]['count_students_accessing'] = $count_students_accessing;
-					$array[$i]['count_students_accessing_percentage'] = round($count_students_accessing / $student_count *100 , 0);
+					$array[$i]['count_students_accessing_percentage'] = round($count_students_accessing / $student_count * 100, 0);
 					$array[$i]['count_students_complete_all_activities_at_50'] = $count_students_complete_all_activities;
-					$array[$i]['count_students_complete_all_activities'] = round($count_students_complete_all_activities / $student_count *100 , 0);;
-					$array[$i]['average_percentage_activities_completed_per_student'] = round($count_students_complete_all_activities/$student_count*100,2);
+					$array[$i]['count_students_complete_all_activities'] = round($count_students_complete_all_activities / $student_count * 100, 0); ;
+					$array[$i]['average_percentage_activities_completed_per_student'] = round($count_students_complete_all_activities / $student_count * 100, 2);
 					$array[$i]['total_time_spent'] = 0;
 					$array[$i]['average_time_spent_per_student'] = 0;
 
@@ -241,7 +241,7 @@ 
 	exit;
 }
 
-$interbreadcrumb[] = array ('url' => 'index.php', 'name' => get_lang('MySpace'));
+$interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('MySpace'));
 
 Display::display_header(get_lang('CurrentCourses'));
 

main/mySpace/user_edit.php

Spacing +14 added lines, -14 removed lines

@@ -89,15 +89,15 @@ 
 	$group[] =& $form->createElement('select', 'auth_source', null, $auth_sources);
 	$group[] =& $form->createElement('static', '', '', '<br />');
 }*/
-$group[] =& $form->createElement('radio', 'password_auto', get_lang('Password'), get_lang('AutoGeneratePassword').'<br />', 1);
-$group[] =& $form->createElement('radio', 'password_auto', 'id="radio_user_password"', null, 0);
-$group[] =& $form->createElement('password', 'password', null, array('onkeydown' => 'javascript: password_switch_radio_button(document.user_add,"password[password_auto]");'));
+$group[] = & $form->createElement('radio', 'password_auto', get_lang('Password'), get_lang('AutoGeneratePassword').'<br />', 1);
+$group[] = & $form->createElement('radio', 'password_auto', 'id="radio_user_password"', null, 0);
+$group[] = & $form->createElement('password', 'password', null, array('onkeydown' => 'javascript: password_switch_radio_button(document.user_add,"password[password_auto]");'));
 $form->addGroup($group, 'password', get_lang('Password'), '');
 
 // Send email
 $group = array();
-$group[] =& $form->createElement('radio', 'send_mail', null, get_lang('Yes'), 1);
-$group[] =& $form->createElement('radio', 'send_mail', null, get_lang('No'), 0);
+$group[] = & $form->createElement('radio', 'send_mail', null, get_lang('Yes'), 1);
+$group[] = & $form->createElement('radio', 'send_mail', null, get_lang('No'), 0);
 $form->addGroup($group, 'mail', get_lang('SendMailToNewUser'), '&nbsp;');
 
 // Set default values
@@ -108,7 +108,7 @@ 
 
 $form->setDefaults($defaults);
 // Submit button
-$select_level = array ();
+$select_level = array();
 $html_results_enabled[] = $form->addButtonUpdate(get_lang('Update'), 'submit', true);
 $form->addGroup($html_results_enabled);
 // Validate form
@@ -169,17 +169,17 @@ 
 			}
 
 			$emailbody = get_lang('Dear')." ".stripslashes(api_get_person_name($userInfo['firstname'], $userInfo['lastname'])).",\n\n".
-                get_lang('YouAreReg')." ". api_get_setting('siteName') ." ".get_lang('WithTheFollowingSettings')."\n\n".
-                get_lang('Username')." : ". $username ."\n". get_lang('Pass')." : ".stripslashes($password)."\n\n" .
-                get_lang('Address') ." ". api_get_setting('siteName') ." ".
-                get_lang('Is') ." : ".$portal_url."\n\n".
-                get_lang('Problem'). "\n\n".
+                get_lang('YouAreReg')." ".api_get_setting('siteName')." ".get_lang('WithTheFollowingSettings')."\n\n".
+                get_lang('Username')." : ".$username."\n".get_lang('Pass')." : ".stripslashes($password)."\n\n".
+                get_lang('Address')." ".api_get_setting('siteName')." ".
+                get_lang('Is')." : ".$portal_url."\n\n".
+                get_lang('Problem')."\n\n".
                 get_lang('SignatureFormula').",\n\n".
                 api_get_person_name(api_get_setting('administratorName'), api_get_setting('administratorSurname'))."\n".
-                get_lang('Manager'). " ".
+                get_lang('Manager')." ".
                 api_get_setting('siteName')."\nT. ".
-                api_get_setting('administratorTelephone')."\n" .
-                get_lang('Email') ." : ".api_get_setting('emailAdministrator');
+                api_get_setting('administratorTelephone')."\n".
+                get_lang('Email')." : ".api_get_setting('emailAdministrator');
 
 			api_mail_html(
                 api_get_person_name($userInfo['firstname'], $userInfo['lastname'], null, PERSON_NAME_EMAIL_ADDRESS),