symfony
diff --git a/‎src/Symfony/Component/Workflow/CHANGELOG.md
Lines changed: 5 additions & 0 deletions b/‎src/Symfony/Component/Workflow/CHANGELOG.md
Lines changed: 5 additions & 0 deletions
diff --git a/‎src/Symfony/Component/Workflow/Event/Event.php
Lines changed: 1 addition & 1 deletion b/‎src/Symfony/Component/Workflow/Event/Event.php
Lines changed: 1 addition & 1 deletion
diff --git a/‎src/Symfony/Component/Workflow/Tests/WorkflowTest.php
Lines changed: 2 additions & 0 deletions b/‎src/Symfony/Component/Workflow/Tests/WorkflowTest.php
Lines changed: 2 additions & 0 deletions
diff --git a/‎src/Symfony/Component/Workflow/Workflow.php
Lines changed: 7 additions & 3 deletions b/‎src/Symfony/Component/Workflow/Workflow.php
Lines changed: 7 additions & 3 deletions
@@ -1,6 +1,11 @@
 CHANGELOG
 =========
 
+4.3.0
+-----
+
+ * Trigger `entered` event for subject entering in the Workflow for the first time
+
 4.1.0
 -----
 
 
@@ -35,7 +35,7 @@ class Event extends BaseEvent
      * @param Transition        $transition
      * @param WorkflowInterface $workflow
      */
-    public function __construct($subject, Marking $marking, Transition $transition, $workflow = null)
+    public function __construct($subject, Marking $marking, Transition $transition = null, $workflow = null)
     {
         $this->subject = $subject;
         $this->marking = $marking;
 
@@ -373,6 +373,8 @@ public function testApplyWithEventDispatcher()
         $workflow = new Workflow($definition, new MultipleStateMarkingStore(), $eventDispatcher, 'workflow_name');
 
         $eventNameExpected = array(
+            'workflow.entered',
+            'workflow.workflow_name.entered',
             'workflow.guard',
             'workflow.workflow_name.guard',
             'workflow.workflow_name.guard.t1',
 
@@ -61,6 +61,8 @@ public function getMarking($subject)
 
             // update the subject with the new marking
             $this->markingStore->setMarking($subject, $marking);
+
+            $this->entered($subject, null, $marking);
         }
 
         // check that the subject has a known place
@@ -323,7 +325,7 @@ private function enter($subject, Transition $transition, Marking $marking): void
         }
     }
 
-    private function entered($subject, Transition $transition, Marking $marking): void
+    private function entered($subject, Transition $transition = null, Marking $marking): void
     {
         if (null === $this->dispatcher) {
             return;
@@ -334,8 +336,10 @@ private function entered($subject, Transition $transition, Marking $marking): vo
         $this->dispatcher->dispatch('workflow.entered', $event);
         $this->dispatcher->dispatch(sprintf('workflow.%s.entered', $this->name), $event);
 
-        foreach ($transition->getTos() as $place) {
-            $this->dispatcher->dispatch(sprintf('workflow.%s.entered.%s', $this->name, $place), $event);
+        if ($transition) {
+            foreach ($transition->getTos() as $place) {
+                $this->dispatcher->dispatch(sprintf('workflow.%s.entered.%s', $this->name, $place), $event);
+            }
         }
     }
Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ class Event extends BaseEvent`
`35`	`35`	`* @param Transition $transition`
`36`	`36`	`* @param WorkflowInterface $workflow`
`37`	`37`	`*/`
`38`		`- public function __construct($subject, Marking $marking, Transition $transition, $workflow = null)`
	`38`	`+ public function __construct($subject, Marking $marking, Transition $transition = null, $workflow = null)`
`39`	`39`	`{`
`40`	`40`	`$this->subject = $subject;`
`41`	`41`	`$this->marking = $marking;`
Original file line number	Diff line number	Diff line change
`@@ -61,6 +61,8 @@ public function getMarking($subject)`
`61`	`61`
`62`	`62`	`// update the subject with the new marking`
`63`	`63`	`$this->markingStore->setMarking($subject, $marking);`
	`64`	`+`
	`65`	`+ $this->entered($subject, null, $marking);`
`64`	`66`	`}`
`65`	`67`
`66`	`68`	`// check that the subject has a known place`
`@@ -323,7 +325,7 @@ private function enter($subject, Transition $transition, Marking $marking): void`
`323`	`325`	`}`
`324`	`326`	`}`
`325`	`327`
`326`		`- private function entered($subject, Transition $transition, Marking $marking): void`
	`328`	`+ private function entered($subject, Transition $transition = null, Marking $marking): void`
`327`	`329`	`{`
`328`	`330`	`if (null === $this->dispatcher) {`
`329`	`331`	`return;`
`@@ -334,8 +336,10 @@ private function entered($subject, Transition $transition, Marking $marking): vo`
`334`	`336`	`$this->dispatcher->dispatch('workflow.entered', $event);`
`335`	`337`	`$this->dispatcher->dispatch(sprintf('workflow.%s.entered', $this->name), $event);`
`336`	`338`
`337`		`- foreach ($transition->getTos() as $place) {`
`338`		`- $this->dispatcher->dispatch(sprintf('workflow.%s.entered.%s', $this->name, $place), $event);`
	`339`	`+ if ($transition) {`
	`340`	`+ foreach ($transition->getTos() as $place) {`
	`341`	`+ $this->dispatcher->dispatch(sprintf('workflow.%s.entered.%s', $this->name, $place), $event);`
	`342`	`+ }`
`339`	`343`	`}`
`340`	`344`	`}`
`341`	`345`