Merge pull request #79 from isobar-nz/fix-session-current-step

Fix get current step
silverstripe · Nov 28, 2019 · 41d8927 · 41d8927
2 parents a57bfe7 + 2574126
commit 41d8927
Showing 1 changed file with 8 additions and 8 deletions.
diff --git a/src/Forms/MultiForm.php b/src/Forms/MultiForm.php
@@ -229,14 +229,14 @@ public function getCurrentStep()
         $currentStep = null;
         $StepID = $this->controller->getRequest()->getVar('StepID');
         if (isset($StepID)) {
-            $currentStep = DataObject::get_one(
-                MultiFormStep::class,
-                [
-                    'SessionID' => $this->session->ID,
-                    'ID' => $StepID
-                ]
-            );
-        } elseif ($this->session->CurrentStepID) {
+            $currentStep = MultiFormStep::get()->filter([
+                'SessionID' => $this->session->ID,
+                'ID' => $StepID
+            ])->first();
+        }
+
+        // if current step doesn't exist and no session current step then get the current step
+        if (!$currentStep && $this->session->CurrentStepID) {
             $currentStep = $this->session->CurrentStep();
         }