Merge branch '3.4'

* 3.4:
  [3.4] Remove useless docblocks
  [3.3] More docblock fixes
  [2.7] More docblock fixes
  [TwigBridge] Fix BC break due required twig environment
  Random fixes
  Docblock fixes
  [DI] Fix cannot bind env var
  Fix some signatures in PHP-DSLs
  [HttpKernel] Enhance deprecation message
  bumped Symfony version to 3.4.0
  updated VERSION for 3.4.0-BETA3
  updated CHANGELOG for 3.4.0-BETA3
  [SecurityBundle] Fix the datacollector to properly support decision.object being null

Author: nicolas-grekas

CHANGELOG-3.4.md

@@ -7,6 +7,27 @@ in 3.4 minor versions.
 To get the diff for a specific change, go to https://github.com/symfony/symfony/commit/XXX where XXX is the change hash
 To get the diff between two versions, go to https://github.com/symfony/symfony/compare/v3.4.0...v3.4.1
 
+* 3.4.0-BETA3 (2017-11-05)
+
+ * bug #24531 [HttpFoundation] Fix forward-compat of NativeSessionStorage with PHP 7.2 (sroze)
+ * bug #24828 [DI] Fix the "almost-circular refs" fix (nicolas-grekas)
+ * bug #24665 Fix dump panel hidden when closing a dump (julienfalque)
+ * bug #24802 [TwigBridge] [Bootstrap 4] Fix hidden errors (ostrolucky)
+ * bug #24816 [Serializer] Fix extra attributes when no group specified (ogizanagi)
+ * bug #24822 [DI] Fix "almost-circular" dependencies handling (nicolas-grekas)
+ * bug #24821 symfony/form auto-enables symfony/validator, even when not present (weaverryan)
+ * bug #24824 [FrameworkBundle][Config] fix: do not add resource checkers for no-debug (dmaicher)
+ * bug #24814 [Intl] Make intl-data tests pass and save language aliases again (jakzal)
+ * bug #24810 [Serializer] readd default argument value (xabbuh)
+ * bug #24809 [Config] Fix dump of config references for deprecated nodes (chalasr)
+ * bug #24796 [PhpUnitBridge] Fixed fatal error in CoverageListener when something goes wrong in Test::setUpBeforeClass (lyrixx)
+ * bug #24774 [HttpKernel] Let the storage manage the session starts (sroze)
+ * bug #24735 [VarDumper] fix trailling comma when dumping an exception (Simperfit)
+ * bug #24770 [Validator] Fix TraceableValidator is reset on data collector instantiation (ogizanagi)
+ * bug #24764 [HttpFoundation] add Early Hints to Reponse to fix test (Simperfit)
+ * bug #24759 Removes \n or space when $context/$extra are empty (kirkmadera)
+ * bug #24758 Throwing exception if redis and predis unavailable (aequasi)
+
 * 3.4.0-BETA2 (2017-10-30)
 
  * bug #24728 [Bridge\Twig] fix bootstrap checkbox_row to render properly & remove spaceless (arkste)

src/Symfony/Bridge/Doctrine/DependencyInjection/CompilerPass/RegisterMappingsPass.php

@@ -69,7 +69,7 @@ abstract class RegisterMappingsPass implements CompilerPassInterface
      * only do anything if the parameter is present. (But regardless of the
      * value of that parameter.
      *
-     * @var string
+     * @var string|false
      */
     protected $enabledParameter;
 

src/Symfony/Bridge/Doctrine/Tests/Fixtures/Type/StringWrapper.php

@@ -13,9 +13,6 @@
 
 class StringWrapper
 {
-    /**
-     * @var string
-     */
     private $string;
 
     /**

src/Symfony/Bridge/PhpUnit/Legacy/SymfonyTestsListenerTrait.php

@@ -315,7 +315,7 @@ public function handleError($type, $msg, $file, $line, $context = array())
     }
 
     /**
-     * @param Test $test
+     * @param TestCase $test
      *
      * @return bool
      */

src/Symfony/Bridge/Twig/Command/DebugCommand.php

@@ -135,7 +135,7 @@ private function getLoaderPaths()
 
         $loaderPaths = array();
         foreach ($loader->getNamespaces() as $namespace) {
-            $paths = array_map(function ($path) use ($namespace) {
+            $paths = array_map(function ($path) {
                 if (null !== $this->projectDir && 0 === strpos($path, $this->projectDir)) {
                     $path = ltrim(substr($path, strlen($this->projectDir)), DIRECTORY_SEPARATOR);
                 }

src/Symfony/Bridge/Twig/DataCollector/TwigDataCollector.php

@@ -31,7 +31,7 @@ class TwigDataCollector extends DataCollector implements LateDataCollectorInterf
     private $twig;
     private $computed;
 
-    public function __construct(Profile $profile, Environment $twig)
+    public function __construct(Profile $profile, Environment $twig = null)
     {
         $this->profile = $profile;
         $this->twig = $twig;
@@ -62,6 +62,10 @@ public function lateCollect()
         $this->data['profile'] = serialize($this->profile);
         $this->data['template_paths'] = array();
 
+        if (null === $this->twig) {
+            return;
+        }
+
         $templateFinder = function (Profile $profile) use (&$templateFinder) {
             if ($profile->isTemplate()) {
                 try {

src/Symfony/Bridge/Twig/Extension/CodeExtension.php

@@ -196,7 +196,7 @@ public function formatFile($file, $line, $text = null)
      * @param string $file An absolute file path
      * @param int    $line The line number
      *
-     * @return string A link of false
+     * @return string|false A link or false
      */
     public function getFileLink($file, $line)
     {

src/Symfony/Bundle/FrameworkBundle/CacheWarmer/SerializerCacheWarmer.php

@@ -80,7 +80,7 @@ private function extractSupportedLoaders(array $loaders)
             if ($loader instanceof XmlFileLoader || $loader instanceof YamlFileLoader) {
                 $supportedLoaders[] = $loader;
             } elseif ($loader instanceof LoaderChain) {
-                $supportedLoaders = array_merge($supportedLoaders, $this->extractSupportedLoaders($loader->getDelegatedLoaders()));
+                $supportedLoaders = array_merge($supportedLoaders, $this->extractSupportedLoaders($loader->getLoaders()));
             }
         }
 

src/Symfony/Bundle/FrameworkBundle/CacheWarmer/ValidatorCacheWarmer.php

@@ -91,7 +91,7 @@ private function extractSupportedLoaders(array $loaders)
             if ($loader instanceof XmlFileLoader || $loader instanceof YamlFileLoader) {
                 $supportedLoaders[] = $loader;
             } elseif ($loader instanceof LoaderChain) {
-                $supportedLoaders = array_merge($supportedLoaders, $this->extractSupportedLoaders($loader->getDelegatedLoaders()));
+                $supportedLoaders = array_merge($supportedLoaders, $this->extractSupportedLoaders($loader->getLoaders()));
             }
         }
 

src/Symfony/Bundle/FrameworkBundle/Client.php

@@ -62,7 +62,7 @@ public function getKernel()
     /**
      * Gets the profile associated with the current Response.
      *
-     * @return HttpProfile A Profile instance
+     * @return HttpProfile|false A Profile instance
      */
     public function getProfile()
     {