Merge branch '4.3' into 4.4

* 4.3:
  cs fix
  Fix return statements
  [TwigBridge] add missing dep
  Add false type to ChoiceListFactoryInterface::createView $label argument
  Update UPGRADE guide of 4.3 for EventDispatcher
  [SecurityBundle] display the correct class name on the deprecated notice

Author: nicolas-grekas

Controller/ControllerTrait.php

@@ -352,7 +352,7 @@ protected function getDoctrine(): ManagerRegistry
     /**
      * Get a user from the Security Token Storage.
      *
-     * @return mixed
+     * @return object|null
      *
      * @throws \LogicException If SecurityBundle is not available
      *
@@ -367,12 +367,12 @@ protected function getUser()
         }
 
         if (null === $token = $this->container->get('security.token_storage')->getToken()) {
-            return;
+            return null;
         }
 
         if (!\is_object($user = $token->getUser())) {
             // e.g. anonymous authentication
-            return;
+            return null;
         }
 
         return $user;

Templating/GlobalVariables.php

@@ -40,7 +40,7 @@ public function __construct(ContainerInterface $container)
     public function getToken()
     {
         if (!$this->container->has('security.token_storage')) {
-            return;
+            return null;
         }
 
         return $this->container->get('security.token_storage')->getToken();

Templating/Helper/CodeHelper.php

@@ -126,7 +126,7 @@ public function fileExcerpt($file, $line)
 
                 // Check if the file is an application/octet-stream (eg. Phar file) because highlight_file cannot parse these files
                 if ('application/octet-stream' === $finfo->file($file, FILEINFO_MIME_TYPE)) {
-                    return;
+                    return '';
                 }
             }