diff --git a/module/custom/config.php b/module/custom/config.php index 9b42284642..c95872d315 100644 --- a/module/custom/config.php +++ b/module/custom/config.php @@ -56,8 +56,8 @@ $config->custom->fieldList['build'] = 'scmPath,filePath,desc'; $config->custom->fieldList['bug']['create'] = 'module,project,deadline,type,os,browser,severity,pri,steps,keywords'; $config->custom->fieldList['bug']['edit'] = 'plan,project,assignedTo,deadline,type,os,browser,severity,pri,steps,keywords'; $config->custom->fieldList['bug']['resolve'] = 'resolvedBuild,resolvedDate,assignedTo,comment'; -$config->custom->fieldList['testcase']['create'] = 'stage,story,pri,precondition,keywords'; -$config->custom->fieldList['testcase']['edit'] = 'stage,story,pri,precondition,keywords,status'; +$config->custom->fieldList['testcase']['create'] = 'stage,story,pri,precondition,keywords,module'; +$config->custom->fieldList['testcase']['edit'] = 'stage,story,pri,precondition,keywords,status,module'; $config->custom->fieldList['testsuite'] = 'desc'; $config->custom->fieldList['caselib'] = 'desc'; $config->custom->fieldList['testcase']['createcase'] = 'lib,stage,pri,precondition,keywords'; diff --git a/module/testcase/model.php b/module/testcase/model.php index dfc14e2135..514112aaf5 100644 --- a/module/testcase/model.php +++ b/module/testcase/model.php @@ -179,7 +179,7 @@ class testcaseModel extends model foreach(explode(',', $this->config->testcase->create->requiredFields) as $field) { $field = trim($field); - if($field and empty($data[$i]->$field)) return print(js::alert(sprintf($this->lang->error->notempty, $this->lang->testcase->$field))); + if($field and empty($data[$i]->$field)) return helper::end(js::alert(sprintf($this->lang->error->notempty, $this->lang->testcase->$field))); } } @@ -1018,7 +1018,7 @@ class testcaseModel extends model } else { - return print(js::error('case#' . $caseID . dao::getError(true))); + return helper::end(js::error('case#' . $caseID . dao::getError(true))); } } @@ -2005,8 +2005,8 @@ class testcaseModel extends model /** * Build test case menu. * - * @param object $case - * @param string $type + * @param object $case + * @param string $type * @access public * @return string */ @@ -2019,7 +2019,7 @@ class testcaseModel extends model /** * Build test case view menu. * - * @param object $case + * @param object $case * @access public * @return string */ @@ -2067,11 +2067,11 @@ class testcaseModel extends model { if(!isonlybody()) $menu .= $this->buildMenu('testcase', 'edit', $params, $case, 'view', '', '', 'showinonlybody'); if(!$case->isLibCase && $case->auto != 'unit') - { + { $menu .= $this->buildMenu('testcase', 'create', "productID=$case->product&branch=$case->branch&moduleID=$case->module&from=testcase¶m=$case->id", $case, 'view', 'copy'); } if($case->isLibCase && common::hasPriv('caselib', 'createCase')) - { + { echo html::a(helper::createLink('caselib', 'createCase', "libID=$case->lib&moduleID=$case->module¶m=$case->id"), "", '', "class='btn' title='{$this->lang->testcase->copy}'"); } @@ -2084,7 +2084,7 @@ class testcaseModel extends model /** * Build test case browse menu. * - * @param object $case + * @param object $case * @access public * @return string */