bug #32331 [Workflow] only decorate when an event dispatcher was passed (xabbuh)

lyrixx · lyrixx · commit b7a75e459efb · 2019-07-03T10:37:46.000+02:00
This PR was merged into the 4.3 branch. Discussion ---------- [Workflow] only decorate when an event dispatcher was passed | Q | A | ------------- | --- | Branch? | 4.3 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #32330 | License | MIT | Doc PR | Commits ------- f19f28a only decorate when an event dispatcher was passed
diff --git a/src/Symfony/Component/Workflow/Workflow.php b/src/Symfony/Component/Workflow/Workflow.php
@@ -43,7 +43,7 @@ public function __construct(Definition $definition, MarkingStoreInterface $marki
     {
         $this->definition = $definition;
         $this->markingStore = $markingStore ?: new MultipleStateMarkingStore();
-        $this->dispatcher = LegacyEventDispatcherProxy::decorate($dispatcher);
+        $this->dispatcher = null !== $dispatcher ? LegacyEventDispatcherProxy::decorate($dispatcher) : null;
         $this->name = $name;
     }
 

Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ public function __construct(Definition $definition, MarkingStoreInterface $marki`
`43`	`43`	`{`
`44`	`44`	`$this->definition = $definition;`
`45`	`45`	`$this->markingStore = $markingStore ?: new MultipleStateMarkingStore();`
`46`		`- $this->dispatcher = LegacyEventDispatcherProxy::decorate($dispatcher);`
	`46`	`+ $this->dispatcher = null !== $dispatcher ? LegacyEventDispatcherProxy::decorate($dispatcher) : null;`
`47`	`47`	`$this->name = $name;`
`48`	`48`	`}`
`49`	`49`