From 910353b77cabadd7e396d9a7af9b42eb23a03e29 Mon Sep 17 00:00:00 2001
From: Julio Montoya <gugli100@gmail.com>
Date: Tue, 14 Aug 2018 09:33:08 +0200
Subject: [PATCH] Minor - format code

---
 config/packages/dev/web_profiler.yaml             | 2 +-
 main/admin/user_edit.php                          | 1 -
 src/CoreBundle/EventListener/LegacyListener.php   | 4 ++--
 src/CoreBundle/Framework/PageController.php       | 8 +++-----
 src/CourseBundle/EventListener/CourseListener.php | 2 --
 src/UserBundle/Entity/User.php                    | 3 +--
 6 files changed, 7 insertions(+), 13 deletions(-)

diff --git a/config/packages/dev/web_profiler.yaml b/config/packages/dev/web_profiler.yaml
index 1e44b3592c..805f7b11f5 100644
--- a/config/packages/dev/web_profiler.yaml
+++ b/config/packages/dev/web_profiler.yaml
@@ -1,6 +1,6 @@
 web_profiler:
   toolbar: true
-  intercept_redirects: true
+  intercept_redirects: false
 framework:
   profiler:
     only_exceptions: false
diff --git a/main/admin/user_edit.php b/main/admin/user_edit.php
index d5f9bcdaa2..9bd867145e 100755
--- a/main/admin/user_edit.php
+++ b/main/admin/user_edit.php
@@ -329,7 +329,6 @@ $returnParams = $extraField->addElements(
 );
 $jqueryReadyContent = $returnParams['jquery_ready_content'];
 
-
 // the $jquery_ready_content variable collects all functions that will be load in the $(document).ready javascript function
 $htmlHeadXtra[] = '<script>
 $(document).ready(function(){
diff --git a/src/CoreBundle/EventListener/LegacyListener.php b/src/CoreBundle/EventListener/LegacyListener.php
index 8827e56d98..ecaf87f287 100644
--- a/src/CoreBundle/EventListener/LegacyListener.php
+++ b/src/CoreBundle/EventListener/LegacyListener.php
@@ -120,14 +120,14 @@ class LegacyListener
             $session->set('is_platformAdmin', $isAdmin);
             $session->set('is_allowedCreateCourse', $allowedCreateCourse);
 
-            $_admin = [
+            $adminInfo = [
                 'email' => api_get_setting('emailAdministrator'),
                 'surname' => api_get_setting('administratorSurname'),
                 'name' => api_get_setting('administratorName'),
                 'telephone' => api_get_setting('administratorTelephone'),
             ];
 
-            $twig->addGlobal('_admin', $_admin);
+            $twig->addGlobal('_admin', $adminInfo);
 
             $theme = api_get_visual_theme();
             $twig->addGlobal('favico', \Template::getPortalIcon($theme));
diff --git a/src/CoreBundle/Framework/PageController.php b/src/CoreBundle/Framework/PageController.php
index 701f7fd572..c73f03efcd 100644
--- a/src/CoreBundle/Framework/PageController.php
+++ b/src/CoreBundle/Framework/PageController.php
@@ -1042,7 +1042,7 @@ class PageController
             }
         }
 
-        $load_directories_preview = api_get_setting('document.show_documents_preview') == 'true' ? true : false;
+        $load_directories_preview = api_get_setting('document.show_documents_preview') === 'true' ? true : false;
         $sessions_with_no_category = $html;
 
         if (isset($session_categories) && !empty($session_categories)) {
@@ -1125,16 +1125,14 @@ class PageController
                                 $params['title'] = $session_link;
 
                                 $moved_status = \SessionManager::getSessionChangeUserReason(
-                                    isset($session['moved_status']) ? $session['moved_status'] : ''
+                                    $session['moved_status'] ?? ''
                                 );
                                 $moved_status = isset($moved_status) && !empty($moved_status) ? ' ('.$moved_status.')' : null;
 
                                 $params['subtitle'] = isset($session['coach_info']) ? $session['coach_info']['complete_name'] : null.$moved_status;
                                 //$params['dates'] = $session['date_message'];
 
-                                $params['dates'] = \SessionManager::parseSessionDates(
-                                    $session
-                                );
+                                $params['dates'] = \SessionManager::parseSessionDates($session);
                                 $params['right_actions'] = '';
                                 if (api_is_platform_admin()) {
                                     $params['right_actions'] .=
diff --git a/src/CourseBundle/EventListener/CourseListener.php b/src/CourseBundle/EventListener/CourseListener.php
index 7de55be9e9..7f6115ebfe 100644
--- a/src/CourseBundle/EventListener/CourseListener.php
+++ b/src/CourseBundle/EventListener/CourseListener.php
@@ -5,8 +5,6 @@ namespace Chamilo\CourseBundle\EventListener;
 
 use Chamilo\CoreBundle\Controller\LegacyController;
 use Chamilo\CoreBundle\Entity\Course;
-use Chamilo\CoreBundle\Entity\Session;
-
 use Chamilo\CoreBundle\Framework\Container;
 use Chamilo\CoreBundle\Security\Authorization\Voter\CourseVoter;
 use Chamilo\CoreBundle\Security\Authorization\Voter\GroupVoter;
diff --git a/src/UserBundle/Entity/User.php b/src/UserBundle/Entity/User.php
index 48494ee04c..61d5d9248b 100644
--- a/src/UserBundle/Entity/User.php
+++ b/src/UserBundle/Entity/User.php
@@ -1868,11 +1868,10 @@ class User extends BaseUser implements ThemeUser, EquatableInterface //implement
         return true;
     }
 
-
     /**
      * @return string
      */
-    public function getPictureLegacy()
+    public function getPictureLegacy() : string
     {
         $id = $this->id;