[Workflow] Added Registry::has() to check if a workflow exists

lyrixx · lyrixx · commit 11866e1afd33 · 2019-11-25T14:41:16.000+01:00
diff --git a/src/Symfony/Component/Workflow/CHANGELOG.md b/src/Symfony/Component/Workflow/CHANGELOG.md
@@ -5,6 +5,7 @@ CHANGELOG
 -----
 
  * Added context to `TransitionException` and its child classes whenever they are thrown in `Workflow::apply()`
+ * Added `Registry::has()` to check if a workflow exists
 
 5.0.0
 -----
diff --git a/src/Symfony/Component/Workflow/Registry.php b/src/Symfony/Component/Workflow/Registry.php
@@ -27,14 +27,25 @@ public function addWorkflow(WorkflowInterface $workflow, WorkflowSupportStrategy
         $this->workflows[] = [$workflow, $supportStrategy];
     }
 
+    public function has(object $subject, string $workflowName = null): bool
+    {
+        foreach ($this->workflows as [$workflow, $supportStrategy]) {
+            if ($this->supports($workflow, $supportStrategy, $subject, $workflowName)) {
+                return true;
+            }
+        }
+
+        return false;
+    }
+
     /**
      * @return Workflow
      */
     public function get(object $subject, string $workflowName = null)
     {
         $matched = [];
 
-        foreach ($this->workflows as list($workflow, $supportStrategy)) {
+        foreach ($this->workflows as [$workflow, $supportStrategy]) {
             if ($this->supports($workflow, $supportStrategy, $subject, $workflowName)) {
                 $matched[] = $workflow;
             }
@@ -61,7 +72,7 @@ public function get(object $subject, string $workflowName = null)
     public function all(object $subject): array
     {
         $matched = [];
-        foreach ($this->workflows as list($workflow, $supportStrategy)) {
+        foreach ($this->workflows as [$workflow, $supportStrategy]) {
             if ($supportStrategy->supports($workflow, $subject)) {
                 $matched[] = $workflow;
             }
diff --git a/src/Symfony/Component/Workflow/Tests/RegistryTest.php b/src/Symfony/Component/Workflow/Tests/RegistryTest.php
@@ -28,6 +28,16 @@ protected function tearDown(): void
         $this->registry = null;
     }
 
+    public function testHasWithMatch()
+    {
+        $this->assertTrue($this->registry->has(new Subject1()));
+    }
+
+    public function testHasWithoutMatch()
+    {
+        $this->assertFalse($this->registry->has(new Subject1(), 'nope'));
+    }
+
     public function testGetWithSuccess()
     {
         $workflow = $this->registry->get(new Subject1());
