From 35f05020c82d122f6b273d8d2125cff7fd1c9f17 Mon Sep 17 00:00:00 2001
From: CD <1103614279@qq.com>
Date: Tue, 27 Dec 2022 14:46:49 +0800
Subject: [PATCH 1/3] =?UTF-8?q?=E9=A1=B9=E7=9B=AE=E5=88=97=E8=A1=A8?=
 =?UTF-8?q?=E4=BB=A5=E5=8F=8A=E8=AF=A6=E6=83=85=E4=B8=AD=E5=93=81=E7=89=8C?=
 =?UTF-8?q?=E7=9A=84=E5=9B=9E=E6=98=BE?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../com/jero/modules/project/entity/ProjectLibraryBase.java   | 3 +++
 .../modules/project/mapper/xml/ProjectLibraryBaseMapper.xml   | 4 +++-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/jero-boot/jero-boot-modules/src/main/java/com/jero/modules/project/entity/ProjectLibraryBase.java b/jero-boot/jero-boot-modules/src/main/java/com/jero/modules/project/entity/ProjectLibraryBase.java
index 7442085f7..ef761dbd0 100644
--- a/jero-boot/jero-boot-modules/src/main/java/com/jero/modules/project/entity/ProjectLibraryBase.java
+++ b/jero-boot/jero-boot-modules/src/main/java/com/jero/modules/project/entity/ProjectLibraryBase.java
@@ -138,6 +138,9 @@ public class ProjectLibraryBase implements Comparable<ProjectLibraryBase> {
     //品牌
     private String brand;
 
+    @TableField(exist = false)
+    private String brandText;
+
     @Override
     public int compareTo(ProjectLibraryBase o) {
         return Integer.valueOf(this.projectVersion)-Integer.valueOf(o.projectVersion);//升序
diff --git a/jero-boot/jero-boot-modules/src/main/java/com/jero/modules/project/mapper/xml/ProjectLibraryBaseMapper.xml b/jero-boot/jero-boot-modules/src/main/java/com/jero/modules/project/mapper/xml/ProjectLibraryBaseMapper.xml
index b504c0987..a483480a0 100644
--- a/jero-boot/jero-boot-modules/src/main/java/com/jero/modules/project/mapper/xml/ProjectLibraryBaseMapper.xml
+++ b/jero-boot/jero-boot-modules/src/main/java/com/jero/modules/project/mapper/xml/ProjectLibraryBaseMapper.xml
@@ -51,11 +51,13 @@
             plb.software_version,
             plb.sort,
             plb.flag,
-            plb.brand
+            plb.brand,
+            sdi.item_text brandText
         from project_library_base as plb
         left join project_name_info as pni on  plb.project_name_id=pni.id
         left join sys_user as studiouser on plb.studio_engineer=studiouser.id
         left join project_year_name_info as pyni on plb.year_name_id=pyni.id
+        left join sys_dict_item as sdi on sdi.item_value = plb.brand
     </sql>
 
     <select id="queryPageList" resultType="com.jero.modules.project.entity.ProjectLibraryBase">

From 1ccb5b66632176f26398b1c7e5a158a4c0ab9c55 Mon Sep 17 00:00:00 2001
From: zhn <947514737@qq.com>
Date: Tue, 27 Dec 2022 15:51:18 +0800
Subject: [PATCH 2/3] =?UTF-8?q?=E5=8F=82=E6=95=B0=E5=AF=BC=E5=87=BA?=
 =?UTF-8?q?=E6=A8=A1=E6=9D=BF=E7=AE=A1=E7=90=86--=E8=A1=A8=E5=A4=B4?=
 =?UTF-8?q?=E9=80=82=E7=94=A8=E5=9C=B0=E5=8C=BA=E5=AD=97=E6=AE=B5=E6=8E=92?=
 =?UTF-8?q?=E5=BA=8F?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../controller/ParamsExportTemplateEOController.java      | 8 ++++----
 .../cert/report/entity/ParamsExportTemplateEO.java        | 2 +-
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/jero-boot/jero-boot-modules/src/main/java/com/jero/modules/cert/report/controller/ParamsExportTemplateEOController.java b/jero-boot/jero-boot-modules/src/main/java/com/jero/modules/cert/report/controller/ParamsExportTemplateEOController.java
index 6cdfcd34e..8a569854c 100644
--- a/jero-boot/jero-boot-modules/src/main/java/com/jero/modules/cert/report/controller/ParamsExportTemplateEOController.java
+++ b/jero-boot/jero-boot-modules/src/main/java/com/jero/modules/cert/report/controller/ParamsExportTemplateEOController.java
@@ -125,19 +125,19 @@ public class ParamsExportTemplateEOController extends JeroController<ParamsExpor
 						String regionName = regionDictItemList.stream().filter(e -> e.getItemValue().equals(s)).map(f -> f.getItemText()).collect(Collectors.joining(","));
 						regionBuilder.append(regionName).append(",");
 					}
-					paramsExportTemplateEOTemp.setRegionName(regionBuilder.substring(0, regionBuilder.toString().length() - 1));
+					paramsExportTemplateEOTemp.setRegion_dictText(regionBuilder.substring(0, regionBuilder.toString().length() - 1));
 				}else{
-					paramsExportTemplateEOTemp.setRegionName("");
+					paramsExportTemplateEOTemp.setRegion_dictText("");
 				}
 			}
 			Collator comparator = Collator.getInstance(Locale.CHINESE);
 			if (OrderEnum.POSITIVE.getValue().equals(paramsExportTemplateEO.getOrderBy())) {
 				Collections.sort(list, (e1, e2) -> {
-					return comparator.compare(e1.getRegionName(), e2.getRegionName());
+					return comparator.compare(e1.getRegion_dictText(), e2.getRegion_dictText());
 				});
 			} else if (OrderEnum.REVERSE.getValue().equals(paramsExportTemplateEO.getOrderBy())) {
 				Collections.sort(list, (e1, e2) -> {
-					return comparator.compare(e2.getRegionName(), e1.getRegionName());
+					return comparator.compare(e2.getRegion_dictText(), e1.getRegion_dictText());
 				});
 			}
 			pageList = getPages(pageNo, pageSize, list);
diff --git a/jero-boot/jero-boot-modules/src/main/java/com/jero/modules/cert/report/entity/ParamsExportTemplateEO.java b/jero-boot/jero-boot-modules/src/main/java/com/jero/modules/cert/report/entity/ParamsExportTemplateEO.java
index 37e482426..f78516765 100644
--- a/jero-boot/jero-boot-modules/src/main/java/com/jero/modules/cert/report/entity/ParamsExportTemplateEO.java
+++ b/jero-boot/jero-boot-modules/src/main/java/com/jero/modules/cert/report/entity/ParamsExportTemplateEO.java
@@ -94,6 +94,6 @@ public class ParamsExportTemplateEO implements Serializable {
     //适用地区
     private String region;
     @TableField(exist = false)
-    private String regionName;
+    private String region_dictText;
 
 }

From ca1fb97fc576d73dd9029e45d6e7cf0e78dad967 Mon Sep 17 00:00:00 2001
From: CD <1103614279@qq.com>
Date: Tue, 27 Dec 2022 16:11:22 +0800
Subject: [PATCH 3/3] =?UTF-8?q?=E5=89=8D=E7=AB=AF=E5=93=81=E7=89=8C?=
 =?UTF-8?q?=E5=9B=9E=E6=98=BE=20+=20=E5=90=8E=E7=AB=AF=E5=A4=A7=E5=88=97?=
 =?UTF-8?q?=E8=A1=A8=E4=BF=AE=E6=94=B9=E6=9D=83=E9=99=90=E7=AD=9B=E9=80=89?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../system/controller/SysUserController.java  |  2 +-
 .../modules/system/enums/ProjectRoleEnum.java |  1 +
 .../service/impl/SysUserServiceImpl.java      | 39 +++++++++++--------
 .../impl/ProjectLibraryBaseServiceImpl.java   | 16 ++++----
 .../components/ProjectDetails.vue             |  6 +--
 .../projectLibrary/index.vue                  |  2 +-
 .../src/views/system/modules/UserModal.vue    |  6 +--
 7 files changed, 37 insertions(+), 35 deletions(-)

diff --git a/jero-boot/jero-boot-module-system/src/main/java/com/jero/modules/system/controller/SysUserController.java b/jero-boot/jero-boot-module-system/src/main/java/com/jero/modules/system/controller/SysUserController.java
index f9425d9d1..7735f4266 100644
--- a/jero-boot/jero-boot-module-system/src/main/java/com/jero/modules/system/controller/SysUserController.java
+++ b/jero-boot/jero-boot-module-system/src/main/java/com/jero/modules/system/controller/SysUserController.java
@@ -252,7 +252,7 @@ public class SysUserController {
 
                     sysUserService.editUserWithRole(user, roles);
                     if (ObjectUtils.isNotEmpty(roles)) {
-                        //如果角色包含viewer，设置责任领域权限和品牌权限
+                        //如果角色包含viewer或者品牌管理员，设置责任领域权限和品牌权限
                         sysUserService.setBrandAndDutyTerritory(roles, dutyTerritories, brands, projectUserBrandList, projectUserDutyTerritoryList, id);
                     }
                     result.success("修改成功!");
diff --git a/jero-boot/jero-boot-module-system/src/main/java/com/jero/modules/system/enums/ProjectRoleEnum.java b/jero-boot/jero-boot-module-system/src/main/java/com/jero/modules/system/enums/ProjectRoleEnum.java
index 4a62c010a..aaf412c9c 100644
--- a/jero-boot/jero-boot-module-system/src/main/java/com/jero/modules/system/enums/ProjectRoleEnum.java
+++ b/jero-boot/jero-boot-module-system/src/main/java/com/jero/modules/system/enums/ProjectRoleEnum.java
@@ -11,6 +11,7 @@ public enum ProjectRoleEnum {
     MANAGER("领导角色","11","R&H Manager"),
     ADMIN("系统管理员","12","admin"),
     VIEWER("Viewer","13","Viewer"),
+    BRAND_ADMINISTRATOR("品牌管理员","14","BrandAdministrator"),
     ;
 
     String name;
diff --git a/jero-boot/jero-boot-module-system/src/main/java/com/jero/modules/system/service/impl/SysUserServiceImpl.java b/jero-boot/jero-boot-module-system/src/main/java/com/jero/modules/system/service/impl/SysUserServiceImpl.java
index 798a03a79..7fa268907 100644
--- a/jero-boot/jero-boot-module-system/src/main/java/com/jero/modules/system/service/impl/SysUserServiceImpl.java
+++ b/jero-boot/jero-boot-module-system/src/main/java/com/jero/modules/system/service/impl/SysUserServiceImpl.java
@@ -713,28 +713,33 @@ public class SysUserServiceImpl extends ServiceImpl<SysUserMapper, SysUser> impl
 	@Override
 	public void setBrandAndDutyTerritory(String roles, String dutyTerritories, String brands, List<ProjectUserBrand> projectUserBrandList, List<ProjectUserDutyTerritory> projectUserDutyTerritoryList, String userId) {
 		List<String> roleCodes = sysRoleService.listByIds(Arrays.asList(roles.split(","))).stream().map(SysRole::getRoleCode).collect(Collectors.toList());
-		if (roleCodes.contains(ProjectRoleEnum.VIEWER.getCode())) {
+		if (roleCodes.contains(ProjectRoleEnum.VIEWER.getCode())||roleCodes.contains(ProjectRoleEnum.BRAND_ADMINISTRATOR.getCode())) {
+			//viewer设置对应的责任领域
+			if (roleCodes.contains(ProjectRoleEnum.VIEWER.getCode())) {
+				//先删后加
+				QueryWrapper<ProjectUserDutyTerritory> dutyTerritoryDelWrapper = new QueryWrapper<>();
+				dutyTerritoryDelWrapper.eq("user_id", userId);
+				projectUserDutyTerritoryService.remove(dutyTerritoryDelWrapper);
+
+				if (ObjectUtils.isNotEmpty(dutyTerritories)) {
+					List<String> dictItemValue = sysDictItemService.selectItemsByDictCode(DictCodeEnum.DUTY_TERRITORY.getValue()).stream().map(SysDictItem::getItemValue).collect(Collectors.toList());
+					for (String dutyTerritory : dutyTerritories.split(",")) {
+						//校验责任领域是否合法
+						if (dictItemValue.contains(dutyTerritory)) {
+							ProjectUserDutyTerritory projectUserDutyTerritory = new ProjectUserDutyTerritory();
+							projectUserDutyTerritory.setUserId(userId);
+							projectUserDutyTerritory.setDutyTerritory(dutyTerritory);
+							projectUserDutyTerritoryList.add(projectUserDutyTerritory);
+						}
+					}
+				}
+			}
+
 			//先删后加
 			QueryWrapper<ProjectUserBrand> brandDelWrapper = new QueryWrapper<>();
 			brandDelWrapper.eq("user_id", userId);
 			projectUserBrandService.remove(brandDelWrapper);
 
-			QueryWrapper<ProjectUserDutyTerritory> dutyTerritoryDelWrapper = new QueryWrapper<>();
-			dutyTerritoryDelWrapper.eq("user_id", userId);
-			projectUserDutyTerritoryService.remove(dutyTerritoryDelWrapper);
-
-			if (ObjectUtils.isNotEmpty(dutyTerritories)) {
-				List<String> dictItemValue = sysDictItemService.selectItemsByDictCode(DictCodeEnum.DUTY_TERRITORY.getValue()).stream().map(SysDictItem::getItemValue).collect(Collectors.toList());
-				for (String dutyTerritory : dutyTerritories.split(",")) {
-					//校验责任领域是否合法
-					if (dictItemValue.contains(dutyTerritory)) {
-						ProjectUserDutyTerritory projectUserDutyTerritory = new ProjectUserDutyTerritory();
-						projectUserDutyTerritory.setUserId(userId);
-						projectUserDutyTerritory.setDutyTerritory(dutyTerritory);
-						projectUserDutyTerritoryList.add(projectUserDutyTerritory);
-					}
-				}
-			}
 			if (ObjectUtils.isNotEmpty(brands)) {
 				List<String> dictItemValue = sysDictItemService.selectItemsByDictCode(DictCodeEnum.BRAND.getValue()).stream().map(SysDictItem::getItemValue).collect(Collectors.toList());
 				for (String brand : brands.split(",")) {
diff --git a/jero-boot/jero-boot-modules/src/main/java/com/jero/modules/project/service/impl/ProjectLibraryBaseServiceImpl.java b/jero-boot/jero-boot-modules/src/main/java/com/jero/modules/project/service/impl/ProjectLibraryBaseServiceImpl.java
index ece289216..6e0e0c9b8 100644
--- a/jero-boot/jero-boot-modules/src/main/java/com/jero/modules/project/service/impl/ProjectLibraryBaseServiceImpl.java
+++ b/jero-boot/jero-boot-modules/src/main/java/com/jero/modules/project/service/impl/ProjectLibraryBaseServiceImpl.java
@@ -768,14 +768,12 @@ public class ProjectLibraryBaseServiceImpl extends ServiceImpl<ProjectLibraryBas
 		LoginUser loginUser = (LoginUser) SecurityUtils.getSubject().getPrincipal();
 		Integer isViewer = sysRoleMapper.queryUserRoleByCode(ProjectRoleEnum.VIEWER.getName(), loginUser.getId());
 		Set<String> projectIds = new HashSet<>();
-		if (isViewer == 0) {
-			//如果不是viewer，根据项目用户权限表筛选有权访问的项目
-			Set<String> permissionProjectIds = projectUserPermissionMapper.queryProjectIdsByUserId(loginUser.getId());
-			if (ObjectUtils.isNotEmpty(permissionProjectIds)) {
-				projectIds.addAll(permissionProjectIds);
-			}
+		//根据项目用户权限表筛选有权访问的项目
+		Set<String> permissionProjectIds = projectUserPermissionMapper.queryProjectIdsByUserId(loginUser.getId());
+		if (ObjectUtils.isNotEmpty(permissionProjectIds)) {
+			projectIds.addAll(permissionProjectIds);
 		}
-		if (sysRoleMapper.queryUserRoleByCode(ProjectRoleEnum.VIEWER.getName(), loginUser.getId()) > 0) {
+		if (isViewer > 0) {
 			//如果是Viewer，根据配置的品牌筛选可访问的项目
 			List<String> brandTypes = projectUserBrandService.queryBrandByUserId(loginUser.getId());
 			if (ObjectUtils.isNotEmpty(brandTypes)) {
@@ -874,10 +872,10 @@ public class ProjectLibraryBaseServiceImpl extends ServiceImpl<ProjectLibraryBas
 
 		List<ProjectLibraryBase> resultLast = new ArrayList<>();
 		//权限筛选
-		if (isViewer != 0) {
+		if (sysUserService.isAdministrator()) {
+			//管理员查看全部
 			resultLast = resultNew;
 		} else {
-
 			for (ProjectLibraryBase projectLibraryBase : resultNew) {
 				//判断父项目是否有权限
 				if (!projectIds.contains(projectLibraryBase.getId())) {
diff --git a/jero-web/src/views/projectManagement/components/ProjectDetails.vue b/jero-web/src/views/projectManagement/components/ProjectDetails.vue
index ad4a6cc24..bed29b591 100644
--- a/jero-web/src/views/projectManagement/components/ProjectDetails.vue
+++ b/jero-web/src/views/projectManagement/components/ProjectDetails.vue
@@ -61,10 +61,8 @@
       </div>
       <div class="text-field">
         <span class="text-field-left" :title="$t('brand')">{{$t('brand')}}</span>
-        <span class="text-field-right text-field-right-color"
-              :title="queryForm.brand"
-              @click="urlClick(queryForm.brand)"
-        >{{queryForm.brand}}</span>
+        <span class="text-field-right" :title="queryForm.brandText"
+        >{{queryForm.brandText}}</span>
       </div>
     </div>
     <div class="content-text">
diff --git a/jero-web/src/views/projectManagement/projectLibrary/index.vue b/jero-web/src/views/projectManagement/projectLibrary/index.vue
index 8615a0fe6..444b1ee8b 100644
--- a/jero-web/src/views/projectManagement/projectLibrary/index.vue
+++ b/jero-web/src/views/projectManagement/projectLibrary/index.vue
@@ -246,7 +246,7 @@
             align: 'center',
             width: 180,
             ellipsis: true,
-            dataIndex: 'brand'
+            dataIndex: 'brandText'
           },
           {
             title: this.$t('explain'),
diff --git a/jero-web/src/views/system/modules/UserModal.vue b/jero-web/src/views/system/modules/UserModal.vue
index 7a36a6149..638fdd786 100644
--- a/jero-web/src/views/system/modules/UserModal.vue
+++ b/jero-web/src/views/system/modules/UserModal.vue
@@ -85,7 +85,7 @@
 <!--                              :placeholder="$t('PleaseSelect')+$t('brand')"-->
 <!--                              :type="'select'"-->
 <!--                              :triggerChange="false" :dictCode="'brand'"/>-->
-          <a-select allowClear :placeholder="$t('PleaseSelect')+$t('engineeringInterfacePerson')"
+          <a-select allowClear mode="multiple" :getPopupContainer="triggerNode => triggerNode.parentNode" :placeholder="$t('PleaseSelect')+$t('engineeringInterfacePerson')"
                     v-model="brands">
             <a-select-option v-for="(item, key) in brandsList" :key="key" :value="item.value">
                     <span style="display: inline-block;width: 100%" :title=" item.text || item.label ">
@@ -413,7 +413,7 @@
             this.dutyTerritories = res.dutyTerritoryList.join(',')
           }
           if(res.brandList){
-            this.brands = res.brandList.join(',')
+            this.brands = res.brandList
           }
         })
       },
@@ -462,7 +462,7 @@
           ids: this.userId,
           selectedroles: selectedRole.join(','),
           dutyTerritories: dutyTerritorie,
-          brands: brand
+          brands: brand.join(',')
         }
         postAction('/sys/user/setRole', param).then((res) => {
           if (res.success) {