[Workflow] Deprecated DefinitionBuilder::setInitialPlace()

Added missing part of #30468

Author: lyrixx

CHANGELOG.md

@@ -28,6 +28,7 @@ CHANGELOG
  * Dispatch `CompletedEvent` on `workflow.completed`
  * Dispatch `AnnounceEvent` on `workflow.announce`
  * Added support for many `initialPlaces`
+ * Deprecated `DefinitionBuilder::setInitialPlace()` method, use `DefinitionBuilder::setInitialPlaces()` instead.
  * Deprecated the `MultipleStateMarkingStore` class, use the `MethodMarkingStore` instead.
  * Deprecated the `SingleStateMarkingStore` class, use the `MethodMarkingStore` instead.
 

Definition.php

@@ -48,13 +48,13 @@ public function __construct(array $places, array $transitions, $initialPlaces =
     }
 
     /**
-     * @deprecated since Symfony 4.3. Use the getInitialPlaces() instead.
+     * @deprecated since Symfony 4.3. Use getInitialPlaces() instead.
      *
      * @return string|null
      */
     public function getInitialPlace()
     {
-        @trigger_error(sprintf('Calling %s::getInitialPlace() is deprecated. Call %s::getInitialPlaces() instead.', __CLASS__, __CLASS__));
+        @trigger_error(sprintf('Calling %s::getInitialPlace() is deprecated since Symfony 4.3. Call getInitialPlaces() instead.', __CLASS__), E_USER_DEPRECATED);
 
         if (!$this->initialPlaces) {
             return null;

DefinitionBuilder.php

@@ -24,7 +24,7 @@ class DefinitionBuilder
 {
     private $places = [];
     private $transitions = [];
-    private $initialPlace;
+    private $initialPlaces;
     private $metadataStore;
 
     /**
@@ -42,7 +42,7 @@ public function __construct(array $places = [], array $transitions = [])
      */
     public function build()
     {
-        return new Definition($this->places, $this->transitions, $this->initialPlace, $this->metadataStore);
+        return new Definition($this->places, $this->transitions, $this->initialPlaces, $this->metadataStore);
     }
 
     /**
@@ -54,20 +54,36 @@ public function clear()
     {
         $this->places = [];
         $this->transitions = [];
-        $this->initialPlace = null;
+        $this->initialPlaces = null;
         $this->metadataStore = null;
 
         return $this;
     }
 
     /**
+     * @deprecated since Symfony 4.3. Use setInitialPlaces() instead.
+     *
      * @param string $place
      *
      * @return $this
      */
     public function setInitialPlace($place)
     {
-        $this->initialPlace = $place;
+        @trigger_error(sprintf('Calling %s::setInitialPlace() is deprecated since Symfony 4.3. Call setInitialPlaces() instead.', __CLASS__), E_USER_DEPRECATED);
+
+        $this->initialPlaces = $place;
+
+        return $this;
+    }
+
+    /**
+     * @param string|string[]|null $initialPlaces
+     *
+     * @return $this
+     */
+    public function setInitialPlaces($initialPlaces)
+    {
+        $this->initialPlaces = $initialPlaces;
 
         return $this;
     }
@@ -80,7 +96,7 @@ public function setInitialPlace($place)
     public function addPlace($place)
     {
         if (!$this->places) {
-            $this->initialPlace = $place;
+            $this->initialPlaces = $place;
         }
 
         $this->places[$place] = $place;

Tests/DefinitionBuilderTest.php

@@ -9,6 +9,7 @@
 
 class DefinitionBuilderTest extends TestCase
 {
+    /** @group legacy */
     public function testSetInitialPlace()
     {
         $builder = new DefinitionBuilder(['a', 'b']);
@@ -18,6 +19,15 @@ public function testSetInitialPlace()
         $this->assertEquals(['b'], $definition->getInitialPlaces());
     }
 
+    public function testSetInitialPlaces()
+    {
+        $builder = new DefinitionBuilder(['a', 'b']);
+        $builder->setInitialPlaces('b');
+        $definition = $builder->build();
+
+        $this->assertEquals(['b'], $definition->getInitialPlaces());
+    }
+
     public function testAddTransition()
     {
         $places = range('a', 'b');