[Workflow] Fixed initial places when no places are configured

lyrixx · lyrixx · commit 8f3a2e58195f · 2019-04-06T12:03:54.000+02:00
I introduced a BC break in #30468 and this PR fix it. With the full stack framework, when one does not configure the initial_place(s) the DIC set `[]` for the initial values. So it removes the initials values guessed in `Definition::addPlace()`
diff --git a/src/Symfony/Component/Workflow/Definition.php b/src/Symfony/Component/Workflow/Definition.php
@@ -92,9 +92,9 @@ public function getMetadataStore(): MetadataStoreInterface
         return $this->metadataStore;
     }
 
-    private function setInitialPlaces($places = null)
+    private function setInitialPlaces(array $places = null)
     {
-        if (null === $places) {
+        if (!$places) {
             return;
         }
 

Original file line number	Diff line number	Diff line change
`@@ -92,9 +92,9 @@ public function getMetadataStore(): MetadataStoreInterface`
`92`	`92`	`return $this->metadataStore;`
`93`	`93`	`}`
`94`	`94`
`95`		`- private function setInitialPlaces($places = null)`
	`95`	`+ private function setInitialPlaces(array $places = null)`
`96`	`96`	`{`
`97`		`- if (null === $places) {`
	`97`	`+ if (!$places) {`
`98`	`98`	`return;`
`99`	`99`	`}`
`100`	`100`