chamilo / chamilo-lms

main/inc/ajax/link.ajax.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
             // set URL and other appropriate options
             $defaults = array(
                 CURLOPT_URL => $url,
-                CURLOPT_FOLLOWLOCATION => true,         // follow redirects accept youtube.com
+                CURLOPT_FOLLOWLOCATION => true, // follow redirects accept youtube.com
                 CURLOPT_HEADER => 0,
                 CURLOPT_RETURNTRANSFER => true,
                 CURLOPT_TIMEOUT => 4

main/inc/ajax/session.ajax.php

Spacing +2 added lines, -2 removed lines

@@ -15,7 +15,7 @@ 
             $list_sessions = SessionManager::get_sessions_by_user($user_id, true);
             if (!empty($list_sessions)) {
                 foreach ($list_sessions as $session_item) {
-                    echo $session_item['session_name'] . '<br />';
+                    echo $session_item['session_name'].'<br />';
                 }
             } else {
                 echo get_lang('NoSessionsForThisUser');
@@ -29,7 +29,7 @@ 
                 [
                     's.name' => [
                         'operator' => 'LIKE',
-                        'value' => "%" . $_REQUEST['q'] . "%"
+                        'value' => "%".$_REQUEST['q']."%"
                     ]
                 ]
             );

main/inc/ajax/nanogong.ajax.php

Switch Indentation +38 added lines, -38 removed lines

@@ -30,44 +30,44 @@
 }
 
 switch ($action) {
-	case 'get_file':
-		if ($nano->get_param_value('user_id') == api_get_user_id() || api_is_allowed_to_edit()) {
-			$file_path = $nano->load_filename_if_exists();
-			//$file_path = '/var/www/rocio/courses/GREAT123/exercises/0/2/5/1/1-0-1-2-5-38.mp3';
-			if ($file_path) {
-				$info = pathinfo($file_path);
-				$user_info = api_get_user_info($nano->params['user_id']);
-				$name = get_lang('Quiz').'-'.$user_info['firstname'].'-'.$user_info['lastname'].'.'.$info['extension'];
-				$download = true;
-				if (isset($_REQUEST['download'])  && $_REQUEST['download'] == 0) {
-					$download = false;
-				}
-				DocumentManager::file_send_for_download($file_path, $download);
-				exit;
-			}
-		}
-		break;
-	case 'show_audio':
-		if (!$is_nano) {
-			echo $nano->return_js($_REQUEST);
-		}
-		echo $nano->show_audio_file($is_nano);
-		break;
-	case 'delete':
-		$return = $nano->delete_files();
-		if ($return == 1) {
-			//cant' do this because the post that nano send doesnt take into account the session
-			Display::display_confirmation_message(get_lang('FileDeleted'));
-		} else {
-			Display::display_confirmation_message(get_lang('FileNotFound'));
-		}
-		break;
-	case 'show_form':
-		api_protect_course_script(true);
-		Display::display_reduced_header();
-		echo $nano->return_js($_REQUEST);
-		echo $nano->return_form();
-		break;
+	    case 'get_file':
+		    if ($nano->get_param_value('user_id') == api_get_user_id() || api_is_allowed_to_edit()) {
+			    $file_path = $nano->load_filename_if_exists();
+			    //$file_path = '/var/www/rocio/courses/GREAT123/exercises/0/2/5/1/1-0-1-2-5-38.mp3';
+			    if ($file_path) {
+				    $info = pathinfo($file_path);
+				    $user_info = api_get_user_info($nano->params['user_id']);
+				    $name = get_lang('Quiz').'-'.$user_info['firstname'].'-'.$user_info['lastname'].'.'.$info['extension'];
+				    $download = true;
+				    if (isset($_REQUEST['download'])  && $_REQUEST['download'] == 0) {
+					    $download = false;
+				    }
+				    DocumentManager::file_send_for_download($file_path, $download);
+				    exit;
+			    }
+		    }
+		    break;
+	    case 'show_audio':
+		    if (!$is_nano) {
+			    echo $nano->return_js($_REQUEST);
+		    }
+		    echo $nano->show_audio_file($is_nano);
+		    break;
+	    case 'delete':
+		    $return = $nano->delete_files();
+		    if ($return == 1) {
+			    //cant' do this because the post that nano send doesnt take into account the session
+			    Display::display_confirmation_message(get_lang('FileDeleted'));
+		    } else {
+			    Display::display_confirmation_message(get_lang('FileNotFound'));
+		    }
+		    break;
+	    case 'show_form':
+		    api_protect_course_script(true);
+		    Display::display_reduced_header();
+		    echo $nano->return_js($_REQUEST);
+		    echo $nano->return_form();
+		    break;
     case 'save_file':
     	//User access same as upload.php
 

Spacing +2 added lines, -2 removed lines

@@ -11,7 +11,7 @@ 
 $action = $_REQUEST['a'];
 
 unset($_REQUEST['a']);
-$js_path 		= api_get_path(WEB_LIBRARY_PATH).'javascript/';
+$js_path = api_get_path(WEB_LIBRARY_PATH).'javascript/';
 
 //Fix in order to add the exe_id
 
@@ -39,7 +39,7 @@ 
 				$user_info = api_get_user_info($nano->params['user_id']);
 				$name = get_lang('Quiz').'-'.$user_info['firstname'].'-'.$user_info['lastname'].'.'.$info['extension'];
 				$download = true;
-				if (isset($_REQUEST['download'])  && $_REQUEST['download'] == 0) {
+				if (isset($_REQUEST['download']) && $_REQUEST['download'] == 0) {
 					$download = false;
 				}
 				DocumentManager::file_send_for_download($file_path, $download);

Indentation +67 added lines, -67 removed lines

@@ -16,84 +16,84 @@
 //Fix in order to add the exe_id
 
 if (isset($_REQUEST['from_htaccess'])) {
-	if (isset($_REQUEST['file'])) {
-		$fileinfo = pathinfo($_REQUEST['file']);
-		$items = explode('-', $fileinfo['filename']);
-		$_REQUEST['exe_id'] = $items[5];
-	}
+    if (isset($_REQUEST['file'])) {
+        $fileinfo = pathinfo($_REQUEST['file']);
+        $items = explode('-', $fileinfo['filename']);
+        $_REQUEST['exe_id'] = $items[5];
+    }
 }
 $nano = new Nanogong($_REQUEST);
 $is_nano = false;
 
 if (isset($_REQUEST['is_nano'])) {
-	$is_nano = true;
+    $is_nano = true;
 }
 
 switch ($action) {
-	case 'get_file':
-		if ($nano->get_param_value('user_id') == api_get_user_id() || api_is_allowed_to_edit()) {
-			$file_path = $nano->load_filename_if_exists();
-			//$file_path = '/var/www/rocio/courses/GREAT123/exercises/0/2/5/1/1-0-1-2-5-38.mp3';
-			if ($file_path) {
-				$info = pathinfo($file_path);
-				$user_info = api_get_user_info($nano->params['user_id']);
-				$name = get_lang('Quiz').'-'.$user_info['firstname'].'-'.$user_info['lastname'].'.'.$info['extension'];
-				$download = true;
-				if (isset($_REQUEST['download'])  && $_REQUEST['download'] == 0) {
-					$download = false;
-				}
-				DocumentManager::file_send_for_download($file_path, $download);
-				exit;
-			}
-		}
-		break;
-	case 'show_audio':
-		if (!$is_nano) {
-			echo $nano->return_js($_REQUEST);
-		}
-		echo $nano->show_audio_file($is_nano);
-		break;
-	case 'delete':
-		$return = $nano->delete_files();
-		if ($return == 1) {
-			//cant' do this because the post that nano send doesnt take into account the session
-			Display::display_confirmation_message(get_lang('FileDeleted'));
-		} else {
-			Display::display_confirmation_message(get_lang('FileNotFound'));
-		}
-		break;
-	case 'show_form':
-		api_protect_course_script(true);
-		Display::display_reduced_header();
-		echo $nano->return_js($_REQUEST);
-		echo $nano->return_form();
-		break;
+    case 'get_file':
+        if ($nano->get_param_value('user_id') == api_get_user_id() || api_is_allowed_to_edit()) {
+            $file_path = $nano->load_filename_if_exists();
+            //$file_path = '/var/www/rocio/courses/GREAT123/exercises/0/2/5/1/1-0-1-2-5-38.mp3';
+            if ($file_path) {
+                $info = pathinfo($file_path);
+                $user_info = api_get_user_info($nano->params['user_id']);
+                $name = get_lang('Quiz').'-'.$user_info['firstname'].'-'.$user_info['lastname'].'.'.$info['extension'];
+                $download = true;
+                if (isset($_REQUEST['download'])  && $_REQUEST['download'] == 0) {
+                    $download = false;
+                }
+                DocumentManager::file_send_for_download($file_path, $download);
+                exit;
+            }
+        }
+        break;
+    case 'show_audio':
+        if (!$is_nano) {
+            echo $nano->return_js($_REQUEST);
+        }
+        echo $nano->show_audio_file($is_nano);
+        break;
+    case 'delete':
+        $return = $nano->delete_files();
+        if ($return == 1) {
+            //cant' do this because the post that nano send doesnt take into account the session
+            Display::display_confirmation_message(get_lang('FileDeleted'));
+        } else {
+            Display::display_confirmation_message(get_lang('FileNotFound'));
+        }
+        break;
+    case 'show_form':
+        api_protect_course_script(true);
+        Display::display_reduced_header();
+        echo $nano->return_js($_REQUEST);
+        echo $nano->return_form();
+        break;
     case 'save_file':
-    	//User access same as upload.php
+        //User access same as upload.php
 
-    	$return = $nano->upload_file($is_nano);
+        $return = $nano->upload_file($is_nano);
 
-    	if ($is_nano) {
-    		//nano looks for numbers
-	    	if ($return == 1) {
-	    		//cant' do this because the post that nano send doesnt take into account the session
-	    		echo 1; //Display::display_confirmation_message(get_lang('UplUploadSucceeded'));
-	    	} else {
-	    		echo 0;
-	    		//Display::display_warning_message(get_lang('UplUnableToSaveFileFilteredExtension'));
-	    	}
-    	} else {
-    		Display::display_reduced_header();
-    		echo $nano->return_js($_REQUEST);
-    		//normal form
-    		if ($return == 1) {
-    			//cant' do this because the post that nano send doesnt take into account the session
-    			$message = Display::return_message(get_lang('UplUploadSucceeded'), 'confirm');
-    		} else {
-    			$message = Display::return_message(get_lang('UplUnableToSaveFileFilteredExtension'), 'warning');
-    		}
-    		echo $nano->return_form($message);
-    	}
+        if ($is_nano) {
+            //nano looks for numbers
+            if ($return == 1) {
+                //cant' do this because the post that nano send doesnt take into account the session
+                echo 1; //Display::display_confirmation_message(get_lang('UplUploadSucceeded'));
+            } else {
+                echo 0;
+                //Display::display_warning_message(get_lang('UplUnableToSaveFileFilteredExtension'));
+            }
+        } else {
+            Display::display_reduced_header();
+            echo $nano->return_js($_REQUEST);
+            //normal form
+            if ($return == 1) {
+                //cant' do this because the post that nano send doesnt take into account the session
+                $message = Display::return_message(get_lang('UplUploadSucceeded'), 'confirm');
+            } else {
+                $message = Display::return_message(get_lang('UplUnableToSaveFileFilteredExtension'), 'warning');
+            }
+            echo $nano->return_form($message);
+        }
         break;
     default:
         echo '';

main/inc/ajax/online.ajax.php

Spacing +3 added lines, -3 removed lines

@@ -4,7 +4,7 @@ 
 
 $action = $_GET['a'];
 
-switch($action) {
+switch ($action) {
     case 'load_online_user':
         if (isset($_SESSION['who_is_online_counter'])) {
             $_SESSION['who_is_online_counter']++;
@@ -14,8 +14,8 @@ 
         $images_to_show = 9;
 
         $page = intval($_REQUEST['online_page_nr']);
-        $max_page = ceil(who_is_online_count()/$images_to_show);
-        $page_rows = ($page-1)*9;
+        $max_page = ceil(who_is_online_count() / $images_to_show);
+        $page_rows = ($page - 1) * 9;
 
         if (!empty($max_page) && $page <= $max_page) {
             if (isset($_GET['cidReq']) && strlen($_GET['cidReq']) > 0) {

main/inc/ajax/message.ajax.php

Spacing +8 added lines, -8 removed lines

@@ -37,11 +37,11 @@ 
             break;
         }
         $track_online_table      = Database::get_main_table(TABLE_STATISTIC_TRACK_E_ONLINE);
-        $tbl_my_user		     = Database::get_main_table(TABLE_MAIN_USER);
+        $tbl_my_user = Database::get_main_table(TABLE_MAIN_USER);
         $tbl_my_user_friend      = Database::get_main_table(TABLE_MAIN_USER_REL_USER);
-        $tbl_user 			     = Database::get_main_table(TABLE_MAIN_USER);
+        $tbl_user = Database::get_main_table(TABLE_MAIN_USER);
         $tbl_access_url_rel_user = Database :: get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER);
-        $search				     = Database::escape_string($_REQUEST['q']);
+        $search = Database::escape_string($_REQUEST['q']);
 
         $access_url_id           = api_get_multiple_access_url() == 'true' ? api_get_current_access_url_id() : 1;
         $user_id                 = api_get_user_id();
@@ -49,9 +49,9 @@ 
 
         $likeCondition = " AND (firstname LIKE '%$search%' OR lastname LIKE '%$search%' OR email LIKE '%$search%') ";
 
-        if (api_get_setting('allow_social_tool')=='true' && api_get_setting('allow_message_tool') == 'true') {
+        if (api_get_setting('allow_social_tool') == 'true' && api_get_setting('allow_message_tool') == 'true') {
             // All users
-            if (api_get_setting('allow_send_message_to_all_platform_users') == 'true' || api_is_platform_admin() ) {
+            if (api_get_setting('allow_send_message_to_all_platform_users') == 'true' || api_is_platform_admin()) {
                 if ($access_url_id != 0) {
                     $sql = "SELECT DISTINCT u.user_id as id, u.firstname, u.lastname, u.email
                             FROM $tbl_user u LEFT JOIN $tbl_access_url_rel_user r ON u.user_id = r.user_id
@@ -97,7 +97,7 @@ 
                                 $likeCondition";
                 }
             }
-        } elseif (api_get_setting('allow_social_tool')=='false' && api_get_setting('allow_message_tool')=='true') {
+        } elseif (api_get_setting('allow_social_tool') == 'false' && api_get_setting('allow_message_tool') == 'true') {
             if (api_get_setting('allow_send_message_to_all_platform_users') == 'true') {
                 $sql = "SELECT DISTINCT u.user_id as id, u.firstname, u.lastname, u.email
                         FROM $tbl_user u LEFT JOIN $tbl_access_url_rel_user r ON u.user_id = r.user_id
@@ -108,7 +108,7 @@ 
                             $likeCondition ";
             } else {
                 $time_limit = api_get_setting('time_limit_whosonline');
-                $online_time = time() - $time_limit*60;
+                $online_time = time() - $time_limit * 60;
                 $limit_date	 = api_get_utc_datetime($online_time);
                 $sql = "SELECT SELECT DISTINCT u.user_id as id, u.firstname, u.lastname, u.email
                         FROM $tbl_my_user u INNER JOIN $track_online_table t
@@ -117,7 +117,7 @@ 
                         $likeCondition";
             }
         }
-        $sql .=' LIMIT 20';
+        $sql .= ' LIMIT 20';
         $result = Database::query($sql);
 
         $showEmail = api_get_setting('show_email_addresses');

main/inc/ajax/admin.ajax.php

Spacing +13 added lines, -13 removed lines

@@ -50,9 +50,9 @@ 
             $url = api_remove_trailing_slash(preg_replace('/https?:\/\//i', '', $urlInfo['url']));
             $cleanUrl = str_replace('/', '-', $url);
 
-            $newUrlDir = api_get_path(SYS_APP_PATH) . "home/$cleanUrl/admin/";
+            $newUrlDir = api_get_path(SYS_APP_PATH)."home/$cleanUrl/admin/";
         } else {
-            $newUrlDir = api_get_path(SYS_APP_PATH) . "home/admin/";
+            $newUrlDir = api_get_path(SYS_APP_PATH)."home/admin/";
         }
 
         if (!file_exists($newUrlDir)) {
@@ -166,15 +166,15 @@ 
             }
 
             if ($system_version != $version_info) {
-                $output = '<br /><span style="color:red">' . get_lang('YourVersionNotUpToDate') . '. '.get_lang('LatestVersionIs').' <b>Chamilo '.$version_info.'</b>. '.get_lang('YourVersionIs').' <b>Chamilo '.$system_version. '</b>. '.str_replace('http://www.chamilo.org', '<a href="http://www.chamilo.org">http://www.chamilo.org</a>', get_lang('PleaseVisitOurWebsite')).'</span>';
+                $output = '<br /><span style="color:red">'.get_lang('YourVersionNotUpToDate').'. '.get_lang('LatestVersionIs').' <b>Chamilo '.$version_info.'</b>. '.get_lang('YourVersionIs').' <b>Chamilo '.$system_version.'</b>. '.str_replace('http://www.chamilo.org', '<a href="http://www.chamilo.org">http://www.chamilo.org</a>', get_lang('PleaseVisitOurWebsite')).'</span>';
             } else {
                 $output = '<br /><span style="color:green">'.get_lang('VersionUpToDate').': Chamilo '.$version_info.'</span>';
             }
         } else {
-            $output = '<span style="color:red">' . get_lang('ImpossibleToContactVersionServerPleaseTryAgain') . '</span>';
+            $output = '<span style="color:red">'.get_lang('ImpossibleToContactVersionServerPleaseTryAgain').'</span>';
         }
     } else {
-        $output = '<span style="color:red">' . get_lang('AllowurlfopenIsSetToOff') . '</span>';
+        $output = '<span style="color:red">'.get_lang('AllowurlfopenIsSetToOff').'</span>';
     }
     return $output;
 }
@@ -197,17 +197,17 @@ 
     $getdata_str = count($getdata) ? '?' : '';
 
     foreach ($getdata as $k => $v) {
-                $getdata_str .= urlencode($k) .'='. urlencode($v) . '&';
+                $getdata_str .= urlencode($k).'='.urlencode($v).'&';
     }
 
     $crlf = "\r\n";
-    $req = $verb .' '. $uri . $getdata_str .' HTTP/1.1' . $crlf;
-    $req .= 'Host: '. $ip . $crlf;
-    $req .= 'User-Agent: Mozilla/5.0 Firefox/3.6.12' . $crlf;
-    $req .= 'Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8' . $crlf;
-    $req .= 'Accept-Language: en-us,en;q=0.5' . $crlf;
-    $req .= 'Accept-Encoding: deflate' . $crlf;
-    $req .= 'Accept-Charset: utf-8;q=0.7,*;q=0.7' . $crlf;
+    $req = $verb.' '.$uri.$getdata_str.' HTTP/1.1'.$crlf;
+    $req .= 'Host: '.$ip.$crlf;
+    $req .= 'User-Agent: Mozilla/5.0 Firefox/3.6.12'.$crlf;
+    $req .= 'Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8'.$crlf;
+    $req .= 'Accept-Language: en-us,en;q=0.5'.$crlf;
+    $req .= 'Accept-Encoding: deflate'.$crlf;
+    $req .= 'Accept-Charset: utf-8;q=0.7,*;q=0.7'.$crlf;
 
     $req .= $crlf;
 

main/inc/ajax/exercise.ajax.php

Indentation +1 added lines, -1 removed lines

@@ -488,7 +488,7 @@
                     $remind_list
                 );
 
-                 // Destruction of the Question object
+                    // Destruction of the Question object
                 unset($objQuestionTmp);
                 if ($debug) {
                     error_log(" -- end question -- ");

Spacing +16 added lines, -16 removed lines

@@ -30,10 +30,10 @@ 
         $exercise_id = intval($_GET['exercise_id']);
         $page = intval($_REQUEST['page']); //page
         $limit = intval($_REQUEST['rows']); //quantity of rows
-        $sidx = $_REQUEST['sidx'];         //index to filter
-        $sord = $_REQUEST['sord'];         //asc or desc
+        $sidx = $_REQUEST['sidx']; //index to filter
+        $sord = $_REQUEST['sord']; //asc or desc
 
-        if (!in_array($sord, array('asc','desc'))) {
+        if (!in_array($sord, array('asc', 'desc'))) {
             $sord = 'desc';
         }
         // get index row - i.e. user click to sort $sord = $_GET['sord'];
@@ -62,7 +62,7 @@ 
         $total_pages = 0;
         if ($count > 0) {
             if (!empty($limit)) {
-                $total_pages = ceil($count/$limit);
+                $total_pages = ceil($count / $limit);
             }
         }
 
@@ -124,7 +124,7 @@ 
         $response->page = $page;
         $response->total = $total_pages;
         $response->records = $count;
-        $i=0;
+        $i = 0;
 
         if (!empty($results)) {
             foreach ($results as $row) {
@@ -138,22 +138,22 @@ 
                             GROUP by question_id
                         ) as count_table";
                 $result_count = Database::query($sql);
-                $count_questions = Database::fetch_array($result_count,'ASSOC');
+                $count_questions = Database::fetch_array($result_count, 'ASSOC');
                 $count_questions = $count_questions['count_question_id'];
 
                 $row['count_questions'] = $count_questions;
 
                 $response->rows[$i]['id'] = $row['exe_id'];
-                $remaining = strtotime($row['start_date'])+($oExe->expired_time*60) - strtotime(api_get_utc_datetime(time()));
-                $h = floor($remaining/3600);
-                $m = floor(($remaining - ($h*3600))/60);
-                $s = ($remaining - ($h*3600) - ($m*60));
+                $remaining = strtotime($row['start_date']) + ($oExe->expired_time * 60) - strtotime(api_get_utc_datetime(time()));
+                $h = floor($remaining / 3600);
+                $m = floor(($remaining - ($h * 3600)) / 60);
+                $s = ($remaining - ($h * 3600) - ($m * 60));
                 $array = array(
                     $row['firstname'],
                     $row['lastname'],
-                    api_format_date($row['start_date'], DATE_TIME_FORMAT_LONG).' ['.($h>0?$h.':':'').sprintf("%02d",$m).':'.sprintf("%02d",$s).']',
+                    api_format_date($row['start_date'], DATE_TIME_FORMAT_LONG).' ['.($h > 0 ? $h.':' : '').sprintf("%02d", $m).':'.sprintf("%02d", $s).']',
                     $row['count_questions'],
-                    round($row['score']*100).'%'
+                    round($row['score'] * 100).'%'
                 );
                 $response->rows[$i]['cell'] = $array;
                 $i++;
@@ -209,7 +209,7 @@ 
         break;
     case 'add_question_to_reminder':
         /** @var Exercise $objExercise */
-        $objExercise  = $_SESSION['objExercise'];
+        $objExercise = $_SESSION['objExercise'];
         if (empty($objExercise)) {
             echo 0;
             exit;
@@ -362,8 +362,8 @@ 
                 $objQuestionTmp = Question::read($my_question_id, $course_id);
 
                 // Getting free choice data.
-                if ($objQuestionTmp->type  == FREE_ANSWER && $type == 'all') {
-                    $my_choice = isset($_REQUEST['free_choice'][$my_question_id]) && !empty($_REQUEST['free_choice'][$my_question_id]) ? $_REQUEST['free_choice'][$my_question_id]: null;
+                if ($objQuestionTmp->type == FREE_ANSWER && $type == 'all') {
+                    $my_choice = isset($_REQUEST['free_choice'][$my_question_id]) && !empty($_REQUEST['free_choice'][$my_question_id]) ? $_REQUEST['free_choice'][$my_question_id] : null;
                 }
 
                 if ($type == 'all') {
@@ -407,7 +407,7 @@ 
                         $session_id,
                         $my_question_id
                     );
-                    if ($objQuestionTmp->type  == HOT_SPOT) {
+                    if ($objQuestionTmp->type == HOT_SPOT) {
                         Event::delete_attempt_hotspot(
                             $exe_id,
                             api_get_user_id(),

main/inc/ajax/course.ajax.php

Indentation +6 added lines, -6 removed lines

@@ -258,13 +258,13 @@
                 ), 'first');
 
                 $coachName = api_get_person_name($userResult['firstname'], $userResult['lastname']);
-           }
+            }
 
-           $courses[] = array(
-               'id' => $courseId,
-               'name' => $course['title'],
-               'coachName' => $coachName,
-           );
+            $courses[] = array(
+                'id' => $courseId,
+                'name' => $course['title'],
+                'coachName' => $coachName,
+            );
         }
 
         echo json_encode($courses);

Spacing +7 added lines, -7 removed lines

@@ -73,7 +73,7 @@ 
                     0, //howMany
                     1, //$orderby = 1
                     'ASC',
-                    -1,  //visibility
+                    -1, //visibility
                     $_GET['q'],
                     null, //$urlId
                     true //AlsoSearchCode
@@ -92,7 +92,7 @@ 
 
                 if (!empty($course['category_code'])) {
                     $parents = getParentsToString($course['category_code']);
-                    $title = $parents . $course['title'];
+                    $title = $parents.$course['title'];
                 }
 
                 $results['items'][] = array(
@@ -172,7 +172,7 @@ 
                     INNER JOIN $session_course_user r ON u.user_id = r.user_id
                     WHERE session_id = %d AND c_id =  '%s'
                     AND (u.firstname LIKE '%s' OR u.username LIKE '%s' OR u.lastname LIKE '%s')";
-            $needle = '%' . $_GET['q'] . '%';
+            $needle = '%'.$_GET['q'].'%';
             $sql_query = sprintf($sql, $_GET['session_id'], $course['real_id'], $needle, $needle, $needle);
 
             $result = Database::query($sql_query);
@@ -191,11 +191,11 @@ 
     case 'search_exercise_by_course':
         if (api_is_platform_admin()) {
             $course = api_get_course_info_by_id($_GET['course_id']);
-            $session_id = (!empty($_GET['session_id'])) ?  intval($_GET['session_id']) : 0 ;
+            $session_id = (!empty($_GET['session_id'])) ? intval($_GET['session_id']) : 0;
             $exercises = ExerciseLib::get_all_exercises($course, $session_id, false, $_GET['q'], true, 3);
 
             foreach ($exercises as $exercise) {
-                $data[] = array('id' => $exercise['id'], 'text' => html_entity_decode($exercise['title']) );
+                $data[] = array('id' => $exercise['id'], 'text' => html_entity_decode($exercise['title']));
             }
             if (!empty($data)) {
                 $data[] = array('id' => 'T', 'text' => 'TODOS');
@@ -220,11 +220,11 @@ 
                 $sql,
                 intval($_GET['course_id']),
                 intval($_GET['session_id']),
-                '%' . Database::escape_string($_GET['q']).'%'
+                '%'.Database::escape_string($_GET['q']).'%'
             );
             $result = Database::query($sql_query);
             while ($survey = Database::fetch_assoc($result)) {
-                $survey['title'] .= ($survey['anonymous'] == 1) ? ' (' . get_lang('Anonymous') . ')' : '';
+                $survey['title'] .= ($survey['anonymous'] == 1) ? ' ('.get_lang('Anonymous').')' : '';
                 $data[] = array(
                     'id' => $survey['id'],
                     'text' => strip_tags(html_entity_decode($survey['title']))

main/inc/ajax/course_home.ajax.php

Braces +9 added lines, -6 removed lines

@@ -121,8 +121,9 @@ 
             $new_session_list = UserManager::get_personal_session_course_list(api_get_user_id());
             $my_session_list  = array();
             foreach($new_session_list as $item) {
-                if (!empty($item['id_session']))
-                    $my_session_list[] = $item['id_session'];
+                if (!empty($item['id_session'])) {
+                                    $my_session_list[] = $item['id_session'];
+                }
             }
             if (!in_array($session_id, $my_session_list)) {
             	break;
@@ -254,8 +255,9 @@ 
             $new_session_list = UserManager::get_personal_session_course_list(api_get_user_id());
             $my_session_list  = array();
             foreach($new_session_list as $item) {
-                if (!empty($item['id_session']))
-                    $my_session_list[] = $item['id_session'];
+                if (!empty($item['id_session'])) {
+                                    $my_session_list[] = $item['id_session'];
+                }
             }
             if (!in_array($session_id, $my_session_list)) {
                 break;
@@ -384,8 +386,9 @@ 
             $new_session_list = UserManager::get_personal_session_course_list(api_get_user_id());
             $my_session_list  = array();
             foreach($new_session_list as $item) {
-                if (!empty($item['id_session']))
-                    $my_session_list[] = $item['id_session'];
+                if (!empty($item['id_session'])) {
+                                    $my_session_list[] = $item['id_session'];
+                }
             }
             if (!in_array($session_id, $my_session_list)) {
                 break;

Indentation +21 added lines, -21 removed lines

@@ -63,12 +63,12 @@ 
             echo json_encode($response_data);
         }
         break;
-	case 'show_course_information' :
-		require_once '../global.inc.php';
+    case 'show_course_information' :
+        require_once '../global.inc.php';
 
-		// Get the name of the database course.
-		$tbl_course_description = Database::get_course_table(TABLE_COURSE_DESCRIPTION);
-		$course_info = api_get_course_info($_GET['code']);
+        // Get the name of the database course.
+        $tbl_course_description = Database::get_course_table(TABLE_COURSE_DESCRIPTION);
+        $course_info = api_get_course_info($_GET['code']);
 
         if (
             api_get_setting('course_catalog_hide_private') === 'true' &&
@@ -78,24 +78,24 @@ 
             break;
         }
 
-		$sql = "SELECT * FROM $tbl_course_description
+        $sql = "SELECT * FROM $tbl_course_description
 		        WHERE c_id = ".$course_info['real_id']." AND session_id = 0
 		        ORDER BY id";
-		$result = Database::query($sql);
-		if (Database::num_rows($result) > 0 ) {
-		    while ($description = Database::fetch_object($result)) {
-			    $descriptions[$description->id] = $description;
-		    }
+        $result = Database::query($sql);
+        if (Database::num_rows($result) > 0 ) {
+            while ($description = Database::fetch_object($result)) {
+                $descriptions[$description->id] = $description;
+            }
             // Function that displays the details of the course description in html.
-		    echo CourseManager::get_details_course_description_html(
+            echo CourseManager::get_details_course_description_html(
                 $descriptions,
                 api_get_system_encoding(),
                 false
             );
-		} else {
-		    echo get_lang('NoDescription');
-		}
-	    break;
+        } else {
+            echo get_lang('NoDescription');
+        }
+        break;
     case 'session_courses_lp_default':
         /**
          * @todo this functions need to belong to a class or a special
@@ -110,7 +110,7 @@ 
         $sidx  = isset($_REQUEST['sidx']) && !empty($_REQUEST['sidx']) ? $_REQUEST['sidx'] : 'id';
         $sord  = $_REQUEST['sord'];    //asc or desc
         if (!in_array($sord, array('asc','desc'))) {
-        	$sord = 'desc';
+            $sord = 'desc';
         }
         $session_id  = intval($_REQUEST['session_id']);
         $course_id   = intval($_REQUEST['course_id']);
@@ -124,7 +124,7 @@ 
                     $my_session_list[] = $item['id_session'];
             }
             if (!in_array($session_id, $my_session_list)) {
-            	break;
+                break;
             }
         }
 
@@ -445,7 +445,7 @@ 
                     $date = '-';
                 }
 
-                 //Checking LP publicated and expired_on dates
+                    //Checking LP publicated and expired_on dates
                 if (!empty($lp_item['publicated_on']) && $lp_item['publicated_on'] != '0000-00-00 00:00:00') {
                     if ($now < api_strtotime($lp_item['publicated_on'], 'UTC')) {
                         continue;
@@ -499,7 +499,7 @@ 
 
         echo json_encode($response);
         break;
-	default:
-		echo '';
+    default:
+        echo '';
 }
 exit;

Spacing +52 added lines, -52 removed lines

@@ -30,7 +30,7 @@ 
                     null,
                     true
                 );
-                $na_image   = str_replace('.png', '_na.png', $tool_image);
+                $na_image = str_replace('.png', '_na.png', $tool_image);
             }
 
             if (isset($tool_info['custom_icon']) && !empty($tool_info['custom_icon'])) {
@@ -38,20 +38,20 @@ 
                 $na_image = CourseHome::getCustomWebIconPath().CourseHome::getDisableIcon($tool_info['custom_icon']);
             }
 
-            $requested_image = ($tool_visibility == 0 ) ? $tool_image : $na_image;
-            $requested_class = ($tool_visibility == 0 ) ? 'visible' : 'invisible';
-            $requested_message = ($tool_visibility == 0 ) ? 'is_active' : 'is_inactive';
-            $requested_view = ($tool_visibility == 0 ) ? 'visible.png' : 'invisible.png';
-            $requested_visible = ($tool_visibility == 0 ) ? 1 : 0;
+            $requested_image = ($tool_visibility == 0) ? $tool_image : $na_image;
+            $requested_class = ($tool_visibility == 0) ? 'visible' : 'invisible';
+            $requested_message = ($tool_visibility == 0) ? 'is_active' : 'is_inactive';
+            $requested_view = ($tool_visibility == 0) ? 'visible.png' : 'invisible.png';
+            $requested_visible = ($tool_visibility == 0) ? 1 : 0;
 
-            $requested_view = ($tool_visibility == 0 ) ? 'visible.png' : 'invisible.png';
-            $requested_visible = ($tool_visibility == 0 ) ? 1 : 0;
+            $requested_view = ($tool_visibility == 0) ? 'visible.png' : 'invisible.png';
+            $requested_visible = ($tool_visibility == 0) ? 1 : 0;
 
             // HIDE AND REACTIVATE TOOL
             if ($_GET["id"] == strval(intval($_GET["id"]))) {
                 $sql = "UPDATE $tool_table SET
                         visibility = $requested_visible
-                        WHERE c_id = $course_id AND id='" . intval($_GET['id']) . "'";
+                        WHERE c_id = $course_id AND id='".intval($_GET['id'])."'";
                 Database::query($sql);
             }
             $response_data = array(
@@ -82,7 +82,7 @@ 
 		        WHERE c_id = ".$course_info['real_id']." AND session_id = 0
 		        ORDER BY id";
 		$result = Database::query($sql);
-		if (Database::num_rows($result) > 0 ) {
+		if (Database::num_rows($result) > 0) {
 		    while ($description = Database::fetch_object($result)) {
 			    $descriptions[$description->id] = $description;
 		    }
@@ -104,12 +104,12 @@ 
 
         require_once '../global.inc.php';
         $now = time();
-        $page  = intval($_REQUEST['page']);     //page
-        $limit = intval($_REQUEST['rows']);     // quantity of rows
+        $page  = intval($_REQUEST['page']); //page
+        $limit = intval($_REQUEST['rows']); // quantity of rows
         //index to filter
         $sidx  = isset($_REQUEST['sidx']) && !empty($_REQUEST['sidx']) ? $_REQUEST['sidx'] : 'id';
-        $sord  = $_REQUEST['sord'];    //asc or desc
-        if (!in_array($sord, array('asc','desc'))) {
+        $sord  = $_REQUEST['sord']; //asc or desc
+        if (!in_array($sord, array('asc', 'desc'))) {
         	$sord = 'desc';
         }
         $session_id  = intval($_REQUEST['session_id']);
@@ -119,7 +119,7 @@ 
         if (!api_is_platform_admin()) {
             $new_session_list = UserManager::get_personal_session_course_list(api_get_user_id());
             $my_session_list  = array();
-            foreach($new_session_list as $item) {
+            foreach ($new_session_list as $item) {
                 if (!empty($item['id_session']))
                     $my_session_list[] = $item['id_session'];
             }
@@ -128,7 +128,7 @@ 
             }
         }
 
-        $start = $limit*$page - $limit;
+        $start = $limit * $page - $limit;
         $course_list = SessionManager::get_course_list_by_session_id($session_id);
         $count = 0;
         $temp = array();
@@ -140,7 +140,7 @@ 
             $item['title'] = Display::url($item['title'], $course_url, array('target' => SESSION_LINK_TARGET));
 
             foreach ($flat_list as $lp_id => $lp_item) {
-                $temp[$count]['id']= $lp_id;
+                $temp[$count]['id'] = $lp_id;
 
                 $lp = new learnpath($item['code'], $lp_id, api_get_user_id());
                 if ($lp->progress_db == 100) {
@@ -203,27 +203,27 @@ 
         }
         $temp = msort($temp, $sidx, $sord);
 
-        $i =0;
+        $i = 0;
         $response = new stdClass();
-        foreach($temp as $key=>$row) {
+        foreach ($temp as $key=>$row) {
             $row = $row['cell'];
             if (!empty($row)) {
-                if ($key >= $start  && $key < ($start + $limit)) {
-                    $response->rows[$i]['id']= $key;
-                    $response->rows[$i]['cell']=array($row[0], $row[1], $row[2]);
+                if ($key >= $start && $key < ($start + $limit)) {
+                    $response->rows[$i]['id'] = $key;
+                    $response->rows[$i]['cell'] = array($row[0], $row[1], $row[2]);
                     $i++;
                 }
             }
         }
 
-        if($count > 0 && $limit > 0) {
-            $total_pages = ceil($count/$limit);
+        if ($count > 0 && $limit > 0) {
+            $total_pages = ceil($count / $limit);
         } else {
             $total_pages = 0;
         }
-        $response->total    = $total_pages;
+        $response->total = $total_pages;
         if ($page > $total_pages) {
-            $response->page= $total_pages;
+            $response->page = $total_pages;
         } else {
             $response->page = $page;
         }
@@ -234,13 +234,13 @@ 
         require_once '../global.inc.php';
         $now = time();
 
-        $page  = intval($_REQUEST['page']);     //page
-        $limit = intval($_REQUEST['rows']);     // quantity of rows
+        $page  = intval($_REQUEST['page']); //page
+        $limit = intval($_REQUEST['rows']); // quantity of rows
         $sidx  = isset($_REQUEST['sidx']) && !empty($_REQUEST['sidx']) ? $_REQUEST['sidx'] : 'course';
         $sidx = str_replace(array('week desc,', ' '), '', $sidx);
 
-        $sord  = $_REQUEST['sord'];    //asc or desc
-        if (!in_array($sord, array('asc','desc'))) {
+        $sord  = $_REQUEST['sord']; //asc or desc
+        if (!in_array($sord, array('asc', 'desc'))) {
             $sord = 'desc';
         }
 
@@ -251,7 +251,7 @@ 
         if (!api_is_platform_admin()) {
             $new_session_list = UserManager::get_personal_session_course_list(api_get_user_id());
             $my_session_list  = array();
-            foreach($new_session_list as $item) {
+            foreach ($new_session_list as $item) {
                 if (!empty($item['id_session']))
                     $my_session_list[] = $item['id_session'];
             }
@@ -260,7 +260,7 @@ 
             }
         }
 
-        $start = $limit*$page - $limit;
+        $start = $limit * $page - $limit;
         $course_list = SessionManager::get_course_list_by_session_id($session_id);
 
         $count = 0;
@@ -272,7 +272,7 @@ 
                 }
             }
 
-            $list = new LearnpathList(api_get_user_id(),$item['code'], $session_id, 'publicated_on DESC');
+            $list = new LearnpathList(api_get_user_id(), $item['code'], $session_id, 'publicated_on DESC');
             $flat_list = $list->get_flat_list();
             $lps[$item['code']] = $flat_list;
             $item['title'] = Display::url(
@@ -282,7 +282,7 @@ 
             );
 
             foreach ($flat_list as $lp_id => $lp_item) {
-                $temp[$count]['id']= $lp_id;
+                $temp[$count]['id'] = $lp_id;
                 $lp_url = api_get_path(WEB_CODE_PATH).'newscorm/lp_controller.php?cidReq='.$item['code'].'&id_session='.$session_id.'&lp_id='.$lp_id.'&action=view';
 
                 $last_date = Tracking::get_last_connection_date_on_the_course(
@@ -345,20 +345,20 @@ 
         }
 
         $response = new stdClass();
-        $i =0;
-        foreach($temp as $key=>$row) {
+        $i = 0;
+        foreach ($temp as $key=>$row) {
             $row = $row['cell'];
             if (!empty($row)) {
-                if ($key >= $start  && $key < ($start + $limit)) {
-                    $response->rows[$i]['id']= $key;
-                    $response->rows[$i]['cell']=array($row[0], $row[1], $row[2],$row[3]);
+                if ($key >= $start && $key < ($start + $limit)) {
+                    $response->rows[$i]['id'] = $key;
+                    $response->rows[$i]['cell'] = array($row[0], $row[1], $row[2], $row[3]);
                     $i++;
                 }
             }
         }
 
         if ($count > 0 && $limit > 0) {
-            $total_pages = ceil($count/$limit);
+            $total_pages = ceil($count / $limit);
         } else {
             $total_pages = 0;
         }
@@ -374,12 +374,12 @@ 
     case 'session_courses_lp_by_course':
         require_once '../global.inc.php';
         $now = time();
-        $page  = intval($_REQUEST['page']);     //page
-        $limit = intval($_REQUEST['rows']);     // quantity of rows
+        $page  = intval($_REQUEST['page']); //page
+        $limit = intval($_REQUEST['rows']); // quantity of rows
         $sidx = isset($_REQUEST['sidx']) && !empty($_REQUEST['sidx']) ? $_REQUEST['sidx'] : 'id';
         $sidx = str_replace(array('course asc,', ' '), '', $sidx);
 
-        $sord = $_REQUEST['sord'];    //asc or desc
+        $sord = $_REQUEST['sord']; //asc or desc
         if (!in_array($sord, array('asc', 'desc'))) {
             $sord = 'desc';
         }
@@ -390,7 +390,7 @@ 
         if (!api_is_platform_admin()) {
             $new_session_list = UserManager::get_personal_session_course_list(api_get_user_id());
             $my_session_list  = array();
-            foreach($new_session_list as $item) {
+            foreach ($new_session_list as $item) {
                 if (!empty($item['id_session']))
                     $my_session_list[] = $item['id_session'];
             }
@@ -399,7 +399,7 @@ 
             }
         }
 
-        $start = $limit*$page - $limit;
+        $start = $limit * $page - $limit;
         $course_list = SessionManager::get_course_list_by_session_id($session_id);
         $count = 0;
 
@@ -410,15 +410,15 @@ 
                 }
             }
 
-            $list = new LearnpathList(api_get_user_id(),$item['code'],$session_id);
+            $list = new LearnpathList(api_get_user_id(), $item['code'], $session_id);
             $flat_list = $list->get_flat_list();
             $lps[$item['code']] = $flat_list;
             $item['title'] = Display::url(
                 $item['title'],
                 api_get_path(WEB_COURSE_PATH).$item['directory'].'/?id_session='.$session_id, array('target'=>SESSION_LINK_TARGET)
             );
-            foreach($flat_list as $lp_id => $lp_item) {
-                $temp[$count]['id']= $lp_id;
+            foreach ($flat_list as $lp_id => $lp_item) {
+                $temp[$count]['id'] = $lp_id;
                 $lp_url = api_get_path(WEB_CODE_PATH).'newscorm/lp_controller.php?cidReq='.$item['code'].'&id_session='.$session_id.'&lp_id='.$lp_id.'&action=view';
                 $last_date = Tracking::get_last_connection_date_on_the_course(
                     api_get_user_id(),
@@ -472,13 +472,13 @@ 
         $temp = msort($temp, $sidx, $sord);
 
         $response = new stdClass();
-        $i =0;
+        $i = 0;
         foreach ($temp as $key=>$row) {
             $row = $row['cell'];
             if (!empty($row)) {
-                if ($key >= $start  && $key < ($start + $limit)) {
-                    $response->rows[$i]['id']= $key;
-                    $response->rows[$i]['cell']=array($row[0], $row[1], $row[2]);
+                if ($key >= $start && $key < ($start + $limit)) {
+                    $response->rows[$i]['id'] = $key;
+                    $response->rows[$i]['cell'] = array($row[0], $row[1], $row[2]);
                     $i++;
                 }
             }