Minor - format code

main/admin/init_cas_user_extra_field_from_ldap.php

@@ -199,7 +199,7 @@ foreach ($databaseUsers as $user) {
             echo "\033[2K\r$ldapUsernameAttribute: $ldapUsername, $ldapCASUserAttribute: $ldapCASUser, ";
             $problems = [];
             if ($username === $ldapUsername) {
-                true; // fine
+                true;
             } elseif (in_array(
                 strtolower(trim($username)),
                 [strtolower(trim($ldapUsername)), strtolower(trim($ldapCASUser))]

main/inc/lib/extra_field_value.lib.php

@@ -771,7 +771,7 @@ class ExtraFieldValue extends Model
                 ON (s.field_id = sf.id)
                 WHERE
                     item_id = '$item_id'  AND
-                    variable = '".$field_variable."' AND
+                    variable = '$field_variable' AND
                     sf.extra_field_type = $extraFieldType
                 ";
         if ($filterByVisibility) {
@@ -804,10 +804,7 @@ class ExtraFieldValue extends Model
                         $result = $field_option->get($options[0]);
 
                         if (!empty($result)) {
-                            $result['value'] = $result['display_text']
-                                .'→'
-                                .$options[1];
-                        }
+                            $result['value'] = $result['display_text'].'→'.$options[1];                        }
                     }
                 }
                 if ($result['field_type'] == ExtraField::FIELD_TYPE_TRIPLE_SELECT) {
@@ -818,7 +815,6 @@ class ExtraFieldValue extends Model
                         foreach ($optionIds as $optionId) {
                             $objEfOption = new ExtraFieldOption('user');
                             $optionInfo = $objEfOption->get($optionId);
-
                             $optionValues[] = $optionInfo['display_text'];
                         }
 

main/inc/lib/userportal.lib.php

@@ -237,8 +237,6 @@ class IndexManager
 
             if (trim($home_top_temp) == '' && api_is_platform_admin()) {
                 $home_top_temp = get_lang('PortalHomepageDefaultIntroduction');
-            } else {
-                $home_top_temp;
             }
             $open = str_replace('{rel_path}', api_get_path(REL_PATH), $home_top_temp);
             $html = api_to_system_encoding($open, api_detect_encoding(strip_tags($open)));

plugin/jcapture/applet.php

@@ -17,7 +17,7 @@ session_write_close();
 header('Content-Type: text/html; charset=utf-8');
 $hostName = "http".($_SERVER['HTTPS'] ? 's' : null).'://'.$_SERVER['HTTP_HOST'];
 $imageFormat = "PNG";
-$cookies;
+$cookies = '';
 foreach (array_keys($_COOKIE) as $cookieName) {
     $cookies .= bin2hex($cookieName)."=".bin2hex($_COOKIE[$cookieName]).";";
 }
@@ -73,7 +73,7 @@ $edid = $_GET["edid"];
     <PARAM NAME = "host" VALUE="<?php echo $hostName; ?>">
     <PARAM NAME = "pageName" VALUE="<?php echo $pageName; ?>">
     <PARAM NAME = "edid" VALUE="<?php echo $edid; ?>">
-    <PARAM NAME = CODEBASE VALUE = "/somenonexistingcodebase" >    
+    <PARAM NAME = CODEBASE VALUE = "/somenonexistingcodebase" >
     <PARAM NAME = "authtok" VALUE="<?php echo auth_createToken(); ?>">
 
 Java 2 Standard Edition v 1.7 or above is required for this applet.<br/>

plugin/migrationmoodle/src/Transformer/Property/ScormScoTrackData.php

@@ -1,4 +1,5 @@
 <?php
+
 /* For licensing terms, see /license.txt */
 
 namespace Chamilo\PluginBundle\MigrationMoodle\Transformer\Property;
@@ -7,13 +8,11 @@ use Chamilo\PluginBundle\MigrationMoodle\Interfaces\TransformPropertyInterface;
 
 /**
  * Class ScormScoTrackData.
- *
- * @package Chamilo\PluginBundle\MigrationMoodle\Transformer\Property
  */
 class ScormScoTrackData implements TransformPropertyInterface
 {
-    const SEPARATOR_COMPONENTS = '|@|';
-    const SEPARATOR_VALUES = '==>>';
+    public const SEPARATOR_COMPONENTS = '|@|';
+    public const SEPARATOR_VALUES = '==>>';
 
     /**
      * {@inheritdoc}
@@ -97,8 +96,6 @@ class ScormScoTrackData implements TransformPropertyInterface
         if (empty($value)) {
             return 'none';
         }
-
-        $value;
     }
 
     /**