chamilo / chamilo-lms

main/admin/system_announcements.php

Spacing +20 added lines, -20 removed lines

@@ -14,7 +14,7 @@ 
 
 // Setting the section (for the tabs).
 $this_section = SECTION_PLATFORM_ADMIN;
-$_SESSION['this_section']=$this_section;
+$_SESSION['this_section'] = $this_section;
 
 $action = isset($_GET['action']) ? $_GET['action'] : null;
 $action_todo = false;
@@ -35,12 +35,12 @@ 
 }
 
 if (!empty($action)) {
-    $interbreadcrumb[] = array ("url" => "system_announcements.php", "name" => get_lang('SystemAnnouncements'));
+    $interbreadcrumb[] = array("url" => "system_announcements.php", "name" => get_lang('SystemAnnouncements'));
     if ($action == 'add') {
-        $interbreadcrumb[] = array ("url" => '#', "name" => get_lang('AddAnnouncement'));
+        $interbreadcrumb[] = array("url" => '#', "name" => get_lang('AddAnnouncement'));
     }
     if ($action == 'edit') {
-        $interbreadcrumb[] = array ("url" => '#', "name" => get_lang('Edit'));
+        $interbreadcrumb[] = array("url" => '#', "name" => get_lang('Edit'));
     }
 } else {
     $tool_name = get_lang('SystemAnnouncements');
@@ -66,7 +66,7 @@ 
 }
 
 // Actions
-switch($action) {
+switch ($action) {
     case 'make_visible':
     case 'make_invisible':
         $status = false;
@@ -81,7 +81,7 @@ 
         Display :: display_confirmation_message(get_lang('AnnouncementDeleted'));
         break;
     case 'delete_selected':
-        foreach($_POST['id'] as $index => $id) {
+        foreach ($_POST['id'] as $index => $id) {
             SystemAnnouncementManager :: delete_announcement($id);
         }
         Display :: display_confirmation_message(get_lang('AnnouncementDeleted'));
@@ -137,11 +137,11 @@ 
     $language_list = api_get_languages();
     $language_list_with_keys = array();
     $language_list_with_keys['all'] = get_lang('All');
-    for ($i=0; $i<count($language_list['name']) ; $i++) {
+    for ($i = 0; $i < count($language_list['name']); $i++) {
         $language_list_with_keys[$language_list['folder'][$i]] = $language_list['name'][$i];
     }
 
-    $form->addElement('select', 'lang',get_lang('Language'), $language_list_with_keys);
+    $form->addElement('select', 'lang', get_lang('Language'), $language_list_with_keys);
     $form->addHtmlEditor(
         'content',
         get_lang('Content'),
@@ -157,9 +157,9 @@ 
 
     $group = array();
 
-    $group[]= $form->createElement('checkbox', 'visible_teacher', null, get_lang('Teacher'));
-    $group[]= $form->createElement('checkbox', 'visible_student', null, get_lang('Student'));
-    $group[]= $form->createElement('checkbox', 'visible_guest', null, get_lang('Guest'));
+    $group[] = $form->createElement('checkbox', 'visible_teacher', null, get_lang('Teacher'));
+    $group[] = $form->createElement('checkbox', 'visible_student', null, get_lang('Student'));
+    $group[] = $form->createElement('checkbox', 'visible_guest', null, get_lang('Guest'));
 
     $form->addGroup($group, null, get_lang('Visible'), '');
 
@@ -182,15 +182,15 @@ 
 
     $form->addElement('checkbox', 'send_mail', null, get_lang('SendMail'));
 
-    if (isset($_REQUEST['action']) && $_REQUEST['action']=='add') {
+    if (isset($_REQUEST['action']) && $_REQUEST['action'] == 'add') {
         $form->addElement('checkbox', 'add_to_calendar', null, get_lang('AddToCalendar'));
-        $text=get_lang('AddNews');
-        $class='add';
+        $text = get_lang('AddNews');
+        $class = 'add';
         $form->addElement('hidden', 'action', 'add');
 
     } elseif (isset($_REQUEST['action']) && $_REQUEST['action'] == 'edit') {
-        $text=get_lang('EditNews');
-        $class='save';
+        $text = get_lang('EditNews');
+        $class = 'save';
         $form->addElement('hidden', 'action', 'edit');
     }
     $form->addElement('checkbox', 'send_email_test', null, get_lang('SendOnlyAnEmailToMySelfToTest'));
@@ -280,7 +280,7 @@ 
 
 if ($show_announcement_list) {
     $announcements = SystemAnnouncementManager :: get_all_announcements();
-    $announcement_data = array ();
+    $announcement_data = array();
     foreach ($announcements as $index => $announcement) {
         $row = array();
         $row[] = $announcement->id;
@@ -288,9 +288,9 @@ 
         $row[] = $announcement->title;
         $row[] = api_convert_and_format_date($announcement->date_start);
         $row[] = api_convert_and_format_date($announcement->date_end);
-        $row[] = "<a href=\"?id=".$announcement->id."&person=".SystemAnnouncementManager::VISIBLE_TEACHER."&action=". ($announcement->visible_teacher ? 'make_invisible' : 'make_visible')."\">".Display::return_icon(($announcement->visible_teacher  ? 'eyes.png' : 'eyes-close.png'), get_lang('ShowOrHide'))."</a>";
-        $row[] = "<a href=\"?id=".$announcement->id."&person=".SystemAnnouncementManager::VISIBLE_STUDENT."&action=". ($announcement->visible_student  ? 'make_invisible' : 'make_visible')."\">".Display::return_icon(($announcement->visible_student  ? 'eyes.png' : 'eyes-close.png'), get_lang('ShowOrHide'))."</a>";
-        $row[] = "<a href=\"?id=".$announcement->id."&person=".SystemAnnouncementManager::VISIBLE_GUEST."&action=". ($announcement->visible_guest ? 'make_invisible' : 'make_visible')."\">".Display::return_icon(($announcement->visible_guest  ? 'eyes.png' : 'eyes-close.png'), get_lang('ShowOrHide'))."</a>";
+        $row[] = "<a href=\"?id=".$announcement->id."&person=".SystemAnnouncementManager::VISIBLE_TEACHER."&action=".($announcement->visible_teacher ? 'make_invisible' : 'make_visible')."\">".Display::return_icon(($announcement->visible_teacher ? 'eyes.png' : 'eyes-close.png'), get_lang('ShowOrHide'))."</a>";
+        $row[] = "<a href=\"?id=".$announcement->id."&person=".SystemAnnouncementManager::VISIBLE_STUDENT."&action=".($announcement->visible_student ? 'make_invisible' : 'make_visible')."\">".Display::return_icon(($announcement->visible_student ? 'eyes.png' : 'eyes-close.png'), get_lang('ShowOrHide'))."</a>";
+        $row[] = "<a href=\"?id=".$announcement->id."&person=".SystemAnnouncementManager::VISIBLE_GUEST."&action=".($announcement->visible_guest ? 'make_invisible' : 'make_visible')."\">".Display::return_icon(($announcement->visible_guest ? 'eyes.png' : 'eyes-close.png'), get_lang('ShowOrHide'))."</a>";
 
         $row[] = $announcement->lang;
         $row[] = "<a href=\"?action=edit&id=".$announcement->id."\">".Display::return_icon('edit.png', get_lang('Edit'), array(), ICON_SIZE_SMALL)."</a> <a href=\"?action=delete&id=".$announcement->id."\"  onclick=\"javascript:if(!confirm('".addslashes(api_htmlentities(get_lang("ConfirmYourChoice"), ENT_QUOTES))."')) return false;\">".Display::return_icon('delete.png', get_lang('Delete'), array(), ICON_SIZE_SMALL)."</a>";

main/admin/specific_fields.php

Indentation +7 added lines, -7 removed lines

@@ -43,15 +43,15 @@ 
 $table->set_column_filter(3, 'edit_filter');
 
 function edit_filter($id,$url_params,$row) {
-	global $charset;
-	$return = '<a href="specific_fields_add.php?action=edit&field_id='.$row[0].'">'.Display::return_icon('edit.gif',get_lang('Edit')).'</a>';
-	$return .= ' <a href="'.api_get_self().'?action=delete&field_id='.$row[0].'" onclick="javascript:if(!confirm('."'".addslashes(api_htmlentities(get_lang("ConfirmYourChoice"),ENT_QUOTES,$charset))."'".')) return false;">'.Display::return_icon('delete.gif',get_lang('Delete')).'</a>';
-	return $return;
+    global $charset;
+    $return = '<a href="specific_fields_add.php?action=edit&field_id='.$row[0].'">'.Display::return_icon('edit.gif',get_lang('Edit')).'</a>';
+    $return .= ' <a href="'.api_get_self().'?action=delete&field_id='.$row[0].'" onclick="javascript:if(!confirm('."'".addslashes(api_htmlentities(get_lang("ConfirmYourChoice"),ENT_QUOTES,$charset))."'".')) return false;">'.Display::return_icon('delete.gif',get_lang('Delete')).'</a>';
+    return $return;
 }
 
 if ($_REQUEST['action'] == 'delete') {
-	delete_specific_field($_REQUEST['field_id']);
-	header('Location: specific_fields.php?message='.get_lang('FieldRemoved'));
+    delete_specific_field($_REQUEST['field_id']);
+    header('Location: specific_fields.php?message='.get_lang('FieldRemoved'));
 }
 
 // Start output
@@ -61,7 +61,7 @@ 
 echo Display::display_normal_message(get_lang('SpecificSearchFieldsIntro'));
 
 if(!empty($_GET['message'])) {
-  Display::display_confirmation_message($_GET['message']);
+    Display::display_confirmation_message($_GET['message']);
 }
 
 echo '<div class="actions">';

Spacing +14 added lines, -14 removed lines

@@ -15,36 +15,36 @@ 
 api_protect_admin_script();
 
 // Breadcrumb
-$interbreadcrumb[] = array ('url' => 'index.php', 'name' => get_lang('PlatformAdmin'));
-$interbreadcrumb[] = array ('url' => 'settings.php?category=Search', 'name' => get_lang('PlatformConfigSettings'));
+$interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('PlatformAdmin'));
+$interbreadcrumb[] = array('url' => 'settings.php?category=Search', 'name' => get_lang('PlatformConfigSettings'));
 
 $libpath = api_get_path(LIBRARY_PATH);
 
 include_once $libpath.'specific_fields_manager.lib.php';
 
 // Create an add-field box
-$form = new FormValidator('add_field','post','','',null,false);
-$renderer =& $form->defaultRenderer();
+$form = new FormValidator('add_field', 'post', '', '', null, false);
+$renderer = & $form->defaultRenderer();
 $renderer->setCustomElementTemplate('<span>{element}</span> ');
-$form->addElement('static','search_advanced_link',null,'<a href="specific_fields_add.php">'.Display::return_icon('fieldadd.gif').get_lang('AddSpecificSearchField').'</a>');
+$form->addElement('static', 'search_advanced_link', null, '<a href="specific_fields_add.php">'.Display::return_icon('fieldadd.gif').get_lang('AddSpecificSearchField').'</a>');
 
 // Create a sortable table with specific fields data
-$column_show = array(1,1,1);
-$column_order = array(3,2,1);
+$column_show = array(1, 1, 1);
+$column_order = array(3, 2, 1);
 $extra_fields = get_specific_field_list();
 $number_of_extra_fields = count($extra_fields);
 
-$table = new SortableTableFromArrayConfig($extra_fields,2,50,'',$column_show,$column_order);
-$table->set_header(0, '&nbsp;', false,null,'width="2%"', 'style="display:none"');
+$table = new SortableTableFromArrayConfig($extra_fields, 2, 50, '', $column_show, $column_order);
+$table->set_header(0, '&nbsp;', false, null, 'width="2%"', 'style="display:none"');
 $table->set_header(1, get_lang('Code'), TRUE, 'width="10%"');
 $table->set_header(2, get_lang('Name'));
-$table->set_header(3, get_lang('Modify'),false,'width="10%"');
+$table->set_header(3, get_lang('Modify'), false, 'width="10%"');
 $table->set_column_filter(3, 'edit_filter');
 
-function edit_filter($id,$url_params,$row) {
+function edit_filter($id, $url_params, $row) {
 	global $charset;
-	$return = '<a href="specific_fields_add.php?action=edit&field_id='.$row[0].'">'.Display::return_icon('edit.gif',get_lang('Edit')).'</a>';
-	$return .= ' <a href="'.api_get_self().'?action=delete&field_id='.$row[0].'" onclick="javascript:if(!confirm('."'".addslashes(api_htmlentities(get_lang("ConfirmYourChoice"),ENT_QUOTES,$charset))."'".')) return false;">'.Display::return_icon('delete.gif',get_lang('Delete')).'</a>';
+	$return = '<a href="specific_fields_add.php?action=edit&field_id='.$row[0].'">'.Display::return_icon('edit.gif', get_lang('Edit')).'</a>';
+	$return .= ' <a href="'.api_get_self().'?action=delete&field_id='.$row[0].'" onclick="javascript:if(!confirm('."'".addslashes(api_htmlentities(get_lang("ConfirmYourChoice"), ENT_QUOTES, $charset))."'".')) return false;">'.Display::return_icon('delete.gif', get_lang('Delete')).'</a>';
 	return $return;
 }
 
@@ -60,7 +60,7 @@ 
 Display::display_header(get_lang('SpecificSearchFields'));
 echo Display::display_normal_message(get_lang('SpecificSearchFieldsIntro'));
 
-if(!empty($_GET['message'])) {
+if (!empty($_GET['message'])) {
   Display::display_confirmation_message($_GET['message']);
 }
 

main/admin/skills_profile.php

Spacing +8 added lines, -8 removed lines

@@ -24,7 +24,7 @@ 
 $skill_profile   = new SkillProfile();
 $skill_rel_user  = new SkillRelUser();
 
-$url  = api_get_path(WEB_AJAX_PATH).'skill.ajax.php';
+$url = api_get_path(WEB_AJAX_PATH).'skill.ajax.php';
 
 $tpl = new Template(get_lang('Skills'));
 
@@ -55,16 +55,16 @@ 
         $_SESSION['skills'] = $skills;
 
     } else {
-        $skills = isset($_SESSION['skills']) ? $_SESSION['skills']: array();
+        $skills = isset($_SESSION['skills']) ? $_SESSION['skills'] : array();
     }
 } else {
-    $skills = isset($_SESSION['skills']) ? $_SESSION['skills']: array();
+    $skills = isset($_SESSION['skills']) ? $_SESSION['skills'] : array();
 }
 
 $user_list = array();
 $count_skills = count($skills);
 
-$users  = $skill_rel_user->get_user_by_skills($skills);
+$users = $skill_rel_user->get_user_by_skills($skills);
 
 if (!empty($users)) {
     foreach ($users as $user) {
@@ -72,7 +72,7 @@ 
         $user_list[$user['user_id']]['user'] = $user_info;
         $my_user_skills = $skill_rel_user->get_user_skills($user['user_id']);
         $user_skills = array();
-        $found_counts = 0 ;
+        $found_counts = 0;
         foreach ($my_user_skills as $my_skill) {
             $found = false;
             if (in_array($my_skill['skill_id'], $skills)) {
@@ -89,7 +89,7 @@ 
         $user_list[$user['user_id']]['total_found_skills'] = $found_counts;
     }
     $ordered_user_list = array();
-    foreach($user_list as $user_id => $user_data) {
+    foreach ($user_list as $user_id => $user_data) {
         $ordered_user_list[$user_data['total_found_skills']][] = $user_data;
     }
     if (!empty($ordered_user_list)) {
@@ -103,14 +103,14 @@ 
 
 if (!empty($skills)) {
     $counter = 0;
-    foreach($skills as $hidden_skill_id) {
+    foreach ($skills as $hidden_skill_id) {
         $form->addElement('hidden', 'hidden_skills[]', $hidden_skill_id);
         $counter++;
     }
 }
 
 if (!empty($skills)) {
-    foreach($skills as $my_skill) {
+    foreach ($skills as $my_skill) {
         $total_skills_to_search[$my_skill] = $my_skill;
     }
 }

main/admin/extra_field_options.php

Spacing +15 added lines, -15 removed lines

@@ -18,7 +18,7 @@ 
 $htmlHeadXtra[] = api_get_jqgrid_js();
 
 // setting breadcrumbs
-$interbreadcrumb[]=array('url' => 'index.php','name' => get_lang('PlatformAdmin'));
+$interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('PlatformAdmin'));
 
 $tool_name = null;
 
@@ -39,23 +39,23 @@ 
 $token = Security::get_token();
 
 if ($action == 'add') {
-    $interbreadcrumb[]=array('url' => 'extra_fields.php?type='.$extra_field->type,'name' => $extra_field->pageName);
-    $interbreadcrumb[]=array('url' => 'extra_fields.php?type='.$extra_field->type.'&action=edit&id='.$extra_field_info['id'],'name' => $extra_field_info['display_text']);
-    $interbreadcrumb[]=array('url' => 'extra_field_options.php?type='.$extra_field->type.'&field_id='.$extra_field_info['id'], 'name' => get_lang('EditExtraFieldOptions'));
-    $interbreadcrumb[]=array('url' => '#','name' => get_lang('Add'));
+    $interbreadcrumb[] = array('url' => 'extra_fields.php?type='.$extra_field->type, 'name' => $extra_field->pageName);
+    $interbreadcrumb[] = array('url' => 'extra_fields.php?type='.$extra_field->type.'&action=edit&id='.$extra_field_info['id'], 'name' => $extra_field_info['display_text']);
+    $interbreadcrumb[] = array('url' => 'extra_field_options.php?type='.$extra_field->type.'&field_id='.$extra_field_info['id'], 'name' => get_lang('EditExtraFieldOptions'));
+    $interbreadcrumb[] = array('url' => '#', 'name' => get_lang('Add'));
 } elseif ($action == 'edit') {
-    $interbreadcrumb[]=array('url' => 'extra_fields.php?type='.$extra_field->type,'name' => $extra_field->pageName);
-    $interbreadcrumb[]=array('url' => 'extra_fields.php?type='.$extra_field->type.'&action=edit&id='.$extra_field_info['id'],'name' => $extra_field_info['display_text']);
-    $interbreadcrumb[]=array('url' => 'extra_field_options.php?type='.$extra_field->type.'&field_id='.$extra_field_info['id'], 'name' => get_lang('EditExtraFieldOptions'));
+    $interbreadcrumb[] = array('url' => 'extra_fields.php?type='.$extra_field->type, 'name' => $extra_field->pageName);
+    $interbreadcrumb[] = array('url' => 'extra_fields.php?type='.$extra_field->type.'&action=edit&id='.$extra_field_info['id'], 'name' => $extra_field_info['display_text']);
+    $interbreadcrumb[] = array('url' => 'extra_field_options.php?type='.$extra_field->type.'&field_id='.$extra_field_info['id'], 'name' => get_lang('EditExtraFieldOptions'));
 
-    $interbreadcrumb[]=array('url' => '#','name' => get_lang('Edit'));
+    $interbreadcrumb[] = array('url' => '#', 'name' => get_lang('Edit'));
 } else {
-    $interbreadcrumb[]=array('url' => 'extra_fields.php?type='.$extra_field->type,'name' => $extra_field->pageName);
-    $interbreadcrumb[]=array(
+    $interbreadcrumb[] = array('url' => 'extra_fields.php?type='.$extra_field->type, 'name' => $extra_field->pageName);
+    $interbreadcrumb[] = array(
         'url' =>  'extra_fields.php?type='.$extra_field->type.'&action=edit&id='.$extra_field_info['id'],
         'name' => $extra_field_info['display_text']
     );
-    $interbreadcrumb[]=array('url' => '#','name' => get_lang('EditExtraFieldOptions'));
+    $interbreadcrumb[] = array('url' => '#', 'name' => get_lang('EditExtraFieldOptions'));
 }
 
 //jqgrid will use this URL to do the selects
@@ -108,12 +108,12 @@ 
 
 //With this function we can add actions to the jgrid (edit, delete, etc)
 $action_links = 'function action_formatter(cellvalue, options, rowObject) {
-    return \'<a href="?action=edit&'.$params.'&id=\'+options.rowId+\'">'.Display::return_icon('edit.png',get_lang('Edit'),'',ICON_SIZE_SMALL).'</a>'.
-    '&nbsp;<a onclick="javascript:if(!confirm('."\'".addslashes(get_lang("ConfirmYourChoice"))."\'".')) return false;"  href="?sec_token='.$token.'&action=delete&'.$params.'&id=\'+options.rowId+\'">'.Display::return_icon('delete.png',get_lang('Delete'),'',ICON_SIZE_SMALL).'</a>'.
+    return \'<a href="?action=edit&'.$params.'&id=\'+options.rowId+\'">'.Display::return_icon('edit.png', get_lang('Edit'), '', ICON_SIZE_SMALL).'</a>'.
+    '&nbsp;<a onclick="javascript:if(!confirm('."\'".addslashes(get_lang("ConfirmYourChoice"))."\'".')) return false;"  href="?sec_token='.$token.'&action=delete&'.$params.'&id=\'+options.rowId+\'">'.Display::return_icon('delete.png', get_lang('Delete'), '', ICON_SIZE_SMALL).'</a>'.
     '\';
  }';
 
-$htmlHeadXtra[]='<script>
+$htmlHeadXtra[] = '<script>
 $(function() {
     // grid definition see the $obj->display() function
     '.Display::grid_js(

main/admin/add_users_to_usergroup.php

Braces +8 added lines, -2 removed lines

@@ -345,7 +345,10 @@ 
 $searchForm->display();
 echo '</div>';
 ?>
-<form name="formulaire" method="post" action="<?php echo api_get_self(); ?>?id=<?php echo $id; if(!empty($_GET['add'])) echo '&add=true' ; ?>" style="margin:0px;">
+<form name="formulaire" method="post" action="<?php echo api_get_self(); ?>?id=<?php echo $id; if(!empty($_GET['add'])) {
+    echo '&add=true' ;
+}
+?>" style="margin:0px;">
 <?php
 echo '<legend>'.$tool_name.': '.$data['name'].'</legend>';
 
@@ -420,7 +423,10 @@ 
     ?>
     <br />
       <label class="control-label">
-          <input type="checkbox" <?php if ($user_with_any_group) echo 'checked="checked"';?> onchange="checked_in_no_group(this.checked);" name="user_with_any_group" id="user_with_any_group_id">
+          <input type="checkbox" <?php if ($user_with_any_group) {
+    echo 'checked="checked"';
+}
+?> onchange="checked_in_no_group(this.checked);" name="user_with_any_group" id="user_with_any_group_id">
           <?php echo get_lang('UsersRegisteredInAnyGroup'); ?>
       </label>
     </div>

Spacing +23 added lines, -23 removed lines

@@ -18,8 +18,8 @@ 
 api_protect_admin_script(true);
 
 // setting breadcrumbs
-$interbreadcrumb[]= array('url' => 'index.php','name' => get_lang('PlatformAdmin'));
-$interbreadcrumb[]= array('url' => 'usergroups.php','name' => get_lang('Classes'));
+$interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('PlatformAdmin'));
+$interbreadcrumb[] = array('url' => 'usergroups.php', 'name' => get_lang('Classes'));
 
 // Database Table Definitions
 
@@ -91,14 +91,14 @@ 
 
 </script>';
 
-$form_sent  = 0;
+$form_sent = 0;
 
 $extra_field_list = UserManager::get_extra_fields();
 $new_field_list = array();
 if (is_array($extra_field_list)) {
     foreach ($extra_field_list as $extra_field) {
         //if is enabled to filter and is a "<select>" field type
-        if ($extra_field[8]==1 && $extra_field[2]==4 ) {
+        if ($extra_field[8] == 1 && $extra_field[2] == 4) {
             $new_field_list[] = array(
                 'name'=> $extra_field[3],
                 'variable' => $extra_field[1], 'data'=> $extra_field[9]
@@ -146,7 +146,7 @@ 
         foreach ($users as $user) {
             $data[] = array($user['username'], $groupInfo['name']);
         }
-        $filename = 'export_user_class_' . api_get_local_time();
+        $filename = 'export_user_class_'.api_get_local_time();
         Export::arrayToCsv($data, $filename);
         exit;
     }
@@ -156,11 +156,11 @@ 
 // Filter by Extra Fields
 $use_extra_fields = false;
 if (is_array($extra_field_list)) {
-    if (is_array($new_field_list) && count($new_field_list)>0 ) {
+    if (is_array($new_field_list) && count($new_field_list) > 0) {
         foreach ($new_field_list as $new_field) {
             $varname = 'field_'.$new_field['variable'];
             if (UserManager::is_extra_field_available($new_field['variable'])) {
-                if (isset($_POST[$varname]) && $_POST[$varname]!='0') {
+                if (isset($_POST[$varname]) && $_POST[$varname] != '0') {
                     $use_extra_fields = true;
                     $extra_field_result[] = UserManager::get_extra_user_data_by_value(
                         $new_field['variable'],
@@ -174,10 +174,10 @@ 
 
 if ($use_extra_fields) {
     $final_result = array();
-    if (count($extra_field_result)>1) {
-        for ($i=0; $i<count($extra_field_result)-1; $i++) {
-            if (is_array($extra_field_result[$i+1])) {
-                $final_result = array_intersect($extra_field_result[$i], $extra_field_result[$i+1]);
+    if (count($extra_field_result) > 1) {
+        for ($i = 0; $i < count($extra_field_result) - 1; $i++) {
+            if (is_array($extra_field_result[$i + 1])) {
+                $final_result = array_intersect($extra_field_result[$i], $extra_field_result[$i + 1]);
             }
         }
     } else {
@@ -196,7 +196,7 @@ 
 
 $searchForm = new FormValidator('search', 'get', api_get_self().'?id='.$id);
 $searchForm->addHeader(get_lang('AdvancedSearch'));
-$renderer =& $searchForm->defaultRenderer();
+$renderer = & $searchForm->defaultRenderer();
 
 $searchForm->addElement('hidden', 'id', $id);
 foreach ($filters as $param) {
@@ -345,13 +345,13 @@ 
 $searchForm->display();
 echo '</div>';
 ?>
-<form name="formulaire" method="post" action="<?php echo api_get_self(); ?>?id=<?php echo $id; if(!empty($_GET['add'])) echo '&add=true' ; ?>" style="margin:0px;">
+<form name="formulaire" method="post" action="<?php echo api_get_self(); ?>?id=<?php echo $id; if (!empty($_GET['add'])) echo '&add=true'; ?>" style="margin:0px;">
 <?php
 echo '<legend>'.$tool_name.': '.$data['name'].'</legend>';
 
 
     if (is_array($extra_field_list)) {
-        if (is_array($new_field_list) && count($new_field_list)>0) {
+        if (is_array($new_field_list) && count($new_field_list) > 0) {
             echo '<h3>'.get_lang('FilterByUser').'</h3>';
             foreach ($new_field_list as $new_field) {
                 echo $new_field['name'];
@@ -359,7 +359,7 @@ 
                 echo '&nbsp;<select name="'.$varname.'">';
                 echo '<option value="0">--'.get_lang('Select').'--</option>';
                 foreach ($new_field['data'] as $option) {
-                    $checked='';
+                    $checked = '';
                     if (isset($_POST[$varname])) {
                         if ($_POST[$varname] == $option[1]) {
                             $checked = 'selected="true"';
@@ -386,15 +386,15 @@ 
         <?php if ($data['group_type'] == UserGroup::SOCIAL_CLASS) { ?>
         <select name="relation" id="relation">
             <option value=""><?php echo get_lang('SelectARelationType')?></option>
-            <option value="<?php echo GROUP_USER_PERMISSION_ADMIN ?>" <?php echo ((isset($relation) && $relation == GROUP_USER_PERMISSION_ADMIN)?'selected=selected':'') ?> >
+            <option value="<?php echo GROUP_USER_PERMISSION_ADMIN ?>" <?php echo ((isset($relation) && $relation == GROUP_USER_PERMISSION_ADMIN) ? 'selected=selected' : '') ?> >
                 <?php echo get_lang('Admin') ?></option>
-            <option value="<?php echo GROUP_USER_PERMISSION_READER ?>" <?php echo ((isset($relation) && $relation == GROUP_USER_PERMISSION_READER)?'selected=selected':'') ?> >
+            <option value="<?php echo GROUP_USER_PERMISSION_READER ?>" <?php echo ((isset($relation) && $relation == GROUP_USER_PERMISSION_READER) ? 'selected=selected' : '') ?> >
                 <?php echo get_lang('Reader') ?></option>
-            <option value="<?php echo GROUP_USER_PERMISSION_PENDING_INVITATION ?>" <?php echo ((isset($relation) && $relation == GROUP_USER_PERMISSION_PENDING_INVITATION)?'selected=selected':'') ?> >
+            <option value="<?php echo GROUP_USER_PERMISSION_PENDING_INVITATION ?>" <?php echo ((isset($relation) && $relation == GROUP_USER_PERMISSION_PENDING_INVITATION) ? 'selected=selected' : '') ?> >
                 <?php echo get_lang('PendingInvitation') ?></option>
-            <option value="<?php echo GROUP_USER_PERMISSION_MODERATOR ?>" <?php echo ((isset($relation) && $relation == GROUP_USER_PERMISSION_MODERATOR)?'selected=selected':'') ?> >
+            <option value="<?php echo GROUP_USER_PERMISSION_MODERATOR ?>" <?php echo ((isset($relation) && $relation == GROUP_USER_PERMISSION_MODERATOR) ? 'selected=selected' : '') ?> >
                 <?php echo get_lang('Moderator') ?></option>
-            <option value="<?php echo GROUP_USER_PERMISSION_HRM ?>" <?php echo ((isset($relation) && $relation == GROUP_USER_PERMISSION_HRM)?'selected=selected':'') ?> >
+            <option value="<?php echo GROUP_USER_PERMISSION_HRM ?>" <?php echo ((isset($relation) && $relation == GROUP_USER_PERMISSION_HRM) ? 'selected=selected' : '') ?> >
                 <?php echo get_lang('Drh') ?></option>
         </select>
         <?php } ?>
@@ -414,13 +414,13 @@ 
         'elements_not_in_name',
         $elements_not_in,
         '',
-        array('class'=>'col-md-7', 'multiple'=>'multiple','id'=>'elements_not_in','size'=>'15px'),
+        array('class'=>'col-md-7', 'multiple'=>'multiple', 'id'=>'elements_not_in', 'size'=>'15px'),
         false
     );
     ?>
     <br />
       <label class="control-label">
-          <input type="checkbox" <?php if ($user_with_any_group) echo 'checked="checked"';?> onchange="checked_in_no_group(this.checked);" name="user_with_any_group" id="user_with_any_group_id">
+          <input type="checkbox" <?php if ($user_with_any_group) echo 'checked="checked"'; ?> onchange="checked_in_no_group(this.checked);" name="user_with_any_group" id="user_with_any_group_id">
           <?php echo get_lang('UsersRegisteredInAnyGroup'); ?>
       </label>
     </div>
@@ -444,7 +444,7 @@ 
             'elements_in_name[]',
             $elements_in,
             '',
-            array('class'=>'col-md-7', 'multiple'=>'multiple','id'=>'elements_in','size'=>'15px'),
+            array('class'=>'col-md-7', 'multiple'=>'multiple', 'id'=>'elements_in', 'size'=>'15px'),
             false
         );
         unset($sessionUsersList);

main/admin/system_status.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 $this_section = SECTION_PLATFORM_ADMIN;
 // User permissions
 api_protect_admin_script();
-$interbreadcrumb[] = array ('url' => 'index.php', 'name' => get_lang('PlatformAdmin'));
+$interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('PlatformAdmin'));
 Display :: display_header(get_lang('SystemStatus'));
 $diag = new Diagnoser();
 $diag->show_html();

main/admin/special_exports.php

Indentation +65 added lines, -65 removed lines

@@ -28,8 +28,8 @@ 
 require_once '../coursecopy/classes/CourseSelectForm.class.php';
 
 if (function_exists('ini_set')) {
-	api_set_memory_limit('256M');
-	ini_set('max_execution_time',0);
+    api_set_memory_limit('256M');
+    ini_set('max_execution_time',0);
 }
 
 // Displaying the header
@@ -50,36 +50,36 @@ 
 if ((isset($_POST['action']) && $_POST['action'] == 'course_select_form') ||
     (isset ($_POST['backup_option']) && $_POST['backup_option'] == 'full_backup')
 ) {
-	$export = false;
-	if (isset ($_POST['action']) && $_POST['action'] == 'course_select_form') {
-		$FileZip = create_zip();
-		$to_group_id = 0;
-		$sql_session = "SELECT id, name FROM $tbl_session ";
-		$query_session = Database::query($sql_session);
-		$ListSession = array();
-		while ($rows_session = Database::fetch_assoc($query_session)) {
-			$ListSession[$rows_session['id']] = $rows_session['name'];
-		}
+    $export = false;
+    if (isset ($_POST['action']) && $_POST['action'] == 'course_select_form') {
+        $FileZip = create_zip();
+        $to_group_id = 0;
+        $sql_session = "SELECT id, name FROM $tbl_session ";
+        $query_session = Database::query($sql_session);
+        $ListSession = array();
+        while ($rows_session = Database::fetch_assoc($query_session)) {
+            $ListSession[$rows_session['id']] = $rows_session['name'];
+        }
 
         $groupCondition = " props.to_group_id = $to_group_id";
         if (empty($to_group_id)) {
             $groupCondition = " (props.to_group_id = 0 OR props.to_group_id IS NULL)";
         }
 
-		$zip_folder=new PclZip($FileZip['TEMP_FILE_ZIP']);
-		if(!isset($_POST['resource']) || count($_POST['resource']) == 0 ) {
-			Display::display_error_message(get_lang('ErrorMsgSpecialExport'));
-		} else {
-			$Resource = $_POST['resource'];
+        $zip_folder=new PclZip($FileZip['TEMP_FILE_ZIP']);
+        if(!isset($_POST['resource']) || count($_POST['resource']) == 0 ) {
+            Display::display_error_message(get_lang('ErrorMsgSpecialExport'));
+        } else {
+            $Resource = $_POST['resource'];
 
-			foreach ($Resource as $Code_course => $Sessions) {
-				$_course 		= api_get_course_info($Code_course);
-				$tbl_document 	= Database::get_course_table(TABLE_DOCUMENT);
-				$tbl_property 	= Database::get_course_table(TABLE_ITEM_PROPERTY);
-				$course_id 		= $_course['real_id'];
+            foreach ($Resource as $Code_course => $Sessions) {
+                $_course 		= api_get_course_info($Code_course);
+                $tbl_document 	= Database::get_course_table(TABLE_DOCUMENT);
+                $tbl_property 	= Database::get_course_table(TABLE_ITEM_PROPERTY);
+                $course_id 		= $_course['real_id'];
 
-				//Add item to the zip file course
-				$sql = "SELECT path FROM $tbl_document AS docs, $tbl_property AS props
+                //Add item to the zip file course
+                $sql = "SELECT path FROM $tbl_document AS docs, $tbl_property AS props
                         WHERE props.tool='".TOOL_DOCUMENT."'
 						AND docs.id=props.ref
 						AND docs.path LIKE '".$querypath."/%'
@@ -89,8 +89,8 @@ 
 						AND $groupCondition
 						AND docs.c_id = $course_id
 						AND props.c_id = $course_id";
-				$query = Database::query($sql );
-				while ($rows_course_file = Database::fetch_assoc($query)) {
+                $query = Database::query($sql );
+                while ($rows_course_file = Database::fetch_assoc($query)) {
                     $zip_folder->add(
                         $FileZip['PATH_COURSE'].$_course['directory']."/document".$rows_course_file['path'],
                         PCLZIP_OPT_ADD_PATH,
@@ -98,12 +98,12 @@ 
                         PCLZIP_OPT_REMOVE_PATH,
                         $FileZip['PATH_COURSE'].$_course['directory']."/document".$FileZip['PATH_REMOVE']
                     );
-				}
+                }
 
-				foreach ($Sessions as $IdSession => $value){
-					$session_id = Security::remove_XSS($IdSession);
-					//Add tem to the zip file session course
-					$sql_session_doc = "SELECT path FROM $tbl_document AS docs, $tbl_property AS props
+                foreach ($Sessions as $IdSession => $value){
+                    $session_id = Security::remove_XSS($IdSession);
+                    //Add tem to the zip file session course
+                    $sql_session_doc = "SELECT path FROM $tbl_document AS docs, $tbl_property AS props
 						WHERE props.tool='".TOOL_DOCUMENT."'
 							AND docs.id=props.ref
 							AND docs.path LIKE '".$querypath."/%'
@@ -113,8 +113,8 @@ 
 							AND $groupCondition
 							AND docs.c_id = $course_id
 							AND props.c_id = $course_id";
-					$query_session_doc = Database::query($sql_session_doc);
-					while ($rows_course_session_file = Database::fetch_assoc($query_session_doc)) {
+                    $query_session_doc = Database::query($sql_session_doc);
+                    while ($rows_course_session_file = Database::fetch_assoc($query_session_doc)) {
                         $zip_folder->add(
                             $FileZip['PATH_COURSE'].$_course['directory'].'/document'.$rows_course_session_file['path'],
                             PCLZIP_OPT_ADD_PATH,
@@ -122,15 +122,15 @@ 
                             PCLZIP_OPT_REMOVE_PATH,
                             $FileZip['PATH_COURSE'].$_course['directory'].'/document'.$FileZip['PATH_REMOVE']
                         );
-					}
-				}
-			}
-			$name = rename_zip($FileZip);
-			$export = true;
-		}
-	} else {
-		$name = fullexportspecial();
-	}
+                    }
+                }
+            }
+            $name = rename_zip($FileZip);
+            $export = true;
+        }
+    } else {
+        $name = fullexportspecial();
+    }
 ?>
 <!-- Manual download <script language="JavaScript">
  // setTimeout(\'download_backup()\',2000);
@@ -143,23 +143,23 @@ 
 }
 
 if ($export && $name) {
-	Display::display_confirmation_message(get_lang('BackupCreated'));
-	echo '<br /><a class="btn btn-default" href="'.api_get_path(WEB_CODE_PATH).'course_info/download.php?archive='.urlencode($name).'&session=true">'.get_lang('Download').'</a>';
+    Display::display_confirmation_message(get_lang('BackupCreated'));
+    echo '<br /><a class="btn btn-default" href="'.api_get_path(WEB_CODE_PATH).'course_info/download.php?archive='.urlencode($name).'&session=true">'.get_lang('Download').'</a>';
 } else {
-	// Display forms especial export
-	if (isset ($_POST['backup_option']) && $_POST['backup_option'] == 'select_items') {
-		$cb = new CourseBuilder();
-		$course = $cb->build_session_course();
-		if($course === false){
-			Display::display_error_message(get_lang('ErrorMsgSpecialExport'));
-			form_special_export();
-		} else {
-			Display::display_normal_message(get_lang('ToExportSpecialSelect'));
-			CourseSelectForm :: display_form_session_export($course);
-		}
-	} else {
-		form_special_export();
-	}
+    // Display forms especial export
+    if (isset ($_POST['backup_option']) && $_POST['backup_option'] == 'select_items') {
+        $cb = new CourseBuilder();
+        $course = $cb->build_session_course();
+        if($course === false){
+            Display::display_error_message(get_lang('ErrorMsgSpecialExport'));
+            form_special_export();
+        } else {
+            Display::display_normal_message(get_lang('ToExportSpecialSelect'));
+            CourseSelectForm :: display_form_session_export($course);
+        }
+    } else {
+        form_special_export();
+    }
 }
 
 /* FOOTER */
@@ -200,10 +200,10 @@ 
     }
     $temp_zip_file = $temp_zip_dir."/".md5(time()).".zip";  //create zipfile of given directory
     return array('PATH' => $path,
-                 'PATH_TEMP_ARCHIVE' => $temp_zip_dir,
-                 'PATH_COURSE' => $sys_course_path,
-                 'TEMP_FILE_ZIP' => $temp_zip_file,
-                 'PATH_REMOVE' => $remove_dir);
+                    'PATH_TEMP_ARCHIVE' => $temp_zip_dir,
+                    'PATH_COURSE' => $sys_course_path,
+                    'TEMP_FILE_ZIP' => $temp_zip_file,
+                    'PATH_REMOVE' => $remove_dir);
 }
 
 function rename_zip($FileZip) {
@@ -287,9 +287,9 @@ 
                 $query_session_doc = Database::query($sql_session_doc);
                 while ($rows_course_session_file = Database::fetch_assoc($query_session_doc)) {
                     $zip_folder->add($FileZip['PATH_COURSE'].$_course['directory'].'/document'.$rows_course_session_file['path'],
-                         PCLZIP_OPT_ADD_PATH, $_course['directory']."/".$rows_session['name'],
-                         PCLZIP_OPT_REMOVE_PATH, $FileZip['PATH_COURSE'].$_course['directory'].'/document'.$FileZip['PATH_REMOVE']
-                     );
+                            PCLZIP_OPT_ADD_PATH, $_course['directory']."/".$rows_session['name'],
+                            PCLZIP_OPT_REMOVE_PATH, $FileZip['PATH_COURSE'].$_course['directory'].'/document'.$FileZip['PATH_REMOVE']
+                        );
                 }
             }
         }

Spacing +34 added lines, -34 removed lines

@@ -15,7 +15,7 @@ 
 // setting the section (for the tabs)
 $this_section = SECTION_PLATFORM_ADMIN;
 // setting breadcrumbs
-$interbreadcrumb[] = array ("url" => 'index.php', "name" => get_lang('PlatformAdmin'));
+$interbreadcrumb[] = array("url" => 'index.php', "name" => get_lang('PlatformAdmin'));
 // Access restrictions
 api_protect_admin_script(true);
 $nameTools = get_lang('SpecialExports');
@@ -29,7 +29,7 @@ 
 
 if (function_exists('ini_set')) {
 	api_set_memory_limit('256M');
-	ini_set('max_execution_time',0);
+	ini_set('max_execution_time', 0);
 }
 
 // Displaying the header
@@ -41,7 +41,7 @@ 
 if (count($_POST) == 0) {
     Display::display_normal_message(get_lang('SpecialExportsIntroduction'));
 }
-$error =0;
+$error = 0;
 /* MAIN CODE */
 
 $tbl_session = Database::get_main_table(TABLE_MAIN_SESSION);
@@ -66,17 +66,17 @@ 
             $groupCondition = " (props.to_group_id = 0 OR props.to_group_id IS NULL)";
         }
 
-		$zip_folder=new PclZip($FileZip['TEMP_FILE_ZIP']);
-		if(!isset($_POST['resource']) || count($_POST['resource']) == 0 ) {
+		$zip_folder = new PclZip($FileZip['TEMP_FILE_ZIP']);
+		if (!isset($_POST['resource']) || count($_POST['resource']) == 0) {
 			Display::display_error_message(get_lang('ErrorMsgSpecialExport'));
 		} else {
 			$Resource = $_POST['resource'];
 
 			foreach ($Resource as $Code_course => $Sessions) {
-				$_course 		= api_get_course_info($Code_course);
+				$_course = api_get_course_info($Code_course);
 				$tbl_document 	= Database::get_course_table(TABLE_DOCUMENT);
 				$tbl_property 	= Database::get_course_table(TABLE_ITEM_PROPERTY);
-				$course_id 		= $_course['real_id'];
+				$course_id = $_course['real_id'];
 
 				//Add item to the zip file course
 				$sql = "SELECT path FROM $tbl_document AS docs, $tbl_property AS props
@@ -89,7 +89,7 @@ 
 						AND $groupCondition
 						AND docs.c_id = $course_id
 						AND props.c_id = $course_id";
-				$query = Database::query($sql );
+				$query = Database::query($sql);
 				while ($rows_course_file = Database::fetch_assoc($query)) {
                     $zip_folder->add(
                         $FileZip['PATH_COURSE'].$_course['directory']."/document".$rows_course_file['path'],
@@ -100,7 +100,7 @@ 
                     );
 				}
 
-				foreach ($Sessions as $IdSession => $value){
+				foreach ($Sessions as $IdSession => $value) {
 					$session_id = Security::remove_XSS($IdSession);
 					//Add tem to the zip file session course
 					$sql_session_doc = "SELECT path FROM $tbl_document AS docs, $tbl_property AS props
@@ -150,7 +150,7 @@ 
 	if (isset ($_POST['backup_option']) && $_POST['backup_option'] == 'select_items') {
 		$cb = new CourseBuilder();
 		$course = $cb->build_session_course();
-		if($course === false){
+		if ($course === false) {
 			Display::display_error_message(get_lang('ErrorMsgSpecialExport'));
 			form_special_export();
 		} else {
@@ -166,12 +166,12 @@ 
 Display::display_footer();
 
 function form_special_export() {
-    $form = new FormValidator('special_exports','post');
+    $form = new FormValidator('special_exports', 'post');
     $renderer = $form->defaultRenderer();
     $renderer->setCustomElementTemplate('<div>{element}</div> ');
-    $form->addElement('radio', 'backup_option', '',  get_lang('SpecialCreateFullBackup'), 'full_backup');
-    $form->addElement('radio', 'backup_option', '',  get_lang('SpecialLetMeSelectItems'), 'select_items');
-    $form->addElement('html','<br />');
+    $form->addElement('radio', 'backup_option', '', get_lang('SpecialCreateFullBackup'), 'full_backup');
+    $form->addElement('radio', 'backup_option', '', get_lang('SpecialLetMeSelectItems'), 'select_items');
+    $form->addElement('html', '<br />');
     $form->addButtonExport(get_lang('CreateBackup'));
     $form->add_progress_bar();
     $values['backup_option'] = 'full_backup';
@@ -179,26 +179,26 @@ 
     $form->display();
 }
 
-function create_zip(){
+function create_zip() {
     $path = '';
-    if(empty($path)) { $path='/'; }
-    $remove_dir = ($path!='/') ? substr($path,0,strlen($path) - strlen(basename($path))) : '/';
+    if (empty($path)) { $path = '/'; }
+    $remove_dir = ($path != '/') ? substr($path, 0, strlen($path) - strlen(basename($path))) : '/';
     $sys_archive_path = api_get_path(SYS_ARCHIVE_PATH);
     $sys_course_path = api_get_path(SYS_COURSE_PATH);
     $temp_zip_dir = $sys_archive_path."temp";
-    if(!is_dir($temp_zip_dir)) {
+    if (!is_dir($temp_zip_dir)) {
         mkdir($temp_zip_dir, api_get_permissions_for_new_directories());
     } else {
-        $handle=opendir($temp_zip_dir);
-        while (false!==($file = readdir($handle))) {
+        $handle = opendir($temp_zip_dir);
+        while (false !== ($file = readdir($handle))) {
             if ($file != "." && $file != "..") {
-                $Diff = (time() - filemtime("$temp_zip_dir/$file"))/60/60;  //the "age" of the file in hours
-                if ($Diff > 4) unlink("$temp_zip_dir/$file");   //delete files older than 4 hours
+                $Diff = (time() - filemtime("$temp_zip_dir/$file")) / 60 / 60; //the "age" of the file in hours
+                if ($Diff > 4) unlink("$temp_zip_dir/$file"); //delete files older than 4 hours
             }
         }
         closedir($handle);
     }
-    $temp_zip_file = $temp_zip_dir."/".md5(time()).".zip";  //create zipfile of given directory
+    $temp_zip_file = $temp_zip_dir."/".md5(time()).".zip"; //create zipfile of given directory
     return array('PATH' => $path,
                  'PATH_TEMP_ARCHIVE' => $temp_zip_dir,
                  'PATH_COURSE' => $sys_course_path,
@@ -207,17 +207,17 @@ 
 }
 
 function rename_zip($FileZip) {
-    Event::event_download(($FileZip['PATH'] == '/')?'full_export_'.date('Ymd').'.zip (folder)': basename($FileZip['PATH']).'.zip (folder)');
-    $name = ($FileZip['PATH']=='/')? 'full_export_'.date('Ymd').'.zip':basename($FileZip['PATH']).'.zip';
-    if(file_exists($FileZip['PATH_TEMP_ARCHIVE'].'/'.$name)){ unlink($FileZip['PATH_TEMP_ARCHIVE'].'/'.$name); }
-    if(file_exists($FileZip['TEMP_FILE_ZIP'])) {
+    Event::event_download(($FileZip['PATH'] == '/') ? 'full_export_'.date('Ymd').'.zip (folder)' : basename($FileZip['PATH']).'.zip (folder)');
+    $name = ($FileZip['PATH'] == '/') ? 'full_export_'.date('Ymd').'.zip' : basename($FileZip['PATH']).'.zip';
+    if (file_exists($FileZip['PATH_TEMP_ARCHIVE'].'/'.$name)) { unlink($FileZip['PATH_TEMP_ARCHIVE'].'/'.$name); }
+    if (file_exists($FileZip['TEMP_FILE_ZIP'])) {
         rename($FileZip['TEMP_FILE_ZIP'], $FileZip['PATH_TEMP_ARCHIVE'].'/'.$name);
         return $name;
     } else { return false; }
 
 }
 
-function fullexportspecial(){
+function fullexportspecial() {
     global $tbl_session, $tbl_session_course, $export;
     $FileZip = create_zip();
     $to_group_id = 0;
@@ -235,10 +235,10 @@ 
     }
 
 
-    if (count($list_course) >0 ) {
-        foreach($list_course as $_course) {
-            if($FileZip['PATH'] == '/') {
-                $querypath=''; // to prevent ...path LIKE '//%'... in query
+    if (count($list_course) > 0) {
+        foreach ($list_course as $_course) {
+            if ($FileZip['PATH'] == '/') {
+                $querypath = ''; // to prevent ...path LIKE '//%'... in query
             } else {
                 $querypath = $FileZip['PATH'];
             }
@@ -255,7 +255,7 @@ 
                     AND $groupCondition
                     AND docs.c_id = $course_id
                     AND props.c_id = $course_id";
-            $query = Database::query($sql );
+            $query = Database::query($sql);
             while ($rows_course_file = Database::fetch_assoc($query)) {
                 $rows_course_file['path'];
                 $zip_folder->add($FileZip['PATH_COURSE'].$_course['directory']."/document".$rows_course_file['path'],
@@ -295,7 +295,7 @@ 
         }
 
         $name = rename_zip($FileZip);
-        if ($name === false){
+        if ($name === false) {
             $export = false;
             return false;
         } else {

Braces +4 added lines, -1 removed lines

@@ -193,7 +193,10 @@
         while (false!==($file = readdir($handle))) {
             if ($file != "." && $file != "..") {
                 $Diff = (time() - filemtime("$temp_zip_dir/$file"))/60/60;  //the "age" of the file in hours
-                if ($Diff > 4) unlink("$temp_zip_dir/$file");   //delete files older than 4 hours
+                if ($Diff > 4) {
+                    unlink("$temp_zip_dir/$file");
+                }
+                //delete files older than 4 hours
             }
         }
         closedir($handle);

main/notebook/index.php

Indentation +12 added lines, -12 removed lines

@@ -37,12 +37,12 @@ 
 
 // Tool name
 if (isset($_GET['action']) && $_GET['action'] == 'addnote') {
-	$tool = 'NoteAddNew';
-	$interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('ToolNotebook'));
+    $tool = 'NoteAddNew';
+    $interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('ToolNotebook'));
 }
 if (isset($_GET['action']) && $_GET['action'] == 'editnote') {
-	$tool = 'ModifyNote';
-	$interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('ToolNotebook'));
+    $tool = 'ModifyNote';
+    $interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('ToolNotebook'));
 }
 
 // Displaying the header
@@ -53,16 +53,16 @@ 
 
 // Action handling: Adding a note
 if (isset($_GET['action']) && $_GET['action'] == 'addnote') {
-	if (api_get_session_id() != 0 && !api_is_allowed_to_session_edit(false, true)) {
-		api_not_allowed();
-	}
+    if (api_get_session_id() != 0 && !api_is_allowed_to_session_edit(false, true)) {
+        api_not_allowed();
+    }
 
-	if (!empty($_GET['isStudentView'])) {
-		NotebookManager::display_notes();
-		exit;
-	}
+    if (!empty($_GET['isStudentView'])) {
+        NotebookManager::display_notes();
+        exit;
+    }
 
-	$_SESSION['notebook_view'] = 'creation_date';
+    $_SESSION['notebook_view'] = 'creation_date';
 
     $form = new FormValidator(
         'note',

Spacing +3 added lines, -3 removed lines

@@ -10,7 +10,7 @@ 
 // Including the global initialization file
 require_once '../inc/global.inc.php';
 
-$current_course_tool  = TOOL_NOTEBOOK;
+$current_course_tool = TOOL_NOTEBOOK;
 
 // The section (tabs)
 $this_section = SECTION_COURSES;
@@ -96,7 +96,7 @@ 
         NotebookManager::display_notes();
     } else {
         echo '<div class="actions">';
-        echo '<a href="index.php">'.Display::return_icon('back.png',get_lang('BackToNotesList'),'',ICON_SIZE_MEDIUM).'</a>';
+        echo '<a href="index.php">'.Display::return_icon('back.png', get_lang('BackToNotesList'), '', ICON_SIZE_MEDIUM).'</a>';
         echo '</div>';
         $token = Security::get_token();
         $form->addElement('hidden', 'sec_token');
@@ -147,7 +147,7 @@ 
     } else {
         echo '<div class="actions">';
         echo '<a href="index.php">'.
-            Display::return_icon('back.png',get_lang('BackToNotesList'),'',ICON_SIZE_MEDIUM).'</a>';
+            Display::return_icon('back.png', get_lang('BackToNotesList'), '', ICON_SIZE_MEDIUM).'</a>';
         echo '</div>';
         $token = Security::get_token();
         $form->addElement('hidden', 'sec_token');

main/announcements/download.php

Spacing +6 added lines, -6 removed lines

@@ -28,7 +28,7 @@ 
 $doc_url = str_replace(' ', '+', $doc_url);
 $doc_url = str_replace('/..', '', $doc_url); //echo $doc_url;
 
-if (strpos($doc_url,'../') OR strpos($doc_url,'/..')) {
+if (strpos($doc_url, '../') OR strpos($doc_url, '/..')) {
     $doc_url = '';
 }
 
@@ -43,7 +43,7 @@ 
     //remove last slash if present
     //$doc_url = ($doc_url{strlen($doc_url)-1}=='/')?substr($doc_url,0,strlen($doc_url)-1):$doc_url;
     //mod_rewrite can change /some/path/ to /some/path// in some cases, so clean them all off (René)
-    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
@@ -62,12 +62,12 @@ 
 $sql = "SELECT filename FROM $tbl_announcement_attachment
   	  	WHERE c_id = $course_id AND path LIKE BINARY '$doc_url'";
 
-$result= Database::query($sql);
+$result = Database::query($sql);
 if (Database::num_rows($result) > 0) {
-    $row= Database::fetch_array($result);
-    $title = str_replace(' ','_', $row['filename']);
+    $row = Database::fetch_array($result);
+    $title = str_replace(' ', '_', $row['filename']);
     if (Security::check_abs_path($full_file_name,
-        api_get_path(SYS_COURSE_PATH) . api_get_course_path() . '/upload/announcements/')
+        api_get_path(SYS_COURSE_PATH).api_get_course_path().'/upload/announcements/')
     ) {
         DocumentManager::file_send_for_download($full_file_name, true, $title);
     }

Braces +3 added lines, -1 removed lines

@@ -43,7 +43,9 @@
     //remove last slash if present
     //$doc_url = ($doc_url{strlen($doc_url)-1}=='/')?substr($doc_url,0,strlen($doc_url)-1):$doc_url;
     //mod_rewrite can change /some/path/ to /some/path// in some cases, so clean them all off (René)
-    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