Merge pull request #22 from vlsoprun/master

#21 Does not work multiple "from"

Author: brexis

src/WorkflowRegistry.php

@@ -94,7 +94,9 @@ public function addFromArray($name, array $workflowData)
                 $transitionName = $transition['name'];
             }
 
-            $builder->addTransition(new Transition($transitionName, $transition['from'], $transition['to']));
+            foreach ((array)$transition['from'] as $form) {
+                $builder->addTransition(new Transition($transitionName, $form, $transition['to']));
+            }
         }
 
         $definition = $builder->build();

tests/WorkflowRegistryTest.php

@@ -125,4 +125,44 @@ public function testIfTransitionsWithSameNameCanBothBeUsed()
 
         $this->assertTrue($workflow->can($subject, 't1'));
     }
+
+    public function testWhenMultipleFromIsUsed()
+    {
+        $config = [
+            'straight' => [
+                'type'        => 'state_machine',
+                'supports'    => ['Tests\Fixtures\TestObject'],
+                'places'      => ['a', 'b', 'c'],
+                'transitions' => [
+                    [
+                        'name' => 't1',
+                        'from' => 'a',
+                        'to'   => 'b',
+                    ],
+                    [
+                        'name' => 't2',
+                        'from' => [
+                            'a',
+                            'b'
+                        ],
+                        'to'   => 'c',
+                    ],
+                ],
+            ],
+        ];
+
+        $registry = new WorkflowRegistry($config);
+        $subject = new TestObject;
+        $workflow = $registry->get($subject);
+
+        $markingStoreProp = new ReflectionProperty(Workflow::class, 'markingStore');
+        $markingStoreProp->setAccessible(true);
+
+        $markingStore = $markingStoreProp->getValue($workflow);
+
+        $this->assertTrue($workflow instanceof StateMachine);
+        $this->assertTrue($markingStore instanceof SingleStateMarkingStore);
+        $this->assertTrue($workflow->can($subject, 't1'));
+        $this->assertTrue($workflow->can($subject, 't2'));
+    }
 }