diff --git a/module/testcase/model.php b/module/testcase/model.php index 9c32697450..541f0cde08 100644 --- a/module/testcase/model.php +++ b/module/testcase/model.php @@ -1090,12 +1090,13 @@ class testcaseModel extends model foreach($requiredFields as $requiredField) { $requiredField = trim($requiredField); - if(empty($caseData->$requiredField)) die(js::alert(sprintf($this->lang->testcase->noRequire, $key, $this->lang->testcase->$requiredField))); + if(empty($caseData->$requiredField)) dao::$errors[] = sprintf($this->lang->testcase->noRequire, $key, $this->lang->testcase->$requiredField); } } $cases[$key] = $caseData; } + if(dao::isError()) die(js::error(dao::getError())); $forceNotReview = $this->forceNotReview(); foreach($cases as $key => $caseData) diff --git a/module/testsuite/model.php b/module/testsuite/model.php index 95a4e77f81..9bada58ab6 100644 --- a/module/testsuite/model.php +++ b/module/testsuite/model.php @@ -720,12 +720,13 @@ class testsuiteModel extends model foreach($requiredFields as $requiredField) { $requiredField = trim($requiredField); - if(empty($caseData->$requiredField)) die(js::alert(sprintf($this->lang->testcase->noRequire, $key, $this->lang->testcase->$requiredField))); + if(empty($caseData->$requiredField)) dao::$errors[] = sprintf($this->lang->testcase->noRequire, $key, $this->lang->testcase->$requiredField); } } $cases[$key] = $caseData; } + if(dao::isError()) die(js::error(dao::getError())); $forceNotReview = $this->testcase->forceNotReview(); foreach($cases as $key => $caseData)