[Form] apply automatically step=1 for datetime-local input

Author: ottaviano

Extension/Core/Type/DateTimeType.php

@@ -203,6 +203,14 @@ public function buildView(FormView $view, FormInterface $form, array $options)
         //  * the html5 is set to true
         if ($options['html5'] && 'single_text' === $options['widget'] && self::HTML5_FORMAT === $options['format']) {
             $view->vars['type'] = 'datetime-local';
+
+            // we need to force the browser to display the seconds by
+            // adding the HTML attribute step if not already defined.
+            // Otherwise the browser will not display and so not send the seconds
+            // therefore the value will always be considered as invalid.
+            if ($options['with_seconds'] && !isset($view->vars['attr']['step'])) {
+                $view->vars['attr']['step'] = 1;
+            }
         }
     }
 

Tests/Extension/Core/Type/DateTimeTypeTest.php

@@ -444,6 +444,37 @@ public function testDontPassHtml5TypeIfNotSingleText()
         $this->assertArrayNotHasKey('type', $view->vars);
     }
 
+    public function testSingleTextWidgetWithSecondsShouldHaveRightStepAttribute()
+    {
+        $view = $this->factory
+            ->create(static::TESTED_TYPE, null, [
+                'widget' => 'single_text',
+                'with_seconds' => true,
+            ])
+            ->createView()
+        ;
+
+        $this->assertArrayHasKey('step', $view->vars['attr']);
+        $this->assertEquals(1, $view->vars['attr']['step']);
+    }
+
+    public function testSingleTextWidgetWithSecondsShouldNotOverrideStepAttribute()
+    {
+        $view = $this->factory
+            ->create(static::TESTED_TYPE, null, [
+                'widget' => 'single_text',
+                'with_seconds' => true,
+                'attr' => [
+                    'step' => 30,
+                ],
+            ])
+            ->createView()
+        ;
+
+        $this->assertArrayHasKey('step', $view->vars['attr']);
+        $this->assertEquals(30, $view->vars['attr']['step']);
+    }
+
     public function testDateTypeChoiceErrorsBubbleUp()
     {
         $error = new FormError('Invalid!');