Merge branch '7.4' into 8.0

* 7.4:
  [TypeInfo] Simple array should be array type
  Handle signals on text input
  [TwigBridge] Fix form constraint
  [Runtime] Reuse the already created Request object when the app needs it as argument returns a kernel
  [Config] Fix array shape generation for backed enums
  [Config] Define `TreeBuilder` default generic type
  Update validators.el.xlf
  Fix MoneyType: add missing step attribute when html5=true
  [JsonStreamer] fix invalid json output for list of self
  [Console] Preserve `--help` option when a command is not found
  [FrameworkBundle] Fix using `FailedMessages*Command` with `SigningSerializer`
  [Lock] Fix unserializing already serialized Key payloads
  [HttpClient] CachingHttpClient must run after UriTemplate and Scoping
  Only register PhpConfigReferenceDumpPass in dev env with debug flag enabled
  [Messenger] Fix PHP 8.5 deprecation for pgsqlGetNotify() in PostgreSQL transport
  chore: PHP CS Fixer - do not use deprecated sets in config
  verify spanish translations with state needs-review-translation
  [Security] Fix OIDC discovery when using multiple HttpClient instances
  [DependencyInjection] Allow manual bindings on parameters with #[Target]

Author: nicolas-grekas

Extension/Core/Type/MoneyType.php

@@ -51,6 +51,12 @@ public function buildView(FormView $view, FormInterface $form, array $options):
 
         if ($options['html5']) {
             $view->vars['type'] = 'number';
+
+            if (!isset($view->vars['attr']['step'])) {
+                $view->vars['attr']['step'] = 'any';
+            }
+        } else {
+            $view->vars['attr']['inputmode'] = 0 === $options['scale'] ? 'numeric' : 'decimal';
         }
     }
 

Tests/Extension/Core/Type/MoneyTypeTest.php

@@ -126,6 +126,46 @@ public function testHtml5EnablesSpecificFormatting()
         $this->assertSame('number', $form->createView()->vars['type']);
     }
 
+    public function testHtml5AddsStepAttributeIfNotSet()
+    {
+        $form = $this->factory->create(static::TESTED_TYPE, null, [
+            'html5' => true,
+        ]);
+        $view = $form->createView();
+        $this->assertSame('any', $view->vars['attr']['step']);
+
+        $form = $this->factory->create(static::TESTED_TYPE, null, [
+            'html5' => false,
+            'scale' => 2,
+        ]);
+        $view = $form->createView();
+        $this->assertSame('decimal', $view->vars['attr']['inputmode']);
+
+        $form = $this->factory->create(static::TESTED_TYPE, null, [
+            'html5' => false,
+            'scale' => 0,
+        ]);
+        $view = $form->createView();
+        $this->assertSame('numeric', $view->vars['attr']['inputmode']);
+    }
+
+    public function testHtml5DoesNotOverrideUserProvidedStep()
+    {
+        $form = $this->factory->create(static::TESTED_TYPE, null, [
+            'html5' => true,
+            'attr' => ['step' => '0.01'],
+        ]);
+        $view = $form->createView();
+        $this->assertSame('0.01', $view->vars['attr']['step']);
+
+        $form = $this->factory->create(static::TESTED_TYPE, null, [
+            'html5' => false,
+            'scale' => 2,
+        ]);
+        $view = $form->createView();
+        $this->assertSame('decimal', $view->vars['attr']['inputmode']);
+    }
+
     public function testDefaultInput()
     {
         $form = $this->factory->create(static::TESTED_TYPE, null, ['divisor' => 100]);