chamilo / chamilo-lms

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

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

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

Indentation +93 added lines, -93 removed lines

@@ -18,10 +18,10 @@ 
 $this_section = SECTION_COURSES;
 
 if (!api_is_allowed_to_edit(false, true)) {
-	Display :: display_header(get_lang('ToolSurvey'));
-	Display :: display_error_message(get_lang('NotAllowed'), false);
-	Display :: display_footer();
-	exit;
+    Display :: display_header(get_lang('ToolSurvey'));
+    Display :: display_error_message(get_lang('NotAllowed'), false);
+    Display :: display_footer();
+    exit;
 }
 
 // Database table definitions
@@ -37,15 +37,15 @@ 
 $survey_id = Security::remove_XSS($_GET['survey_id']);
 $survey_data = SurveyManager::get_survey($survey_id);
 if (empty($survey_data)) {
-	Display :: display_header(get_lang('ToolSurvey'));
-	Display :: display_error_message(get_lang('InvallidSurvey'), false);
-	Display :: display_footer();
-	exit;
+    Display :: display_header(get_lang('ToolSurvey'));
+    Display :: display_error_message(get_lang('InvallidSurvey'), false);
+    Display :: display_footer();
+    exit;
 }
 
 $urlname = strip_tags(api_substr(api_html_entity_decode($survey_data['title'], ENT_QUOTES), 0, 40));
 if (api_strlen(strip_tags($survey_data['title'])) > 40) {
-	$urlname .= '...';
+    $urlname .= '...';
 }
 
 // Breadcrumbs
@@ -80,23 +80,23 @@ 
 		WHERE c_id = $course_id AND code='".Database::escape_string($survey_data['code'])."'";
 $result = Database::query($sql);
 if (Database::num_rows($result) > 1) {
-	Display::display_warning_message(get_lang('IdenticalSurveycodeWarning'));
+    Display::display_warning_message(get_lang('IdenticalSurveycodeWarning'));
 }
 
 // Invited / answered message
 if ($survey_data['invited'] > 0 && !isset($_POST['submit'])) {
-	$message  = '<a href="'.api_get_path(WEB_CODE_PATH).'survey/survey_invitation.php?view=answered&survey_id='.$survey_data['survey_id'].'">'.$survey_data['answered'].'</a> ';
-	$message .= get_lang('HaveAnswered').' ';
-	$message .= '<a href="'.api_get_path(WEB_CODE_PATH).'survey/survey_invitation.php?view=invited&survey_id='.$survey_data['survey_id'].'">'.$survey_data['invited'].'</a> ';
-	$message .= get_lang('WereInvited');
-	Display::display_normal_message($message, false);
+    $message  = '<a href="'.api_get_path(WEB_CODE_PATH).'survey/survey_invitation.php?view=answered&survey_id='.$survey_data['survey_id'].'">'.$survey_data['answered'].'</a> ';
+    $message .= get_lang('HaveAnswered').' ';
+    $message .= '<a href="'.api_get_path(WEB_CODE_PATH).'survey/survey_invitation.php?view=invited&survey_id='.$survey_data['survey_id'].'">'.$survey_data['invited'].'</a> ';
+    $message .= get_lang('WereInvited');
+    Display::display_normal_message($message, false);
 }
 
 // Building the form for publishing the survey
 $form = new FormValidator(
-	'publish_form',
-	'post',
-	api_get_self().'?survey_id='.$survey_id.'&'.api_get_cidreq()
+    'publish_form',
+    'post',
+    api_get_self().'?survey_id='.$survey_id.'&'.api_get_cidreq()
 );
 $form->addElement('header', '', $tool_name);
 
@@ -109,7 +109,7 @@ 
 );
 $possible_users = array();
 foreach ($complete_user_list as & $user) {
-	$possible_users[$user['user_id']] = api_get_person_name($user['firstname'], $user['lastname']);
+    $possible_users[$user['user_id']] = api_get_person_name($user['firstname'], $user['lastname']);
 }
 
 
@@ -125,10 +125,10 @@ 
 
 // Additional users
 $form->addElement(
-	'textarea',
-	'additional_users',
-	array(get_lang('AdditonalUsers'), get_lang('AdditonalUsersComment')),
-	array('rows' => 5)
+    'textarea',
+    'additional_users',
+    array(get_lang('AdditonalUsers'), get_lang('AdditonalUsersComment')),
+    array('rows' => 5)
 );
 
 $form->addElement('html', '<div id="check_mail">');
@@ -141,15 +141,15 @@ 
 $form->addText('mail_title', get_lang('MailTitle'), false);
 // The text of the mail
 $form->addHtmlEditor(
-	'mail_text',
-	array(get_lang('MailText'), get_lang('UseLinkSyntax')),
-	false,
-	array('ToolbarSet' => 'Survey', 'Height' => '150')
+    'mail_text',
+    array(get_lang('MailText'), get_lang('UseLinkSyntax')),
+    false,
+    array('ToolbarSet' => 'Survey', 'Height' => '150')
 );
 $form->addElement('html', '</div>');
 // You cab send a reminder to unanswered people if the survey is not anonymous
 if ($survey_data['anonymous'] != 1) {
-	$form->addElement('checkbox', 'remindUnAnswered', '', get_lang('RemindUnanswered'));
+    $form->addElement('checkbox', 'remindUnAnswered', '', get_lang('RemindUnanswered'));
 }
 // Allow resending to all selected users
 $form->addElement('checkbox', 'resend_to_all', '', get_lang('ReminderResendToAllUsers'));
@@ -163,11 +163,11 @@ 
 }*/
 $portal_url = api_get_path(WEB_PATH);
 if (api_is_multiple_url_enabled()) {
-	$access_url_id = api_get_current_access_url_id();
-	if ($access_url_id != -1) {
-		$url = api_get_access_url($access_url_id);
-		$portal_url = $url['url'];
-	}
+    $access_url_id = api_get_current_access_url_id();
+    if ($access_url_id != -1) {
+        $url = api_get_access_url($access_url_id);
+        $portal_url = $url['url'];
+    }
 }
 
 // Show the URL that can be used by users to fill a survey without invitation
@@ -177,56 +177,56 @@ 
 $form->addElement('label', null, $auto_survey_link);
 
 if ($form->validate()) {
-   	$values = $form->exportValues();
+        $values = $form->exportValues();
     if (isset($values['send_mail']) && $values['send_mail'] == 1) {
         if (empty($values['mail_title']) || empty($values['mail_text'])) {
             Display :: display_error_message(get_lang('FormHasErrorsPleaseComplete'));
             // Getting the invited users
-        	$defaults = SurveyUtil::get_invited_users($survey_data['code']);
-
-        	// Getting the survey mail text
-        	if (!empty($survey_data['reminder_mail'])) {
-        		$defaults['mail_text'] = $survey_data['reminder_mail'];
-        	} else {
-        		$defaults['mail_text'] = $survey_data['invite_mail'];
-        	}
-        	$defaults['mail_title'] = $survey_data['mail_subject'];
-        	$defaults['send_mail'] = 1;
-        	$form->setDefaults($defaults);
+            $defaults = SurveyUtil::get_invited_users($survey_data['code']);
+
+            // Getting the survey mail text
+            if (!empty($survey_data['reminder_mail'])) {
+                $defaults['mail_text'] = $survey_data['reminder_mail'];
+            } else {
+                $defaults['mail_text'] = $survey_data['invite_mail'];
+            }
+            $defaults['mail_title'] = $survey_data['mail_subject'];
+            $defaults['send_mail'] = 1;
+            $form->setDefaults($defaults);
             $form->display();
             return;
         }
     }
     // Save the invitation mail
-	SurveyUtil::save_invite_mail(
-		$values['mail_text'],
-		$values['mail_title'],
-		!empty($survey_data['invite_mail'])
-	);
+    SurveyUtil::save_invite_mail(
+        $values['mail_text'],
+        $values['mail_title'],
+        !empty($survey_data['invite_mail'])
+    );
 
     $resendAll = isset($values['resend_to_all']) ? $values['resend_to_all'] : '';
     $sendMail = isset($values['send_mail']) ? $values['send_mail'] : '';
     $remindUnAnswered = isset($values['remindUnAnswered']) ? $values['remindUnAnswered'] : '';
 
-	// Saving the invitations for the course users
-	$count_course_users = SurveyUtil::saveInvitations(
-		$values['users'],
-		$values['mail_title'],
-		$values['mail_text'],
+    // Saving the invitations for the course users
+    $count_course_users = SurveyUtil::saveInvitations(
+        $values['users'],
+        $values['mail_title'],
+        $values['mail_text'],
         $resendAll,
         $sendMail,
         $remindUnAnswered
-	);
+    );
 
-	// 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
-	$additional_users = explode(';', $temp);
-	for ($i = 0; $i < count($additional_users); $i++) {
-		$additional_users[$i] = trim($additional_users[$i]);
-	}
+    // 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
+    $additional_users = explode(';', $temp);
+    for ($i = 0; $i < count($additional_users); $i++) {
+        $additional_users[$i] = trim($additional_users[$i]);
+    }
 
-	$counter_additional_users = SurveyUtil::saveInvitations(
+    $counter_additional_users = SurveyUtil::saveInvitations(
         $additional_users,
         $values['mail_title'],
         $values['mail_text'],
@@ -235,43 +235,43 @@ 
         $remindUnAnswered
     );
 
-	// Updating the invited field in the survey table
-	SurveyUtil::update_count_invited($survey_data['code']);
-	$total_count = $count_course_users + $counter_additional_users;
+    // Updating the invited field in the survey table
+    SurveyUtil::update_count_invited($survey_data['code']);
+    $total_count = $count_course_users + $counter_additional_users;
     $table_survey = Database :: get_course_table(TABLE_SURVEY);
-	// Counting the number of people that are invited
-	$sql = "SELECT * FROM $table_survey
+    // Counting the number of people that are invited
+    $sql = "SELECT * FROM $table_survey
 	        WHERE
 	        	c_id = $course_id AND
 	        	code = '".Database::escape_string($survey_data['code'])."'
 			";
-	$result = Database::query($sql);
-	$row = Database::fetch_array($result);
-	$total_invited = $row['invited'];
+    $result = Database::query($sql);
+    $row = Database::fetch_array($result);
+    $total_invited = $row['invited'];
     if ($total_invited > 0) {
-    	$message  = '<a href="'.api_get_path(WEB_CODE_PATH).'survey/survey_invitation.php?view=answered&survey_id='.$survey_data['survey_id'].'">'.
-			$survey_data['answered'].'</a> ';
-    	$message .= get_lang('HaveAnswered').' ';
-    	$message .= '<a href="'.api_get_path(WEB_CODE_PATH).'survey/survey_invitation.php?view=invited&survey_id='.$survey_data['survey_id'].'">'.
-			$total_invited.'</a> ';
-    	$message .= get_lang('WereInvited');
-    	Display::display_normal_message($message, false);
-    	Display::display_confirmation_message($total_count.' '.get_lang('InvitationsSend'));
+        $message  = '<a href="'.api_get_path(WEB_CODE_PATH).'survey/survey_invitation.php?view=answered&survey_id='.$survey_data['survey_id'].'">'.
+            $survey_data['answered'].'</a> ';
+        $message .= get_lang('HaveAnswered').' ';
+        $message .= '<a href="'.api_get_path(WEB_CODE_PATH).'survey/survey_invitation.php?view=invited&survey_id='.$survey_data['survey_id'].'">'.
+            $total_invited.'</a> ';
+        $message .= get_lang('WereInvited');
+        Display::display_normal_message($message, false);
+        Display::display_confirmation_message($total_count.' '.get_lang('InvitationsSend'));
     }
 } else {
-	// Getting the invited users
-	$defaults = SurveyUtil::get_invited_users($survey_data['code']);
-
-	// Getting the survey mail text
-	if (!empty($survey_data['reminder_mail'])) {
-		$defaults['mail_text'] = $survey_data['reminder_mail'];
-	} else {
-		$defaults['mail_text'] = $survey_data['invite_mail'];
-	}
-	$defaults['mail_title'] = $survey_data['mail_subject'];
-	$defaults['send_mail'] = 1;
-
-	$form->setDefaults($defaults);
+    // Getting the invited users
+    $defaults = SurveyUtil::get_invited_users($survey_data['code']);
+
+    // Getting the survey mail text
+    if (!empty($survey_data['reminder_mail'])) {
+        $defaults['mail_text'] = $survey_data['reminder_mail'];
+    } else {
+        $defaults['mail_text'] = $survey_data['invite_mail'];
+    }
+    $defaults['mail_title'] = $survey_data['mail_subject'];
+    $defaults['send_mail'] = 1;
+
+    $form->setDefaults($defaults);
     $form->display();
 }
 Display :: display_footer();

main/survey/fillsurvey.php

Indentation +1 added lines, -2 removed lines

@@ -6,8 +6,7 @@
 * @author unknown, the initial survey that did not make it in 1.8 because of bad code
 * @author Patrick Cool <patrick.cool@UGent.be>, Ghent University: cleanup, refactoring and rewriting large parts of the code
 * @author Julio Montoya Armas <gugli100@gmail.com>, Chamilo: Personality Test modification and rewriting large parts of the code as well
-
-* @todo check if the user already filled the survey and if this is the case then the answers have to be updated and not stored again.
+ * @todo check if the user already filled the survey and if this is the case then the answers have to be updated and not stored again.
 * @todo performance could be improved if not the survey_id was stored with the invitation but the survey_code
  */
 // Unsetting the course id (because it is in the URL)

Spacing +6 added lines, -6 removed lines

@@ -112,7 +112,7 @@ 
         $now = api_get_utc_datetime();
         if (Database :: num_rows($result) == 0) {
             $params = [
-                'c_id' => $course_id ,
+                'c_id' => $course_id,
                 'survey_code' => $surveyCode,
                 'user' => $userid,
                 'invitation_code' => $autoInvitationcode,
@@ -471,7 +471,7 @@ 
     $jquery_ready_content = $returnParams['jquery_ready_content'];
 
 // the $jquery_ready_content variable collects all functions that will be load in the $(document).ready javascript function
-    $htmlHeadXtra[] ='<script>
+    $htmlHeadXtra[] = '<script>
     $(document).ready(function(){
         '.$jquery_ready_content.'
     });
@@ -904,16 +904,16 @@ 
                             $group2 = $groups[0];
                             $secondary .= " OR ( survey_group_sec1 = '$group1' AND  survey_group_sec2 = '$group2') ";
                             $secondary .= " OR ( survey_group_sec1 = '$group2' AND survey_group_sec2 = '$group1' ) ";
-                            $combi.= $group1.' - '.$group2." or ".$group2.' - '.$group1.'<br />';
+                            $combi .= $group1.' - '.$group2." or ".$group2.' - '.$group1.'<br />';
                         } else {
                             if ($i != 0) {
                                 $secondary .= " OR ( survey_group_sec1 = '$group1' AND  survey_group_sec2 = '$group2') ";
                                 $secondary .= " OR ( survey_group_sec1 = '$group2' AND survey_group_sec2 = '$group1' ) ";
-                                $combi.= $group1.' - '.$group2." or ".$group2.' - '.$group1.'<br />';
+                                $combi .= $group1.' - '.$group2." or ".$group2.' - '.$group1.'<br />';
                             } else {
                                 $secondary .= " ( survey_group_sec1 = '$group1' AND  survey_group_sec2 = '$group2') ";
                                 $secondary .= " OR ( survey_group_sec1 = '$group2' AND survey_group_sec2 = '$group1' ) ";
-                                $combi.= $group1.' - '.$group2." or ".$group2.' - '.$group1.'<br />';
+                                $combi .= $group1.' - '.$group2." or ".$group2.' - '.$group1.'<br />';
                             }
                         }
                     }
@@ -1132,7 +1132,7 @@ 
                         $questions[$row['sort']]['survey_group_sec1'] = $row['survey_group_sec1'];
                         $questions[$row['sort']]['survey_group_sec2'] = $row['survey_group_sec2'];
                         $questions[$row['sort']]['survey_group_pri'] = $row['survey_group_pri'];
-                    }  else {
+                    } else {
                         // If the type is a page break we are finished loading the questions for this page
                         break;
                     }

Braces +1 added lines, -1 removed lines

@@ -1132,7 +1132,7 @@
                         $questions[$row['sort']]['survey_group_sec1'] = $row['survey_group_sec1'];
                         $questions[$row['sort']]['survey_group_sec2'] = $row['survey_group_sec2'];
                         $questions[$row['sort']]['survey_group_pri'] = $row['survey_group_pri'];
-                    }  else {
+                    } else {
                         // If the type is a page break we are finished loading the questions for this page
                         break;
                     }

main/survey/ch_yesno.php

Spacing +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
                 $class = 'radio-inline';
             }
 
-            $name = 'question' . $questionData['question_id'];
+            $name = 'question'.$questionData['question_id'];
 
             $form->addRadio(
                 $name,

main/survey/reporting.php

Indentation +3 added lines, -3 removed lines

@@ -83,15 +83,15 @@
 
 /** @todo this has to be moved to a more appropriate place (after the display_header of the code)*/
 if (!api_is_allowed_to_edit(false, true) || $isDrhOfCourse) {
-	Display :: display_header(get_lang('ToolSurvey'));
+    Display :: display_header(get_lang('ToolSurvey'));
     // Show error message if the survey can be seen only by tutors
     if ($survey_data['visible_results'] != SURVEY_VISIBLE_TUTOR) {
         SurveyUtil::handle_reporting_actions($survey_data, $people_filled);
     } else {
         Display :: display_error_message(get_lang('NotAllowed'), false);
     }
-	Display :: display_footer();
-	exit;
+    Display :: display_footer();
+    exit;
 }
 
 // Database table definitions

Spacing +6 added lines, -6 removed lines

@@ -149,7 +149,7 @@ 
 // Actions bar
 echo '<div class="actions">';
 echo '<a href="'.api_get_path(WEB_CODE_PATH).'survey/survey.php?survey_id='.$survey_id.'">'.
-    Display::return_icon('back.png', get_lang('BackToSurvey'),'',ICON_SIZE_MEDIUM).'</a>';
+    Display::return_icon('back.png', get_lang('BackToSurvey'), '', ICON_SIZE_MEDIUM).'</a>';
 echo '</div>';
 
 // Content
@@ -159,15 +159,15 @@ 
 ) {
     $myweb_survey_id = $survey_id;
     echo '<div class="sectiontitle"><a href="'.api_get_path(WEB_CODE_PATH).'survey/reporting.php?action=questionreport&amp;survey_id='.$myweb_survey_id.'&amp;'.$cidReq.'&amp;single_page=1">'.
-        Display::return_icon('survey_reporting_overall.png',get_lang('QuestionsOverallReport')).' '.get_lang('QuestionsOverallReport').'</a></div><div class="sectioncomment">'.get_lang('QuestionsOverallReportDetail').' </div>';
+        Display::return_icon('survey_reporting_overall.png', get_lang('QuestionsOverallReport')).' '.get_lang('QuestionsOverallReport').'</a></div><div class="sectioncomment">'.get_lang('QuestionsOverallReportDetail').' </div>';
     echo '<div class="sectiontitle"><a href="'.api_get_path(WEB_CODE_PATH).'survey/reporting.php?action=questionreport&amp;survey_id='.$myweb_survey_id.'&'.$cidReq.'">'.
-        Display::return_icon('survey_reporting_question.gif',get_lang('DetailedReportByQuestion')).' '.get_lang('DetailedReportByQuestion').'</a></div><div class="sectioncomment">'.get_lang('DetailedReportByQuestionDetail').' </div>';
+        Display::return_icon('survey_reporting_question.gif', get_lang('DetailedReportByQuestion')).' '.get_lang('DetailedReportByQuestion').'</a></div><div class="sectioncomment">'.get_lang('DetailedReportByQuestionDetail').' </div>';
     echo '<div class="sectiontitle"><a href="'.api_get_path(WEB_CODE_PATH).'survey/reporting.php?action=userreport&amp;survey_id='.$myweb_survey_id.'&'.$cidReq.'">'.
-        Display::return_icon('survey_reporting_user.gif',get_lang('DetailedReportByUser')).' '.get_lang('DetailedReportByUser').'</a></div><div class="sectioncomment">'.get_lang('DetailedReportByUserDetail').'.</div>';
+        Display::return_icon('survey_reporting_user.gif', get_lang('DetailedReportByUser')).' '.get_lang('DetailedReportByUser').'</a></div><div class="sectioncomment">'.get_lang('DetailedReportByUserDetail').'.</div>';
     echo '<div class="sectiontitle"><a href="'.api_get_path(WEB_CODE_PATH).'survey/reporting.php?action=comparativereport&amp;survey_id='.$myweb_survey_id.'&'.$cidReq.'">'.
-        Display::return_icon('survey_reporting_comparative.gif',get_lang('ComparativeReport')).' '.get_lang('ComparativeReport').'</a></div><div class="sectioncomment">'.get_lang('ComparativeReportDetail').'.</div>';
+        Display::return_icon('survey_reporting_comparative.gif', get_lang('ComparativeReport')).' '.get_lang('ComparativeReport').'</a></div><div class="sectioncomment">'.get_lang('ComparativeReportDetail').'.</div>';
     echo '<div class="sectiontitle"><a href="reporting.php?action=completereport&amp;survey_id='.$myweb_survey_id.'&'.$cidReq.'">'.
-        Display::return_icon('survey_reporting_complete.gif',get_lang('CompleteReport')).' '.get_lang('CompleteReport').'</a></div><div class="sectioncomment">'.get_lang('CompleteReportDetail').'</div>';
+        Display::return_icon('survey_reporting_complete.gif', get_lang('CompleteReport')).' '.get_lang('CompleteReport').'</a></div><div class="sectioncomment">'.get_lang('CompleteReportDetail').'</div>';
 }
 
 // Footer

main/survey/ch_multiplechoice.php

Spacing +2 added lines, -2 removed lines

@@ -31,7 +31,7 @@ 
             foreach ($formData['answers'] as $key => $value) {
                 $this->getForm()->addHtmlEditor('answers['.$key.']', null, false, false, $config);
 
-                if ($key < $total-1) {
+                if ($key < $total - 1) {
                     //$this->getForm()->addButton("move_down[$key]", get_lang('Down'));
                 }
 
@@ -39,7 +39,7 @@ 
                     //$this->getForm()->addButton("move_up[$key]", get_lang('Up'));
                 }
 
-                if ($total> 2) {
+                if ($total > 2) {
                     $this->getForm()->addButton("delete_answer[$key]", get_lang('Delete'), 'trash', 'danger');
                 }
             }