chamilo / chamilo-lms

src/Chamilo/CoreBundle/Component/Editor/CkEditor/Toolbar/TestMatching.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
             ['Cut', 'Copy', 'Paste', 'PasteText', 'PasteFromWord'],
             ['Undo', 'Redo', '-', 'SelectAll', 'Find', '-', 'RemoveFormat'],
             ['Link', 'Unlink', 'Anchor', 'Glossary'],
-            ['Image', 'Mapping', 'Video', 'Oembed', 'Youtube', 'Flash', 'Audio', 'leaflet', 'Smiley', 'SpecialChar', 'Asciimath',],
+            ['Image', 'Mapping', 'Video', 'Oembed', 'Youtube', 'Flash', 'Audio', 'leaflet', 'Smiley', 'SpecialChar', 'Asciimath', ],
             '/',
             ['Table', '-', 'CreateDiv'],
             ['BulletedList', 'NumberedList', 'HorizontalRule', '-', 'Outdent', 'Indent', 'Blockquote'],

plugin/openmeetings/lib/openmeetings_rest_service.php

Spacing +13 added lines, -13 removed lines

@@ -40,31 +40,31 @@ 
         // $request
         
         // Initialize the session by passing the request as a parameter
-        $session = curl_init ( $request );
+        $session = curl_init($request);
         
         // Set curl options by passing session and flags
         // CURLOPT_HEADER allows us to receive the HTTP header
-        curl_setopt ( $session, CURLOPT_HEADER, true );
+        curl_setopt($session, CURLOPT_HEADER, true);
         
         // CURLOPT_RETURNTRANSFER will return the response
-        curl_setopt ( $session, CURLOPT_RETURNTRANSFER, true );
+        curl_setopt($session, CURLOPT_RETURNTRANSFER, true);
         
         // Make the request
-        $response = curl_exec ( $session );
+        $response = curl_exec($session);
         
         // Close the curl session
-        curl_close ( $session );
+        curl_close($session);
         
         // Confirm that the request was transmitted to the OpenMeetings! Image Search Service
-        if (! $response) {
-            die ( "Request OpenMeetings! OpenMeetings Service failed and no response was returned in ".__CLASS__.'::'.__FUNCTION__.'()' );
+        if (!$response) {
+            die ("Request OpenMeetings! OpenMeetings Service failed and no response was returned in ".__CLASS__.'::'.__FUNCTION__.'()');
         }
         
         // Create an array to store the HTTP response codes
-        $status_code = array ();
+        $status_code = array();
         
         // Use regular expressions to extract the code from the header
-        preg_match ( '/\d\d\d/', $response, $status_code );
+        preg_match('/\d\d\d/', $response, $status_code);
         $bt = debug_backtrace();
         $caller = array_shift($bt);
         $extra = ' (from '.$caller['file'].' at line '.$caller['line'].') ';
@@ -74,24 +74,24 @@ 
                 // Success
                 break;
             case 503 :
-                error_log( 'Your call to OpenMeetings Web Services '.$extra.' failed and returned an HTTP status of 503.
+                error_log('Your call to OpenMeetings Web Services '.$extra.' failed and returned an HTTP status of 503.
                                  That means: Service unavailable. An internal problem prevented us from returning data to you.' );
                 return false;
                 break;
             case 403 :
-                error_log( 'Your call to OpenMeetings Web Services '.$extra.' failed and returned an HTTP status of 403.
+                error_log('Your call to OpenMeetings Web Services '.$extra.' failed and returned an HTTP status of 403.
                                  That means: Forbidden. You do not have permission to access this resource, or are over your rate limit.' );
                 return false;
                 break;
             case 400 :
                 // You may want to fall through here and read the specific XML error
-                error_log( 'Your call to OpenMeetings Web Services '.$extra.' failed and returned an HTTP status of 400.
+                error_log('Your call to OpenMeetings Web Services '.$extra.' failed and returned an HTTP status of 400.
                                  That means:  Bad request. The parameters passed to the service did not match as expected.   
                                  The exact error is returned in the XML response.' );
                 return false;
                 break;
             default:
-                error_log( 'Your call to OpenMeetings Web Services '.$extra.' returned an unexpected HTTP status of: ' . $status_code [0] . " Request " . $request );
+                error_log('Your call to OpenMeetings Web Services '.$extra.' returned an unexpected HTTP status of: '.$status_code [0]." Request ".$request);
                 return false;
         }
         

plugin/dashboard/block_daily/block_daily.class.php

Spacing +21 added lines, -21 removed lines

@@ -72,19 +72,19 @@ 
         $data = array();
         $content = $this->get_content_html();
         $html = '<div class="panel panel-default" id="intro">
-		                <div class="panel-heading">' . get_lang('GradebookAndAttendances') . '
+		                <div class="panel-heading">' . get_lang('GradebookAndAttendances').'
 		                    <div class="pull-right"><a class="btn btn-danger btn-xs" onclick="javascript:if(!confirm(\'' . addslashes(
                 api_htmlentities(
                     get_lang('ConfirmYourChoice'),
                     ENT_QUOTES,
                     $charset
                 )
-            ) . '\')) return false;" href="index.php?action=disable_block&path=' . $this->path . '">
+            ).'\')) return false;" href="index.php?action=disable_block&path='.$this->path.'">
                     <em class="fa fa-times"></em>
                         </a></div>
 		                </div>
 		                <div class="panel-body">
-		                   ' . $content . '
+		                   ' . $content.'
 		                </div>
 		            </div>
 				';
@@ -102,17 +102,17 @@ 
     {
         $course_data = $this->get_course_information_data();
         //$content = '<div style="margin:10px;">';
-        $content = '<h4>' . get_lang(
+        $content = '<h4>'.get_lang(
                 'YourCourseList'
-            ) . '</h4>';
+            ).'</h4>';
         $data_table = null;
         if (!empty($course_data)) {
             $data_table .= '<table class="data_table" width:"95%">';
             $data_table .= '<tr>
-	    						<th>' . get_lang('CourseTitle') . '</th>
-	    						<th width="20%">' . get_lang('NbStudents') . '</th>
-	    						<th width="20%">' . get_lang('Evaluation') . '</th>
-	    						<th width="20%">' . get_lang('ToolAttendance') . '</th>
+	    						<th>' . get_lang('CourseTitle').'</th>
+	    						<th width="20%">' . get_lang('NbStudents').'</th>
+	    						<th width="20%">' . get_lang('Evaluation').'</th>
+	    						<th width="20%">' . get_lang('ToolAttendance').'</th>
 	    					</tr>';
             $i = 1;
             foreach ($course_data as $course) {
@@ -121,12 +121,12 @@ 
                 } else {
                     $class_tr = 'row_even';
                 }
-                $data_table .= '<tr class="' . $class_tr . '">';
+                $data_table .= '<tr class="'.$class_tr.'">';
                 if (!isset($course[3])) {
                     $course[3] = get_lang('NotAvailable');
                 }
                 foreach ($course as $cell) {
-                    $data_table .= '<td align="right">' . $cell . '</td>';
+                    $data_table .= '<td align="right">'.$cell.'</td>';
                 }
                 $data_table .= '</tr>';
                 $i++;
@@ -138,7 +138,7 @@ 
         $content .= $data_table;
         if (!empty($course_data)) {
             $content .= '<div style="text-align:right;margin-top:10px;">
-            <a href="' . api_get_path(WEB_CODE_PATH) . 'mySpace/course.php">' . get_lang('SeeMore') . '</a></div>';
+            <a href="' . api_get_path(WEB_CODE_PATH).'mySpace/course.php">'.get_lang('SeeMore').'</a></div>';
         }
         //$content .= '</div>';
         return $content;
@@ -176,7 +176,7 @@ 
             $table_course = Database::get_course_table(TABLE_ATTENDANCE);
 
             $sql = "SELECT id, name, attendance_qualify_max FROM $table_course
-                    WHERE c_id = " . $course_info['real_id'] . " AND active = 1 AND session_id = 0";
+                    WHERE c_id = ".$course_info['real_id']." AND active = 1 AND session_id = 0";
             $rs = Database::query($sql);
             $attendance = array();
             $attendances = array();
@@ -188,7 +188,7 @@ 
                 $attendance['course_code'] = $course_info['code'];
 
                 if ($attendance['done'] != '0') {
-                    $attendances[] = '<a href="' . api_get_path(WEB_PATH).'main/attendance/index.php?cidReq=' . $attendance['course_code'] . '&action=attendance_sheet_print&attendance_id=' . $attendance['id']. '">' . Display::return_icon('printmgr.gif', get_lang('Print')).'</a>';
+                    $attendances[] = '<a href="'.api_get_path(WEB_PATH).'main/attendance/index.php?cidReq='.$attendance['course_code'].'&action=attendance_sheet_print&attendance_id='.$attendance['id'].'">'.Display::return_icon('printmgr.gif', get_lang('Print')).'</a>';
                 } else {
                     $attendances[] = get_lang("NotAvailable");
                 }
@@ -200,7 +200,7 @@ 
             // Number of students
 
             $sql = "SELECT user_id FROM $tbl_course_user as course_rel_user
-                    WHERE course_rel_user.status=" . STUDENT . " AND course_rel_user.c_id=$courseId";
+                    WHERE course_rel_user.status=".STUDENT." AND course_rel_user.c_id=$courseId";
             $rs = Database::query($sql);
             $users = array();
             while ($row = Database::fetch_array($rs)) {
@@ -211,11 +211,11 @@ 
             }
 
             if (!empty($tematic_advance)) {
-                $tematic_advance_progress = '<a title="' . get_lang(
+                $tematic_advance_progress = '<a title="'.get_lang(
                         'GoToThematicAdvance'
-                    ) . '" href="' . api_get_path(
+                    ).'" href="'.api_get_path(
                         WEB_CODE_PATH
-                    ) . 'attendance/index.php?cidReq=' . $course_code . '&action=attendance_sheet_print&attendance_id=">' . $tematic_advance . '%</a>';
+                    ).'attendance/index.php?cidReq='.$course_code.'&action=attendance_sheet_print&attendance_id=">'.$tematic_advance.'%</a>';
             } else {
                 $tematic_advance_progress = '0%';
             }
@@ -224,8 +224,8 @@ 
             $tbl_grade_categories = Database::get_main_table(
                 TABLE_MAIN_GRADEBOOK_CATEGORY
             );
-            $sql = "SELECT id from " . $tbl_grade_categories . "
-                    WHERE course_code ='" . $course_code . "'";
+            $sql = "SELECT id from ".$tbl_grade_categories."
+                    WHERE course_code ='" . $course_code."'";
             $rs = Database::query($sql);
             $category = null;
             while ($row = Database::fetch_array($rs)) {
@@ -238,7 +238,7 @@ 
                 if (count($eval) > 0) {
                     $i = 0;
                     foreach ($eval as $item) {
-                        $score .= '<a href="' . api_get_path(WEB_PATH).'main/gradebook/gradebook_view_result.php?export=pdf&cat_code=' . $cat[0]->get_id() . '&official_code=' . $cat[0]->get_course_code() . '&selecteval=' . $item->get_id(). '">' . $item->get_name() . '</a>';
+                        $score .= '<a href="'.api_get_path(WEB_PATH).'main/gradebook/gradebook_view_result.php?export=pdf&cat_code='.$cat[0]->get_id().'&official_code='.$cat[0]->get_course_code().'&selecteval='.$item->get_id().'">'.$item->get_name().'</a>';
                         if (count($eval) - 1 != $i) {
                             $score .= ', ';
                         }

plugin/dashboard/block_global_info/block_global_info.class.php

Spacing +5 added lines, -5 removed lines

@@ -29,10 +29,10 @@ 
     /**
      * Constructor
      */
-    public function __construct ($user_id)
+    public function __construct($user_id)
     {
-        $this->user_id 		= $user_id;
-        $this->path 		= 'block_global_info';
+        $this->user_id = $user_id;
+        $this->path = 'block_global_info';
         if ($this->is_block_visible_for_user($user_id)) {
             //$this->courses = CourseManager::get_courses_followed_by_drh($user_id);
         }
@@ -69,7 +69,7 @@ 
     	$html = '
     	            <div class="panel panel-default" id="intro">
     	                <div class="panel-heading">'.get_lang('GlobalPlatformInformation').'
-    	                    <div class="pull-right"><a class="btn btn-danger btn-xs" onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang('ConfirmYourChoice'),ENT_QUOTES,$charset)).'\')) return false;" href="index.php?action=disable_block&path='.$this->path.'">
+    	                    <div class="pull-right"><a class="btn btn-danger btn-xs" onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang('ConfirmYourChoice'), ENT_QUOTES, $charset)).'\')) return false;" href="index.php?action=disable_block&path='.$this->path.'">
     	                    <em class="fa fa-times"></em>
     	                    </a></div>
     	                </div>
@@ -98,7 +98,7 @@ 
              $data_table = '<table class="table table-bordered" width="95%">';
              $i = 1;
              foreach ($global_data as $data) {
-                 if ($i%2 == 0) {
+                 if ($i % 2 == 0) {
                      $class_tr = 'row_odd';
                  } else {
                      $class_tr = 'row_even';

plugin/buycourses/src/buy_course_plugin.class.php

Spacing +27 added lines, -27 removed lines

@@ -124,7 +124,7 @@ 
             return false;

         }

 

-        require_once api_get_path(SYS_PLUGIN_PATH) . 'buycourses/database.php';

+        require_once api_get_path(SYS_PLUGIN_PATH).'buycourses/database.php';

     }

 

     /**

@@ -173,10 +173,10 @@ 
             $item = $this->getItemByProduct(intval($productId), $productType);

             $return['html'] = '<div class="buycourses-price">';

             if ($item) {

-                $return['html'] .= '<span class="label label-primary"><b>'. $item['iso_code'] .' ' . $item['price'] . '</b></span>';

+                $return['html'] .= '<span class="label label-primary"><b>'.$item['iso_code'].' '.$item['price'].'</b></span>';

                 $return['verificator'] = true;

             } else {

-                $return['html'] .= '<span class="label label-primary"><b>'. $this->get_lang('Free'). '</b></span>';

+                $return['html'] .= '<span class="label label-primary"><b>'.$this->get_lang('Free').'</b></span>';

                 $return['verificator'] = false;

             }

             $return['html'] .= '</div>';

@@ -195,15 +195,15 @@ 
      */

     public function returnBuyCourseButton($productId, $productType)

     {

-        $url = api_get_path(WEB_PLUGIN_PATH) .

-            'buycourses/src/process.php?i=' .

-            intval($productId) .

-            '&t=' .

+        $url = api_get_path(WEB_PLUGIN_PATH).

+            'buycourses/src/process.php?i='.

+            intval($productId).

+            '&t='.

             $productType

         ;

 

-        $html = ' <a class="btn btn-success btn-sm" title="' . $this->get_lang('Buy') . '" href="' . $url . '">' .

-            Display::returnFontAwesomeIcon('fa fa-shopping-cart') . '</a>';

+        $html = ' <a class="btn btn-success btn-sm" title="'.$this->get_lang('Buy').'" href="'.$url.'">'.

+            Display::returnFontAwesomeIcon('fa fa-shopping-cart').'</a>';

 

         return $html;

     }

@@ -748,7 +748,7 @@ 
         );

 

         if (!empty($sessionImage)) {

-            $sessionInfo['image'] = api_get_path(WEB_UPLOAD_PATH) . $sessionImage['value'];

+            $sessionInfo['image'] = api_get_path(WEB_UPLOAD_PATH).$sessionImage['value'];

         }

 

         $sessionCourses = $session->getCourses();

@@ -1201,7 +1201,7 @@ 
 

         $str = '';

 

-        srand((double)microtime() * 1000000);

+        srand((double) microtime() * 1000000);

 

         for ($i = 0; $i < $length; $i++) {

             $numbers = rand(0, strlen($salt) - 1);

@@ -1543,8 +1543,8 @@ 
      */

     public function getPayouts($status = self::PAYOUT_STATUS_PENDING, $payoutId = false, $userId = false)

     {

-        $condition = ($payoutId) ? 'AND p.id = '. intval($payoutId) : '';

-        $condition2 = ($userId) ? ' AND p.user_id = ' . intval($userId) : '';

+        $condition = ($payoutId) ? 'AND p.id = '.intval($payoutId) : '';

+        $condition2 = ($userId) ? ' AND p.user_id = '.intval($userId) : '';

         $typeResult = ($condition) ? 'first' : 'all';

         $payoutsTable = Database::get_main_table(self::TABLE_PAYPAL_PAYOUTS);

         $saleTable = Database::get_main_table(self::TABLE_SALE);

@@ -1571,14 +1571,14 @@ 
             INNER JOIN $saleTable s ON s.id = p.sale_id

             INNER JOIN $currencyTable c ON s.currency_id = c.id

             LEFT JOIN  $extraFieldValues efv ON p.user_id = efv.item_id 

-            AND field_id = " . intval($paypalExtraField['id']) . "

+            AND field_id = ".intval($paypalExtraField['id'])."

         ";

 

         $payouts = Database::select(

             "p.* , u.firstname, u.lastname, efv.value as paypal_account, s.reference as sale_reference, s.price as item_price, c.iso_code",

             "$payoutsTable p $innerJoins",

             [

-                'where' => ['p.status = ? '.$condition . ' ' .$condition2 => $status]

+                'where' => ['p.status = ? '.$condition.' '.$condition2 => $status]

             ],

             $typeResult

         );

@@ -1641,7 +1641,7 @@ 
         $platformCommission = $this->getPlatformCommission();

 

         $sale = $this->getSale($saleId);

-        $teachersCommission = number_format((floatval($sale['price']) * intval($platformCommission['commission']))/100, 2);

+        $teachersCommission = number_format((floatval($sale['price']) * intval($platformCommission['commission'])) / 100, 2);

 

 

         $beneficiaries = $this->getBeneficiariesBySale($saleId);

@@ -1653,7 +1653,7 @@ 
                     'payout_date' => getdate(),

                     'sale_id' => intval($saleId),

                     'user_id' => $beneficiary['user_id'],

-                    'commission' => number_format((floatval($teachersCommission) * intval($beneficiary['commissions']))/100, 2),

+                    'commission' => number_format((floatval($teachersCommission) * intval($beneficiary['commissions'])) / 100, 2),

                     'status' => self::PAYOUT_STATUS_PENDING

                 ]

             );

@@ -2251,17 +2251,17 @@ 
      */

     public function getPath($var)

     {

-        $pluginPath = api_get_path(WEB_PLUGIN_PATH) . 'buycourses/';

+        $pluginPath = api_get_path(WEB_PLUGIN_PATH).'buycourses/';

         $paths = [

-            'SERVICE_IMAGES' => $pluginPath . 'uploads/services/images/',

-            'SRC' => $pluginPath . 'src/',

-            'VIEW' => $pluginPath . 'view/',

-            'UPLOADS' => $pluginPath . 'uploads/',

-            'LANGUAGES' => $pluginPath . 'lang/',

-            'RESOURCES' => $pluginPath . 'resources/',

-            'RESOURCES_IMG' => $pluginPath . 'resources/img/',

-            'RESOURCES_CSS' => $pluginPath . 'resources/css/',

-            'RESOURCES_JS' => $pluginPath . 'resources/js/',

+            'SERVICE_IMAGES' => $pluginPath.'uploads/services/images/',

+            'SRC' => $pluginPath.'src/',

+            'VIEW' => $pluginPath.'view/',

+            'UPLOADS' => $pluginPath.'uploads/',

+            'LANGUAGES' => $pluginPath.'lang/',

+            'RESOURCES' => $pluginPath.'resources/',

+            'RESOURCES_IMG' => $pluginPath.'resources/img/',

+            'RESOURCES_CSS' => $pluginPath.'resources/css/',

+            'RESOURCES_JS' => $pluginPath.'resources/js/',

         ];

 

         return $paths[$var];

src/Chamilo/CoreBundle/Component/Utils/ChamiloApi.php

Spacing +11 added lines, -11 removed lines

@@ -74,14 +74,14 @@ 
         $themeDir = \Template::getThemeDir($theme);
         $customLogoPath = "$themeDir/images/header-logo-custom$accessUrlId.png";
 
-        if (file_exists(api_get_path(SYS_PUBLIC_PATH) . "css/$customLogoPath")) {
-            return api_get_path(WEB_CSS_PATH) . $customLogoPath;
+        if (file_exists(api_get_path(SYS_PUBLIC_PATH)."css/$customLogoPath")) {
+            return api_get_path(WEB_CSS_PATH).$customLogoPath;
         }
 
         $originalLogoPath = "$themeDir/images/header-logo.png";
 
-        if (file_exists(api_get_path(SYS_CSS_PATH) . $originalLogoPath)) {
-            return api_get_path(WEB_CSS_PATH) . $originalLogoPath;
+        if (file_exists(api_get_path(SYS_CSS_PATH).$originalLogoPath)) {
+            return api_get_path(WEB_CSS_PATH).$originalLogoPath;
         }
 
         return null;
@@ -101,10 +101,10 @@ 
         $siteName = api_get_setting('siteName');
 
         if ($logoPath === null) {
-            $headerLogo = \Display::url($siteName, api_get_path(WEB_PATH) . 'index.php');
+            $headerLogo = \Display::url($siteName, api_get_path(WEB_PATH).'index.php');
 
             if (!empty($institutionUrl) && !empty($institution)) {
-                $headerLogo .= ' - ' . \Display::url($institution, $institutionUrl);
+                $headerLogo .= ' - '.\Display::url($institution, $institutionUrl);
             }
 
             $courseInfo = api_get_course_info();
@@ -127,7 +127,7 @@ 
 
         $image = \Display::img($logoPath, $institution, $imageAttributes);
 
-        return \Display::url($image, api_get_path(WEB_PATH) . 'index.php');
+        return \Display::url($image, api_get_path(WEB_PATH).'index.php');
     }
 
     /**
@@ -139,9 +139,9 @@ 
     public static function stripGivenTags($string, $tags)
     {
         foreach ($tags as $tag) {
-            $string2 = preg_replace('#</' . $tag . '[^>]*>#i', ' ', $string);
+            $string2 = preg_replace('#</'.$tag.'[^>]*>#i', ' ', $string);
             if ($string2 != $string) {
-                $string = preg_replace('/<' . $tag . '[^>]*>/i', ' ', $string2);
+                $string = preg_replace('/<'.$tag.'[^>]*>/i', ' ', $string2);
             }
         }
 
@@ -162,9 +162,9 @@ 
         sscanf($time, "%d:%d:%d", $hours, $minutes, $seconds);
         $timeSeconds = isset($seconds) ? $hours * 3600 + $minutes * 60 + $seconds : $hours * 60 + $minutes;
         if ($operation) {
-            $date->add(new \DateInterval('PT' . $timeSeconds . 'S'));
+            $date->add(new \DateInterval('PT'.$timeSeconds.'S'));
         } else {
-            $date->sub(new \DateInterval('PT' . $timeSeconds . 'S'));
+            $date->sub(new \DateInterval('PT'.$timeSeconds.'S'));
         }
 
         return $date->format('Y-m-d H:i:s');

src/Chamilo/CoreBundle/Component/Editor/Editor.php

Spacing +1 added lines, -1 removed lines

@@ -131,7 +131,7 @@
             case 'integer':
                 //no break
             case 'double':
-                return (string)$var;
+                return (string) $var;
                 //no break
             case 'resource':
                 //no break

main/inc/ajax/course_home.ajax.php

Spacing +43 added lines, -43 removed lines

@@ -48,19 +48,19 @@ 
                 $na_image = CourseHome::getCustomWebIconPath().CourseHome::getDisableIcon($tool_info['custom_icon']);
             }
 
-            $requested_image = ($tool_visibility == 0 ) ? $tool_image : $na_image;
-            $requested_class = ($tool_visibility == 0 ) ? '' : 'text-muted';
-            $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_image = ($tool_visibility == 0) ? $tool_image : $na_image;
+            $requested_class = ($tool_visibility == 0) ? '' : 'text-muted';
+            $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;
 
             // 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(
@@ -112,12 +112,12 @@ 
          */
         require_once __DIR__.'/../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']);
@@ -137,7 +137,7 @@ 
             }
         }
 
-        $start = $limit*$page - $limit;
+        $start = $limit * $page - $limit;
         $course_list = SessionManager::get_course_list_by_session_id($session_id);
         $count = 0;
         $temp = array();
@@ -149,14 +149,14 @@ 
             $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) {
                     continue;
                 }
 
-                $lp_url = api_get_path(WEB_CODE_PATH) . 'lp/lp_controller.php?cidReq='.$item['code'].'&id_session='.$session_id.'&lp_id='.$lp_id.'&action=view';
+                $lp_url = api_get_path(WEB_CODE_PATH).'lp/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(),
@@ -212,25 +212,25 @@ 
         }
         $temp = msort($temp, $sidx, $sord);
 
-        $i =0;
+        $i = 0;
         $response = new stdClass();
         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++;
                 }
             }
         }
         $total_pages = 0;
         if ($count > 0 && $limit > 0) {
-            $total_pages = ceil($count/$limit);
+            $total_pages = ceil($count / $limit);
         }
         $response->total = $total_pages;
         if ($page > $total_pages) {
-            $response->page= $total_pages;
+            $response->page = $total_pages;
         } else {
             $response->page = $page;
         }
@@ -240,12 +240,12 @@ 
     case 'session_courses_lp_by_week':
         require_once __DIR__.'/../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';
         }
 
@@ -293,8 +293,8 @@ 
             );
 
             foreach ($flat_list as $lp_id => $lp_item) {
-                $temp[$count]['id']= $lp_id;
-                $lp_url = api_get_path(WEB_CODE_PATH) . 'lp/lp_controller.php?cidReq='.$item['code'].'&id_session='.$session_id.'&lp_id='.$lp_id.'&action=view';
+                $temp[$count]['id'] = $lp_id;
+                $lp_url = api_get_path(WEB_CODE_PATH).'lp/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(),
@@ -356,20 +356,20 @@ 
         }
 
         $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],$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++;
                 }
             }
         }
         $total_pages = 0;
         if ($count > 0 && $limit > 0) {
-            $total_pages = ceil($count/$limit);
+            $total_pages = ceil($count / $limit);
         }
         $response->total = $total_pages;
         if ($page > $total_pages) {
@@ -383,12 +383,12 @@ 
     case 'session_courses_lp_by_course':
         require_once __DIR__.'/../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';
         }
@@ -409,7 +409,7 @@ 
             }
         }
 
-        $start = $limit*$page - $limit;
+        $start = $limit * $page - $limit;
         $course_list = SessionManager::get_course_list_by_session_id($session_id);
         $count = 0;
 
@@ -431,9 +431,9 @@ 
                 $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;
-                $lp_url = api_get_path(WEB_CODE_PATH) . 'lp/lp_controller.php?cidReq='.$item['code'].'&id_session='.$session_id.'&lp_id='.$lp_id.'&action=view';
+            foreach ($flat_list as $lp_id => $lp_item) {
+                $temp[$count]['id'] = $lp_id;
+                $lp_url = api_get_path(WEB_CODE_PATH).'lp/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(),
                     $item,
@@ -485,13 +485,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++;
                 }
             }

main/inc/lib/events.lib.php

Spacing +9 added lines, -11 removed lines

@@ -78,9 +78,7 @@ 
         Database::query($sql);
 
         // Auto subscribe
-        $user_status = $userInfo['status']  == SESSIONADMIN ? 'sessionadmin' :
-            $userInfo['status'] == COURSEMANAGER ? 'teacher' :
-                $userInfo['status'] == DRH ? 'DRH' : 'student';
+        $user_status = $userInfo['status'] == SESSIONADMIN ? 'sessionadmin' : $userInfo['status'] == COURSEMANAGER ? 'teacher' : $userInfo['status'] == DRH ? 'DRH' : 'student';
         $autoSubscribe = api_get_setting($user_status.'_autosubscribe');
         if ($autoSubscribe) {
             $autoSubscribe = explode('|', $autoSubscribe);
@@ -337,7 +335,7 @@ 
         global $debug;
 
         if ($debug) error_log('Called to update_event_exercice');
-        if ($debug) error_log('duration:' . $duration);
+        if ($debug) error_log('duration:'.$duration);
 
         if ($exeid != '') {
             /*
@@ -385,7 +383,7 @@ 
         		   status = '".$status."',
         		   questions_to_check = '".$remind_list."',
         		   data_tracking = '".implode(',', $question_list)."',
-                   user_ip = '" . Database::escape_string(api_get_real_ip()) . "'
+                   user_ip = '" . Database::escape_string(api_get_real_ip())."'
         		 WHERE exe_id = '".Database::escape_string($exeid)."'";
             $res = Database::query($sql);
 
@@ -1069,15 +1067,15 @@ 
 
         if (!empty($exe_list) && is_array($exe_list) && count($exe_list) > 0) {
             $sql = "DELETE FROM $track_e_exercises
-                WHERE exe_id IN (" . implode(',', $exe_list) . ")";
+                WHERE exe_id IN (".implode(',', $exe_list).")";
             Database::query($sql);
 
             $sql = "DELETE FROM $track_attempts
-                WHERE exe_id IN (" . implode(',', $exe_list) . ")";
+                WHERE exe_id IN (".implode(',', $exe_list).")";
             Database::query($sql);
 
             $sql = "DELETE FROM $recording_table
-                WHERE exe_id IN (" . implode(',', $exe_list) . ")";
+                WHERE exe_id IN (".implode(',', $exe_list).")";
             Database::query($sql);
         }
 
@@ -1119,7 +1117,7 @@ 
             self::addEvent(
                 LOG_EXERCISE_RESULT_DELETE,
                 LOG_EXERCISE_AND_USER_ID,
-                $exercise_id . '-' . $user_id,
+                $exercise_id.'-'.$user_id,
                 null,
                 null,
                 $course_id,
@@ -1725,7 +1723,7 @@ 
         self::addEvent(
             LOG_QUESTION_RESULT_DELETE,
             LOG_EXERCISE_ATTEMPT_QUESTION_ID,
-            $exe_id . '-' . $question_id,
+            $exe_id.'-'.$question_id,
             null,
             null,
             $courseId,
@@ -1762,7 +1760,7 @@ 
         self::addEvent(
             LOG_QUESTION_RESULT_DELETE,
             LOG_EXERCISE_ATTEMPT_QUESTION_ID,
-            $exe_id . '-' . $question_id,
+            $exe_id.'-'.$question_id,
             null,
             null,
             $courseId,