minor symfony#20462 [Workfow] Rename current MarkingStore (lyrixx)

This PR was merged into the 3.2-dev branch.

Discussion
----------

[Workfow] Rename current MarkingStore

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | yes (in un-released branch)
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT
| Doc PR        | -

* 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).

---

Note: I did that to ease the on-boarding

Commits
-------

08464c9 [Workfow] Rename current MarkingStore

Author: lyrixx

src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Configuration.php

@@ -241,10 +241,9 @@ private function addWorkflowSection(ArrayNodeDefinition $rootNode)
                                 ->defaultValue('workflow')
                             ->end()
                             ->arrayNode('marking_store')
-                                ->isRequired()
                                 ->children()
                                     ->enumNode('type')
-                                        ->values(array('property_accessor', 'scalar'))
+                                        ->values(array('multiple_state', 'single_state'))
                                     ->end()
                                     ->arrayNode('arguments')
                                         ->beforeNormalization()

src/Symfony/Bundle/FrameworkBundle/Resources/config/workflow.xml

@@ -18,8 +18,8 @@
             <argument /> <!-- name -->
         </service>
 
-        <service id="workflow.marking_store.property_accessor" class="Symfony\Component\Workflow\MarkingStore\PropertyAccessorMarkingStore" abstract="true" />
-        <service id="workflow.marking_store.scalar" class="Symfony\Component\Workflow\MarkingStore\ScalarMarkingStore" abstract="true" />
+        <service id="workflow.marking_store.multiple_state" class="Symfony\Component\Workflow\MarkingStore\MultipleStateMarkingStore" abstract="true" />
+        <service id="workflow.marking_store.single_state" class="Symfony\Component\Workflow\MarkingStore\SingleStateMarkingStore" abstract="true" />
 
         <service id="workflow.registry" class="Symfony\Component\Workflow\Registry" />
 

src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/Fixtures/php/workflow.php

@@ -6,7 +6,7 @@
     'workflows' => array(
         'my_workflow' => array(
             'marking_store' => array(
-                'type' => 'property_accessor',
+                'type' => 'multiple_state',
             ),
             'supports' => array(
                 FrameworkExtensionTest::class,

src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/Fixtures/xml/workflow.xml

@@ -10,7 +10,7 @@
         <framework:workflows>
             <framework:workflow name="my_workflow">
                 <framework:marking-store>
-                    <framework:type>property_accessor</framework:type>
+                    <framework:type>multiple_state</framework:type>
                     <framework:arguments>a</framework:arguments>
                     <framework:arguments>a</framework:arguments>
                 </framework:marking-store>

src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/Fixtures/yml/workflow.yml

@@ -2,7 +2,7 @@ framework:
     workflows:
         my_workflow:
             marking_store:
-                type: property_accessor
+                type: multiple_state
             supports:
                 - Symfony\Bundle\FrameworkBundle\Tests\DependencyInjection\FrameworkExtensionTest
             places:

src/Symfony/Component/Workflow/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]>
  */

src/Symfony/Component/Workflow/MarkingStore/PropertyAccessorMarkingStore.php renamed to src/Symfony/Component/Workflow/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

src/Symfony/Component/Workflow/MarkingStore/ScalarMarkingStore.php renamed to src/Symfony/Component/Workflow/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

src/Symfony/Component/Workflow/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);
     }
 }

src/Symfony/Component/Workflow/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');