fix(processor): remove resumable from the main interface (#4684)

Author: soyuka

src/Doctrine/Common/State/Processor.php

@@ -31,11 +31,6 @@ public function __construct(ManagerRegistry $managerRegistry)
         $this->managerRegistry = $managerRegistry;
     }
 
-    public function resumable(?string $operationName = null, array $context = []): bool
-    {
-        return false;
-    }
-
     public function supports($data, array $identifiers = [], ?string $operationName = null, array $context = []): bool
     {
         return null !== $this->getManager($data);

src/State/ChainProcessor.php

@@ -35,17 +35,6 @@ public function __construct(iterable $processors)
         $this->processors = $processors;
     }
 
-    public function resumable(?string $operationName = null, array $context = []): bool
-    {
-        foreach ($this->processors as $processor) {
-            if ($processor->resumable($operationName, $context)) {
-                return $processor->resumable($operationName, $context);
-            }
-        }
-
-        return false;
-    }
-
     public function supports($data, array $identifiers = [], ?string $operationName = null, array $context = []): bool
     {
         foreach ($this->processors as $processor) {

src/State/ProcessorInterface.php

@@ -20,12 +20,6 @@
  */
 interface ProcessorInterface
 {
-    /**
-     * Should we continue calling the next Processor or stop after this one?
-     * Defaults to stop the ChainProcessor if this interface is not implemented.
-     */
-    public function resumable(?string $operationName = null, array $context = []): bool;
-
     /**
      * Whether this state handler supports the class/identifier tuple.
      *

src/Symfony/Bundle/Processor/TraceableChainProcessor.php

@@ -42,7 +42,7 @@ private function traceProcessors($data, array $context = [])
             if (
                 ($this->processorsResponse[\get_class($processor)] = $found ? false : $processor->supports($data, $context))
                 &&
-                !($processor instanceof ProcessorInterface && $processor->resumable()) && !$found
+                !$found
             ) {
                 $found = true;
             }

src/Symfony/Messenger/Processor.php

@@ -67,11 +67,6 @@ private function remove($data, array $context = [])
         );
     }
 
-    public function resumable(?string $operationName = null, array $context = []): bool
-    {
-        return false;
-    }
-
     public function process($data, array $identifiers = [], ?string $operationName = null, array $context = [])
     {
         if (\array_key_exists('operation', $context) && $context['operation']->isDelete()) {

tests/Symfony/Messenger/ProcessorTest.php

@@ -51,14 +51,6 @@ public function testSupport()
         $this->assertTrue($processor->supports(new Dummy(), [], 'create'));
     }
 
-    public function testResumable()
-    {
-        $metadataFactoryProphecy = $this->prophesize(ResourceMetadataCollectionFactoryInterface::class);
-
-        $processor = new Processor($metadataFactoryProphecy->reveal(), $this->prophesize(MessageBusInterface::class)->reveal());
-        $this->assertFalse($processor->resumable());
-    }
-
     public function testSupportWithContext()
     {
         $resourceMetadataFactoryProphecy = $this->prophesize(ResourceMetadataCollectionFactoryInterface::class);