bug #762 Remove Component namespace on state (loic425)

This PR was merged into the 1.11 branch.

Discussion
----------

| Q               | A
| --------------- | -----
| Bug fix?        | no
| New feature?    | no
| BC breaks?      | no
| Deprecations?   | no
| Related tickets | 
| License         | MIT


Commits
-------

395d821 Remove Component namespace on state
b5d040a Fix DI
d74db52 Fix Phpspec tests
2040d33 Add missing test on responder

docs/processors.md

@@ -37,7 +37,7 @@ namespace App\Sylius\State\Processor;
 
 use Sylius\Component\Customer\Model\CustomerInterface;
 use Sylius\Resource\Doctrine\Common\State\PersistProcessor;
-use Sylius\Component\Resource\State\ProcessorInterface;
+use Sylius\Resource\State\ProcessorInterface;
 
 final class CreateCustomerProcessor implements ProcessorInterface
 {

docs/providers.md

@@ -82,6 +82,8 @@ declare(strict_types=1);
 
 namespace App\BoardGameBlog\Infrastructure\Sylius\State\Provider;
 
+use Sylius\Resource\State\ProviderInterface;
+
 final class BoardGameItemProvider implements ProviderInterface
 {
     public function __construct(

src/Bundle/DependencyInjection/SyliusResourceExtension.php

@@ -26,10 +26,10 @@
 use Sylius\Component\Resource\Metadata\Metadata;
 use Sylius\Component\Resource\Metadata\ResourceMetadata;
 use Sylius\Component\Resource\Reflection\ClassReflection;
-use Sylius\Component\Resource\State\ProcessorInterface;
-use Sylius\Component\Resource\State\ProviderInterface;
-use Sylius\Component\Resource\State\ResponderInterface;
 use Sylius\Component\Resource\Twig\Context\Factory\ContextFactoryInterface;
+use Sylius\Resource\State\ProcessorInterface;
+use Sylius\Resource\State\ProviderInterface;
+use Sylius\Resource\State\ResponderInterface;
 use Symfony\Component\Config\FileLocator;
 use Symfony\Component\Config\Loader\LoaderInterface;
 use Symfony\Component\DependencyInjection\ContainerBuilder;

src/Bundle/Resources/config/services/state.xml

@@ -23,41 +23,41 @@
         <service id="Sylius\Component\Resource\Factory\ArgumentParserInterface" alias="sylius.resource_factory.argument_parser" />
 
         <service id="sylius.state.provider.event_dispatcher"
-                 class="Sylius\Component\Resource\State\EventDispatcherProvider"
+                 class="Sylius\Resource\State\Provider\EventDispatcherProvider"
                  decorates="sylius.state.provider"
                  decoration-priority="-32"
         >
             <argument type="service" id=".inner" />
             <argument type="service" id="sylius.dispatcher.operation" />
         </service>
 
-        <service id="sylius.state.provider" class="Sylius\Component\Resource\State\Provider">
+        <service id="sylius.state.provider" class="Sylius\Resource\State\Provider">
             <argument type="tagged_locator" tag="sylius.state_provider" />
         </service>
-        <service id="Sylius\Component\Resource\State\ProviderInterface" alias="sylius.state.provider" />
+        <service id="Sylius\Resource\State\ProviderInterface" alias="sylius.state.provider" />
 
         <service id="sylius.state.provider.event_dispatcher"
-                 class="Sylius\Component\Resource\State\EventDispatcherProvider"
+                 class="Sylius\Resource\State\Provider\EventDispatcherProvider"
                  decorates="sylius.state.provider"
                  decoration-priority="-32"
         >
             <argument type="service" id=".inner" />
             <argument type="service" id="sylius.dispatcher.operation" />
         </service>
 
-        <service id="sylius.state.factory" class="Sylius\Component\Resource\State\Factory">
+        <service id="sylius.state.factory" class="Sylius\Resource\State\Factory">
             <argument type="tagged_locator" tag="sylius.resource_factory" />
             <argument type="service" id="sylius.expression_language.argument_parser.factory" />
         </service>
-        <service id="Sylius\Component\Resource\State\FactoryInterface" alias="sylius.state.factory" />
+        <service id="Sylius\Resource\State\FactoryInterface" alias="sylius.state.factory" />
 
-        <service id="sylius.state.processor" class="Sylius\Component\Resource\State\Processor">
+        <service id="sylius.state.processor" class="Sylius\Resource\State\Processor">
             <argument type="tagged_locator" tag="sylius.state_processor" />
         </service>
-        <service id="Sylius\Component\Resource\State\ProcessorInterface" alias="sylius.state.processor" />
+        <service id="Sylius\Resource\State\ProcessorInterface" alias="sylius.state.processor" />
 
         <service id="sylius.state.processor.event_dispatcher"
-                 class="Sylius\Component\Resource\State\EventDispatcherProcessor"
+                 class="Sylius\Resource\State\Processor\EventDispatcherProcessor"
                  decorates="sylius.state.processor"
                  decoration-priority="-32"
         >
@@ -67,26 +67,26 @@
         </service>
 
         <service id="sylius.state.processor.bulk_aware"
-                 class="Sylius\Component\Resource\State\BulkAwareProcessor"
+                 class="Sylius\Resource\State\Processor\BulkAwareProcessor"
                  decorates="sylius.state.processor"
                  decoration-priority="-64"
         >
             <argument type="service" id=".inner" />
         </service>
 
         <service id="sylius.state.processor.event_dispatcher_bulk_aware"
-                 class="Sylius\Component\Resource\State\EventDispatcherBulkAwareProcessor"
+                 class="Sylius\Resource\State\Processor\EventDispatcherBulkAwareProcessor"
                  decorates="sylius.state.processor.bulk_aware"
                  decoration-priority="-32"
         >
             <argument type="service" id=".inner" />
             <argument type="service" id="sylius.dispatcher.operation" />
         </service>
 
-        <service id="sylius.state.responder" class="Sylius\Component\Resource\State\Responder">
+        <service id="sylius.state.responder" class="Sylius\Resource\State\Responder">
             <argument type="tagged_locator" tag="sylius.state_responder" />
         </service>
-        <service id="Sylius\Component\Resource\State\ResponderInterface" alias="sylius.state.responder" />
+        <service id="Sylius\Resource\State\ResponderInterface" alias="sylius.state.responder" />
 
         <service id="Sylius\Component\Resource\Symfony\Request\State\Provider">
             <argument type="tagged_locator" tag="sylius.repository" />

src/Component/Grid/State/RequestGridProvider.php

@@ -19,9 +19,9 @@
 use Sylius\Component\Resource\Grid\View\Factory\GridViewFactoryInterface;
 use Sylius\Component\Resource\Metadata\GridAwareOperationInterface;
 use Sylius\Component\Resource\Metadata\Operation;
-use Sylius\Component\Resource\State\ProviderInterface;
 use Sylius\Resource\Context\Context;
 use Sylius\Resource\Context\Option\RequestOption;
+use Sylius\Resource\State\ProviderInterface;
 
 final class RequestGridProvider implements ProviderInterface
 {

src/Component/StateMachine/State/ApplyStateMachineTransitionProcessor.php

@@ -14,9 +14,9 @@
 namespace Sylius\Component\Resource\StateMachine\State;
 
 use Sylius\Component\Resource\Metadata\Operation;
-use Sylius\Component\Resource\State\ProcessorInterface;
 use Sylius\Component\Resource\StateMachine\OperationStateMachineInterface;
 use Sylius\Resource\Context\Context;
+use Sylius\Resource\State\ProcessorInterface;
 
 final class ApplyStateMachineTransitionProcessor implements ProcessorInterface
 {

src/Component/Symfony/EventListener/FactoryListener.php

@@ -16,8 +16,8 @@
 use Sylius\Component\Resource\Metadata\FactoryAwareOperationInterface;
 use Sylius\Component\Resource\Metadata\Operation;
 use Sylius\Component\Resource\Metadata\Operation\HttpOperationInitiatorInterface;
-use Sylius\Component\Resource\State\FactoryInterface;
 use Sylius\Resource\Context\Initiator\RequestContextInitiatorInterface;
+use Sylius\Resource\State\FactoryInterface;
 use Symfony\Component\HttpKernel\Event\RequestEvent;
 
 final class FactoryListener

src/Component/Symfony/EventListener/ReadListener.php

@@ -15,8 +15,8 @@
 
 use Sylius\Component\Resource\Metadata\CreateOperationInterface;
 use Sylius\Component\Resource\Metadata\Operation\HttpOperationInitiatorInterface;
-use Sylius\Component\Resource\State\ProviderInterface;
 use Sylius\Resource\Context\Initiator\RequestContextInitiatorInterface;
+use Sylius\Resource\State\ProviderInterface;
 use Symfony\Component\HttpKernel\Event\RequestEvent;
 use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
 

src/Component/Symfony/EventListener/RespondListener.php

@@ -14,8 +14,8 @@
 namespace Sylius\Component\Resource\Symfony\EventListener;
 
 use Sylius\Component\Resource\Metadata\Operation\HttpOperationInitiatorInterface;
-use Sylius\Component\Resource\State\ResponderInterface;
 use Sylius\Resource\Context\Initiator\RequestContextInitiatorInterface;
+use Sylius\Resource\State\ResponderInterface;
 use Symfony\Component\HttpFoundation\Response;
 use Symfony\Component\HttpKernel\Event\ViewEvent;
 use Webmozart\Assert\Assert;

src/Component/Symfony/EventListener/WriteListener.php

@@ -14,8 +14,8 @@
 namespace Sylius\Component\Resource\Symfony\EventListener;
 
 use Sylius\Component\Resource\Metadata\Operation\HttpOperationInitiatorInterface;
-use Sylius\Component\Resource\State\ProcessorInterface;
 use Sylius\Resource\Context\Initiator\RequestContextInitiatorInterface;
+use Sylius\Resource\State\ProcessorInterface;
 use Symfony\Component\HttpFoundation\Response;
 use Symfony\Component\HttpKernel\Event\ViewEvent;
 

src/Component/Symfony/Request/State/ApiResponder.php

@@ -17,10 +17,10 @@
 use Sylius\Component\Resource\Metadata\DeleteOperationInterface;
 use Sylius\Component\Resource\Metadata\Operation;
 use Sylius\Component\Resource\Metadata\UpdateOperationInterface;
-use Sylius\Component\Resource\State\ResponderInterface;
 use Sylius\Component\Resource\Symfony\Response\HeadersInitiatorInterface;
 use Sylius\Resource\Context\Context;
 use Sylius\Resource\Context\Option\RequestOption;
+use Sylius\Resource\State\ResponderInterface;
 use Symfony\Component\HttpFoundation\Response;
 use Webmozart\Assert\Assert;
 

src/Component/Symfony/Request/State/Provider.php

@@ -19,11 +19,11 @@
 use Sylius\Component\Resource\Metadata\CollectionOperationInterface;
 use Sylius\Component\Resource\Metadata\Operation;
 use Sylius\Component\Resource\Reflection\CallableReflection;
-use Sylius\Component\Resource\State\ProviderInterface;
 use Sylius\Component\Resource\Symfony\ExpressionLanguage\ArgumentParserInterface;
 use Sylius\Component\Resource\Symfony\Request\RepositoryArgumentResolver;
 use Sylius\Resource\Context\Context;
 use Sylius\Resource\Context\Option\RequestOption;
+use Sylius\Resource\State\ProviderInterface;
 
 final class Provider implements ProviderInterface
 {

src/Component/Symfony/Request/State/Responder.php

@@ -15,9 +15,9 @@
 
 use Psr\Container\ContainerInterface;
 use Sylius\Component\Resource\Metadata\Operation;
-use Sylius\Component\Resource\State\ResponderInterface;
 use Sylius\Resource\Context\Context;
 use Sylius\Resource\Context\Option\RequestOption;
+use Sylius\Resource\State\ResponderInterface;
 
 final class Responder implements ResponderInterface
 {

src/Component/Symfony/Request/State/TwigResponder.php

@@ -18,11 +18,11 @@
 use Sylius\Component\Resource\Metadata\HttpOperation;
 use Sylius\Component\Resource\Metadata\Operation;
 use Sylius\Component\Resource\Metadata\UpdateOperationInterface;
-use Sylius\Component\Resource\State\ResponderInterface;
 use Sylius\Component\Resource\Symfony\Routing\RedirectHandlerInterface;
 use Sylius\Component\Resource\Twig\Context\Factory\ContextFactoryInterface;
 use Sylius\Resource\Context\Context;
 use Sylius\Resource\Context\Option\RequestOption;
+use Sylius\Resource\State\ResponderInterface;
 use Symfony\Component\HttpFoundation\Response;
 use Twig\Environment;
 

src/Component/spec/StateMachine/State/ApplyStateMachineTransitionProcessorSpec.php

@@ -15,10 +15,10 @@
 
 use PhpSpec\ObjectBehavior;
 use Sylius\Component\Resource\Metadata\Create;
-use Sylius\Component\Resource\State\ProcessorInterface;
 use Sylius\Component\Resource\StateMachine\OperationStateMachineInterface;
 use Sylius\Component\Resource\StateMachine\State\ApplyStateMachineTransitionProcessor;
 use Sylius\Resource\Context\Context;
+use Sylius\Resource\State\ProcessorInterface;
 
 final class ApplyStateMachineTransitionProcessorSpec extends ObjectBehavior
 {

src/Component/spec/Symfony/EventListener/FactoryListenerSpec.php

@@ -17,10 +17,10 @@
 use Sylius\Component\Resource\Metadata\Create;
 use Sylius\Component\Resource\Metadata\Operation\HttpOperationInitiatorInterface;
 use Sylius\Component\Resource\Metadata\Update;
-use Sylius\Component\Resource\State\FactoryInterface;
 use Sylius\Component\Resource\Symfony\EventListener\FactoryListener;
 use Sylius\Resource\Context\Context;
 use Sylius\Resource\Context\Initiator\RequestContextInitiatorInterface;
+use Sylius\Resource\State\FactoryInterface;
 use Symfony\Component\HttpFoundation\ParameterBag;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpKernel\Event\RequestEvent;

src/Component/spec/Symfony/EventListener/ReadListenerSpec.php

@@ -19,10 +19,10 @@
 use Sylius\Component\Resource\Metadata\HttpOperation;
 use Sylius\Component\Resource\Metadata\Operation\HttpOperationInitiatorInterface;
 use Sylius\Component\Resource\Metadata\Resource\Factory\ResourceMetadataCollectionFactoryInterface;
-use Sylius\Component\Resource\State\ProviderInterface;
 use Sylius\Component\Resource\Symfony\EventListener\ReadListener;
 use Sylius\Resource\Context\Context;
 use Sylius\Resource\Context\Initiator\RequestContextInitiatorInterface;
+use Sylius\Resource\State\ProviderInterface;
 use Symfony\Component\HttpFoundation\ParameterBag;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpKernel\Event\RequestEvent;

src/Component/spec/Symfony/EventListener/RespondListenerSpec.php

@@ -16,10 +16,10 @@
 use PhpSpec\ObjectBehavior;
 use Sylius\Component\Resource\Metadata\HttpOperation;
 use Sylius\Component\Resource\Metadata\Operation\HttpOperationInitiatorInterface;
-use Sylius\Component\Resource\State\ResponderInterface;
 use Sylius\Component\Resource\Symfony\EventListener\RespondListener;
 use Sylius\Resource\Context\Context;
 use Sylius\Resource\Context\Initiator\RequestContextInitiatorInterface;
+use Sylius\Resource\State\ResponderInterface;
 use Symfony\Component\HttpFoundation\ParameterBag;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;

src/Component/spec/Symfony/EventListener/WriteListenerSpec.php

@@ -17,10 +17,10 @@
 use Prophecy\Argument;
 use Sylius\Component\Resource\Metadata\HttpOperation;
 use Sylius\Component\Resource\Metadata\Operation\HttpOperationInitiatorInterface;
-use Sylius\Component\Resource\State\ProcessorInterface;
 use Sylius\Component\Resource\Symfony\EventListener\WriteListener;
 use Sylius\Resource\Context\Context;
 use Sylius\Resource\Context\Initiator\RequestContextInitiatorInterface;
+use Sylius\Resource\State\ProcessorInterface;
 use Symfony\Component\HttpFoundation\ParameterBag;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;

src/Component/spec/Symfony/Request/State/ResponderSpec.php

@@ -16,10 +16,10 @@
 use PhpSpec\ObjectBehavior;
 use Psr\Container\ContainerInterface;
 use Sylius\Component\Resource\Metadata\HttpOperation;
-use Sylius\Component\Resource\State\ResponderInterface;
 use Sylius\Component\Resource\Symfony\Request\State\Responder;
 use Sylius\Resource\Context\Context;
 use Sylius\Resource\Context\Option\RequestOption;
+use Sylius\Resource\State\ResponderInterface;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
 

src/Component/src/Doctrine/Common/State/PersistProcessor.php

@@ -19,8 +19,8 @@
 use Doctrine\Persistence\ObjectManager as DoctrineObjectManager;
 use Sylius\Component\Resource\Metadata\Operation;
 use Sylius\Component\Resource\Reflection\ClassInfoTrait;
-use Sylius\Component\Resource\State\ProcessorInterface;
 use Sylius\Resource\Context\Context;
+use Sylius\Resource\State\ProcessorInterface;
 
 final class PersistProcessor implements ProcessorInterface
 {

src/Component/src/Doctrine/Common/State/RemoveProcessor.php

@@ -17,8 +17,8 @@
 use Doctrine\Persistence\ObjectManager as DoctrineObjectManager;
 use Sylius\Component\Resource\Metadata\Operation;
 use Sylius\Component\Resource\Reflection\ClassInfoTrait;
-use Sylius\Component\Resource\State\ProcessorInterface;
 use Sylius\Resource\Context\Context;
+use Sylius\Resource\State\ProcessorInterface;
 
 final class RemoveProcessor implements ProcessorInterface
 {

src/Component/State/Factory.php → src/Component/src/State/Factory.php

@@ -11,7 +11,7 @@
 
 declare(strict_types=1);
 
-namespace Sylius\Component\Resource\State;
+namespace Sylius\Resource\State;
 
 use Psr\Container\ContainerInterface;
 use Sylius\Component\Resource\Factory\FactoryInterface as ResourceFactoryInterface;

src/Component/State/FactoryInterface.php → src/Component/src/State/FactoryInterface.php

@@ -11,7 +11,7 @@
 
 declare(strict_types=1);
 
-namespace Sylius\Component\Resource\State;
+namespace Sylius\Resource\State;
 
 use Sylius\Component\Resource\Metadata\Operation;
 use Sylius\Resource\Context\Context;

src/Component/State/Processor.php → src/Component/src/State/Processor.php

@@ -11,7 +11,7 @@
 
 declare(strict_types=1);
 
-namespace Sylius\Component\Resource\State;
+namespace Sylius\Resource\State;
 
 use Psr\Container\ContainerInterface;
 use Sylius\Component\Resource\Metadata\Operation;

src/Component/State/BulkAwareProcessor.php → src/Component/src/State/Processor/BulkAwareProcessor.php

@@ -11,11 +11,12 @@
 
 declare(strict_types=1);
 
-namespace Sylius\Component\Resource\State;
+namespace Sylius\Resource\State\Processor;
 
 use Sylius\Component\Resource\Metadata\BulkOperationInterface;
 use Sylius\Component\Resource\Metadata\Operation;
 use Sylius\Resource\Context\Context;
+use Sylius\Resource\State\ProcessorInterface;
 
 /**
  * @experimental

src/Component/State/EventDispatcherBulkAwareProcessor.php → src/Component/src/State/Processor/EventDispatcherBulkAwareProcessor.php

@@ -11,12 +11,13 @@
 
 declare(strict_types=1);
 
-namespace Sylius\Component\Resource\State;
+namespace Sylius\Resource\State\Processor;
 
 use Sylius\Component\Resource\Metadata\BulkOperationInterface;
 use Sylius\Component\Resource\Metadata\Operation;
 use Sylius\Component\Resource\Symfony\EventDispatcher\OperationEventDispatcherInterface;
 use Sylius\Resource\Context\Context;
+use Sylius\Resource\State\ProcessorInterface;
 
 /**
  * @experimental

src/Component/State/EventDispatcherProcessor.php → src/Component/src/State/Processor/EventDispatcherProcessor.php

@@ -11,14 +11,15 @@
 
 declare(strict_types=1);
 
-namespace Sylius\Component\Resource\State;
+namespace Sylius\Resource\State\Processor;
 
 use Sylius\Component\Resource\Metadata\CreateOperationInterface;
 use Sylius\Component\Resource\Metadata\Operation;
 use Sylius\Component\Resource\ResourceActions;
 use Sylius\Component\Resource\Symfony\EventDispatcher\OperationEventDispatcherInterface;
 use Sylius\Component\Resource\Symfony\EventDispatcher\OperationEventHandlerInterface;
 use Sylius\Resource\Context\Context;
+use Sylius\Resource\State\ProcessorInterface;
 
 /**
  * @experimental

src/Component/State/ProcessorInterface.php → src/Component/src/State/ProcessorInterface.php

@@ -11,7 +11,7 @@
 
 declare(strict_types=1);
 
-namespace Sylius\Component\Resource\State;
+namespace Sylius\Resource\State;
 
 use Sylius\Component\Resource\Metadata\Operation;
 use Sylius\Resource\Context\Context;