MINOR: fixing completed class for current step

silverstripe · Nov 30, 2015 · 1b84a83 · 1b84a83
1 parent d2780c4
commit 1b84a83
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/code/model/MultiForm.php b/code/model/MultiForm.php
@@ -574,12 +574,15 @@ public function getAllStepsLinear() {
 
 		$firstStep = DataObject::get_one(static::$start_step, "\"SessionID\" = {$this->session->ID}");
 		$firstStep->LinkingMode = ($firstStep->ID == $this->getCurrentStep()->ID) ? 'current' : 'link';
-		$firstStep->addExtraClass('completed');
 		$firstStep->setForm($this);
 		$stepsFound->push($firstStep);
 
 		// mark the further steps as non-completed if the first step is the current
-		if ($firstStep->ID == $this->getCurrentStep()->ID) $this->currentStepHasBeenFound = true;
+		if ($firstStep->ID == $this->getCurrentStep()->ID) {
+			$this->currentStepHasBeenFound = true;
+		} else {
+			$firstStep->addExtraClass('completed');
+		}
 
 		$this->getAllStepsRecursive($firstStep, $stepsFound);