Merge pull request #134 from lemberg/issue/126-remove-step-is-not-called

Fix remove Configurer scripts step is never called
lemberg · Jan 16, 2020 · 872c2b2 · 872c2b2
2 parents 79be063 + b92219e
commit 872c2b2
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/Config/Manager/UpdateManager.php b/src/Config/Manager/UpdateManager.php
@@ -5,6 +5,7 @@
 namespace Lemberg\Draft\Environment\Config\Manager;
 
 use Lemberg\Draft\Environment\App;
+use Lemberg\Draft\Environment\Config\AbstractStepInterface;
 use Lemberg\Draft\Environment\Config\Config;
 use Lemberg\Draft\Environment\Config\Update\UpdateStepInterface;
 
@@ -48,9 +49,9 @@ public function update(): void {
   private function filterSteps(): void {
     $lastAppliedUpdateWeight = $this->getLastAppliedUpdateWeight();
 
-    $this->steps = array_filter($this->steps, function (int $weight) use ($lastAppliedUpdateWeight): bool {
-      return $weight > $lastAppliedUpdateWeight;
-    }, ARRAY_FILTER_USE_KEY);
+    $this->steps = array_filter($this->steps, function (AbstractStepInterface $step) use ($lastAppliedUpdateWeight): bool {
+      return $step->getWeight() > $lastAppliedUpdateWeight;
+    });
   }
 
   /**