Merge branch '4.4' into 5.2

* 4.4:
  [Contracts] Add missing `@return` annotations
  [FrameworkBundle] Fixed file operations in Sodium vault seal

Author: nicolas-grekas

src/Symfony/Bundle/FrameworkBundle/Secrets/SodiumVault.php

@@ -90,7 +90,7 @@ public function seal(string $name, string $value): void
         $list = $this->list();
         $list[$name] = null;
         uksort($list, 'strnatcmp');
-        file_put_contents($this->pathPrefix.'list.php', sprintf("<?php\n\nreturn %s;\n", VarExporter::export($list), \LOCK_EX));
+        file_put_contents($this->pathPrefix.'list.php', sprintf("<?php\n\nreturn %s;\n", VarExporter::export($list)), \LOCK_EX);
 
         $this->lastMessage = sprintf('Secret "%s" encrypted in "%s"; you can commit it.', $name, $this->getPrettyPath(\dirname($this->pathPrefix).\DIRECTORY_SEPARATOR));
     }
@@ -142,7 +142,7 @@ public function remove(string $name): bool
 
         $list = $this->list();
         unset($list[$name]);
-        file_put_contents($this->pathPrefix.'list.php', sprintf("<?php\n\nreturn %s;\n", VarExporter::export($list), \LOCK_EX));
+        file_put_contents($this->pathPrefix.'list.php', sprintf("<?php\n\nreturn %s;\n", VarExporter::export($list)), \LOCK_EX);
 
         $this->lastMessage = sprintf('Secret "%s" removed from "%s".', $name, $this->getPrettyPath(\dirname($this->pathPrefix).\DIRECTORY_SEPARATOR));
 

src/Symfony/Contracts/Cache/CacheTrait.php

@@ -27,6 +27,8 @@ trait CacheTrait
 {
     /**
      * {@inheritdoc}
+     *
+     * @return mixed
      */
     public function get(string $key, callable $callback, float $beta = null, array &$metadata = null)
     {

src/Symfony/Contracts/HttpClient/Test/TestHttpServer.php

@@ -18,6 +18,9 @@ class TestHttpServer
 {
     private static $process = [];
 
+    /**
+     * @return Process
+     */
     public static function start(int $port = 8057)
     {
         if (isset(self::$process[$port])) {

src/Symfony/Contracts/Service/ServiceSubscriberTrait.php

@@ -52,6 +52,8 @@ public static function getSubscribedServices(): array
 
     /**
      * @required
+     *
+     * @return ContainerInterface|null
      */
     public function setContainer(ContainerInterface $container)
     {

src/Symfony/Contracts/Service/Test/ServiceLocatorTest.php

@@ -17,6 +17,9 @@
 
 abstract class ServiceLocatorTest extends TestCase
 {
+    /**
+     * @return ContainerInterface
+     */
     protected function getServiceLocator(array $factories)
     {
         return new class($factories) implements ContainerInterface {

src/Symfony/Contracts/Tests/Service/ServiceSubscriberTraitTest.php

@@ -42,7 +42,7 @@ public function aParentService(): Service1
     {
     }
 
-    public function setContainer(ContainerInterface $container)
+    public function setContainer(ContainerInterface $container): ContainerInterface
     {
         return $container;
     }

src/Symfony/Contracts/Translation/Test/TranslatorTest.php

@@ -43,6 +43,9 @@ protected function tearDown(): void
         \Locale::setDefault($this->defaultLocale);
     }
 
+    /**
+     * @return TranslatorInterface
+     */
     public function getTranslator()
     {
         return new class() implements TranslatorInterface {