Merge branch 'main' into 199

Author: njoubert-cleverage

.github/ISSUE_TEMPLATE/bug.md

@@ -0,0 +1,58 @@
+---
+name: Bug report
+about: Report a task, transformer or engine behaving incorrectly
+title: ''
+labels: bug
+assignees: ''
+---
+
+## Description
+
+<!-- What is wrong, in one or two sentences. List each symptom when there are several (wrong output, exception, silent failure...) -->
+
+### Reproduction
+
+<!-- A minimal process configuration reproducing the bug, ideally a demo.* process from https://github.com/cleverage/process-bundle-demo -->
+
+```yaml
+clever_age_process:
+    configurations:
+        demo.<name>:
+            tasks:
+                <task>:
+                    service: '@CleverAge\ProcessBundle\Task\...'
+                    options: {}
+                    outputs: [debug]
+                debug:
+                    service: '@CleverAge\ProcessBundle\Task\Debug\DebugTask'
+```
+
+<!-- Input data, command and actual output -->
+
+```
+$ bin/console cleverage:process:execute demo.<name>
+```
+
+Expected: `<expected output>`
+
+Tested on `main` (`<commit>`), PHP x.y, Symfony x.y.
+
+### Cause
+
+<!-- Optional: where the bug comes from, with links to the relevant lines -->
+
+### Proposed fix
+
+<!-- Optional: how to fix it, and which documentation must be updated -->
+
+## Requirements
+
+* Documentation updates
+  - [ ] Reference
+  - [ ] Cookbooks
+  - [ ] Changelog
+* [ ] Unit tests
+
+## Breaking changes
+
+<!-- Behaviour changes introduced by the fix, or "None" with the reason (e.g. every configuration of this task currently fails) -->

.github/ISSUE_TEMPLATE/config.yml

@@ -0,0 +1 @@
+blank_issues_enabled: true

.github/ISSUE_TEMPLATE.md .github/ISSUE_TEMPLATE/issue.md.github/ISSUE_TEMPLATE.md renamed to .github/ISSUE_TEMPLATE/issue.md

@@ -1,3 +1,11 @@
+---
+name: Feature request / other
+about: Propose an enhancement, a refactoring or any other change
+title: ''
+labels: ''
+assignees: ''
+---
+
 ## Description
 
 <!-- Please describe the issue here -->

CHANGELOG.md

@@ -9,6 +9,7 @@ Latest
 * [#192](https://github.com/cleverage/process-bundle/issues/192) Fix CommandRunnerTask: only pass the `options` option to `Process::setOptions()`, support string `commandline` through `Process::fromShellCommandline()`, validate option types. Update documentation, add tests.
 * [#194](https://github.com/cleverage/process-bundle/issues/194) Fix ProcessLauncherTask: the `process_options` normalizer returned an array despite its scalar return type, so the task always failed with a `TypeError`. Update documentation, add tests.
 * [#189](https://github.com/cleverage/process-bundle/issues/189) Fix EventDispatcherTask: dispatch the event under the `event_name` option (regression since v4.0). `event_name` is now optional: when `null` (default), the event is dispatched under its class name. Update documentation, add tests.
+* [#198](https://github.com/cleverage/process-bundle/issues/198) Fix DateParserTransformer: a `\DateTimeImmutable` input is converted to a `\DateTime` instead of throwing a `TypeError`. Update documentation, add tests.
 * [#199](https://github.com/cleverage/process-bundle/issues/199) Fix PregFilterTransformer: an array `replacement` is no longer cast to the string `"Array"`, and requires an array `pattern`. Update documentation, add tests.
 
 ## Deprecated

docs/reference/transformers/date_parser_transformer.md

@@ -14,6 +14,7 @@ Accepted inputs
 
 * `string`: a date matching the configured `format`
 * `\DateTime`, returned unchanged
+* `\DateTimeImmutable`, converted to a `\DateTime` (same date, time and timezone)
 * Any falsy value (`null`, `''`, `false`…), returned unchanged
 
 A string that cannot be parsed with the given format throws an `\UnexpectedValueException`.
@@ -55,5 +56,3 @@ Notes
 
 Fields missing from `format` are taken from the current time (e.g. with `Y-m-d`, the time part is the current time). Use
 the `!` or `|` format characters to reset them, e.g. `'!Y-m-d'`.
-
-A `\DateTimeImmutable` input is not returned unchanged: it is passed to `createFromFormat()` and throws a `\TypeError`.

src/Transformer/Date/DateParserTransformer.php

@@ -17,7 +17,7 @@
 use Symfony\Component\OptionsResolver\OptionsResolver;
 
 /**
- * Transformer aiming to take a date as an input (object or a format defined string) to strictly output aa \DateTime.
+ * Transformer aiming to take a date as an input (object or a format defined string) to strictly output a \DateTime.
  *
  * @example in YML config
  * transformers:
@@ -32,6 +32,10 @@ public function transform(mixed $value, array $options = []): mixed
             return $value;
         }
 
+        if ($value instanceof \DateTimeImmutable) {
+            return \DateTime::createFromImmutable($value);
+        }
+
         $date = \DateTime::createFromFormat($options['format'], $value);
 
         if (!$date) {

tests/Transformer/Date/DateParserTransformerTest.php

@@ -69,6 +69,19 @@ public function testTransformDateTimeObject(): void
         $this->assertSame($value, $transformedValue);
     }
 
+    public function testTransformDateTimeImmutableObject(): void
+    {
+        $transformer = new DateParserTransformer();
+        $value = new \DateTimeImmutable('2023-09-28 12:34:56', new \DateTimeZone('Europe/Paris'));
+        $options = ['format' => 'Y-m-d'];
+
+        $transformedValue = $transformer->transform($value, $options);
+
+        $this->assertInstanceOf(\DateTime::class, $transformedValue);
+        $this->assertEquals($value, \DateTimeImmutable::createFromMutable($transformedValue));
+        $this->assertSame('Europe/Paris', $transformedValue->getTimezone()->getName());
+    }
+
     public function testGetCode(): void
     {
         $transformer = new DateParserTransformer();