Merge branch '4.4' into 5.0

* 4.4:
  [appveyor] bump cache
  [Twig][Mime] Removed extra quotes in missing package exception message
  [DI] µfix
  Allowing empty secrets to be set
  [DI] add missing property declarations in InlineServiceConfigurator
  [DI] fix detecting short service syntax in yaml
  Supress error from fread when reading a unix pipe
  [HttpClient] Fix scoped client without query option configuration
  [Workflow] Use a strict comparison when retrieving raw marking in MarkingStore
  [Workflow] Use a strict comparison when retrieving raw markin in MarkingStore

Author: nicolas-grekas

Loader/Configurator/AliasConfigurator.php

@@ -20,6 +20,7 @@ class AliasConfigurator extends AbstractServiceConfigurator
 {
     const FACTORY = 'alias';
 
+    use Traits\DeprecateTrait;
     use Traits\PublicTrait;
 
     public function __construct(ServicesConfigurator $parent, Alias $alias)

Loader/Configurator/InlineServiceConfigurator.php

@@ -29,6 +29,10 @@ class InlineServiceConfigurator extends AbstractConfigurator
     use Traits\ParentTrait;
     use Traits\TagTrait;
 
+    private $id = '[inline]';
+    private $allowParent = true;
+    private $path = null;
+
     public function __construct(Definition $definition)
     {
         $this->definition = $definition;

Loader/YamlFileLoader.php

@@ -299,7 +299,7 @@ private function parseDefaults(array &$content, string $file): array
     private function isUsingShortSyntax(array $service): bool
     {
         foreach ($service as $key => $value) {
-            if (\is_string($key) && ('' === $key || '$' !== $key[0])) {
+            if (\is_string($key) && ('' === $key || ('$' !== $key[0] && false === strpos($key, '\\')))) {
                 return false;
             }
         }
@@ -345,7 +345,7 @@ private function parseDefinition(string $id, $service, string $file, array $defa
 
         if (isset($service['alias'])) {
             $this->container->setAlias($id, $alias = new Alias($service['alias']));
-            if (\array_key_exists('public', $service)) {
+            if (isset($service['public'])) {
                 $alias->setPublic($service['public']);
             } elseif (isset($defaults['public'])) {
                 $alias->setPublic($defaults['public']);

Tests/Fixtures/yaml/services_short_syntax.yml

@@ -0,0 +1,6 @@
+services:
+    foo_bar: [1, 2]
+
+    bar_foo:
+        $a: 'a'
+        App\Foo: 'foo'

Tests/Loader/YamlFileLoaderTest.php

@@ -205,6 +205,17 @@ public function testLoadServices()
         $this->assertEquals(['decorated', 'decorated.pif-pouf', 5, ContainerInterface::IGNORE_ON_INVALID_REFERENCE], $services['decorator_service_with_name_and_priority_and_on_invalid']->getDecoratedService());
     }
 
+    public function testLoadShortSyntax()
+    {
+        $container = new ContainerBuilder();
+        $loader = new YamlFileLoader($container, new FileLocator(self::$fixturesPath.'/yaml'));
+        $loader->load('services_short_syntax.yml');
+        $services = $container->getDefinitions();
+
+        $this->assertSame([1, 2], $services['foo_bar']->getArguments());
+        $this->assertSame(['$a' => 'a', 'App\Foo' => 'foo'], $services['bar_foo']->getArguments());
+    }
+
     public function testDeprecatedAliases()
     {
         $container = new ContainerBuilder();