chamilo / chamilo-lms

main/dashboard/dashboard.php

Indentation +13 added lines, -13 removed lines

@@ -14,23 +14,23 @@ 
 $views = array('blocks', 'list');
 
 if(isset($_GET['view']) && in_array($_GET['view'], $views)){
-	$dashboard_view = $_GET['view'];
+    $dashboard_view = $_GET['view'];
 }
 
 $link_blocks_view = $link_list_view = null;
 
 if (isset($dashboard_view) && $dashboard_view == 'list') {
-	$link_blocks_view = '<a href="'.api_get_self().'?view=blocks">'.
-		Display::return_icon('blocks.png',get_lang('DashboardBlocks'),'',ICON_SIZE_MEDIUM).'</a>';
+    $link_blocks_view = '<a href="'.api_get_self().'?view=blocks">'.
+        Display::return_icon('blocks.png',get_lang('DashboardBlocks'),'',ICON_SIZE_MEDIUM).'</a>';
 } else {
-	$link_list_view = '<a href="'.api_get_self().'?view=list">'.
-		Display::return_icon('edit.png',get_lang('EditBlocks'),'',ICON_SIZE_MEDIUM).'</a>';
+    $link_list_view = '<a href="'.api_get_self().'?view=list">'.
+        Display::return_icon('edit.png',get_lang('EditBlocks'),'',ICON_SIZE_MEDIUM).'</a>';
 }
 
 $configuration_link = null;
 if (api_is_platform_admin()) {
-	$configuration_link = '<a href="'.api_get_path(WEB_CODE_PATH).'admin/settings.php?category=Plugins">'
-	.Display::return_icon('settings.png',get_lang('ConfigureDashboardPlugin'),'',ICON_SIZE_MEDIUM).'</a>';
+    $configuration_link = '<a href="'.api_get_path(WEB_CODE_PATH).'admin/settings.php?category=Plugins">'
+    .Display::return_icon('settings.png',get_lang('ConfigureDashboardPlugin'),'',ICON_SIZE_MEDIUM).'</a>';
 }
 
 echo '<div class="actions">';
@@ -87,10 +87,10 @@ 
     }
 
 } else {
-	// block dashboard list
-	if (isset($success)) {
-		Display::display_confirmation_message(get_lang('BlocksHaveBeenUpdatedSuccessfully'));
-	}
-	$user_id = api_get_user_id();
-	DashboardManager::display_user_dashboard_list($user_id);
+    // block dashboard list
+    if (isset($success)) {
+        Display::display_confirmation_message(get_lang('BlocksHaveBeenUpdatedSuccessfully'));
+    }
+    $user_id = api_get_user_id();
+    DashboardManager::display_user_dashboard_list($user_id);
 }

Spacing +6 added lines, -6 removed lines

@@ -13,7 +13,7 @@ 
 // menu actions for dashboard views
 $views = array('blocks', 'list');
 
-if(isset($_GET['view']) && in_array($_GET['view'], $views)){
+if (isset($_GET['view']) && in_array($_GET['view'], $views)) {
 	$dashboard_view = $_GET['view'];
 }
 
@@ -21,16 +21,16 @@ 
 
 if (isset($dashboard_view) && $dashboard_view == 'list') {
 	$link_blocks_view = '<a href="'.api_get_self().'?view=blocks">'.
-		Display::return_icon('blocks.png',get_lang('DashboardBlocks'),'',ICON_SIZE_MEDIUM).'</a>';
+		Display::return_icon('blocks.png', get_lang('DashboardBlocks'), '', ICON_SIZE_MEDIUM).'</a>';
 } else {
 	$link_list_view = '<a href="'.api_get_self().'?view=list">'.
-		Display::return_icon('edit.png',get_lang('EditBlocks'),'',ICON_SIZE_MEDIUM).'</a>';
+		Display::return_icon('edit.png', get_lang('EditBlocks'), '', ICON_SIZE_MEDIUM).'</a>';
 }
 
 $configuration_link = null;
 if (api_is_platform_admin()) {
 	$configuration_link = '<a href="'.api_get_path(WEB_CODE_PATH).'admin/settings.php?category=Plugins">'
-	.Display::return_icon('settings.png',get_lang('ConfigureDashboardPlugin'),'',ICON_SIZE_MEDIUM).'</a>';
+	.Display::return_icon('settings.png', get_lang('ConfigureDashboardPlugin'), '', ICON_SIZE_MEDIUM).'</a>';
 }
 
 echo '<div class="actions">';
@@ -56,7 +56,7 @@ 
         if (count($columns) > 0) {
             $columns_name = array_keys($columns);
             // blocks for column 1
-            if (in_array('column_1',$columns_name)) {
+            if (in_array('column_1', $columns_name)) {
                 echo '<div id="column1" class="col-md-6">';
                     foreach ($columns['column_1'] as $content) {
                         echo $content;
@@ -68,7 +68,7 @@ 
                 echo '</div>';
             }
             // blocks for column 2
-            if (in_array('column_2',$columns_name)) {
+            if (in_array('column_2', $columns_name)) {
                 // blocks for column 1
                 echo '<div id="column2" class="col-md-6">';
                     foreach ($columns['column_2'] as $content) {

main/dashboard/dashboard_controller.php

Braces +6 added lines, -2 removed lines

@@ -48,7 +48,9 @@ 
             foreach ($dashboard_blocks as $block) {
 
                 // display only user blocks
-                if (!in_array($block['id'], $user_blocks_id)) continue;
+                if (!in_array($block['id'], $user_blocks_id)) {
+                    continue;
+                }
 
                 $path = $block['path'];
                 $controller_class = $block['controller'];
@@ -61,7 +63,9 @@ 
                     // check if user is allowed to see the block
                     if (method_exists($obj, 'is_block_visible_for_user')) {
                         $is_block_visible_for_user = $obj->is_block_visible_for_user($user_id);
-                        if (!$is_block_visible_for_user) continue;
+                        if (!$is_block_visible_for_user) {
+                            continue;
+                        }
                     }
 
                     $data_block[$path] = $obj->get_block();

main/timeline/index.php

Spacing +15 added lines, -15 removed lines

@@ -23,19 +23,19 @@ 
 
 switch ($action) {
     case 'add':
-        $interbreadcrumb[]=array('url' => 'index.php','name' => get_lang('Timeline'));
-        $interbreadcrumb[]=array('url' => '#','name' => get_lang('Add'));
+        $interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('Timeline'));
+        $interbreadcrumb[] = array('url' => '#', 'name' => get_lang('Add'));
         break;
     case 'edit':
-        $interbreadcrumb[]=array('url' => 'index.php','name' => get_lang('Timeline'));
-        $interbreadcrumb[]=array('url' => '#','name' => get_lang('Edit'));
+        $interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('Timeline'));
+        $interbreadcrumb[] = array('url' => '#', 'name' => get_lang('Edit'));
         break;
     case 'add_item':
-        $interbreadcrumb[]=array('url' => 'index.php','name' => get_lang('Timeline'));
-        $interbreadcrumb[]=array('url' => '#','name' => get_lang('TimelineItem'));
+        $interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('Timeline'));
+        $interbreadcrumb[] = array('url' => '#', 'name' => get_lang('TimelineItem'));
         break;
     default:
-        $interbreadcrumb[]=array('url' => '#','name' => get_lang('Timeline'));
+        $interbreadcrumb[] = array('url' => '#', 'name' => get_lang('Timeline'));
 }
 
 //jqgrid will use this URL to do the selects
@@ -46,8 +46,8 @@ 
 
 //Column config
 $column_model   = array(
-                        array('name'=>'name',           'index'=>'name',        'width'=>'120',   'align'=>'left'),
-                        array('name'=>'actions',        'index'=>'actions',     'width'=>'100',  'align'=>'left', 'sortable'=>'false')
+                        array('name'=>'name', 'index'=>'name', 'width'=>'120', 'align'=>'left'),
+                        array('name'=>'actions', 'index'=>'actions', 'width'=>'100', 'align'=>'left', 'sortable'=>'false')
                        );
 //Autowidth
 $extra_params['autowidth'] = 'true';
@@ -55,11 +55,11 @@ 
 $extra_params['height'] = 'auto';
 
 //With this function we can add actions to the jgrid (edit, delete, etc)
-$htmlHeadXtra[] ='
+$htmlHeadXtra[] = '
 <script>
 $(function() {
     // grid definition see the $timeline->display() function
-    '.Display::grid_js('timelines',  $url,$columns,$column_model,$extra_params, array(), null,true).'
+    '.Display::grid_js('timelines', $url, $columns, $column_model, $extra_params, array(), null, true).'
 });
 </script>';
 
@@ -80,12 +80,12 @@ 
                 $values = $form->exportValues();
                 $res    = $timeline->save($values);
                 if ($res) {
-                    $message = Display::return_message(get_lang('ItemAdded'),'success');
+                    $message = Display::return_message(get_lang('ItemAdded'), 'success');
                 }
             }
             $content = $timeline->listing();
         } else {
-            $actions .= '<a href="'.api_get_self().'">'.Display::return_icon('back.png',get_lang('Back'),'',ICON_SIZE_MEDIUM).'</a>';
+            $actions .= '<a href="'.api_get_self().'">'.Display::return_icon('back.png', get_lang('Back'), '', ICON_SIZE_MEDIUM).'</a>';
             $form->addElement('hidden', 'sec_token');
             $form->setConstants(array('sec_token' => $token));
             $content = $form->return_form();
@@ -106,7 +106,7 @@ 
             }
             $timeline->display();
         } else {
-            $actions = '<a href="'.api_get_self().'">'.Display::return_icon('back.png',get_lang('Back'),'',ICON_SIZE_MEDIUM).'</a>';
+            $actions = '<a href="'.api_get_self().'">'.Display::return_icon('back.png', get_lang('Back'), '', ICON_SIZE_MEDIUM).'</a>';
             $form->addElement('hidden', 'sec_token');
             $form->setConstants(array('sec_token' => $token));
             $content = $form->return_form();
@@ -128,7 +128,7 @@ 
             }
             $timeline->display();
         } else {
-            $actions = '<a href="'.api_get_self().'">'.Display::return_icon('back.png',get_lang('Back'),'',ICON_SIZE_MEDIUM).'</a>';
+            $actions = '<a href="'.api_get_self().'">'.Display::return_icon('back.png', get_lang('Back'), '', ICON_SIZE_MEDIUM).'</a>';
             $form->addElement('hidden', 'sec_token');
             $form->setConstants(array('sec_token' => $token));
             $content = $form->return_form();

Indentation +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
                         array('name'=>'certificate',    'index'=>'certificate', 'width'=>'25', 'align'=>'left', 'sortable'=>'false'),
                         array('name'=>'skills',         'index'=>'skills',      'width'=>'300', 'align'=>'left', 'sortable'=>'false'),
                         array('name'=>'actions',        'index'=>'actions',     'width'=>'30', 'align'=>'left','formatter'=>'action_formatter','sortable'=>'false')
-                       );
+                        );
 //Autowidth
 $extra_params['autowidth'] = 'true';
 //height auto

main/timeline/view.php

Spacing +3 added lines, -3 removed lines

@@ -6,8 +6,8 @@ 
 */
 require_once '../inc/global.inc.php';
 
-$interbreadcrumb[]=array('url' => 'index.php','name' => get_lang('Timeline'));
-$interbreadcrumb[]=array('url' => 'index.php','name' => get_lang('Listing'));
+$interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('Timeline'));
+$interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('Listing'));
 
 $timeline = new Timeline();
 if (empty($_GET['id'])) {
@@ -15,7 +15,7 @@ 
 }
 $url = $timeline->get_url($_GET['id']);
 $item = $timeline->get($_GET['id']);
-$interbreadcrumb[]=array('url' => '#','name' => $item['headline']);
+$interbreadcrumb[] = array('url' => '#', 'name' => $item['headline']);
 
 $htmlHeadXtra[] = api_get_css(api_get_path(WEB_LIBRARY_PATH).'javascript/timeline/timeline.css');
 $htmlHeadXtra[] = api_get_js('timeline/timeline-min.js');

main/blog/download.php

Indentation +7 added lines, -7 removed lines

@@ -33,18 +33,18 @@
 $doc_url = str_replace('/..', '', $doc_url); //echo $doc_url;
 
 if (! isset($_course)) {
-	api_not_allowed(true);
+    api_not_allowed(true);
 }
 $full_file_name = api_get_path(SYS_COURSE_PATH).api_get_course_path().'/upload/blog/'.$doc_url;
 
 //if the rewrite rule asks for a directory, we redirect to the course view
 if (is_dir($full_file_name)) {
-	//remove last slash if present
-	while ($doc_url{$dul = strlen($doc_url)-1}=='/') $doc_url = substr($doc_url,0,$dul);
-	//create the path
-	$document_explorer = api_get_path(WEB_COURSE_PATH).api_get_course_path(); // home course path
-	//redirect
-	header('Location: '.$document_explorer);
+    //remove last slash if present
+    while ($doc_url{$dul = strlen($doc_url)-1}=='/') $doc_url = substr($doc_url,0,$dul);
+    //create the path
+    $document_explorer = api_get_path(WEB_COURSE_PATH).api_get_course_path(); // home course path
+    //redirect
+    header('Location: '.$document_explorer);
     exit;
 }
 

Braces +3 added lines, -1 removed lines

@@ -40,7 +40,9 @@
 //if the rewrite rule asks for a directory, we redirect to the course view
 if (is_dir($full_file_name)) {
 	//remove last slash if present
-	while ($doc_url{$dul = strlen($doc_url)-1}=='/') $doc_url = substr($doc_url,0,$dul);
+	while ($doc_url{$dul = strlen($doc_url)-1}=='/') {
+	    $doc_url = substr($doc_url,0,$dul);
+	}
 	//create the path
 	$document_explorer = api_get_path(WEB_COURSE_PATH).api_get_course_path(); // home course path
 	//redirect

Spacing +3 added lines, -3 removed lines

@@ -12,7 +12,7 @@ 
 session_cache_limiter('public');
 
 require_once '../inc/global.inc.php';
-$this_section=SECTION_COURSES;
+$this_section = SECTION_COURSES;
 
 // IMPORTANT to avoid caching of documents
 header('Expires: Wed, 01 Jan 1990 00:00:00 GMT');
@@ -29,7 +29,7 @@ 
 $doc_url = str_replace(' ', '+', $doc_url);
 $doc_url = str_replace('/..', '', $doc_url); //echo $doc_url;
 
-if (! isset($_course)) {
+if (!isset($_course)) {
 	api_not_allowed(true);
 }
 $full_file_name = api_get_path(SYS_COURSE_PATH).api_get_course_path().'/upload/blog/'.$doc_url;
@@ -37,7 +37,7 @@ 
 //if the rewrite rule asks for a directory, we redirect to the course view
 if (is_dir($full_file_name)) {
 	//remove last slash if present
-	while ($doc_url{$dul = strlen($doc_url)-1}=='/') $doc_url = substr($doc_url,0,$dul);
+	while ($doc_url{$dul = strlen($doc_url) - 1} == '/') $doc_url = substr($doc_url, 0, $dul);
 	//create the path
 	$document_explorer = api_get_path(WEB_COURSE_PATH).api_get_course_path(); // home course path
 	//redirect

main/gradebook/gradebook_edit_cat.php

Spacing +3 added lines, -3 removed lines

@@ -62,7 +62,7 @@ 
         $cat->setGenerateCertificates(false);
     }
 
-    if ($values['hid_parent_id'] == 0 ) {
+    if ($values['hid_parent_id'] == 0) {
         $cat->set_certificate_min_score($values['certif_min_score']);
     }
 
@@ -81,12 +81,12 @@ 
     }
 
     $cat->save();
-    header('Location: '.Security::remove_XSS($_SESSION['gradebook_dest']).'?editcat=&selectcat=' . $cat->get_parent_id().'&'.api_get_cidreq());
+    header('Location: '.Security::remove_XSS($_SESSION['gradebook_dest']).'?editcat=&selectcat='.$cat->get_parent_id().'&'.api_get_cidreq());
     exit;
 }
 $selectcat = isset($_GET['selectcat']) ? Security::remove_XSS($_GET['selectcat']) : '';
 $interbreadcrumb[] = array(
-    'url' => Security::remove_XSS($_SESSION['gradebook_dest']) . '?selectcat=' . $selectcat . '&' . api_get_cidreq(),
+    'url' => Security::remove_XSS($_SESSION['gradebook_dest']).'?selectcat='.$selectcat.'&'.api_get_cidreq(),
     'name' => get_lang('Gradebook')
 );
 $this_section = SECTION_COURSES;

main/gradebook/gradebook_display_certificate.php

Indentation +9 added lines, -9 removed lines

@@ -162,17 +162,17 @@
 
     if ($form->validate()) {
         $officialCode = $form->getSubmitValue('filter');
-         if ($officialCode == 'all') {
+            if ($officialCode == 'all') {
             $certificate_list = GradebookUtils::get_list_users_certificates($cat_id);
         } else {
-             $userList = UserManager::getUsersByOfficialCode($officialCode);
-             if (!empty($userList)) {
-                 $certificate_list = GradebookUtils::get_list_users_certificates(
-                     $cat_id,
-                     $userList
-                 );
-             }
-         }
+                $userList = UserManager::getUsersByOfficialCode($officialCode);
+                if (!empty($userList)) {
+                    $certificate_list = GradebookUtils::get_list_users_certificates(
+                        $cat_id,
+                        $userList
+                    );
+                }
+            }
     } else {
         $certificate_list = GradebookUtils::get_list_users_certificates($cat_id);
     }

Spacing +14 added lines, -14 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;
 
 if (!api_is_student_boss()) {
     api_protect_course_script(true);
@@ -16,9 +16,9 @@ 
 ini_set('max_execution_time', 0);
 
 //extra javascript functions for in html head:
-$htmlHeadXtra[] ="<script>
+$htmlHeadXtra[] = "<script>
 function confirmation() {
-	if (confirm(\" " . trim(get_lang('AreYouSureToDelete')) . " ?\")) {
+	if (confirm(\" " . trim(get_lang('AreYouSureToDelete'))." ?\")) {
 	    return true;
 	} else {
 	    return false;
@@ -31,7 +31,7 @@ 
     api_not_allowed(true);
 }
 
-$cat_id = isset($_GET['cat_id']) ? (int)$_GET['cat_id'] : null;
+$cat_id = isset($_GET['cat_id']) ? (int) $_GET['cat_id'] : null;
 $action = isset($_GET['action']) && $_GET['action'] ? $_GET['action'] : null;
 $filterOfficialCode = isset($_POST['filter']) ? Security::remove_XSS($_POST['filter']) : null;
 $filterOfficialCodeGet = isset($_GET['filter']) ? Security::remove_XSS($_GET['filter']) : null;
@@ -72,8 +72,8 @@ 
 
 $course_code = api_get_course_id();
 
-$interbreadcrumb[] = array('url' => Security::remove_XSS($_SESSION['gradebook_dest']).'?',	'name' => get_lang('Gradebook'));
-$interbreadcrumb[] = array('url' => '#','name' => get_lang('GradebookListOfStudentsCertificates'));
+$interbreadcrumb[] = array('url' => Security::remove_XSS($_SESSION['gradebook_dest']).'?', 'name' => get_lang('Gradebook'));
+$interbreadcrumb[] = array('url' => '#', 'name' => get_lang('GradebookListOfStudentsCertificates'));
 
 $this_section = SECTION_COURSES;
 
@@ -85,7 +85,7 @@ 
         $certificate = new Certificate($_GET['certificate_id']);
         $result = $certificate->delete(true);
         Security::clear_token();
-        if ($result ==true) {
+        if ($result == true) {
             Display::display_confirmation_message(get_lang('CertificateRemoved'));
         } else {
             Display::display_error_message(get_lang('CertificateNotRemoved'));
@@ -104,9 +104,9 @@ 
 
     //with this fix the teacher only can view 1 gradebook
     if (api_is_platform_admin()) {
-        $stud_id= (api_is_allowed_to_edit() ? null : api_get_user_id());
+        $stud_id = (api_is_allowed_to_edit() ? null : api_get_user_id());
     } else {
-        $stud_id= api_get_user_id();
+        $stud_id = api_get_user_id();
     }
 
     $total_weight = $cats[0]->get_weight();
@@ -115,7 +115,7 @@ 
     $alleval = $cats[0]->get_evaluations($stud_id);
     $alllink = $cats[0]->get_links($stud_id);
 
-    $datagen = new GradebookDataGenerator ($allcat,$alleval, $alllink);
+    $datagen = new GradebookDataGenerator($allcat, $alleval, $alllink);
 
     $total_resource_weight = 0;
     if (!empty($datagen)) {
@@ -132,7 +132,7 @@ 
                 $newarray[] = array_slice($data, 1);
             }
 
-            foreach($newarray as $item) {
+            foreach ($newarray as $item) {
                 $total_resource_weight = $total_resource_weight + $item['2'];
             }
         }
@@ -150,7 +150,7 @@ 
 if ($filter === 'true') {
     echo '<br />';
     $options = UserManager::getOfficialCodeGrouped();
-    $options =array_merge(array('all' => get_lang('All')), $options);
+    $options = array_merge(array('all' => get_lang('All')), $options);
     $form = new FormValidator(
         'official_code_filter',
         'POST',
@@ -196,7 +196,7 @@ 
 
 echo $filterForm;
 
-if (count($certificate_list) == 0 ) {
+if (count($certificate_list) == 0) {
     echo Display::display_warning_message(get_lang('NoResultsAvailable'));
 } else {
 
@@ -219,7 +219,7 @@ 
             $certificates = Display::url(get_lang('Certificate'), $url, array('target'=>'_blank', 'class' => 'btn btn-default'));
             echo $certificates;
             echo '<a onclick="return confirmation();" href="gradebook_display_certificate.php?sec_token='.$token.'&cidReq='.$course_code.'&action=delete&cat_id='.$cat_id.'&certificate_id='.$value_certificate['id'].'">
-                    '.Display::return_icon('delete.png',get_lang('Delete')).'
+                    '.Display::return_icon('delete.png', get_lang('Delete')).'
                   </a>';
             echo '</td></tr>';
         }

main/gradebook/gradebook_display_summary.php

Spacing +5 added lines, -5 removed lines

@@ -9,7 +9,7 @@ 
 use ChamiloSession as Session;
 
 require_once '../inc/global.inc.php';
-$current_course_tool  = TOOL_GRADEBOOK;
+$current_course_tool = TOOL_GRADEBOOK;
 
 api_protect_course_script();
 
@@ -21,7 +21,7 @@ 
     api_not_allowed(true);
 }
 
-$cat_id = isset($_GET['selectcat']) ? (int)$_GET['selectcat'] : null;
+$cat_id = isset($_GET['selectcat']) ? (int) $_GET['selectcat'] : null;
 $action = isset($_GET['action']) && $_GET['action'] ? $_GET['action'] : null;
 
 $userList = CourseManager::get_user_list_from_course_code(
@@ -113,8 +113,8 @@ 
 
 $course_code = api_get_course_id();
 
-$interbreadcrumb[] = array('url' => Security::remove_XSS($_SESSION['gradebook_dest']).'?',	'name' => get_lang('Gradebook'));
-$interbreadcrumb[] = array('url' => '#','name' => get_lang('GradebookListOfStudentsReports'));
+$interbreadcrumb[] = array('url' => Security::remove_XSS($_SESSION['gradebook_dest']).'?', 'name' => get_lang('Gradebook'));
+$interbreadcrumb[] = array('url' => '#', 'name' => get_lang('GradebookListOfStudentsReports'));
 
 $this_section = SECTION_COURSES;
 
@@ -131,7 +131,7 @@ 
 }
 echo '</div>';
 
-if (count($userList) == 0 ) {
+if (count($userList) == 0) {
     echo Display::display_warning_message(get_lang('NoResultsAvailable'));
 } else {
     echo '<br /><br /><table class="data_table">';

main/gradebook/gradebook_scoring_system.php

Spacing +13 added lines, -13 removed lines

@@ -13,7 +13,7 @@ 
     api_not_allowed();
 }
 
-$htmlHeadXtra[]= '<script>
+$htmlHeadXtra[] = '<script>
 function plusItem(item) {
     document.getElementById(item).style.display = "inline";
     document.getElementById("plus-"+item).style.display = "none";
@@ -45,10 +45,10 @@ 
 );
 
 $select_cat = intval($_GET['selectcat']);
-$displayscore= ScoreDisplay :: instance();
+$displayscore = ScoreDisplay :: instance();
 $customdisplays = $displayscore->get_custom_score_display_settings();
 
-$nr_items = (count($customdisplays) != '0' )? count($customdisplays) : '1';
+$nr_items = (count($customdisplays) != '0') ? count($customdisplays) : '1';
 $scoreform = new ScoreDisplayForm(
     'scoring_system_form',
     api_get_self().'?selectcat='.$select_cat.'&'.api_get_cidreq()
@@ -57,18 +57,18 @@ 
 if ($scoreform->validate()) {
     $value_export = '';
     $value_export = $scoreform->exportValues();
-    $value_export = isset($value_export) ? $scoreform->exportValues(): '';
+    $value_export = isset($value_export) ? $scoreform->exportValues() : '';
     $values = $value_export;
 
     // create new array of custom display settings
     // this loop also checks if all score ranges are unique
 
-    $scoringdisplay= array ();
+    $scoringdisplay = array();
     $ranges_ok = true;
-    $endscore= isset($values['endscore']) ? $values['endscore'] : null;
-    $displaytext=isset($values['displaytext']) ? $values['displaytext'] : null;
-    for ($counter= 1; $ranges_ok && $counter <= 20; $counter++) {
-        $setting= array ();
+    $endscore = isset($values['endscore']) ? $values['endscore'] : null;
+    $displaytext = isset($values['displaytext']) ? $values['displaytext'] : null;
+    for ($counter = 1; $ranges_ok && $counter <= 20; $counter++) {
+        $setting = array();
         $setting['score'] = $endscore[$counter];
         $setting['display'] = $displaytext[$counter];
         if (!empty($setting['score'])) {
@@ -77,12 +77,12 @@ 
                     $ranges_ok = false;
                 }
             }
-            $scoringdisplay[]= $setting;
+            $scoringdisplay[] = $setting;
         }
     }
 
     if (!$ranges_ok) {
-        header('Location: ' . api_get_self() . '?nouniqueranges=&selectcat=' . $select_cat.'&'.api_get_cidreq());
+        header('Location: '.api_get_self().'?nouniqueranges=&selectcat='.$select_cat.'&'.api_get_cidreq());
         exit;
     }
 
@@ -102,8 +102,8 @@ 
 $this_section = SECTION_COURSES;
 Display :: display_header(get_lang('ScoreEdit'));
 
-if (((isset($_GET['isStudentView']) && $_GET['isStudentView']=='false') ||
-    (isset($_GET['selectcat']) && ($_SESSION['studentview']=='teacherview')))
+if (((isset($_GET['isStudentView']) && $_GET['isStudentView'] == 'false') ||
+    (isset($_GET['selectcat']) && ($_SESSION['studentview'] == 'teacherview')))
 ) {
     if (isset ($_GET['scoringupdated'])) {
         Display::display_confirmation_message(get_lang('ScoringUpdated'), false);