chamilo / chamilo-lms

main/gradebook/gradebook_edit_all.php

Spacing +1 added lines, -1 removed lines

@@ -79,7 +79,7 @@
 
 $submitted = isset($_POST['submitted']) ? $_POST['submitted'] : '';
 if ($submitted == 1) {
-    Display :: display_confirmation_message(get_lang('GradebookWeightUpdated')) . '<br /><br />';
+    Display :: display_confirmation_message(get_lang('GradebookWeightUpdated')).'<br /><br />';
     if (isset($_POST['evaluation'])) {
         $eval_log = new Evaluation();
     }

main/gradebook/exercise_jump.php

Spacing +4 added lines, -4 removed lines

@@ -12,7 +12,7 @@ 
 
 require_once '../inc/global.inc.php';
 api_block_anonymous_users();
-$this_section=SECTION_COURSES;
+$this_section = SECTION_COURSES;
 
 $gradebook = Security::remove_XSS($_GET['gradebook']);
 
@@ -24,7 +24,7 @@ 
 $doExerciseUrl = null;
 
 if (isset($_GET['doexercise'])) {
-    $doExerciseUrl = api_get_path(WEB_CODE_PATH) . 'exercise/overview.php?' . http_build_query([
+    $doExerciseUrl = api_get_path(WEB_CODE_PATH).'exercise/overview.php?'.http_build_query([
         'session_id' => $session_id,
         'cidReq' => $cidReq,
         'gradebook' => $gradebook,
@@ -60,11 +60,11 @@ 
                         // If the exercise was added once redirect to the LP
                         $firstLp = current($exercise->lpList);
                         if (isset($firstLp['lp_id'])) {
-                            $url = api_get_path(WEB_CODE_PATH) . 'newscorm/lp_controller.php?' . api_get_cidreq() . '&lp_id=' . $firstLp['lp_id'] . '&action=view&isStudentView=true';
+                            $url = api_get_path(WEB_CODE_PATH).'newscorm/lp_controller.php?'.api_get_cidreq().'&lp_id='.$firstLp['lp_id'].'&action=view&isStudentView=true';
                         }
                     } else {
                         // If the exercise was added multiple times show the LP list
-                        $url = api_get_path(WEB_CODE_PATH) . 'newscorm/lp_controller.php?' . api_get_cidreq().'&action=list';
+                        $url = api_get_path(WEB_CODE_PATH).'newscorm/lp_controller.php?'.api_get_cidreq().'&action=list';
                     }
                 }
             }

main/gradebook/gradebook_showlog_link.php

Spacing +11 added lines, -11 removed lines

@@ -11,9 +11,9 @@ 
 api_block_anonymous_users();
 GradebookUtils::block_students();
 
-$interbreadcrumb[] = array ('url' => Security::remove_XSS($_SESSION['gradebook_dest']).'?','name' => get_lang('Gradebook'));
-$interbreadcrumb[] = array ('url' => Security::remove_XSS($_SESSION['gradebook_dest']).'?selectcat='.Security::remove_XSS($_GET['selectcat']),'name' => get_lang('Details'));
-$interbreadcrumb[] = array ('url' => 'gradebook_showlog_link.php?visiblelink='.Security::remove_XSS($_GET['visiblelink']).'&selectcat='.Security::remove_XSS($_GET['selectcat']),	'name' => get_lang('GradebookQualifyLog'));
+$interbreadcrumb[] = array('url' => Security::remove_XSS($_SESSION['gradebook_dest']).'?', 'name' => get_lang('Gradebook'));
+$interbreadcrumb[] = array('url' => Security::remove_XSS($_SESSION['gradebook_dest']).'?selectcat='.Security::remove_XSS($_GET['selectcat']), 'name' => get_lang('Details'));
+$interbreadcrumb[] = array('url' => 'gradebook_showlog_link.php?visiblelink='.Security::remove_XSS($_GET['visiblelink']).'&selectcat='.Security::remove_XSS($_GET['selectcat']), 'name' => get_lang('GradebookQualifyLog'));
 $this_section = SECTION_COURSES;
 Display :: display_header('');
 echo '<div class="actions">';
@@ -22,21 +22,21 @@ 
 
 $t_user     = Database :: get_main_table(TABLE_MAIN_USER);
 $t_link_log = Database :: get_main_table(TABLE_MAIN_GRADEBOOK_LINKEVAL_LOG);
-$visible_link=Security::remove_XSS($_GET['visiblelink']);
+$visible_link = Security::remove_XSS($_GET['visiblelink']);
 $evaledit   = EvalLink :: load($visible_link);
 $sql = "SELECT lk.name,lk.description,lk.weight,lk.visible,lk.type,lk.created_at,us.username
         FROM ".$t_link_log." lk inner join ".$t_user." us
         ON lk.user_id_log=us.user_id
         WHERE lk.id_linkeval_log=".$evaledit[0]->get_id()." AND lk.type='link';";
-$result=Database::query($sql);
-$list_info=array();
-while ($row=Database::fetch_row($result)) {
+$result = Database::query($sql);
+$list_info = array();
+while ($row = Database::fetch_row($result)) {
     $list_info[] = $row;
 }
 
-foreach($list_info as $key => $info_log) {
-    $list_info[$key][5]=($info_log[5]) ? api_convert_and_format_date($info_log[5]) : 'N/A';
-    $list_info[$key][3]=($info_log[3]==1) ? get_lang('GradebookVisible') : get_lang('GradebookInvisible');
+foreach ($list_info as $key => $info_log) {
+    $list_info[$key][5] = ($info_log[5]) ? api_convert_and_format_date($info_log[5]) : 'N/A';
+    $list_info[$key][3] = ($info_log[3] == 1) ? get_lang('GradebookVisible') : get_lang('GradebookInvisible');
 }
 
 $parameters = array(
@@ -44,7 +44,7 @@ 
     'selectcat' => Security::remove_XSS($_GET['selectcat']),
 );
 
-$table = new SortableTableFromArrayConfig($list_info, 1,20,'gradebooklink');
+$table = new SortableTableFromArrayConfig($list_info, 1, 20, 'gradebooklink');
 $table->set_additional_parameters($parameters);
 $table->set_header(0, get_lang('GradebookNameLog'));
 $table->set_header(1, get_lang('GradebookDescriptionLog'));

main/gradebook/my_certificates.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
         'actions',
         Display::toolbarButton(
             get_lang('SearchCertificates'),
-            api_get_path(WEB_CODE_PATH) . "gradebook/search.php",
+            api_get_path(WEB_CODE_PATH)."gradebook/search.php",
             'search',
             'info'
         )

main/gradebook/gradebook_edit_link.php

Spacing +11 added lines, -11 removed lines

@@ -11,20 +11,20 @@ 
 GradebookUtils::block_students();
 $tbl_grade_links = Database :: get_main_table(TABLE_MAIN_GRADEBOOK_LINK);
 //selected name of database
-$course_id              = GradebookUtils::get_course_id_by_link_id($_GET['editlink']);
-$tbl_forum_thread 		= Database :: get_course_table(TABLE_FORUM_THREAD);
-$tbl_work 				= Database :: get_course_table(TABLE_STUDENT_PUBLICATION);
-$tbl_attendance 		= Database :: get_course_table(TABLE_ATTENDANCE);
+$course_id = GradebookUtils::get_course_id_by_link_id($_GET['editlink']);
+$tbl_forum_thread = Database :: get_course_table(TABLE_FORUM_THREAD);
+$tbl_work = Database :: get_course_table(TABLE_STUDENT_PUBLICATION);
+$tbl_attendance = Database :: get_course_table(TABLE_ATTENDANCE);
 
-$linkarray 				= LinkFactory :: load($_GET['editlink']);
+$linkarray = LinkFactory :: load($_GET['editlink']);
 /** @var AbstractLink $link */
 $link = $linkarray[0];
 if ($link->is_locked() && !api_is_platform_admin()) {
     api_not_allowed();
 }
 
-$linkcat  = isset($_GET['selectcat']) ? Security::remove_XSS($_GET['selectcat']):'';
-$linkedit = isset($_GET['editlink']) ? Security::remove_XSS($_GET['editlink']):'';
+$linkcat  = isset($_GET['selectcat']) ? Security::remove_XSS($_GET['selectcat']) : '';
+$linkedit = isset($_GET['editlink']) ? Security::remove_XSS($_GET['editlink']) : '';
 
 $session_id = api_get_session_id();
 if ($session_id == 0) {
@@ -39,7 +39,7 @@ 
     null,
     $link,
     'edit_link_form',
-    api_get_self() . '?selectcat=' . $linkcat. '&editlink=' . $linkedit.'&'.api_get_cidreq()
+    api_get_self().'?selectcat='.$linkcat.'&editlink='.$linkedit.'&'.api_get_cidreq()
 );
 if ($form->validate()) {
     $values = $form->exportValues();
@@ -58,7 +58,7 @@ 
     //Update weight for attendance
     $sql = 'SELECT ref_id FROM '.$tbl_grade_links.'
             WHERE id = '.intval($_GET['editlink']).' AND type='.LINK_ATTENDANCE;
-    $rs_attendance  = Database::query($sql);
+    $rs_attendance = Database::query($sql);
     if (Database::num_rows($rs_attendance) > 0) {
         $row_attendance = Database::fetch_array($rs_attendance);
         $attendance_id  = $row_attendance['ref_id'];
@@ -81,11 +81,11 @@ 
               weight='.$final_weight.'
 			  WHERE c_id = '.$course_id.' AND id = (
 			    SELECT ref_id FROM '.$tbl_grade_links.'
-			    WHERE id='.intval($_GET['editlink'] ).' AND type=3
+			    WHERE id='.intval($_GET['editlink']).' AND type=3
               )';
 
     Database::query($sql_t);
-    header('Location: '.$_SESSION['gradebook_dest'].'?linkedited=&selectcat=' . $link->get_category_id().'&'.api_get_cidreq());
+    header('Location: '.$_SESSION['gradebook_dest'].'?linkedited=&selectcat='.$link->get_category_id().'&'.api_get_cidreq());
     exit;
 }
 

main/gradebook/gradebook_add_user.php

Spacing +19 added lines, -19 removed lines

@@ -14,59 +14,59 @@
 api_block_anonymous_users();
 GradebookUtils::block_students();
 
-$evaluation= Evaluation :: load($_GET['selecteval']);
+$evaluation = Evaluation :: load($_GET['selecteval']);
 $newstudents = $evaluation[0]->get_not_subscribed_students();
 
 if (count($newstudents) == '0') {
-	header('Location: gradebook_view_result.php?nouser=&selecteval=' . Security::remove_XSS($_GET['selecteval']).'&'.api_get_cidreq());
+	header('Location: gradebook_view_result.php?nouser=&selecteval='.Security::remove_XSS($_GET['selecteval']).'&'.api_get_cidreq());
 	exit;
 }
-$add_user_form= new EvalForm(EvalForm :: TYPE_ADD_USERS_TO_EVAL,
+$add_user_form = new EvalForm(EvalForm :: TYPE_ADD_USERS_TO_EVAL,
 							 $evaluation[0],
 							 null,
 							 'add_users_to_evaluation',
 							 null,
-							 api_get_self() . '?selecteval=' . Security::remove_XSS($_GET['selecteval']),
+							 api_get_self().'?selecteval='.Security::remove_XSS($_GET['selecteval']),
 							 Security::remove_XSS($_GET['firstletter']),
 							 $newstudents);
 
-if ( isset($_POST['submit_button']) ) {
-	$users= is_array($_POST['add_users']) ? $_POST['add_users'] : array ();
-	foreach ($users as $key => $value){
-		$users[$key]= intval($value);
+if (isset($_POST['submit_button'])) {
+	$users = is_array($_POST['add_users']) ? $_POST['add_users'] : array();
+	foreach ($users as $key => $value) {
+		$users[$key] = intval($value);
 	}
 
 	if (count($users) == 0) {
-		header('Location: ' . api_get_self() . '?erroroneuser=&selecteval=' .Security::remove_XSS($_GET['selecteval']));
+		header('Location: '.api_get_self().'?erroroneuser=&selecteval='.Security::remove_XSS($_GET['selecteval']));
 		exit;
 	} else {
 		foreach ($users as $user_id) {
-			$result= new Result();
+			$result = new Result();
 			$result->set_user_id($user_id);
 			$result->set_evaluation_id($_GET['selecteval']);
 			$result->add();
 			}
 		}
-	header('Location: gradebook_view_result.php?adduser=&selecteval=' .Security::remove_XSS($_GET['selecteval']).'&'.api_get_cidreq());
+	header('Location: gradebook_view_result.php?adduser=&selecteval='.Security::remove_XSS($_GET['selecteval']).'&'.api_get_cidreq());
 	exit;
 	} elseif ($_POST['firstLetterUser']) {
-		$firstletter= $_POST['firstLetterUser'];
+		$firstletter = $_POST['firstLetterUser'];
 		if (!empty ($firstletter)) {
-			header('Location: ' . api_get_self() . '?firstletter=' . Security::remove_XSS($firstletter) . '&selecteval=' . Security::remove_XSS($_GET['selecteval']));
+			header('Location: '.api_get_self().'?firstletter='.Security::remove_XSS($firstletter).'&selecteval='.Security::remove_XSS($_GET['selecteval']));
 			exit;
 		}
 }
 
-$interbreadcrumb[]= array ('url' => Security::remove_XSS($_SESSION['gradebook_dest']),'name' => get_lang('Gradebook'));
-$interbreadcrumb[]= array(
-	'url' => 'gradebook_view_result.php?selecteval=' .Security::remove_XSS($_GET['selecteval']).'&'.api_get_cidreq(),
+$interbreadcrumb[] = array('url' => Security::remove_XSS($_SESSION['gradebook_dest']), 'name' => get_lang('Gradebook'));
+$interbreadcrumb[] = array(
+	'url' => 'gradebook_view_result.php?selecteval='.Security::remove_XSS($_GET['selecteval']).'&'.api_get_cidreq(),
 	'name' => get_lang('ViewResult')
 );
 Display :: display_header(get_lang('AddUserToEval'));
-if (isset ($_GET['erroroneuser'])){
-	Display :: display_warning_message(get_lang('AtLeastOneUser'),false);
+if (isset ($_GET['erroroneuser'])) {
+	Display :: display_warning_message(get_lang('AtLeastOneUser'), false);
 }
-DisplayGradebook :: display_header_result($evaluation[0], null, 0,0);
+DisplayGradebook :: display_header_result($evaluation[0], null, 0, 0);
 echo '<div class="main">';
 echo $add_user_form->toHtml();
 echo '</div>';

main/gradebook/user_stats.php

Spacing +13 added lines, -13 removed lines

@@ -16,7 +16,7 @@ 
 if (!$isDrhOfCourse) {
     GradebookUtils::block_students();
 }
-$interbreadcrumb[]= array (
+$interbreadcrumb[] = array(
     'url' => $_SESSION['gradebook_dest'],
     'name' => get_lang('Gradebook'
 ));
@@ -27,12 +27,12 @@ 
 $alllinks = $category[0]->get_links($my_user_id, true);
 
 if ($_GET['selectcat'] != null) {
-    $addparams= array (
+    $addparams = array(
         'userid' => $my_user_id,
         'selectcat' => Security::remove_XSS($_GET['selectcat'])
     );
 } else {
-    $addparams= array (
+    $addparams = array(
         'userid' => $my_user_id,
         'selecteval' => Security::remove_XSS($_GET['selecteval'])
     );
@@ -49,7 +49,7 @@ 
         $newarray[] = array_slice($data, 1);
     }
     $userinfo = api_get_user_info($my_user_id);
-    $html .= get_lang('Results').' : '.api_get_person_name($userinfo['firstname'], $userinfo['lastname']).' ('. api_convert_and_format_date(null, DATE_FORMAT_SHORT). ' ' . api_convert_and_format_date(null, TIME_NO_SEC_FORMAT) .')';
+    $html .= get_lang('Results').' : '.api_get_person_name($userinfo['firstname'], $userinfo['lastname']).' ('.api_convert_and_format_date(null, DATE_FORMAT_SHORT).' '.api_convert_and_format_date(null, TIME_NO_SEC_FORMAT).')';
 
     if ($displayscore->is_custom()) {
         $header_names = array(
@@ -95,23 +95,23 @@ 
 $actions = '<div class="actions">';
 
 if (isset($_GET['selectcat'])) {
-    $interbreadcrumb[]= array ('url' => 'gradebook_flatview.php?selectcat=' . Security::remove_XSS($_GET['selectcat']), 'name' => get_lang('FlatView'));
-    $actions.= '<a href=gradebook_flatview.php?selectcat=' .Security::remove_XSS($_GET['selectcat']) . '>' . Display::return_icon('back.png', get_lang('BackTo').' '.get_lang('FlatView'),'',ICON_SIZE_MEDIUM).'</a>';
+    $interbreadcrumb[] = array('url' => 'gradebook_flatview.php?selectcat='.Security::remove_XSS($_GET['selectcat']), 'name' => get_lang('FlatView'));
+    $actions .= '<a href=gradebook_flatview.php?selectcat='.Security::remove_XSS($_GET['selectcat']).'>'.Display::return_icon('back.png', get_lang('BackTo').' '.get_lang('FlatView'), '', ICON_SIZE_MEDIUM).'</a>';
 }
 
 if (isset ($_GET['selecteval'])) {
-    $interbreadcrumb[]= array (
-        'url' => 'gradebook_view_result.php?selecteval=' . Security::remove_XSS($_GET['selecteval']),
+    $interbreadcrumb[] = array(
+        'url' => 'gradebook_view_result.php?selecteval='.Security::remove_XSS($_GET['selecteval']),
         'name' => get_lang('ViewResult'
     ));
-    $actions.= '<a href=gradebook_view_result.php?selecteval=' . Security::remove_XSS($_GET['selecteval']) . '>
-	'.Display::return_icon('back.png', get_lang('BackToEvaluation'),'',ICON_SIZE_MEDIUM).'</a>';
+    $actions .= '<a href=gradebook_view_result.php?selecteval='.Security::remove_XSS($_GET['selecteval']).'>
+	'.Display::return_icon('back.png', get_lang('BackToEvaluation'), '', ICON_SIZE_MEDIUM).'</a>';
 }
 
-$actions.= '<a href="' . api_get_self() . '?exportpdf=&userid='.Security::remove_XSS($_GET['userid']).'&selectcat=' . $category[0]->get_id() . '" target="_blank">
-' . Display::return_icon('pdf.png', get_lang('ExportPDF'),'',ICON_SIZE_MEDIUM).'</a>';
+$actions .= '<a href="'.api_get_self().'?exportpdf=&userid='.Security::remove_XSS($_GET['userid']).'&selectcat='.$category[0]->get_id().'" target="_blank">
+' . Display::return_icon('pdf.png', get_lang('ExportPDF'), '', ICON_SIZE_MEDIUM).'</a>';
 
-$actions.='</div>';
+$actions .= '</div>';
 
 Display :: display_header(get_lang('ResultsPerUser'));
 echo $actions;

main/gradebook/gradebook_flatview.php

Spacing +6 added lines, -6 removed lines

@@ -6,7 +6,7 @@ 
  * @package chamilo.gradebook
  */
 require_once '../inc/global.inc.php';
-$current_course_tool  = TOOL_GRADEBOOK;
+$current_course_tool = TOOL_GRADEBOOK;
 
 api_protect_course_script(true);
 
@@ -72,7 +72,7 @@ 
     null,
     'simple_search_form',
     null,
-    api_get_self() . '?selectcat=' . $category_id
+    api_get_self().'?selectcat='.$category_id
 );
 $values = $simple_search_form->exportValues();
 
@@ -146,9 +146,9 @@ 
     }
 }
 
-if (isset($_GET['exportpdf']))	{
-    $interbreadcrumb[] = array (
-        'url' => api_get_self().'?selectcat=' . Security::remove_XSS($_GET['selectcat']).'&'.api_get_cidreq(),
+if (isset($_GET['exportpdf'])) {
+    $interbreadcrumb[] = array(
+        'url' => api_get_self().'?selectcat='.Security::remove_XSS($_GET['selectcat']).'&'.api_get_cidreq(),
         'name' => get_lang('FlatView')
     );
 
@@ -187,7 +187,7 @@ 
     }
 }
 
-if (isset($_GET['print']))	{
+if (isset($_GET['print'])) {
     $printable_data = GradebookUtils::get_printable_data(
         $cat[0],
         $users,

main/gradebook/get_badges.php

Spacing +3 added lines, -3 removed lines

@@ -33,7 +33,7 @@ 
 foreach ($userSkills as $skill) {
     $skillId = current($skill);
 
-    $assertionUrl = api_get_path(WEB_CODE_PATH) . "badge/assertion.php?";
+    $assertionUrl = api_get_path(WEB_CODE_PATH)."badge/assertion.php?";
     $assertionUrl .= http_build_query(array(
         'user' => $userId,
         'skill' => $skillId,
@@ -51,14 +51,14 @@ 
     $backpack = $configBackpack;
 }
 
-$htmlHeadXtra[] = '<script src="' . $backpack . 'issuer.js"></script>';
+$htmlHeadXtra[] = '<script src="'.$backpack.'issuer.js"></script>';
 
 $tpl = new Template(get_lang('Badges'), false, false);
 
 $tpl->assign(
     'content',
     "<script>"
-    . "$(document).on('ready', function (){ OpenBadges.issue_no_modal(" . json_encode($assertions) . "); });"
+    . "$(document).on('ready', function (){ OpenBadges.issue_no_modal(".json_encode($assertions)."); });"
     . "</script>"
 );