chamilo / chamilo-lms

main/gradebook/gradebook_view_result.php

Spacing +32 added lines, -32 removed lines

@@ -62,7 +62,7 @@ 
         $resultedit[0],
         'edit_result_form',
         null,
-        api_get_self() . '?editres=' . $resultedit[0]->get_id() . '&selecteval=' . $select_eval.'&'.api_get_cidreq()
+        api_get_self().'?editres='.$resultedit[0]->get_id().'&selecteval='.$select_eval.'&'.api_get_cidreq()
     );
 
     if ($edit_res_form->validate()) {
@@ -79,21 +79,21 @@ 
         }
         $result->save();
         unset($result);
-        header('Location: gradebook_view_result.php?selecteval=' . $select_eval . '&editresmessage=&'.api_get_cidreq());
+        header('Location: gradebook_view_result.php?selecteval='.$select_eval.'&editresmessage=&'.api_get_cidreq());
         exit;
     }
 }
 $file_type = null;
 if (isset($_GET['import'])) {
     $interbreadcrumb[] = array(
-        'url' => api_get_path(WEB_CODE_PATH).'gradebook/gradebook_view_result.php?selecteval=' . $select_eval.'&'.api_get_cidreq(),
+        'url' => api_get_path(WEB_CODE_PATH).'gradebook/gradebook_view_result.php?selecteval='.$select_eval.'&'.api_get_cidreq(),
         'name' => get_lang('ViewResult')
     );
     $import_result_form = new DataForm(
         DataForm :: TYPE_IMPORT,
         'import_result_form',
         null,
-        api_get_self() . '?import=&selecteval=' . $select_eval,
+        api_get_self().'?import=&selecteval='.$select_eval,
         '_blank',
         ''
     );
@@ -122,7 +122,7 @@ 
                     if (($importedresult['user_id'] == $allresult->get_user_id())) {
                         if ($importedresult['score'] != $allresult->get_score()) {
                             if (!isset($values['overwrite'])) {
-                                header('Location: '.api_get_path(WEB_CODE_PATH).'gradebook/gradebook_view_result.php?selecteval=' . $select_eval . '&import_score_error=' . $importedresult['user_id']);
+                                header('Location: '.api_get_path(WEB_CODE_PATH).'gradebook/gradebook_view_result.php?selecteval='.$select_eval.'&import_score_error='.$importedresult['user_id']);
                                 exit;
                                 break;
                             } else {
@@ -140,7 +140,7 @@ 
                     }
                 }
                 if ($importedresult['user_id'] == null) {
-                    header('Location: gradebook_view_result.php?selecteval=' . $select_eval . '&incorrectdata=');
+                    header('Location: gradebook_view_result.php?selecteval='.$select_eval.'&incorrectdata=');
                     exit;
                 }
                 $userinfo = api_get_user_info($importedresult['user_id']);
@@ -149,13 +149,13 @@ 
                     $userinfo['official_code'] != $importedresult['official_code']
                 ) {
                     if (!isset($values['ignoreerrors'])) {
-                        header('Location: gradebook_view_result.php?selecteval=' . $select_eval . '&import_user_error=' . $importedresult['user_id']);
+                        header('Location: gradebook_view_result.php?selecteval='.$select_eval.'&import_user_error='.$importedresult['user_id']);
                         exit;
                     }
                 }
                 if ($added != '1') {
                     if ($importedresult['score'] > $eval[0]->get_max()) {
-                        header('Location: gradebook_view_result.php?selecteval=' . $select_eval . '&overwritemax=');
+                        header('Location: gradebook_view_result.php?selecteval='.$select_eval.'&overwritemax=');
                         exit;
                     }
                     $result = new Result();
@@ -174,18 +174,18 @@ 
                 }
             }
         } else {
-            header('Location: ' . api_get_self() . '?import=&selecteval=' . $select_eval . '&importnofile=');
+            header('Location: '.api_get_self().'?import=&selecteval='.$select_eval.'&importnofile=');
             exit;
         }
         if ($overwritescore != 0) {
-            header('Location: ' . api_get_self() . '?selecteval=' . $select_eval. '&importoverwritescore=' . $overwritescore);
+            header('Location: '.api_get_self().'?selecteval='.$select_eval.'&importoverwritescore='.$overwritescore);
             exit;
         }
         if ($nr_results_added == 0) {
-            header('Location: ' . api_get_self() . '?selecteval=' . $select_eval. '&nothingadded=');
+            header('Location: '.api_get_self().'?selecteval='.$select_eval.'&nothingadded=');
             exit;
         }
-        header('Location: ' . api_get_self() . '?selecteval=' . $select_eval . '&importok=');
+        header('Location: '.api_get_self().'?selecteval='.$select_eval.'&importok=');
         exit;
     }
 }
@@ -200,7 +200,7 @@ 
         DataForm :: TYPE_EXPORT,
         'export_result_form',
         null,
-        api_get_self() . '?export=&selecteval=' . $select_eval,
+        api_get_self().'?export=&selecteval='.$select_eval,
         '_blank',
         $locked_status
     );
@@ -211,7 +211,7 @@ 
     if ($export_result_form->validate()) {
         $export = $export_result_form->exportValues();
         $file_type = $export['file_type'];
-        $filename = 'export_results_' . gmdate('Y-m-d_H-i-s');
+        $filename = 'export_results_'.gmdate('Y-m-d_H-i-s');
         $results = Result :: load(null, null, $select_eval);
         $data = array(); //when file type is csv, add a header to the output file
         if ($file_type == 'csv') {
@@ -231,9 +231,9 @@ 
             $datagen = new ResultsDataGenerator($eval[0], $allresults);
 
             // set headers pdf
-            !empty($_user['official_code']) ? $officialcode = $_user['official_code'] . ' - ' : '';
+            !empty($_user['official_code']) ? $officialcode = $_user['official_code'].' - ' : '';
 
-            $h1 = array(get_lang('Teacher'), $officialcode . $_user['firstName'] . ', ' . $_user['lastName']);
+            $h1 = array(get_lang('Teacher'), $officialcode.$_user['firstName'].', '.$_user['lastName']);
             $h2 = array(get_lang('Score'), $eval[0]->get_max());
             $h3 = array(get_lang('Course'), $_course['name']);
             $h4 = array(get_lang('Weight'), $eval[0]->get_weight());
@@ -243,9 +243,9 @@ 
             $header_pdf = array($h1, $h2, $h3, $h4, $h5, $h6);
 
             // set footer pdf
-            $f1 = '<hr />' . get_lang('Drh');
-            $f2 = '<hr />' . get_lang('Teacher');
-            $f3 = '<hr />' . get_lang('Date');
+            $f1 = '<hr />'.get_lang('Drh');
+            $f2 = '<hr />'.get_lang('Teacher');
+            $f3 = '<hr />'.get_lang('Date');
             $footer_pdf = array($f1, $f2, $f3);
 
             // set title pdf
@@ -254,9 +254,9 @@ 
             // set headers data table
             $head_ape_name = '';
             if (api_is_western_name_order()) {
-                $head_ape_name = get_lang('FirstName') . ', ' . get_lang('LastName');
+                $head_ape_name = get_lang('FirstName').', '.get_lang('LastName');
             } else {
-                $head_ape_name = get_lang('LastName') . ', ' . get_lang('FirstName');
+                $head_ape_name = get_lang('LastName').', '.get_lang('FirstName');
             }
 
             $head_table = array(
@@ -291,15 +291,15 @@ 
                 $result[] = $user_info['username'];
 
                 if (api_is_western_name_order()) {
-                    $result[] = $user_info['firstname'] . ', ' . $user_info['lastname'];
+                    $result[] = $user_info['firstname'].', '.$user_info['lastname'];
                 } else {
-                    $result[] = $user_info['lastname'] . ', ' . $user_info['firstname'];
+                    $result[] = $user_info['lastname'].', '.$user_info['firstname'];
                 }
                 if ($number_decimals == null) {
                     if (empty($data['scoreletter']) && !is_numeric($data['score'])) {
                         $result[] = get_lang('DidNotTakeTheExam');
                     } else {
-                        $result[] = api_strtoupper(get_lang('Literal' . $data['scoreletter']));
+                        $result[] = api_strtoupper(get_lang('Literal'.$data['scoreletter']));
                     }
                 } else {
                     if (empty($data['score']) && !is_numeric($data['score'])) {
@@ -350,7 +350,7 @@ 
 if (isset($_GET['resultdelete'])) {
     $result = Result :: load($_GET['resultdelete']);
     $result[0]->delete();
-    header('Location: gradebook_view_result.php?deleteresult=&selecteval=' . $select_eval);
+    header('Location: gradebook_view_result.php?deleteresult=&selecteval='.$select_eval);
     exit;
 }
 
@@ -367,7 +367,7 @@ 
                     $result[0]->delete();
                     $number_of_deleted_results++;
                 }
-                header('Location: gradebook_view_result.php?massdelete=&selecteval=' . $select_eval);
+                header('Location: gradebook_view_result.php?massdelete=&selecteval='.$select_eval);
                 exit;
                 break;
         }
@@ -408,14 +408,14 @@ 
 
 $htmlHeadXtra[] = '<script>
 function confirmationuser() {
-    if (confirm("' . get_lang('DeleteUser') . '?"))
+    if (confirm("' . get_lang('DeleteUser').'?"))
     	{return true;}
     else
     	{return false;}
 }
 
 function confirmationall () {
-    if (confirm("' . get_lang('DeleteAll') . '?"))
+    if (confirm("' . get_lang('DeleteAll').'?"))
     	{return true;}
     else
     	{return false;}
@@ -423,13 +423,13 @@ 
 </script>';
 if (isset($_GET['deleteall'])) {
     $eval[0]->delete_results();
-    header('Location: '.api_get_path(WEB_CODE_PATH).'gradebook/gradebook_view_result.php?allresdeleted=&selecteval=' . $select_eval.'&'.api_get_cidreq());
+    header('Location: '.api_get_path(WEB_CODE_PATH).'gradebook/gradebook_view_result.php?allresdeleted=&selecteval='.$select_eval.'&'.api_get_cidreq());
     exit;
 }
 if (!isset($_GET['export']) && (!isset($_GET['import']))) {
     if (!isset($_GET['selectcat'])) {
         $interbreadcrumb[] = array(
-            'url' => $_SESSION['gradebook_dest'] . '?selectcat=' . $currentcat[0]->get_id().'&'.api_get_cidreq(),
+            'url' => $_SESSION['gradebook_dest'].'?selectcat='.$currentcat[0]->get_id().'&'.api_get_cidreq(),
             'name' => get_lang('Details')
         );
     }
@@ -484,7 +484,7 @@ 
     Display :: display_warning_message(get_lang('OverWriteMax'), false);
 }
 if (isset($_GET['importoverwritescore'])) {
-    Display :: display_confirmation_message(get_lang('ImportOverWriteScore') . ' ' . $_GET['importoverwritescore']);
+    Display :: display_confirmation_message(get_lang('ImportOverWriteScore').' '.$_GET['importoverwritescore']);
 }
 if (isset($_GET['import_user_error'])) {
     $userinfo = api_get_user_info($_GET['import_user_error']);
@@ -497,7 +497,7 @@ 
 }
 if (isset($_GET['import_score_error'])) {
     $userinfo = api_get_user_info($_GET['import_score_error']);
-    Display :: display_warning_message(get_lang('ScoreDoesNotMatch') . ' ' . api_get_person_name($userinfo['firstname'], $userinfo['lastname']));
+    Display :: display_warning_message(get_lang('ScoreDoesNotMatch').' '.api_get_person_name($userinfo['firstname'], $userinfo['lastname']));
 }
 if ($file_type == null) {
     //show the result header

main/gradebook/certificate_report.php

Spacing +4 added lines, -4 removed lines

@@ -17,7 +17,7 @@ 
 api_block_anonymous_users();
 
 $interbreadcrumb[] = array(
-    "url" => api_get_path(WEB_CODE_PATH) . "mySpace/index.php?".api_get_cidreq(),
+    "url" => api_get_path(WEB_CODE_PATH)."mySpace/index.php?".api_get_cidreq(),
     "name" => get_lang("MySpace")
 );
 
@@ -111,7 +111,7 @@ 
     }
 
     if (!is_null($gradebook)) {
-        $exportAllLink = api_get_path(WEB_CODE_PATH) . "gradebook/gradebook_display_certificate.php?";
+        $exportAllLink = api_get_path(WEB_CODE_PATH)."gradebook/gradebook_display_certificate.php?";
         $exportAllLink .= http_build_query(array(
             "action" => "export_all_certificates",
             "cidReq" => $selectedCourseInfo['code'],
@@ -250,7 +250,7 @@ 
 $searchBySessionCourseDateForm = new FormValidator(
     'certificate_report_form',
     'post',
-    api_get_path(WEB_CODE_PATH) . 'gradebook/certificate_report.php'
+    api_get_path(WEB_CODE_PATH).'gradebook/certificate_report.php'
 );
 $searchBySessionCourseDateForm->addSelect('session', get_lang('Sessions'), $sessions, ['id' => 'session']);
 $searchBySessionCourseDateForm->addSelect('course', get_lang('Courses'), $courses, ['id' => 'course']);
@@ -283,7 +283,7 @@ 
     $searchByStudentForm = new FormValidator(
         'certificate_report_form',
         'post',
-        api_get_path(WEB_CODE_PATH) . 'gradebook/certificate_report.php'
+        api_get_path(WEB_CODE_PATH).'gradebook/certificate_report.php'
     );
     $searchByStudentForm->addSelect('student', get_lang('Students'), $students, ['id' => 'student']);
     $searchByStudentForm->addButtonSearch();

main/gradebook/gradebook.php

Spacing +125 added lines, -125 removed lines

@@ -6,7 +6,7 @@ 
  */
 // $cidReset : This is the main difference with gradebook.php, here we say,
 // basically, that we are inside a course, and many things depend from that
-$cidReset= true;
+$cidReset = true;
 $_in_course = false;
 //make sure the destination for scripts is index.php instead of gradebook.php
 require_once '../inc/global.inc.php';
@@ -30,7 +30,7 @@ 
  </script>';
 api_block_anonymous_users();
 
-$htmlHeadXtra[]= '<script type="text/javascript">
+$htmlHeadXtra[] = '<script type="text/javascript">
 function confirmation ()
 {
 	if (confirm("'.get_lang('DeleteAll').'?")) {
@@ -47,10 +47,10 @@ 
 //this is called when there is no data for the course admin
 if (isset ($_GET['createallcategories'])) {
     GradebookUtils::block_students();
-    $coursecat= Category :: get_not_created_course_categories(api_get_user_id());
+    $coursecat = Category :: get_not_created_course_categories(api_get_user_id());
     if (!count($coursecat) == 0) {
         foreach ($coursecat as $row) {
-            $cat= new Category();
+            $cat = new Category();
             $cat->set_name($row[1]);
             $cat->set_course_code($row[0]);
             $cat->set_description(null);
@@ -66,33 +66,33 @@ 
     exit;
 }
 //move a category
-$selectcat=isset($_GET['selectcat']) ?  Security::remove_XSS($_GET['selectcat']) : '';
+$selectcat = isset($_GET['selectcat']) ? Security::remove_XSS($_GET['selectcat']) : '';
 
 if (isset ($_GET['movecat'])) {
     $move_cat = Security::remove_XSS($_GET['movecat']);
     GradebookUtils::block_students();
-    $cats= Category :: load($move_cat);
+    $cats = Category :: load($move_cat);
     if (!isset ($_GET['targetcat'])) {
-        $move_form= new CatForm(CatForm :: TYPE_MOVE,
+        $move_form = new CatForm(CatForm :: TYPE_MOVE,
             $cats[0],
             'move_cat_form',
             null,
-            api_get_self() . '?movecat=' . $move_cat
-            . '&selectcat=' . Security::remove_XSS($_GET['selectcat']));
+            api_get_self().'?movecat='.$move_cat
+            . '&selectcat='.Security::remove_XSS($_GET['selectcat']));
         if ($move_form->validate()) {
-            header('Location: ' . api_get_self() . '?selectcat=' . Security::remove_XSS($_GET['selectcat'])
-                . '&movecat=' . $move_cat
-                . '&targetcat=' . $move_form->exportValue('move_cat'));
+            header('Location: '.api_get_self().'?selectcat='.Security::remove_XSS($_GET['selectcat'])
+                . '&movecat='.$move_cat
+                . '&targetcat='.$move_form->exportValue('move_cat'));
             exit;
         }
     } else {
-        $get_target_cat=Security::remove_XSS($_GET['targetcat']);
-        $targetcat= Category :: load($get_target_cat);
+        $get_target_cat = Security::remove_XSS($_GET['targetcat']);
+        $targetcat = Category :: load($get_target_cat);
         $course_to_crsind = ($cats[0]->get_course_code() != null && $targetcat[0]->get_course_code() == null);
 
         if (!($course_to_crsind && !isset($_GET['confirm']))) {
             $cats[0]->move_to_cat($targetcat[0]);
-            header('Location: ' . api_get_self() . '?categorymoved=&selectcat=' . Security::remove_XSS($_GET['selectcat']));
+            header('Location: '.api_get_self().'?categorymoved=&selectcat='.Security::remove_XSS($_GET['selectcat']));
             exit;
         }
         unset($targetcat);
@@ -103,32 +103,32 @@ 
 //move an evaluation
 if (isset ($_GET['moveeval'])) {
     GradebookUtils::block_students();
-    $get_move_eval=Security::remove_XSS($_GET['moveeval']);
-    $evals= Evaluation :: load($get_move_eval);
+    $get_move_eval = Security::remove_XSS($_GET['moveeval']);
+    $evals = Evaluation :: load($get_move_eval);
     if (!isset ($_GET['targetcat'])) {
 
-        $move_form= new EvalForm(EvalForm :: TYPE_MOVE,
+        $move_form = new EvalForm(EvalForm :: TYPE_MOVE,
             $evals[0],
             null,
             'move_eval_form',
             null,
-            api_get_self() . '?moveeval=' . $get_move_eval
-            . '&selectcat=' . Security::remove_XSS($_GET['selectcat']));
+            api_get_self().'?moveeval='.$get_move_eval
+            . '&selectcat='.Security::remove_XSS($_GET['selectcat']));
 
         if ($move_form->validate()) {
-            header('Location: ' .api_get_self() . '?selectcat=' . Security::remove_XSS($_GET['selectcat'])
-                . '&moveeval=' . $get_move_eval
-                . '&targetcat=' . $move_form->exportValue('move_cat'));
+            header('Location: '.api_get_self().'?selectcat='.Security::remove_XSS($_GET['selectcat'])
+                . '&moveeval='.$get_move_eval
+                . '&targetcat='.$move_form->exportValue('move_cat'));
             exit;
         }
     } else {
-        $get_target_cat=Security::remove_XSS($_GET['targetcat']);
-        $targetcat= Category :: load($get_target_cat);
+        $get_target_cat = Security::remove_XSS($_GET['targetcat']);
+        $targetcat = Category :: load($get_target_cat);
         $course_to_crsind = ($evals[0]->get_course_code() != null && $targetcat[0]->get_course_code() == null);
 
         if (!($course_to_crsind && !isset($_GET['confirm']))) {
             $evals[0]->move_to_cat($targetcat[0]);
-            header('Location: ' . api_get_self() . '?evaluationmoved=&selectcat=' . Security::remove_XSS($_GET['selectcat']));
+            header('Location: '.api_get_self().'?evaluationmoved=&selectcat='.Security::remove_XSS($_GET['selectcat']));
             exit;
         }
         unset ($targetcat);
@@ -139,8 +139,8 @@ 
 //move a link
 if (isset ($_GET['movelink'])) {
     GradebookUtils::block_students();
-    $get_move_link=Security::remove_XSS($_GET['movelink']);
-    $link= LinkFactory :: load($get_move_link);
+    $get_move_link = Security::remove_XSS($_GET['movelink']);
+    $link = LinkFactory :: load($get_move_link);
     $move_form = new LinkForm(
         LinkForm :: TYPE_MOVE,
         null,
@@ -150,10 +150,10 @@ 
         api_get_self().'?movelink='.$get_move_link.'&selectcat='.Security::remove_XSS($_GET['selectcat'])
     );
     if ($move_form->validate()) {
-        $targetcat= Category :: load($move_form->exportValue('move_cat'));
+        $targetcat = Category :: load($move_form->exportValue('move_cat'));
         $link[0]->move_to_cat($targetcat[0]);
         unset ($link);
-        header('Location: ' . api_get_self(). '?linkmoved=&selectcat=' . Security::remove_XSS($_GET['selectcat']));
+        header('Location: '.api_get_self().'?linkmoved=&selectcat='.Security::remove_XSS($_GET['selectcat']));
         exit;
     }
 }
@@ -162,11 +162,11 @@ 
 if (isset ($_GET['visiblecat'])) {
     GradebookUtils::block_students();
     if (isset ($_GET['set_visible'])) {
-        $visibility_command= 1;
+        $visibility_command = 1;
     } else {
-        $visibility_command= 0;
+        $visibility_command = 0;
     }
-    $cats= Category :: load($_GET['visiblecat']);
+    $cats = Category :: load($_GET['visiblecat']);
     $cats[0]->set_visible($visibility_command);
     $cats[0]->save();
     $cats[0]->apply_visibility_to_children();
@@ -181,7 +181,7 @@ 
 }
 if (isset ($_GET['deletecat'])) {
     GradebookUtils::block_students();
-    $cats= Category :: load($_GET['deletecat']);
+    $cats = Category :: load($_GET['deletecat']);
     //delete all categories,subcategories and results
     if ($cats[0] != null) {
         if ($cats[0]->get_id() != 0) {
@@ -196,12 +196,12 @@ 
 if (isset ($_GET['visibleeval'])) {
     GradebookUtils::block_students();
     if (isset ($_GET['set_visible'])) {
-        $visibility_command= 1;
+        $visibility_command = 1;
     } else {
-        $visibility_command= 0;
+        $visibility_command = 0;
     }
 
-    $eval= Evaluation :: load($_GET['visibleeval']);
+    $eval = Evaluation :: load($_GET['visibleeval']);
     $eval[0]->set_visible($visibility_command);
     $eval[0]->save();
     unset ($eval);
@@ -215,7 +215,7 @@ 
 }
 if (isset ($_GET['deleteeval'])) {
     GradebookUtils::block_students();
-    $eval= Evaluation :: load($_GET['deleteeval']);
+    $eval = Evaluation :: load($_GET['deleteeval']);
     if ($eval[0] != null) {
         $eval[0]->delete_with_results();
     }
@@ -226,11 +226,11 @@ 
 if (isset ($_GET['visiblelink'])) {
     GradebookUtils::block_students();
     if (isset ($_GET['set_visible'])) {
-        $visibility_command= 1;
-    }else {
-        $visibility_command= 0;
+        $visibility_command = 1;
+    } else {
+        $visibility_command = 0;
     }
-    $link= LinkFactory :: load($_GET['visiblelink']);
+    $link = LinkFactory :: load($_GET['visiblelink']);
     $link[0]->set_visible($visibility_command);
     $link[0]->save();
     unset ($link);
@@ -246,7 +246,7 @@ 
     GradebookUtils::block_students();
     //fixing #5229
     if (!empty($_GET['deletelink'])) {
-        $link= LinkFactory :: load($_GET['deletelink']);
+        $link = LinkFactory :: load($_GET['deletelink']);
         if ($link[0] != null) {
             $link[0]->delete();
         }
@@ -263,11 +263,11 @@ 
     }
     $button = '<form name="confirm"
 					 method="post"
-					 action="'.api_get_self() .'?confirm='
-        .(isset($_GET['movecat']) ? '&movecat=' . Security::remove_XSS($_GET['movecat'])
-            : '&moveeval=' . Security::remove_XSS($_GET['moveeval']) )
-        .'&selectcat=' . Security::remove_XSS($_GET['selectcat'])
-        .'&targetcat=' . Security::remove_XSS($_GET['targetcat']).'">
+					 action="'.api_get_self().'?confirm='
+        .(isset($_GET['movecat']) ? '&movecat='.Security::remove_XSS($_GET['movecat'])
+            : '&moveeval='.Security::remove_XSS($_GET['moveeval']))
+        .'&selectcat='.Security::remove_XSS($_GET['selectcat'])
+        .'&targetcat='.Security::remove_XSS($_GET['targetcat']).'">
 			   <input type="submit" value="'.'  '.get_lang('Ok').'  '.'">
 			   </form>';
 
@@ -277,7 +277,7 @@ 
 //actions on the sortabletable
 if (isset ($_POST['action'])) {
     GradebookUtils::block_students();
-    $number_of_selected_items= count($_POST['id']);
+    $number_of_selected_items = count($_POST['id']);
     if ($number_of_selected_items == '0') {
         $warning_message = get_lang('NoItemsSelected');
         $filter_warning_msg = false;
@@ -285,19 +285,19 @@ 
     else {
         switch ($_POST['action']) {
             case 'deleted' :
-                $number_of_deleted_categories= 0;
-                $number_of_deleted_evaluations= 0;
-                $number_of_deleted_links= 0;
+                $number_of_deleted_categories = 0;
+                $number_of_deleted_evaluations = 0;
+                $number_of_deleted_links = 0;
                 foreach ($_POST['id'] as $indexstr) {
                     if (api_substr($indexstr, 0, 4) == 'CATE') {
-                        $cats= Category :: load(api_substr($indexstr, 4));
+                        $cats = Category :: load(api_substr($indexstr, 4));
                         if ($cats[0] != null) {
                             $cats[0]->delete_all();
                         }
                         $number_of_deleted_categories++;
                     }
                     if (api_substr($indexstr, 0, 4) == 'EVAL') {
-                        $eval= Evaluation :: load(api_substr($indexstr, 4));
+                        $eval = Evaluation :: load(api_substr($indexstr, 4));
                         if ($eval[0] != null) {
                             $eval[0]->delete_with_results();
                         }
@@ -306,7 +306,7 @@ 
                     if (api_substr($indexstr, 0, 4) == 'LINK') {
                         $id = api_substr($indexstr, 4);
                         if (!empty($id)) {
-                            $link= LinkFactory :: load();
+                            $link = LinkFactory :: load();
                             if ($link[0] != null) {
                                 $link[0]->delete();
                             }
@@ -314,7 +314,7 @@ 
                         }
                     }
                 }
-                $confirmation_message = get_lang('DeletedCategories') . ' : <b>' . $number_of_deleted_categories . '</b><br />' . get_lang('DeletedEvaluations') . ' : <b>' . $number_of_deleted_evaluations . '</b><br />' . get_lang('DeletedLinks') . ' : <b>' . $number_of_deleted_links . '</b><br /><br />' . get_lang('TotalItems') . ' : <b>' . $number_of_selected_items . '</b>';
+                $confirmation_message = get_lang('DeletedCategories').' : <b>'.$number_of_deleted_categories.'</b><br />'.get_lang('DeletedEvaluations').' : <b>'.$number_of_deleted_evaluations.'</b><br />'.get_lang('DeletedLinks').' : <b>'.$number_of_deleted_links.'</b><br /><br />'.get_lang('TotalItems').' : <b>'.$number_of_selected_items.'</b>';
                 $filter_confirm_msg = false;
                 break;
             case 'setvisible' :
@@ -366,7 +366,7 @@ 
 }
 
 if (isset ($_POST['submit']) && isset ($_POST['keyword'])) {
-    header('Location: ' . api_get_self() . '?selectcat=' . Security::remove_XSS($_GET['selectcat'])
+    header('Location: '.api_get_self().'?selectcat='.Security::remove_XSS($_GET['selectcat'])
         . '&search='.Security::remove_XSS($_POST['keyword']));
     exit;
 }
@@ -374,8 +374,8 @@ 
 // DISPLAY HEADERS AND MESSAGES                           -
 if (!isset($_GET['exportpdf']) && !isset($_GET['export_certificate'])) {
     if (isset ($_GET['studentoverview'])) {
-        $interbreadcrumb[]= array (
-            'url' => $_SESSION['gradebook_dest'].'?selectcat=' . Security::remove_XSS($_GET['selectcat'].'&'.api_get_cidreq()),
+        $interbreadcrumb[] = array(
+            'url' => $_SESSION['gradebook_dest'].'?selectcat='.Security::remove_XSS($_GET['selectcat'].'&'.api_get_cidreq()),
             'name' => get_lang('ToolGradebook')
         );
         Display :: display_header(get_lang('FlatView'));
@@ -386,13 +386,13 @@ 
             $gradebook_dest = Security::remove_XSS($_SESSION['gradebook_dest']);
         }
 
-        $interbreadcrumb[]= array ('url' => $gradebook_dest,'name' => get_lang('Gradebook'));
+        $interbreadcrumb[] = array('url' => $gradebook_dest, 'name' => get_lang('Gradebook'));
 
-        if ((isset($_GET['selectcat']) && $_GET['selectcat']>0)) {
+        if ((isset($_GET['selectcat']) && $_GET['selectcat'] > 0)) {
             if (!empty($_GET['course'])) {
-                $interbreadcrumb[]= array ('url' => $gradebook_dest.'selectcat='.Security::remove_XSS($_GET['selectcat']),'name' => get_lang('Details'));
+                $interbreadcrumb[] = array('url' => $gradebook_dest.'selectcat='.Security::remove_XSS($_GET['selectcat']), 'name' => get_lang('Details'));
             } else {
-                $interbreadcrumb[]= array ('url' => $_SESSION['gradebook_dest'].'?selectcat=0','name' => get_lang('Details'));
+                $interbreadcrumb[] = array('url' => $_SESSION['gradebook_dest'].'?selectcat=0', 'name' => get_lang('Details'));
             }
         }
         Display :: display_header('');
@@ -402,56 +402,56 @@ 
 }
 
 if (isset($_GET['categorymoved'])) {
-    Display :: display_confirmation_message(get_lang('CategoryMoved'),false);
+    Display :: display_confirmation_message(get_lang('CategoryMoved'), false);
 }
 if (isset($_GET['evaluationmoved'])) {
-    Display :: display_confirmation_message(get_lang('EvaluationMoved'),false);
+    Display :: display_confirmation_message(get_lang('EvaluationMoved'), false);
 }
 if (isset($_GET['linkmoved'])) {
-    Display :: display_confirmation_message(get_lang('LinkMoved'),false);
+    Display :: display_confirmation_message(get_lang('LinkMoved'), false);
 }
 if (isset ($_GET['addcat'])) {
-    Display :: display_confirmation_message(get_lang('CategoryAdded'),false);
+    Display :: display_confirmation_message(get_lang('CategoryAdded'), false);
 }
 if (isset ($_GET['linkadded'])) {
-    Display :: display_confirmation_message(get_lang('LinkAdded'),false);
+    Display :: display_confirmation_message(get_lang('LinkAdded'), false);
 }
 if (isset ($_GET['addresult'])) {
-    Display :: display_confirmation_message(get_lang('ResultAdded'),false);
+    Display :: display_confirmation_message(get_lang('ResultAdded'), false);
 }
 if (isset ($_GET['editcat'])) {
-    Display :: display_confirmation_message(get_lang('CategoryEdited'),false);
+    Display :: display_confirmation_message(get_lang('CategoryEdited'), false);
 }
 if (isset ($_GET['editeval'])) {
-    Display :: display_confirmation_message(get_lang('EvaluationEdited'),false);
+    Display :: display_confirmation_message(get_lang('EvaluationEdited'), false);
 }
 if (isset ($_GET['linkedited'])) {
-    Display :: display_confirmation_message(get_lang('LinkEdited'),false);
+    Display :: display_confirmation_message(get_lang('LinkEdited'), false);
 }
 if (isset ($_GET['nolinkitems'])) {
-    Display :: display_warning_message(get_lang('NoLinkItems'),false);
+    Display :: display_warning_message(get_lang('NoLinkItems'), false);
 }
 if (isset ($_GET['addallcat'])) {
-    Display :: display_normal_message(get_lang('AddAllCat'),false);
+    Display :: display_normal_message(get_lang('AddAllCat'), false);
 }
 if (isset ($confirmation_message)) {
-    Display :: display_confirmation_message($confirmation_message,$filter_confirm_msg);
+    Display :: display_confirmation_message($confirmation_message, $filter_confirm_msg);
 }
 if (isset ($warning_message)) {
-    Display :: display_warning_message($warning_message,$filter_warning_msg);
+    Display :: display_warning_message($warning_message, $filter_warning_msg);
 }
 if (isset ($move_form)) {
-    Display :: display_normal_message($move_form->toHtml(),false);
+    Display :: display_normal_message($move_form->toHtml(), false);
 }
 // LOAD DATA & DISPLAY TABLE                             -
-$is_platform_admin= api_is_platform_admin();
-$is_course_admin= api_is_allowed_to_edit();
+$is_platform_admin = api_is_platform_admin();
+$is_course_admin = api_is_allowed_to_edit();
 
 //load data for category, evaluation and links
 if (!isset ($_GET['selectcat']) || empty ($_GET['selectcat'])) {
-    $category= 0;
+    $category = 0;
 } else {
-    $category= Security::remove_XSS($_GET['selectcat']);
+    $category = Security::remove_XSS($_GET['selectcat']);
 }
 // search form
 
@@ -462,7 +462,7 @@ 
     null,
     api_get_self().'?selectcat='.$selectcat
 );
-$values= $simple_search_form->exportValues();
+$values = $simple_search_form->exportValues();
 $keyword = '';
 if (isset($_GET['search']) && !empty($_GET['search'])) {
     $keyword = Security::remove_XSS($_GET['search']);
@@ -472,13 +472,13 @@ 
 }
 
 if (!empty($keyword)) {
-    $cats= Category :: load($category);
-    $allcat= array ();
-    if ((isset($_GET['selectcat']) && $_GET['selectcat']==0) && isset($_GET['search'])) {
-        $allcat= $cats[0]->get_subcategories(null);
-        $allcat_info = Category::find_category($keyword,$allcat);
-        $alleval=array();
-        $alllink=array();
+    $cats = Category :: load($category);
+    $allcat = array();
+    if ((isset($_GET['selectcat']) && $_GET['selectcat'] == 0) && isset($_GET['search'])) {
+        $allcat = $cats[0]->get_subcategories(null);
+        $allcat_info = Category::find_category($keyword, $allcat);
+        $alleval = array();
+        $alllink = array();
     } else {
         $alleval = Evaluation::find_evaluations($keyword, $cats[0]->get_id());
         $alllink = LinkFactory::find_links($keyword, $cats[0]->get_id());
@@ -486,13 +486,13 @@ 
 
 } elseif (isset ($_GET['studentoverview'])) {
     //@todo this code seems to be deprecated because the gradebook tab is off
-    $cats= Category :: load($category);
-    $stud_id= (api_is_allowed_to_edit() ? null : api_get_user_id());
-    $allcat= array ();
-    $alleval= $cats[0]->get_evaluations($stud_id, true);
-    $alllink= $cats[0]->get_links($stud_id, true);
+    $cats = Category :: load($category);
+    $stud_id = (api_is_allowed_to_edit() ? null : api_get_user_id());
+    $allcat = array();
+    $alleval = $cats[0]->get_evaluations($stud_id, true);
+    $alllink = $cats[0]->get_links($stud_id, true);
     if (isset ($_GET['exportpdf'])) {
-        $datagen = new GradebookDataGenerator ($allcat,$alleval, $alllink);
+        $datagen = new GradebookDataGenerator($allcat, $alleval, $alllink);
         $header_names = array(
             get_lang('Name'),
             get_lang('Description'),
@@ -500,18 +500,18 @@ 
             get_lang('Date'),
             get_lang('Results'),
         );
-        $data_array = $datagen->get_data(GradebookDataGenerator :: GDG_SORT_NAME,0,null,true);
+        $data_array = $datagen->get_data(GradebookDataGenerator :: GDG_SORT_NAME, 0, null, true);
         $newarray = array();
         foreach ($data_array as $data) {
             $newarray[] = array_slice($data, 1);
         }
-        $pdf= new Cezpdf();
+        $pdf = new Cezpdf();
         $pdf->selectFont(api_get_path(LIBRARY_PATH).'ezpdf/fonts/Courier.afm');
         $pdf->ezSetMargins(30, 30, 50, 30);
         $pdf->ezSetY(810);
-        $pdf->ezText(get_lang('FlatView').' ('. api_convert_and_format_date(null, DATE_FORMAT_SHORT) . ' ' . api_convert_and_format_date(null, TIME_NO_SEC_FORMAT) .')',12,array('justification'=>'center'));
-        $pdf->line(50,790,550,790);
-        $pdf->line(50,40,550,40);
+        $pdf->ezText(get_lang('FlatView').' ('.api_convert_and_format_date(null, DATE_FORMAT_SHORT).' '.api_convert_and_format_date(null, TIME_NO_SEC_FORMAT).')', 12, array('justification'=>'center'));
+        $pdf->line(50, 790, 550, 790);
+        $pdf->line(50, 40, 550, 40);
         $pdf->ezSetY(750);
         $pdf->ezTable(
             $newarray,
@@ -528,71 +528,71 @@ 
         $pdf->ezStream();
         exit;
     }
-} elseif (!empty($_GET['export_certificate'])){
+} elseif (!empty($_GET['export_certificate'])) {
     //@todo this code seems not to be used
     $user_id = strval(intval($_GET['user']));
-    if (!api_is_allowed_to_edit(true,true)) {
+    if (!api_is_allowed_to_edit(true, true)) {
         $user_id = api_get_user_id();
     }
-    $category = Category :: load ($_GET['cat_id']);
+    $category = Category :: load($_GET['cat_id']);
     if ($category[0]->is_certificate_available($user_id)) {
         $user = api_get_user_info($user_id);
         $scoredisplay = ScoreDisplay :: instance();
         $scorecourse = $category[0]->calc_score($user_id);
-        $scorecourse_display = (isset($scorecourse) ? $scoredisplay->display_score($scorecourse,SCORE_AVERAGE) : get_lang('NoResultsAvailable'));
+        $scorecourse_display = (isset($scorecourse) ? $scoredisplay->display_score($scorecourse, SCORE_AVERAGE) : get_lang('NoResultsAvailable'));
 
         $cattotal = Category :: load(0);
-        $scoretotal= $cattotal[0]->calc_score($user_id);
-        $scoretotal_display = (isset($scoretotal) ? $scoredisplay->display_score($scoretotal,SCORE_PERCENT) : get_lang('NoResultsAvailable'));
+        $scoretotal = $cattotal[0]->calc_score($user_id);
+        $scoretotal_display = (isset($scoretotal) ? $scoredisplay->display_score($scoretotal, SCORE_PERCENT) : get_lang('NoResultsAvailable'));
 
         //prepare all necessary variables:
         $organization_name = api_get_setting('Institution');
         $portal_name = api_get_setting('siteName');
         $stud_fn = $user['firstname'];
         $stud_ln = $user['lastname'];
-        $certif_text = sprintf(get_lang('CertificateWCertifiesStudentXFinishedCourseYWithGradeZ'),$organization_name,$stud_fn.' '.$stud_ln,$category[0]->get_name(),$scorecourse_display);
-        $certif_text = str_replace("\\n","\n",$certif_text);
+        $certif_text = sprintf(get_lang('CertificateWCertifiesStudentXFinishedCourseYWithGradeZ'), $organization_name, $stud_fn.' '.$stud_ln, $category[0]->get_name(), $scorecourse_display);
+        $certif_text = str_replace("\\n", "\n", $certif_text);
         $date = api_convert_and_format_date(null, DATE_FORMAT_SHORT);
 
-        $pdf= new Cezpdf('a4','landscape');
+        $pdf = new Cezpdf('a4', 'landscape');
         $pdf->selectFont(api_get_path(LIBRARY_PATH).'ezpdf/fonts/Courier.afm');
         $pdf->ezSetMargins(30, 30, 50, 50);
         //line Y coordinates in landscape mode are upside down (500 is on top, 10 is on the bottom)
-        $pdf->line(50,50,790,50);
-        $pdf->line(50,550,790,550);
+        $pdf->line(50, 50, 790, 50);
+        $pdf->line(50, 550, 790, 550);
         $pdf->ezSetY(450);
         //@todo replace image
         //$pdf->ezImage(api_get_path(SYS_CODE_PATH).'img/dokeos_logo_certif.png',1,400,'','center','');
         $pdf->ezSetY(480);
-        $pdf->ezText($certif_text,28,array('justification'=>'center'));
+        $pdf->ezText($certif_text, 28, array('justification'=>'center'));
         //$pdf->ezSetY(750);
         $pdf->ezSetY(50);
-        $pdf->ezText($date,18,array('justification'=>'center'));
+        $pdf->ezText($date, 18, array('justification'=>'center'));
         $pdf->ezSetY(580);
-        $pdf->ezText($organization_name,22,array('justification'=>'left'));
+        $pdf->ezText($organization_name, 22, array('justification'=>'left'));
         $pdf->ezSetY(580);
-        $pdf->ezText($portal_name,22,array('justification'=>'right'));
+        $pdf->ezText($portal_name, 22, array('justification'=>'right'));
         $pdf->ezStream();
     }
     exit;
 } else {
-    $cats= Category :: load($category);
-    $stud_id= (api_is_allowed_to_edit() ? null : api_get_user_id());
-    $allcat= $cats[0]->get_subcategories($stud_id);
-    $alleval= $cats[0]->get_evaluations($stud_id);
-    $alllink= $cats[0]->get_links($stud_id);
+    $cats = Category :: load($category);
+    $stud_id = (api_is_allowed_to_edit() ? null : api_get_user_id());
+    $allcat = $cats[0]->get_subcategories($stud_id);
+    $alleval = $cats[0]->get_evaluations($stud_id);
+    $alllink = $cats[0]->get_links($stud_id);
 }
-$addparams = array ('selectcat' => $cats[0]->get_id());
+$addparams = array('selectcat' => $cats[0]->get_id());
 if (isset($_GET['search'])) {
     $addparams['search'] = $keyword;
 }
 if (isset ($_GET['studentoverview'])) {
     $addparams['studentoverview'] = '';
 }
-if (isset($allcat_info) && count($allcat_info)>=0 && (isset($_GET['selectcat']) && $_GET['selectcat']==0) && isset($_GET['search']) && strlen(trim($_GET['search']))>0 ) {
-    $allcat=$allcat_info;
+if (isset($allcat_info) && count($allcat_info) >= 0 && (isset($_GET['selectcat']) && $_GET['selectcat'] == 0) && isset($_GET['search']) && strlen(trim($_GET['search'])) > 0) {
+    $allcat = $allcat_info;
 } else {
-    $allcat=$allcat;
+    $allcat = $allcat;
 }
 $gradebooktable = new GradebookTable(
     $cats[0],
@@ -602,7 +602,7 @@ 
     $addparams
 );
 if (((empty ($allcat)) && (empty ($alleval)) && (empty ($alllink)) && (!$is_platform_admin) && ($is_course_admin) && (!isset ($_GET['selectcat']))) && api_is_course_tutor()) {
-    Display :: display_normal_message(get_lang('GradebookWelcomeMessage') . '<br /><br /><form name="createcat" method="post" action="' . api_get_self() . '?createallcategories=1"><input type="submit" value="' . get_lang('CreateAllCat') . '"></form>',false);
+    Display :: display_normal_message(get_lang('GradebookWelcomeMessage').'<br /><br /><form name="createcat" method="post" action="'.api_get_self().'?createallcategories=1"><input type="submit" value="'.get_lang('CreateAllCat').'"></form>', false);
 }
 // Here we are in a sub category
 if ($category != '0') {

main/gradebook/lib/fe/exportgradebook.php

Spacing +21 added lines, -21 removed lines

@@ -12,7 +12,7 @@ 
  * @param	string	View to print as a title for the table
  * @param	string	Course name to print as title for the table
  */
-function print_table($data_array,$header_names,$view,$coursename)
+function print_table($data_array, $header_names, $view, $coursename)
 {
 	$printdata = '<!DOCTYPE html
      PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
@@ -160,7 +160,7 @@ 
 
 	$count_pages = ceil(count($data_table) / $items_per_page);
     $content_table = '';
-    for ($x = 0; $x<$count_pages; $x++) {
+    for ($x = 0; $x < $count_pages; $x++) {
         $content_table .= '<table width="100%" border="1" style="border-collapse:collapse">';
         // header table
         $content_table .= '<tr>';
@@ -169,7 +169,7 @@ 
 
             foreach ($headers_table as $head_table) {
                 if (!empty($head_table[0])) {
-                    $width = (!empty($head_table[1])?$head_table[1].'%':'');
+                    $width = (!empty($head_table[1]) ? $head_table[1].'%' : '');
                     $content_table .= '<th width="'.$width.'">'.$head_table[0].'</th>';
                     $i++;
                 }
@@ -178,16 +178,16 @@ 
         $content_table .= '</tr>';
         // body table
         if (is_array($data_table) && count($data_table) > 0) {
-            $offset = $x*$items_per_page;
-            $data_table = array_slice ($data_table, $offset, count($data_table));
+            $offset = $x * $items_per_page;
+            $data_table = array_slice($data_table, $offset, count($data_table));
             $i = 1;
-            $item = $offset+1;
+            $item = $offset + 1;
             foreach ($data_table as $data) {
                 $content_table .= '<tr>';
-                $content_table .= '<td>'.($item<10?'0'.$item:$item).'</td>';
+                $content_table .= '<td>'.($item < 10 ? '0'.$item : $item).'</td>';
                 foreach ($data as  $key => $content) {
                     if (isset($content)) {
-                        $key == 1 ? $align='align="left"':$align='align="center"';
+                        $key == 1 ? $align = 'align="left"' : $align = 'align="center"';
                         $content_table .= '<td '.$align.' style="padding:4px;" >'.$content.'</td>';
                     }
                 }
@@ -240,10 +240,10 @@ 
                             <h1>'.$title_pdf.'</h1></td></tr>';
         foreach ($headers_pdf as $header_pdf) {
             if (!empty($header_pdf[0]) && !empty($header_pdf[1])) {
-                $header.= '<tr><td><strong>'.$header_pdf[0].'</strong> </td><td>'.$header_pdf[1].'</td></tr>';
+                $header .= '<tr><td><strong>'.$header_pdf[0].'</strong> </td><td>'.$header_pdf[1].'</td></tr>';
             }
         }
-        $header.='</table><br />';
+        $header .= '</table><br />';
     }
 
     // preparing footer pdf
@@ -267,7 +267,7 @@ 
     }
     $items_per_page = 30;
     $count_pages = ceil(count($data_table) / $items_per_page);
-    for ($x = 0; $x<$count_pages; $x++) {
+    for ($x = 0; $x < $count_pages; $x++) {
         $content_table .= '<table width="100%" border="1" style="border-collapse:collapse">';
         // header table
         $content_table .= '<tr>';
@@ -275,7 +275,7 @@ 
         if (is_array($headers_table)) {
             foreach ($headers_table as $head_table) {
                 if (!empty($head_table[0])) {
-                    $width = (!empty($head_table[1])?$head_table[1].'%':'');
+                    $width = (!empty($head_table[1]) ? $head_table[1].'%' : '');
                     $content_table .= '<th width="'.$width.'">'.$head_table[0].'</th>';
                     $i++;
                 }
@@ -285,16 +285,16 @@ 
         // body table
 
         if (is_array($data_table) && count($data_table) > 0) {
-            $offset = $x*$items_per_page;
-            $data_table = array_slice ($data_table, $offset, count($data_table));
+            $offset = $x * $items_per_page;
+            $data_table = array_slice($data_table, $offset, count($data_table));
             $i = 1;
-            $item = $offset+1;
+            $item = $offset + 1;
             foreach ($data_table as $data) {
                 $content_table .= '<tr>';
-                $content_table .= '<td>'.($item<10?'0'.$item:$item).'</td>';
+                $content_table .= '<td>'.($item < 10 ? '0'.$item : $item).'</td>';
                 foreach ($data as  $key => $content) {
                     if (isset($content)) {
-                        $key == 1 ? $align='align="left"':$align='align="center"';
+                        $key == 1 ? $align = 'align="left"' : $align = 'align="center"';
                         $content_table .= '<td '.$align.' style="padding:4px;" >'.$content.'</td>';
                     }
                 }
@@ -312,7 +312,7 @@ 
     $pdf = new PDF();
     $pdf->set_custom_footer($footer);
     $pdf->set_custom_header($headers_in_pdf);
-    $pdf->content_to_pdf($header.$content_table, $css, $title_pdf );
+    $pdf->content_to_pdf($header.$content_table, $css, $title_pdf);
     exit;
 
 }
@@ -327,8 +327,8 @@ 
 function export_pdf($pdf, $newarray, $header_names, $format)
 {
 	$pdf->selectFont(api_get_path(LIBRARY_PATH).'ezpdf/fonts/Courier.afm');
-	$pdf->ezSetCmMargins(0,0,0,0);
-	$pdf->ezSetY(($format=='portrait')?'820':'570');
+	$pdf->ezSetCmMargins(0, 0, 0, 0);
+	$pdf->ezSetY(($format == 'portrait') ? '820' : '570');
 	$pdf->selectFont(api_get_path(LIBRARY_PATH).'ezpdf/fonts/Courier.afm');
     if ($format == 'portrait') {
         $pdf->line(40, 790, 540, 790);
@@ -337,7 +337,7 @@ 
         $pdf->line(40, 540, 790, 540);
         $pdf->line(40, 40, 790, 40);
     }
-    $pdf->ezSetY(($format=='portrait')?'750':'520');
+    $pdf->ezSetY(($format == 'portrait') ? '750' : '520');
     $pdf->ezTable($newarray, $header_names, '', array(
         'showHeadings' => 1,
         'shaded' => 1,

main/messages/outbox.php

Spacing +25 added lines, -25 removed lines

@@ -9,11 +9,11 @@ 
 api_block_anonymous_users();
 
 if (isset($_GET['messages_page_nr'])) {
-	if (api_get_setting('allow_social_tool')=='true' &&
-        api_get_setting('allow_message_tool')=='true'
+	if (api_get_setting('allow_social_tool') == 'true' &&
+        api_get_setting('allow_message_tool') == 'true'
     ) {
 		$social_link = '';
-		if ($_REQUEST['f']=='social') {
+		if ($_REQUEST['f'] == 'social') {
 			$social_link = '&f=social';
 		}
 		header('Location:outbox.php?pager='.Security::remove_XSS($_GET['messages_page_nr']).$social_link.'');
@@ -21,12 +21,12 @@ 
 	}
 }
 
-if (api_get_setting('allow_message_tool')!='true'){
+if (api_get_setting('allow_message_tool') != 'true') {
 	api_not_allowed();
 }
 //jquery thickbox already called from main/inc/header.inc.php
 
-$htmlHeadXtra[]='<script language="javascript">
+$htmlHeadXtra[] = '<script language="javascript">
 function enviar(miforma) {
 	if(confirm("'.get_lang('SureYouWantToDeleteSelectedMessages', '').'"))
 		miforma.submit();
@@ -53,44 +53,44 @@ 
 /*
 		MAIN CODE
 */
-if (isset($_GET['f']) && $_GET['f']=='social') {
+if (isset($_GET['f']) && $_GET['f'] == 'social') {
 	$this_section = SECTION_SOCIAL;
-	$interbreadcrumb[]= array ('url' => api_get_path(WEB_PATH).'main/social/home.php','name' => get_lang('Social'));
-	$interbreadcrumb[]= array ('url' => '#','name' => get_lang('Outbox'));
+	$interbreadcrumb[] = array('url' => api_get_path(WEB_PATH).'main/social/home.php', 'name' => get_lang('Social'));
+	$interbreadcrumb[] = array('url' => '#', 'name' => get_lang('Outbox'));
 } else {
 	$this_section = SECTION_MYPROFILE;
-	$interbreadcrumb[]= array ('url' => api_get_path(WEB_PATH).'main/auth/profile.php','name' => get_lang('Profile'));
-	$interbreadcrumb[]= array ('url' => '#','name' => get_lang('Outbox'));
+	$interbreadcrumb[] = array('url' => api_get_path(WEB_PATH).'main/auth/profile.php', 'name' => get_lang('Profile'));
+	$interbreadcrumb[] = array('url' => '#', 'name' => get_lang('Outbox'));
 }
 
 $actions = '';
 if (api_get_setting('extended_profile') == 'true') {
     if (api_get_setting('allow_social_tool') == 'true' && api_get_setting('allow_message_tool') == 'true') {
-        $actions .=  '<a href="'.api_get_path(WEB_PATH).'main/social/profile.php">'.Display::return_icon('shared_profile.png', get_lang('ViewSharedProfile')).'</a>';
+        $actions .= '<a href="'.api_get_path(WEB_PATH).'main/social/profile.php">'.Display::return_icon('shared_profile.png', get_lang('ViewSharedProfile')).'</a>';
     }
     if (api_get_setting('allow_message_tool') == 'true') {
         //echo '<a href="'.api_get_path(WEB_PATH).'main/messages/inbox.php">'.Display::return_icon('inbox.png').' '.get_lang('Messages').'</a>';
-        $actions .=  '<a href="'.api_get_path(WEB_PATH).'main/messages/new_message.php">'.Display::return_icon('message_new.png',get_lang('ComposeMessage')).'</a>';
-        $actions .=  '<a href="'.api_get_path(WEB_PATH).'main/messages/inbox.php">'.Display::return_icon('inbox.png',get_lang('Inbox')).'</a>';
-        $actions .=  '<a href="'.api_get_path(WEB_PATH).'main/messages/outbox.php">'.Display::return_icon('outbox.png',get_lang('Outbox')).'</a>';
+        $actions .= '<a href="'.api_get_path(WEB_PATH).'main/messages/new_message.php">'.Display::return_icon('message_new.png', get_lang('ComposeMessage')).'</a>';
+        $actions .= '<a href="'.api_get_path(WEB_PATH).'main/messages/inbox.php">'.Display::return_icon('inbox.png', get_lang('Inbox')).'</a>';
+        $actions .= '<a href="'.api_get_path(WEB_PATH).'main/messages/outbox.php">'.Display::return_icon('outbox.png', get_lang('Outbox')).'</a>';
     }
 }
 
 
-$info_delete_outbox =array();
-$info_delete_outbox = isset($_GET['form_delete_outbox']) ? explode(',',$_GET['form_delete_outbox']) : '';
+$info_delete_outbox = array();
+$info_delete_outbox = isset($_GET['form_delete_outbox']) ? explode(',', $_GET['form_delete_outbox']) : '';
 $count_delete_outbox = count($info_delete_outbox) - 1;
 
-if (isset($info_delete_outbox[0]) && trim($info_delete_outbox[0])=='delete') {
+if (isset($info_delete_outbox[0]) && trim($info_delete_outbox[0]) == 'delete') {
     for ($i = 1; $i <= $count_delete_outbox; $i++) {
-		MessageManager::delete_message_by_user_sender(api_get_user_id(),$info_delete_outbox[$i]);
+		MessageManager::delete_message_by_user_sender(api_get_user_id(), $info_delete_outbox[$i]);
 	}
-    $message_box=get_lang('SelectedMessagesDeleted').
+    $message_box = get_lang('SelectedMessagesDeleted').
         '&nbsp
         <br><a href="../social/index.php?#remote-tab-3">'.
         get_lang('BackToOutbox').
         '</a>';
-    Display::display_normal_message(api_xml_http_response_encode($message_box),false);
+    Display::display_normal_message(api_xml_http_response_encode($message_box), false);
     exit;
 }
 
@@ -117,23 +117,23 @@ 
 }
 //MAIN CONTENT
 if ($action == 'delete') {
-    $delete_list_id=array();
+    $delete_list_id = array();
     if (isset($_POST['out'])) {
-        $delete_list_id=$_POST['out'];
+        $delete_list_id = $_POST['out'];
     }
     if (isset($_POST['id'])) {
-        $delete_list_id=$_POST['id'];
+        $delete_list_id = $_POST['id'];
     }
     for ($i = 0; $i < count($delete_list_id); $i++) {
         MessageManager::delete_message_by_user_sender(api_get_user_id(), $delete_list_id[$i]);
     }
-    $delete_list_id=array();
+    $delete_list_id = array();
     $social_right_content .= MessageManager::outbox_display($keyword);
 } elseif ($action == 'deleteone') {
     $delete_list_id = array();
     $id = Security::remove_XSS($_GET['id']);
     MessageManager::delete_message_by_user_sender(api_get_user_id(), $id);
-    $delete_list_id=array();
+    $delete_list_id = array();
     $social_right_content .= MessageManager::outbox_display($keyword);
 } else {
     $social_right_content .= MessageManager::outbox_display($keyword);

main/group/group_creation.php

Spacing +9 added lines, -9 removed lines

@@ -24,7 +24,7 @@ 
         case 'create_groups':
             $groups = array();
 
-            for ($i = 0; $i < $_POST['number_of_groups']; $i ++) {
+            for ($i = 0; $i < $_POST['number_of_groups']; $i++) {
                 $group1['name'] = empty($_POST['group_'.$i.'_name']) ? get_lang('Group').' '.$i : $_POST['group_'.$i.'_name'];
                 $group1['category'] = isset($_POST['group_'.$i.'_category']) ? $_POST['group_'.$i.'_category'] : null;
                 $group1['tutor'] = isset($_POST['group_'.$i.'_tutor']) ? $_POST['group_'.$i.'_tutor'] : null;
@@ -167,16 +167,16 @@ 
 		$form->addGroup($group_el, 'groups', null, "</td><td>", false);
 		// Checkboxes
 		if ($_POST['number_of_groups'] > 1) {
-			$group_el = array ();
+			$group_el = array();
 			$group_el[] = $form->createElement('static', null, null, ' ');
 			if (api_get_setting('allow_group_categories') == 'true') {
 				$group_el[] = $form->createElement('checkbox', 'same_category', null, get_lang('SameForAll'), array('onclick' => "javascript: switch_state('category');"));
 			}
-			$group_el[] = $form->createElement('checkbox', 'same_places', null, get_lang('SameForAll'), array ('onclick' => "javascript: switch_state('places');"));
+			$group_el[] = $form->createElement('checkbox', 'same_places', null, get_lang('SameForAll'), array('onclick' => "javascript: switch_state('places');"));
 			$form->addGroup($group_el, 'groups', null, '</td><td>', false);
 		}
 		// Properties for all groups
-		for ($group_number = 0; $group_number < $_POST['number_of_groups']; $group_number ++) {
+		for ($group_number = 0; $group_number < $_POST['number_of_groups']; $group_number++) {
 			$group_el = array();
 			$group_el[] = $form->createElement('text', 'group_'.$group_number.'_name');
 			if (api_get_setting('allow_group_categories') == 'true') {
@@ -200,14 +200,14 @@ 
 					$prev = '000';
 				} elseif ($group_id < 100) {
 					$prev = '00';
-				} elseif ($group_id<1000) {
+				} elseif ($group_id < 1000) {
 					$prev = '0';
 				} else {
 					$prev = '';
 				}
 			}
 
-			$defaults['group_'.$group_number.'_name'] = get_lang('GroupSingle').' '.$prev.$group_id ++;
+			$defaults['group_'.$group_number.'_name'] = get_lang('GroupSingle').' '.$prev.$group_id++;
 			$form->addGroup($group_el, 'group_'.$group_number, null, '</td><td>', false);
 		}
 		$defaults['action'] = 'create_groups';
@@ -223,8 +223,8 @@ 
 
 	$create_groups_form = new FormValidator('create_groups', 'post', api_get_self().'?'.api_get_cidreq());
 	$create_groups_form->addElement('header', $nameTools);
-    $create_groups_form->addText('number_of_groups',get_lang('NumberOfGroupsToCreate'),null,array('value'=>'1'));
-    $create_groups_form->addButton('submit', get_lang('ProceedToCreateGroup'),'plus','primary');
+    $create_groups_form->addText('number_of_groups', get_lang('NumberOfGroupsToCreate'), null, array('value'=>'1'));
+    $create_groups_form->addButton('submit', get_lang('ProceedToCreateGroup'), 'plus', 'primary');
 	$defaults = array();
 	$defaults['number_of_groups'] = 1;
 	$create_groups_form->setDefaults($defaults);
@@ -234,7 +234,7 @@ 
 	 * Show form to generate subgroups
 	 */
 	if (api_get_setting('allow_group_categories') == 'true' && count(GroupManager :: get_group_list()) > 0) {
-		$base_group_options = array ();
+		$base_group_options = array();
 		$groups = GroupManager :: get_group_list();
 		foreach ($groups as $index => $group) {
 			$number_of_students = GroupManager :: number_of_students($group['id']);

main/exercice/hotpotatoes.php

Spacing +3 added lines, -3 removed lines

@@ -31,7 +31,7 @@ 
 }
 // The breadcrumbs.
 $interbreadcrumb[] = array(
-    'url' => api_get_path(WEB_CODE_PATH) . '.exercice/exercise.php?' . api_get_cidreq(),
+    'url' => api_get_path(WEB_CODE_PATH).'.exercice/exercise.php?'.api_get_cidreq(),
     'name' => get_lang('Exercises')
 );
 
@@ -153,7 +153,7 @@ 
                 $_course,
                 $_FILES['userFile'],
                 $document_sys_path,
-                $uploadPath . '/' . $fld,
+                $uploadPath.'/'.$fld,
                 api_get_user_id(),
                 null,
                 null,
@@ -165,7 +165,7 @@ 
                     $imgparams = $_POST['imgparams'];
                     $checked = CheckImageName($imgparams, $filename);
                     if ($checked) {
-                        $imgcount = $imgcount-1;
+                        $imgcount = $imgcount - 1;
                     } else {
                         $dialogBox .= $filename.' '.get_lang('NameNotEqual');
                         my_delete($document_sys_path.$uploadPath.'/'.$fld.'/'.$filename);

main/session/session_user_edit.php

Spacing +2 added lines, -2 removed lines

@@ -28,7 +28,7 @@ 
 }
 
 
-$interbreadcrumb[] = array('url' => 'session_list.php','name' => get_lang('SessionList'));
+$interbreadcrumb[] = array('url' => 'session_list.php', 'name' => get_lang('SessionList'));
 $interbreadcrumb[] = array(
     'url' => "resume_session.php?id_session=".$sessionId,
     "name" => get_lang('SessionOverview')
@@ -60,7 +60,7 @@ 
     if ($days > 0) {
         $msg = sprintf(get_lang('FirstAccessWasXSessionDurationYEndDateInZDays'), $firstAccessString, $duration, $days);
     } else {
-        $endDateInSeconds = $firstAccess + $duration * 24*60*60;
+        $endDateInSeconds = $firstAccess + $duration * 24 * 60 * 60;
         $last = api_convert_and_format_date($endDateInSeconds, DATE_FORMAT_SHORT);
         $msg = sprintf(get_lang('FirstAccessWasXSessionDurationYEndDateWasZ'), $firstAccessString, $duration, $last);
     }

main/session/resume_session.php

Spacing +6 added lines, -6 removed lines

@@ -29,7 +29,7 @@ 
 $tool_name = get_lang('SessionOverview');
 
 //$interbreadcrumb[] = array('url' => 'index.php','name' => get_lang('PlatformAdmin'));
-$interbreadcrumb[] = array('url' => 'session_list.php','name' => get_lang('SessionList'));
+$interbreadcrumb[] = array('url' => 'session_list.php', 'name' => get_lang('SessionList'));
 
 $orig_param = '&origin=resume_session';
 
@@ -183,7 +183,7 @@ 
 
 		$coachs = array();
 		if (Database::num_rows($rs) > 0) {
-			while($info_coach = Database::fetch_array($rs)) {
+			while ($info_coach = Database::fetch_array($rs)) {
                 $coachs[] = api_get_person_name(
                         $info_coach['firstname'],
                         $info_coach['lastname']
@@ -194,7 +194,7 @@ 
 		}
 
 		if (count($coachs) > 0) {
-			$coach = implode('<br />',$coachs);
+			$coach = implode('<br />', $coachs);
 		} else {
 			$coach = get_lang('None');
 		}
@@ -217,7 +217,7 @@ 
         $downIcon = 'down.png';
         $downUrl = api_get_self().'?id_session='.$sessionId.'&course_id='.$course->getId().'&action=move_down';
 
-        if ($count +1 == count($courses)) {
+        if ($count + 1 == count($courses)) {
             $downIcon = 'down_na.png';
             $downUrl = '#';
         }
@@ -244,7 +244,7 @@ 
 			<td>'.$coach.'</td>
 			<td>'.$numberOfUsers.'</td>
 			<td>
-                <a href="'. $courseUrl . '">'.
+                <a href="'. $courseUrl.'">'.
                 Display::return_icon('course_home.gif', get_lang('Course')).'</a>
                 '.$orderButtons.'
                 <a href="session_course_user_list.php?id_session='.$sessionId.'&course_code='.$course->getCode().'">'.
@@ -309,7 +309,7 @@ 
             array('onclick' => "javascript:if(!confirm('".get_lang('ConfirmYourChoice')."')) return false;")
         );
 
-        $addUserToUrlLink= '';
+        $addUserToUrlLink = '';
         if ($multiple_url_is_on) {
             if ($user['access_url_id'] != $url_id) {
                 $userLink .= ' '.Display::return_icon(