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/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/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/skill_badge_create.php

Spacing +6 added lines, -6 removed lines

@@ -46,10 +46,10 @@ 
         if ($existsBadgesDirectory) {
 
             if (!empty($skill['icon'])) {
-                $iconFileAbsolutePath = $badgePath . $skill['icon'];
+                $iconFileAbsolutePath = $badgePath.$skill['icon'];
 
                 if (Security::check_abs_path($iconFileAbsolutePath, $badgePath)) {
-                    unlink($badgePath . $skill['icon']);
+                    unlink($badgePath.$skill['icon']);
                 }
             }
 
@@ -72,24 +72,24 @@ 
 
     $objSkill->update($params);
 
-    header('Location: ' . api_get_path(WEB_CODE_PATH) . 'admin/skill_badge_list.php');
+    header('Location: '.api_get_path(WEB_CODE_PATH).'admin/skill_badge_list.php');
     exit;
 }
 
 $interbreadcrumb = array(
     array(
-        'url' => api_get_path(WEB_CODE_PATH) . 'admin/index.php',
+        'url' => api_get_path(WEB_CODE_PATH).'admin/index.php',
         'name' => get_lang('Administration')
     ),
     array(
-        'url' => api_get_path(WEB_CODE_PATH) . 'admin/skill_badge.php',
+        'url' => api_get_path(WEB_CODE_PATH).'admin/skill_badge.php',
         'name' => get_lang('Badges')
     )
 );
 
 $toolbar = Display::toolbarButton(
     get_lang('ManageSkills'),
-    api_get_path(WEB_CODE_PATH) . 'admin/skill_list.php',
+    api_get_path(WEB_CODE_PATH).'admin/skill_list.php',
     'list',
     'primary',
     ['title' => get_lang('ManageSkills')]

main/admin/special_exports.php

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 {

main/notebook/index.php

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);
     }

main/survey/survey_invite.php

Spacing +4 added lines, -4 removed lines

@@ -64,7 +64,7 @@ 
 $tool_name = get_lang('SurveyPublication');
 
 // Displaying the header
-Display::display_header($tool_name,'Survey');
+Display::display_header($tool_name, 'Survey');
 
 echo '<script>
 $(function() {
@@ -132,7 +132,7 @@ 
 );
 
 $form->addElement('html', '<div id="check_mail">');
-$form->addElement('checkbox', 'send_mail','', get_lang('SendMail'));
+$form->addElement('checkbox', 'send_mail', '', get_lang('SendMail'));
 $form->addElement('html', '</div>');
 
 $form->addElement('html', '<div id="mail_text_wrapper">');
@@ -219,8 +219,8 @@ 
 	);
 
 	// Saving the invitations for the additional users
-	$values['additional_users'] = $values['additional_users'].';'; 	// This is for the case when you enter only one email
-	$temp = str_replace(',', ';', $values['additional_users']);		// This is to allow , and ; as email separators
+	$values['additional_users'] = $values['additional_users'].';'; // This is for the case when you enter only one email
+	$temp = str_replace(',', ';', $values['additional_users']); // This is to allow , and ; as email separators
 	$additional_users = explode(';', $temp);
 	for ($i = 0; $i < count($additional_users); $i++) {
 		$additional_users[$i] = trim($additional_users[$i]);