[Workfow] Rename current MarkingStore

* ScalarMarkingStore -> SingleStateMarkingStore
* PropertyAccessorMarkingStore -> MultipleStateMarkingStore

And I also made optionnal the `marking_store` config, to let the
componant choose the best marking store depending on the type
(state_machine or workflow).

Author: lyrixx

MarkingStore/MarkingStoreInterface.php

@@ -14,7 +14,11 @@
 use Symfony\Component\Workflow\Marking;
 
 /**
- * MarkingStoreInterface.
+ * MarkingStoreInterface is the interface between the Workflow Component and a
+ * plain old PHP object: the subject.
+ *
+ * It converts the Marking into something understandable by the subject and vice
+ * versa.
  *
  * @author Grégoire Pineau <[email protected]>
  */

MarkingStore/PropertyAccessorMarkingStore.php renamed to MarkingStore/MultipleStateMarkingStore.php

@@ -16,17 +16,21 @@
 use Symfony\Component\Workflow\Marking;
 
 /**
- * PropertyAccessorMarkingStore.
+ * MultipleStateMarkingStore stores the marking into a property of the
+ * subject.
+ *
+ * This store deals with a "multiple state" Marking. It means a subject can be
+ * in many state at the same time.
  *
  * @author Grégoire Pineau <[email protected]>
  */
-class PropertyAccessorMarkingStore implements MarkingStoreInterface
+class MultipleStateMarkingStore implements MarkingStoreInterface
 {
     private $property;
     private $propertyAccessor;
 
     /**
-     * PropertyAccessorMarkingStore constructor.
+     * MultipleStateMarkingStore constructor.
      *
      * @param string                         $property
      * @param PropertyAccessorInterface|null $propertyAccessor

MarkingStore/ScalarMarkingStore.php renamed to MarkingStore/SingleStateMarkingStore.php

@@ -16,17 +16,20 @@
 use Symfony\Component\Workflow\Marking;
 
 /**
- * ScalarMarkingStore.
+ * SingleStateMarkingStore stores the marking into a property of the subject.
+ *
+ * This store deals with a "single state" Marking. It means a subject can be in
+ * one and only state at the same time.
  *
  * @author Grégoire Pineau <[email protected]>
  */
-class ScalarMarkingStore implements MarkingStoreInterface
+class SingleStateMarkingStore implements MarkingStoreInterface
 {
     private $property;
     private $propertyAccessor;
 
     /**
-     * ScalarMarkingStore constructor.
+     * SingleStateMarkingStore constructor.
      *
      * @param string                         $property
      * @param PropertyAccessorInterface|null $propertyAccessor

StateMachine.php

@@ -4,7 +4,7 @@
 
 use Symfony\Component\EventDispatcher\EventDispatcherInterface;
 use Symfony\Component\Workflow\MarkingStore\MarkingStoreInterface;
-use Symfony\Component\Workflow\MarkingStore\ScalarMarkingStore;
+use Symfony\Component\Workflow\MarkingStore\SingleStateMarkingStore;
 
 /**
  * @author Tobias Nyholm <[email protected]>
@@ -13,6 +13,6 @@ class StateMachine extends Workflow
 {
     public function __construct(Definition $definition, MarkingStoreInterface $markingStore = null, EventDispatcherInterface $dispatcher = null, $name = 'unnamed')
     {
-        parent::__construct($definition, $markingStore ?: new ScalarMarkingStore(), $dispatcher, $name);
+        parent::__construct($definition, $markingStore ?: new SingleStateMarkingStore(), $dispatcher, $name);
     }
 }

Tests/EventListener/AuditTrailListenerTest.php

@@ -6,7 +6,7 @@
 use Symfony\Component\EventDispatcher\EventDispatcher;
 use Symfony\Component\Workflow\Definition;
 use Symfony\Component\Workflow\EventListener\AuditTrailListener;
-use Symfony\Component\Workflow\MarkingStore\PropertyAccessorMarkingStore;
+use Symfony\Component\Workflow\MarkingStore\MultipleStateMarkingStore;
 use Symfony\Component\Workflow\Transition;
 use Symfony\Component\Workflow\Workflow;
 
@@ -29,7 +29,7 @@ public function testItWorks()
         $ed = new EventDispatcher();
         $ed->addSubscriber(new AuditTrailListener($logger));
 
-        $workflow = new Workflow($definition, new PropertyAccessorMarkingStore(), $ed);
+        $workflow = new Workflow($definition, new MultipleStateMarkingStore(), $ed);
 
         $workflow->apply($object, 't1');
 

Tests/MarkingStore/PropertyAccessorMarkingStoreTest.php renamed to Tests/MarkingStore/MultipleStateMarkingStoreTest.php

@@ -3,16 +3,16 @@
 namespace Symfony\Component\Workflow\Tests\MarkingStore;
 
 use Symfony\Component\Workflow\Marking;
-use Symfony\Component\Workflow\MarkingStore\PropertyAccessorMarkingStore;
+use Symfony\Component\Workflow\MarkingStore\MultipleStateMarkingStore;
 
-class PropertyAccessorMarkingStoreTest extends \PHPUnit_Framework_TestCase
+class MultipleStateMarkingStoreTest extends \PHPUnit_Framework_TestCase
 {
     public function testGetSetMarking()
     {
         $subject = new \stdClass();
         $subject->myMarks = null;
 
-        $markingStore = new PropertyAccessorMarkingStore('myMarks');
+        $markingStore = new MultipleStateMarkingStore('myMarks');
 
         $marking = $markingStore->getMarking($subject);
 

Tests/MarkingStore/ScalarMarkingStoreTest.php renamed to Tests/MarkingStore/SingleStateMarkingStoreTest.php

@@ -3,16 +3,16 @@
 namespace Symfony\Component\Workflow\Tests\MarkingStore;
 
 use Symfony\Component\Workflow\Marking;
-use Symfony\Component\Workflow\MarkingStore\ScalarMarkingStore;
+use Symfony\Component\Workflow\MarkingStore\SingleStateMarkingStore;
 
-class ScalarMarkingStoreTest extends \PHPUnit_Framework_TestCase
+class SingleStateMarkingStoreTest extends \PHPUnit_Framework_TestCase
 {
     public function testGetSetMarking()
     {
         $subject = new \stdClass();
         $subject->myMarks = null;
 
-        $markingStore = new ScalarMarkingStore('myMarks');
+        $markingStore = new SingleStateMarkingStore('myMarks');
 
         $marking = $markingStore->getMarking($subject);
 

Tests/WorkflowTest.php

@@ -8,7 +8,7 @@
 use Symfony\Component\Workflow\Event\GuardEvent;
 use Symfony\Component\Workflow\Marking;
 use Symfony\Component\Workflow\MarkingStore\MarkingStoreInterface;
-use Symfony\Component\Workflow\MarkingStore\PropertyAccessorMarkingStore;
+use Symfony\Component\Workflow\MarkingStore\MultipleStateMarkingStore;
 use Symfony\Component\Workflow\Transition;
 use Symfony\Component\Workflow\Workflow;
 
@@ -35,7 +35,7 @@ public function testGetMarkingWithEmptyDefinition()
     {
         $subject = new \stdClass();
         $subject->marking = null;
-        $workflow = new Workflow(new Definition(array(), array()), new PropertyAccessorMarkingStore());
+        $workflow = new Workflow(new Definition(array(), array()), new MultipleStateMarkingStore());
 
         $workflow->getMarking($subject);
     }
@@ -49,7 +49,7 @@ public function testGetMarkingWithImpossiblePlace()
         $subject = new \stdClass();
         $subject->marking = null;
         $subject->marking = array('nope' => true);
-        $workflow = new Workflow(new Definition(array(), array()), new PropertyAccessorMarkingStore());
+        $workflow = new Workflow(new Definition(array(), array()), new MultipleStateMarkingStore());
 
         $workflow->getMarking($subject);
     }
@@ -59,7 +59,7 @@ public function testGetMarkingWithEmptyInitialMarking()
         $definition = $this->createComplexWorkflow();
         $subject = new \stdClass();
         $subject->marking = null;
-        $workflow = new Workflow($definition, new PropertyAccessorMarkingStore());
+        $workflow = new Workflow($definition, new MultipleStateMarkingStore());
 
         $marking = $workflow->getMarking($subject);
 
@@ -74,7 +74,7 @@ public function testGetMarkingWithExistingMarking()
         $subject = new \stdClass();
         $subject->marking = null;
         $subject->marking = array('b' => 1, 'c' => 1);
-        $workflow = new Workflow($definition, new PropertyAccessorMarkingStore());
+        $workflow = new Workflow($definition, new MultipleStateMarkingStore());
 
         $marking = $workflow->getMarking($subject);
 
@@ -92,7 +92,7 @@ public function testCanWithUnexistingTransition()
         $definition = $this->createComplexWorkflow();
         $subject = new \stdClass();
         $subject->marking = null;
-        $workflow = new Workflow($definition, new PropertyAccessorMarkingStore());
+        $workflow = new Workflow($definition, new MultipleStateMarkingStore());
 
         $workflow->can($subject, 'foobar');
     }
@@ -102,7 +102,7 @@ public function testCan()
         $definition = $this->createComplexWorkflow();
         $subject = new \stdClass();
         $subject->marking = null;
-        $workflow = new Workflow($definition, new PropertyAccessorMarkingStore());
+        $workflow = new Workflow($definition, new MultipleStateMarkingStore());
 
         $this->assertTrue($workflow->can($subject, 't1'));
         $this->assertFalse($workflow->can($subject, 't2'));
@@ -117,7 +117,7 @@ public function testCanWithGuard()
         $eventDispatcher->addListener('workflow.workflow_name.guard.t1', function (GuardEvent $event) {
             $event->setBlocked(true);
         });
-        $workflow = new Workflow($definition, new PropertyAccessorMarkingStore(), $eventDispatcher, 'workflow_name');
+        $workflow = new Workflow($definition, new MultipleStateMarkingStore(), $eventDispatcher, 'workflow_name');
 
         $this->assertFalse($workflow->can($subject, 't1'));
     }
@@ -131,7 +131,7 @@ public function testApplyWithImpossibleTransition()
         $definition = $this->createComplexWorkflow();
         $subject = new \stdClass();
         $subject->marking = null;
-        $workflow = new Workflow($definition, new PropertyAccessorMarkingStore());
+        $workflow = new Workflow($definition, new MultipleStateMarkingStore());
 
         $workflow->apply($subject, 't2');
     }
@@ -141,7 +141,7 @@ public function testApply()
         $definition = $this->createComplexWorkflow();
         $subject = new \stdClass();
         $subject->marking = null;
-        $workflow = new Workflow($definition, new PropertyAccessorMarkingStore());
+        $workflow = new Workflow($definition, new MultipleStateMarkingStore());
 
         $marking = $workflow->apply($subject, 't1');
 
@@ -157,7 +157,7 @@ public function testApplyWithEventDispatcher()
         $subject = new \stdClass();
         $subject->marking = null;
         $eventDispatcher = new EventDispatcherMock();
-        $workflow = new Workflow($definition, new PropertyAccessorMarkingStore(), $eventDispatcher, 'workflow_name');
+        $workflow = new Workflow($definition, new MultipleStateMarkingStore(), $eventDispatcher, 'workflow_name');
 
         $eventNameExpected = array(
             'workflow.guard',
@@ -194,7 +194,7 @@ public function testGetEnabledTransitions()
         $eventDispatcher->addListener('workflow.workflow_name.guard.t1', function (GuardEvent $event) {
             $event->setBlocked(true);
         });
-        $workflow = new Workflow($definition, new PropertyAccessorMarkingStore(), $eventDispatcher, 'workflow_name');
+        $workflow = new Workflow($definition, new MultipleStateMarkingStore(), $eventDispatcher, 'workflow_name');
 
         $this->assertEmpty($workflow->getEnabledTransitions($subject));
 

Workflow.php

@@ -16,7 +16,7 @@
 use Symfony\Component\Workflow\Event\GuardEvent;
 use Symfony\Component\Workflow\Exception\LogicException;
 use Symfony\Component\Workflow\MarkingStore\MarkingStoreInterface;
-use Symfony\Component\Workflow\MarkingStore\PropertyAccessorMarkingStore;
+use Symfony\Component\Workflow\MarkingStore\MultipleStateMarkingStore;
 
 /**
  * @author Fabien Potencier <[email protected]>
@@ -33,7 +33,7 @@ class Workflow
     public function __construct(Definition $definition, MarkingStoreInterface $markingStore = null, EventDispatcherInterface $dispatcher = null, $name = 'unnamed')
     {
         $this->definition = $definition;
-        $this->markingStore = $markingStore ?: new PropertyAccessorMarkingStore();
+        $this->markingStore = $markingStore ?: new MultipleStateMarkingStore();
         $this->dispatcher = $dispatcher;
         $this->name = $name;
     }