Merge branch '5.4' into 6.0

* 5.4:
  [Notifier][SMSBiuras] Fix CS
  Fix SmsBiurasTransportTest::testTestMode
  [Notifier][SmsBiuras] Simplify test and data provider
  [Notifier] [SMSBiuras] `true`/`false` mismatch for `test_mode` option
  [HttpKernel] Fix message for unresovable arguments of invokable controllers
  ignore const expressions read by phpdocumentor
  [DependencyInjection] Process bindings in ServiceLocatorTagPass

Author: fabpot

Compiler/ServiceLocatorTagPass.php

@@ -40,6 +40,10 @@ protected function processValue(mixed $value, bool $isRoot = false): mixed
             return self::register($this->container, $value->getValues());
         }
 
+        if ($value instanceof Definition) {
+            $value->setBindings(parent::processValue($value->getBindings()));
+        }
+
         if (!$value instanceof Definition || !$value->hasTag('container.service_locator')) {
             return parent::processValue($value, $isRoot);
         }

Tests/Compiler/ServiceLocatorTagPassTest.php

@@ -214,6 +214,18 @@ public function testDefinitionOrderIsTheSame()
 
         static::assertSame(['service-2', 'service-1'], array_keys($factories));
     }
+
+    public function testBindingsAreProcessed()
+    {
+        $container = new ContainerBuilder();
+
+        $definition = $container->register('foo')
+            ->setBindings(['foo' => new ServiceLocatorArgument()]);
+
+        (new ServiceLocatorTagPass())->process($container);
+
+        $this->assertInstanceOf(Reference::class, $definition->getBindings()['foo']->getValues()[0]);
+    }
 }
 
 class Locator