From 970f7979a51908a5ddd5568db906272ce71abf74 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E7=8E=8B=E6=96=87=E6=98=8A?= <wangwenhao_2026@163.com>
Date: Wed, 8 Jul 2026 15:36:36 +0800
Subject: [PATCH] =?UTF-8?q?fix(hrm):=20=E4=BF=AE=E5=A4=8D=E9=83=A8?=
 =?UTF-8?q?=E9=97=A8=E8=B4=9F=E8=B4=A3=E4=BA=BA=E9=85=8D=E7=BD=AE=E5=8A=9F?=
 =?UTF-8?q?=E8=83=BD=EF=BC=8C=E4=BC=98=E5=8C=96=E9=83=A8=E9=97=A8=E5=85=B3?=
 =?UTF-8?q?=E8=81=94=E9=80=BB=E8=BE=91?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

1. 修复员工部门ID优先使用自定义字段而非用户绑定部门的逻辑
2. 新增部门负责人弹窗配置功能，支持直接编辑部门负责人、联系方式
3. 调整部门关联查询的JOIN条件，使用COALESCE正确获取部门ID
4. 为无负责人的部门标签添加点击阻止冒泡，避免触发树节点展开
---
 .../resources/mapper/HrmEmployeeMapper.xml    |  8 +-
 ruoyi-ui/src/views/hrm/employee/index.vue     | 77 ++++++++++++++++++-
 2 files changed, 78 insertions(+), 7 deletions(-)

diff --git a/fad-hrm/src/main/resources/mapper/HrmEmployeeMapper.xml b/fad-hrm/src/main/resources/mapper/HrmEmployeeMapper.xml
index c6d8593..e5338fd 100644
--- a/fad-hrm/src/main/resources/mapper/HrmEmployeeMapper.xml
+++ b/fad-hrm/src/main/resources/mapper/HrmEmployeeMapper.xml
@@ -51,13 +51,15 @@
     <sql id="selectVo">
         SELECT
             e.emp_id, e.user_id, e.emp_no, e.emp_name, e.gender, e.mobile, e.email, e.id_no,
-            e.hire_date, e.employment_type, e.status, su.dept_id, e.post_id, e.remark,
+            e.hire_date, e.employment_type, e.status,
+            COALESCE(e.dept_id, su.dept_id) AS dept_id,
+            e.post_id, e.remark,
             e.create_by, e.create_time, e.update_by, e.update_time,
             d.dept_name,
             p.post_name
         FROM hrm_employee e
-        left join sys_user su on su.user_id = e.user_id
-        LEFT JOIN sys_dept d ON su.dept_id = d.dept_id AND d.del_flag = 0
+        LEFT JOIN sys_user su ON su.user_id = e.user_id
+        LEFT JOIN sys_dept d ON d.dept_id = COALESCE(e.dept_id, su.dept_id) AND d.del_flag = 0
         LEFT JOIN sys_post p ON e.post_id = p.post_id
         WHERE e.del_flag = 0
     </sql>
diff --git a/ruoyi-ui/src/views/hrm/employee/index.vue b/ruoyi-ui/src/views/hrm/employee/index.vue
index ec630e1..8894d45 100644
--- a/ruoyi-ui/src/views/hrm/employee/index.vue
+++ b/ruoyi-ui/src/views/hrm/employee/index.vue
@@ -18,7 +18,7 @@
               <span v-if="node.data.leader" class="leader-tag">
                 ({{ node.data.leader }})
               </span>
-              <span v-else class="leader-tag" style="color: red;" @click="goDept">
+              <span v-else class="leader-tag" style="color: red; cursor: pointer;" @click.stop="handleConfigLeader(node.data)">
                 请配置负责人
               </span>
             </span>
@@ -75,6 +75,29 @@
       </el-card>
     </div>
 
+    <!-- 配置部门负责人弹窗 -->
+    <el-dialog :title="'配置部门负责人 - ' + deptForm.deptName" :visible.sync="deptDialogVisible" width="450px"
+      append-to-body>
+      <el-form ref="deptFormRef" :model="deptForm" :rules="deptRules" label-width="100px" size="small">
+        <el-form-item label="部门名称">
+          <el-input :value="deptForm.deptName" disabled />
+        </el-form-item>
+        <el-form-item label="负责人" prop="leader">
+          <el-input v-model="deptForm.leader" placeholder="请输入负责人姓名" clearable />
+        </el-form-item>
+        <el-form-item label="联系电话">
+          <el-input v-model="deptForm.phone" placeholder="请输入联系电话" clearable maxlength="11" />
+        </el-form-item>
+        <el-form-item label="邮箱">
+          <el-input v-model="deptForm.email" placeholder="请输入邮箱地址" clearable />
+        </el-form-item>
+      </el-form>
+      <div slot="footer" class="dialog-footer">
+        <el-button @click="deptDialogVisible = false">取消</el-button>
+        <el-button type="primary" :loading="deptSubmitting" @click="handleDeptSubmit">确定</el-button>
+      </div>
+    </el-dialog>
+
     <!-- Add/Edit Dialog -->
     <el-dialog :title="dialogTitle" :visible.sync="dialogVisible" width="700px" append-to-body>
       <el-form ref="empFormRef" :model="empForm" :rules="empRules" label-width="100px" size="small">
@@ -165,7 +188,7 @@
 </template>
 
 <script>
-import { listDept } from '@/api/system/dept'
+import { listDept, updateDept } from '@/api/system/dept'
 import { listPost } from '@/api/system/post'
 import { selectUser } from '@/api/system/user'
 import { listEmployee, addEmployee, updateEmployee, delEmployee } from '@/api/hrm/employee'
@@ -201,6 +224,22 @@ export default {
         deptId: [{ required: true, message: '请选择主部门', trigger: 'change' }],
         postId: [{ required: true, message: '请选择主岗位', trigger: 'change' }],
         status: [{ required: true, message: '请选择状态', trigger: 'change' }]
+      },
+      // 部门负责人配置
+      deptDialogVisible: false,
+      deptSubmitting: false,
+      deptForm: {
+        deptId: undefined,
+        deptName: '',
+        leader: '',
+        phone: '',
+        email: '',
+        parentId: undefined,
+        orderNum: 0,
+        status: '0'
+      },
+      deptRules: {
+        leader: [{ required: true, message: '请输入负责人姓名', trigger: 'blur' }]
       }
     }
   },
@@ -218,8 +257,38 @@ export default {
         this.deptLoading = false
       })
     },
-    goDept() {
-      this.$router.push('/renshi/dept')
+    // 点击"请配置负责人"，弹出对话框直接编辑部门负责人
+    handleConfigLeader(data) {
+      this.deptForm = {
+        deptId: data.deptId,
+        deptName: data.deptName,
+        leader: data.leader || '',
+        phone: data.phone || '',
+        email: data.email || '',
+        parentId: data.parentId,
+        orderNum: data.orderNum || 0,
+        status: data.status || '0'
+      }
+      this.deptDialogVisible = true
+      this.$nextTick(() => {
+        if (this.$refs.deptFormRef) {
+          this.$refs.deptFormRef.clearValidate()
+        }
+      })
+    },
+    // 提交部门负责人配置
+    handleDeptSubmit() {
+      this.$refs.deptFormRef.validate(valid => {
+        if (!valid) return
+        this.deptSubmitting = true
+        updateDept(this.deptForm).then(() => {
+          this.$message.success('部门负责人配置成功')
+          this.deptDialogVisible = false
+          this.loadDeptTree()
+        }).finally(() => {
+          this.deptSubmitting = false
+        })
+      })
     },
     buildTree(list) {
       const map = {}