bug #61912 [FrameworkBundle] Fix support of dumping workflow when workflow is decorated by TraceableWorkflow (lyrixx)

nicolas-grekas · nicolas-grekas · commit 26f9bbb5f62d · 2025-10-01T16:18:58.000+02:00
This PR was merged into the 7.3 branch. Discussion ---------- [FrameworkBundle] Fix support of dumping workflow when workflow is decorated by TraceableWorkflow | Q | A | ------------- | --- | Branch? | 7.3 (the bug is not on 6.4, it has been introduced by 6baa3d2) | Bug fix? | yes | New feature? | no | Deprecations? | no | Issues | Fix #60215 | License | MIT Commits ------- c035ec4 [FrameworkBundle] Fix support of dumping workflow when workflow is decorated by TraceableWorkflow
diff --git a/src/Symfony/Bundle/FrameworkBundle/Command/WorkflowDumpCommand.php b/src/Symfony/Bundle/FrameworkBundle/Command/WorkflowDumpCommand.php
@@ -21,6 +21,7 @@
 use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\DependencyInjection\ServiceLocator;
+use Symfony\Component\Workflow\Debug\TraceableWorkflow;
 use Symfony\Component\Workflow\Dumper\GraphvizDumper;
 use Symfony\Component\Workflow\Dumper\MermaidDumper;
 use Symfony\Component\Workflow\Dumper\PlantUmlDumper;
@@ -78,6 +79,9 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             throw new InvalidArgumentException(\sprintf('The workflow named "%s" cannot be found.', $workflowName));
         }
         $workflow = $this->workflows->get($workflowName);
+        if ($workflow instanceof TraceableWorkflow) {
+            $workflow = $workflow->getInner();
+        }
         $type = $workflow instanceof StateMachine ? 'state_machine' : 'workflow';
         $definition = $workflow->getDefinition();
 
diff --git a/src/Symfony/Component/Workflow/Debug/TraceableWorkflow.php b/src/Symfony/Component/Workflow/Debug/TraceableWorkflow.php
@@ -89,6 +89,11 @@ public function getCalls(): array
         return $this->calls;
     }
 
+    public function getInner(): WorkflowInterface
+    {
+        return $this->workflow;
+    }
+
     private function callInner(string $method, array $args): mixed
     {
         if ($this->disabled?->__invoke()) {

Original file line number	Diff line number	Diff line change
`@@ -89,6 +89,11 @@ public function getCalls(): array`
`89`	`89`	`return $this->calls;`
`90`	`90`	`}`
`91`	`91`
	`92`	`+ public function getInner(): WorkflowInterface`
	`93`	`+ {`
	`94`	`+ return $this->workflow;`
	`95`	`+ }`
	`96`	`+`
`92`	`97`	`private function callInner(string $method, array $args): mixed`
`93`	`98`	`{`
`94`	`99`	`if ($this->disabled?->__invoke()) {`