Added TODO items to fix later

This commit is contained in:
Sean Harvey 2008-05-14 10:57:25 +00:00
parent 927e974724
commit 6cfe0680c0

View File

@ -138,12 +138,10 @@ abstract class MultiForm extends Form {
// Determine whether we use the current step, or create one if it doesn't exist // Determine whether we use the current step, or create one if it doesn't exist
if(isset($_GET['StepID'])) { if(isset($_GET['StepID'])) {
$stepID = (int)$_GET['StepID']; $stepID = (int)$_GET['StepID'];
$step = DataObject::get_one('MultiFormStep', "SessionID = {$this->session->ID} AND ID = {$stepID}"); $currentStep = DataObject::get_one('MultiFormStep', "SessionID = {$this->session->ID} AND ID = {$stepID}");
if($step) { // @TODO if you set a wrong ID, then it ends up at this point with a non-object error.
$currentStep = $step;
}
// @TODO if you set a wrong ID, then it ends up at this point with a non-object error.
} elseif($this->session->CurrentStepID) { } elseif($this->session->CurrentStepID) {
// @TODO if you set a wrong ID, then it ends up at this point with a non-object error.
$currentStep = $this->session->CurrentStep(); $currentStep = $this->session->CurrentStep();
} else { } else {
// @TODO fix the fact that you can continually refresh on the first step creating new records // @TODO fix the fact that you can continually refresh on the first step creating new records