From 4f8cc9df7bfe5cda4435c02a6a5250fb26a80da2 Mon Sep 17 00:00:00 2001
From: xieqiyu <xieqiyu@easycorp.ltd>
Date: Wed, 15 Nov 2023 10:34:54 +0800
Subject: [PATCH] * Fix bug of edit testcase error tip.

---
 module/testcase/ui/edit.html.php | 105 ++++++++++++++++++-------------
 1 file changed, 62 insertions(+), 43 deletions(-)

diff --git a/module/testcase/ui/edit.html.php b/module/testcase/ui/edit.html.php
index 9dad0ff4b9..266753fa9f 100644
--- a/module/testcase/ui/edit.html.php
+++ b/module/testcase/ui/edit.html.php
@@ -116,11 +116,14 @@ detailBody
         section
         (
             set::title($lang->testcase->precondition),
-            textarea
+            formGroup
             (
-                set::name('precondition'),
-                set::value($case->precondition),
-                set::rows(2)
+                textarea
+                (
+                    set::name('precondition'),
+                    set::value($case->precondition),
+                    set::rows(2)
+                )
             )
         ),
         section
@@ -191,33 +194,36 @@ detailBody
             item
             (
                 set::name($lang->testcase->module),
-                inputGroup
+                formGroup
                 (
-                    set::id('moduleIdBox'),
-                    picker
+                    inputGroup
                     (
-                        setID('module'),
-                        set::name('module'),
-                        set::items($moduleOptionMenu),
-                        set::value($case->module),
-                        set::required(true)
-                    ),
-                    span
-                    (
-                        set('class', 'input-group-addon' . (count($moduleOptionMenu) > 1 ? ' hidden' : '')),
-                        a
+                        set::id('moduleIdBox'),
+                        picker
                         (
-                            set('class', 'mr-2'),
-                            set('href', $createModuleLink),
-                            set('data-toggle', 'modal'),
-                            set('data-size', 'lg'),
-                            $lang->tree->manage
+                            setID('module'),
+                            set::name('module'),
+                            set::items($moduleOptionMenu),
+                            set::value($case->module),
+                            set::required(true)
                         ),
-                        a
+                        span
                         (
-                            set('id', 'refresh'),
-                            set('class', 'text-black'),
-                            icon('refresh')
+                            set('class', 'input-group-addon' . (count($moduleOptionMenu) > 1 ? ' hidden' : '')),
+                            a
+                            (
+                                set('class', 'mr-2'),
+                                set('href', $createModuleLink),
+                                set('data-toggle', 'modal'),
+                                set('data-size', 'lg'),
+                                $lang->tree->manage
+                            ),
+                            a
+                            (
+                                set('id', 'refresh'),
+                                set('class', 'text-black'),
+                                icon('refresh')
+                            )
                         )
                     )
                 )
@@ -225,7 +231,7 @@ detailBody
             !$isLibCase ? item
             (
                 set::name($lang->testcase->story),
-                div
+                formGroup
                 (
                     set::id('storyIdBox'),
                     picker
@@ -283,41 +289,54 @@ detailBody
             item
             (
                 set::name($lang->testcase->stage),
-                picker
+                formGroup
                 (
-                    set::name('stage[]'),
-                    set::items($lang->testcase->stageList),
-                    set::value($case->stage),
-                    set::multiple(true)
+                    picker
+                    (
+                        set::name('stage[]'),
+                        set::items($lang->testcase->stageList),
+                        set::value($case->stage),
+                        set::multiple(true)
+                    )
                 )
             ),
             item
             (
                 set::name($lang->testcase->pri),
-                priPicker
+                formGroup
                 (
-                    set::name('pri'),
-                    set::items($priList),
-                    set::value($case->pri)
+                    priPicker
+                    (
+                        set::name('pri'),
+                        set::items($priList),
+                        set::value($case->pri)
+                    )
                 )
             ),
             item
             (
                 set::name($lang->testcase->status),
-                !$forceNotReview && $case->status == 'wait' ? $lang->testcase->statusList[$case->status] : picker
+                !$forceNotReview && $case->status == 'wait' ? $lang->testcase->statusList[$case->status] :
+                formGroup
                 (
-                    set::name('status'),
-                    set::items($lang->testcase->statusList),
-                    set::value($case->status)
+                    picker
+                    (
+                        set::name('status'),
+                        set::items($lang->testcase->statusList),
+                        set::value($case->status)
+                    )
                 )
             ),
             item
             (
                 set::name($lang->testcase->keywords),
-                input
+                formGroup
                 (
-                    set::name('keywords'),
-                    set::value($case->keywords)
+                    input
+                    (
+                        set::name('keywords'),
+                        set::value($case->keywords)
+                    )
                 )
             ),
             (!$isLibCase && hasPriv('testcase', 'linkCases')) ? item