removed skippable option for steps

Author: Tobias Wojtylak

spec/Factory/MultiStepFactorySpec.php

@@ -27,7 +27,6 @@ function it_can_create_step(StepRequiredCheckerInterface $stepRequiredChecker)
             'slug'                => 'TestSlug',
             'template'            => 'TestTemplate',
             'controller'          => 'TestController',
-            'skippable'           => true,
             'stepRequiredChecker' => $stepRequiredChecker
         ];
 
@@ -37,7 +36,6 @@ function it_can_create_step(StepRequiredCheckerInterface $stepRequiredChecker)
         $step->getSlug()->shouldReturn('TestSlug');
         $step->getTemplate()->shouldReturn('TestTemplate');
         $step->getControllerAction()->shouldReturn('TestController');
-        $step->isSkippable()->shouldReturn(true);
         $step->getStepRequiredChecker()->shouldReturn($stepRequiredChecker);
     }
 }

spec/Model/MultiStepSpec.php

@@ -61,17 +61,6 @@ function it_has_controller_action()
         $this->getControllerAction()->shouldReturn($value);
     }
 
-    function it_is_not_skippable()
-    {
-        $this->isSkippable()->shouldReturn(false);
-    }
-
-    function it_is_skippable()
-    {
-        $this->setSkippable(true);
-        $this->isSkippable()->shouldReturn(true);
-    }
-
     function it_has_step_required_checker(StepRequiredCheckerInterface $stepRequiredChecker)
     {
         $this->setStepRequiredChecker($stepRequiredChecker);

src/DependencyInjection/Configuration.php

@@ -39,7 +39,6 @@ public function getConfigTreeBuilder()
                                                 'solutionDrive\MultiStepBundle\Controller\DefaultStepController::renderAction'
                                             )
                                         ->end()
-                                        ->booleanNode('skippable')->defaultFalse()->end()
                                         ->scalarNode('stepRequiredChecker')
                                             ->defaultValue('sd.multistep.step_checker.default')
                                         ->end()

src/Factory/MultiStepFactory.php

@@ -26,7 +26,6 @@ public function createFromConfig(string $id, array $config): MultiStepInterface
         $step->setSlug($config['slug']);
         $step->setTemplate($config['template']);
         $step->setControllerAction($config['controller']);
-        $step->setSkippable((bool) $config['skippable']);
         $step->setStepRequiredChecker($config['stepRequiredChecker']);
         return $step;
     }

src/Model/MultiStep.php

@@ -28,9 +28,6 @@ class MultiStep implements MultiStepInterface
     /** @var string */
     private $controllerAction = '';
 
-    /** @var bool */
-    private $skippable = false;
-
     /** @var StepRequiredCheckerInterface */
     private $stepRequiredChecker;
 
@@ -84,16 +81,6 @@ public function setControllerAction(string $controllerAction): void
         $this->controllerAction = $controllerAction;
     }
 
-    public function isSkippable(): bool
-    {
-        return $this->skippable;
-    }
-
-    public function setSkippable(bool $skippable): void
-    {
-        $this->skippable = $skippable;
-    }
-
     public function getStepRequiredChecker(): StepRequiredCheckerInterface
     {
         return $this->stepRequiredChecker;

src/Model/MultiStepInterface.php

@@ -32,10 +32,6 @@ public function getControllerAction(): string;
 
     public function setControllerAction(string $controllerAction): void;
 
-    public function isSkippable(): bool;
-
-    public function setSkippable(bool $skippable): void;
-
     public function getStepRequiredChecker(): StepRequiredCheckerInterface;
 
     public function setStepRequiredChecker(StepRequiredCheckerInterface $stepRequiredChecker): void;