From e2f7c102964799cfe2fb9435ee8543f426a64f97 Mon Sep 17 00:00:00 2001 From: daitingting Date: Wed, 25 May 2022 01:03:06 +0000 Subject: [PATCH 1/3] * Set default value in getFieldControl function, finish task #54954. --- module/bug/view/batchcreate.html.php | 21 +++------------------ module/story/view/batchcreate.html.php | 7 +------ module/task/view/batchcreate.html.php | 14 ++------------ module/testcase/view/batchcreate.html.php | 4 ---- 4 files changed, 6 insertions(+), 40 deletions(-) diff --git a/module/bug/view/batchcreate.html.php b/module/bug/view/batchcreate.html.php index f0f4267291..dd09aa4bb5 100755 --- a/module/bug/view/batchcreate.html.php +++ b/module/bug/view/batchcreate.html.php @@ -131,12 +131,7 @@ '> loadModel('flow'); - foreach($extendFields as $extendField) - { - $object = new stdclass(); - $object->{$extendField->field} = $extendField->default; - echo "control == 'select' or $extendField->control == 'multi-select') ? " style='overflow:visible'" : '') . ">" . $this->flow->getFieldControl($extendField, $object, $extendField->field . "[$i]") . ""; - } + foreach($extendFields as $extendField) echo "control == 'select' or $extendField->control == 'multi-select') ? " style='overflow:visible'" : '') . ">" . $this->flow->getFieldControl($extendField, '', $extendField->field . "[$i]") . ""; ?> @@ -186,12 +181,7 @@ '> loadModel('flow'); - foreach($extendFields as $extendField) - { - $object = new stdclass(); - $object->{$extendField->field} = $extendField->default; - echo "control == 'select' or $extendField->control == 'multi-select') ? " style='overflow:visible'" : '') . ">" . $this->flow->getFieldControl($extendField, $object, $extendField->field . "[$i]") . ""; - } + foreach($extendFields as $extendField) echo "control == 'select' or $extendField->control == 'multi-select') ? " style='overflow:visible'" : '') . ">" . $this->flow->getFieldControl($extendField, '', $extendField->field . "[$i]") . ""; ?> @@ -240,12 +230,7 @@ '> loadModel('flow'); - foreach($extendFields as $extendField) - { - $object = new stdclass(); - $object->{$extendField->field} = $extendField->default; - echo "control == 'select' or $extendField->control == 'multi-select') ? " style='overflow:visible'" : '') . ">" . $this->flow->getFieldControl($extendField, $object, $extendField->field . "[%s]") . ""; - } + foreach($extendFields as $extendField) echo "control == 'select' or $extendField->control == 'multi-select') ? " style='overflow:visible'" : '') . ">" . $this->flow->getFieldControl($extendField, '', $extendField->field . "[%s]") . ""; ?> diff --git a/module/story/view/batchcreate.html.php b/module/story/view/batchcreate.html.php index 6c9f1ff6c0..794781efed 100755 --- a/module/story/view/batchcreate.html.php +++ b/module/story/view/batchcreate.html.php @@ -116,12 +116,7 @@ '> loadModel('flow'); - foreach($extendFields as $extendField) - { - $object = new stdclass(); - $object->{$extendField->field} = $extendField->default; - echo "control == 'select' or $extendField->control == 'multi-select') ? " style='overflow:visible'" : '') . ">" . $this->flow->getFieldControl($extendField, $object, $extendField->field . '[$id]') . ""; - } + foreach($extendFields as $extendField) echo "control == 'select' or $extendField->control == 'multi-select') ? " style='overflow:visible'" : '') . ">" . $this->flow->getFieldControl($extendField, '', $extendField->field . '[$id]') . ""; ?> diff --git a/module/task/view/batchcreate.html.php b/module/task/view/batchcreate.html.php index 43756f432f..40ea08c359 100755 --- a/module/task/view/batchcreate.html.php +++ b/module/task/view/batchcreate.html.php @@ -168,12 +168,7 @@ >task->priList, $pri, 'class=form-control');?> loadModel('flow'); - foreach($extendFields as $extendField) - { - $object = new stdclass(); - $object->{$extendField->field} = $extendField->default; - echo "control == 'select' or $extendField->control == 'multi-select') ? " style='overflow:visible'" : '') . ">" . $this->flow->getFieldControl($extendField, $object, $extendField->field . "[$i]") . ""; - } + foreach($extendFields as $extendField) echo "control == 'select' or $extendField->control == 'multi-select') ? " style='overflow:visible'" : '') . ">" . $this->flow->getFieldControl($extendField, $object, $extendField->field . "[$i]") . ""; ?> @@ -242,12 +237,7 @@ >task->priList, $pri, 'class=form-control');?> loadModel('flow'); - foreach($extendFields as $extendField) - { - $object = new stdclass(); - $object->{$extendField->field} = $extendField->default; - echo "control == 'select' or $extendField->control == 'multi-select') ? " style='overflow:visible'" : '') . ">" . $this->flow->getFieldControl($extendField, $object, $extendField->field . "[%s]") . ""; - } + foreach($extendFields as $extendField) echo "control == 'select' or $extendField->control == 'multi-select') ? " style='overflow:visible'" : '') . ">" . $this->flow->getFieldControl($extendField, $object, $extendField->field . "[%s]") . ""; ?> diff --git a/module/testcase/view/batchcreate.html.php b/module/testcase/view/batchcreate.html.php index 3a225bceb2..d6bc3cc07c 100644 --- a/module/testcase/view/batchcreate.html.php +++ b/module/testcase/view/batchcreate.html.php @@ -111,8 +111,6 @@ $this->loadModel('flow'); foreach($extendFields as $extendField) { - $object = new stdclass(); - $object->{$extendField->field} = $extendField->default; echo "control == 'select' or $extendField->control == 'multi-select') ? " style='overflow:visible'" : '') . ">" . $this->flow->getFieldControl($extendField, $object, $extendField->field . "[$i]") . ""; } ?> @@ -160,8 +158,6 @@ $this->loadModel('flow'); foreach($extendFields as $extendField) { - $object = new stdclass(); - $object->{$extendField->field} = $extendField->default; echo "control == 'select' or $extendField->control == 'multi-select') ? " style='overflow:visible'" : '') . ">" . $this->flow->getFieldControl($extendField, $object, $extendField->field . "[%s]") . ""; } ?> From 3a3549078f0115f25e17c4a77b0657accf3b95ad Mon Sep 17 00:00:00 2001 From: daitingting Date: Wed, 25 May 2022 01:19:49 +0000 Subject: [PATCH 2/3] * Remove useless object param. --- module/testcase/view/batchcreate.html.php | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/module/testcase/view/batchcreate.html.php b/module/testcase/view/batchcreate.html.php index d6bc3cc07c..804a882dd7 100644 --- a/module/testcase/view/batchcreate.html.php +++ b/module/testcase/view/batchcreate.html.php @@ -109,10 +109,7 @@ '>testcase->reviewList, $needReview, "class='form-control'");?> loadModel('flow'); - foreach($extendFields as $extendField) - { - echo "control == 'select' or $extendField->control == 'multi-select') ? " style='overflow:visible'" : '') . ">" . $this->flow->getFieldControl($extendField, $object, $extendField->field . "[$i]") . ""; - } + foreach($extendFields as $extendField) echo "control == 'select' or $extendField->control == 'multi-select') ? " style='overflow:visible'" : '') . ">" . $this->flow->getFieldControl($extendField, '', $extendField->field . "[$i]") . ""; ?> @@ -156,10 +153,7 @@ '>testcase->reviewList, $needReview, "class='form-control chosen'");?> loadModel('flow'); - foreach($extendFields as $extendField) - { - echo "control == 'select' or $extendField->control == 'multi-select') ? " style='overflow:visible'" : '') . ">" . $this->flow->getFieldControl($extendField, $object, $extendField->field . "[%s]") . ""; - } + foreach($extendFields as $extendField) echo "control == 'select' or $extendField->control == 'multi-select') ? " style='overflow:visible'" : '') . ">" . $this->flow->getFieldControl($extendField, '', $extendField->field . "[%s]") . ""; ?> From 79af02f18ac685dc7e1833487e1bfd711a107923 Mon Sep 17 00:00:00 2001 From: daitingting Date: Wed, 25 May 2022 01:25:53 +0000 Subject: [PATCH 3/3] * Remove useless object param. --- module/task/view/batchcreate.html.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/module/task/view/batchcreate.html.php b/module/task/view/batchcreate.html.php index 40ea08c359..60b617f62d 100755 --- a/module/task/view/batchcreate.html.php +++ b/module/task/view/batchcreate.html.php @@ -168,7 +168,7 @@ >task->priList, $pri, 'class=form-control');?> loadModel('flow'); - foreach($extendFields as $extendField) echo "control == 'select' or $extendField->control == 'multi-select') ? " style='overflow:visible'" : '') . ">" . $this->flow->getFieldControl($extendField, $object, $extendField->field . "[$i]") . ""; + foreach($extendFields as $extendField) echo "control == 'select' or $extendField->control == 'multi-select') ? " style='overflow:visible'" : '') . ">" . $this->flow->getFieldControl($extendField, '', $extendField->field . "[$i]") . ""; ?> @@ -237,7 +237,7 @@ >task->priList, $pri, 'class=form-control');?> loadModel('flow'); - foreach($extendFields as $extendField) echo "control == 'select' or $extendField->control == 'multi-select') ? " style='overflow:visible'" : '') . ">" . $this->flow->getFieldControl($extendField, $object, $extendField->field . "[%s]") . ""; + foreach($extendFields as $extendField) echo "control == 'select' or $extendField->control == 'multi-select') ? " style='overflow:visible'" : '') . ">" . $this->flow->getFieldControl($extendField, '', $extendField->field . "[%s]") . ""; ?>