交易市场平台（供应商）

baqing-admin/src/main/java/com/ruoyi/web/modules/farming/service/impl/BizTechResourceServiceImpl.java

@@ -208,6 +208,7 @@ public class BizTechResourceServiceImpl implements IBizTechResourceService
         row.setAccountAssigned(TechResourceRules.ACCOUNT_ASSIGNED_YES);
         row.setSysUserId(user.getUserId());
         row.setAssignedLoginName(user.getUserName());
+        row.setUpdateBy(operator);
         return bizTechResourceMapper.updateBizTechResource(row);
     }
 

baqing-admin/src/main/java/com/ruoyi/web/modules/farming/support/TechResourceAccountAssigner.java

@@ -2,16 +2,18 @@ package com.ruoyi.web.modules.farming.support;
 
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Component;
+import com.ruoyi.common.core.domain.entity.SysRole;
 import com.ruoyi.common.core.domain.entity.SysUser;
 import com.ruoyi.common.exception.ServiceException;
 import com.ruoyi.common.utils.SecurityUtils;
 import com.ruoyi.common.utils.StringUtils;
+import com.ruoyi.system.mapper.SysRoleMapper;
 import com.ruoyi.system.service.ISysConfigService;
 import com.ruoyi.system.service.ISysUserService;
 import com.ruoyi.web.modules.farming.domain.BizTechResource;
 
 /**
- * 为专家资源创建平台账号。
+ * 为专家资源创建平台账号并绑定角色。
  */
 @Component
 public class TechResourceAccountAssigner
@@ -22,6 +24,9 @@ public class TechResourceAccountAssigner
     @Autowired
     private ISysConfigService configService;
 
+    @Autowired
+    private SysRoleMapper sysRoleMapper;
+
     public SysUser createAccount(BizTechResource resource, String operator)
     {
         String loginName = TechResourceLoginNameGenerator.generate(
@@ -44,7 +49,7 @@ public class TechResourceAccountAssigner
         user.setPassword(SecurityUtils.encryptPassword(initPwd));
         user.setStatus("0");
         user.setCreateBy(operator);
-        user.setRoleIds(new Long[] { TechResourceRules.ROLE_ID_EXPERT });
+        user.setRoleIds(new Long[] { resolveRoleId() });
         int rows = sysUserService.insertUser(user);
         if (rows <= 0 || user.getUserId() == null)
         {
@@ -67,4 +72,15 @@ public class TechResourceAccountAssigner
         }
         sysUserService.deleteUserById(resource.getSysUserId());
     }
+
+    private long resolveRoleId()
+    {
+        SysRole role = sysRoleMapper.checkRoleKeyUnique(TechResourceRules.ROLE_KEY_EXPERT);
+        if (role == null || role.getRoleId() == null)
+        {
+            throw new ServiceException(String.format(TechResourceRules.MSG_ROLE_NOT_FOUND,
+                    TechResourceRules.ROLE_KEY_EXPERT));
+        }
+        return role.getRoleId();
+    }
 }

baqing-admin/src/main/java/com/ruoyi/web/modules/farming/support/TechResourceLoginNameGenerator.java

@@ -1,88 +1,18 @@
 package com.ruoyi.web.modules.farming.support;
 
-import net.sourceforge.pinyin4j.PinyinHelper;
-import net.sourceforge.pinyin4j.format.HanyuPinyinCaseType;
-import net.sourceforge.pinyin4j.format.HanyuPinyinOutputFormat;
-import net.sourceforge.pinyin4j.format.HanyuPinyinToneType;
-import net.sourceforge.pinyin4j.format.HanyuPinyinVCharType;
-import com.ruoyi.common.utils.StringUtils;
+import com.ruoyi.web.modules.trading.support.TradingPartnerLoginNameGenerator;
 
 /**
- * 分配账号默认登录名：专家姓名拼音（小写全拼）+ 资源 id。
+ * 分配账号默认登录名：专家姓名汉字拼音首字母（小写）+ 资源 id。
  */
 public final class TechResourceLoginNameGenerator
 {
-    private static final HanyuPinyinOutputFormat FORMAT = new HanyuPinyinOutputFormat();
-
-    static
-    {
-        FORMAT.setCaseType(HanyuPinyinCaseType.LOWERCASE);
-        FORMAT.setToneType(HanyuPinyinToneType.WITHOUT_TONE);
-        FORMAT.setVCharType(HanyuPinyinVCharType.WITH_V);
-    }
-
     private TechResourceLoginNameGenerator()
     {
     }
 
     public static String generate(String resourceName, Long id)
     {
-        if (id == null)
-        {
-            throw new IllegalArgumentException("id is required");
-        }
-        String base = toPinyinBase(StringUtils.nvl(resourceName, ""));
-        if (base.isEmpty())
-        {
-            base = "user";
-        }
-        return base + id;
-    }
-
-    static String toPinyinBase(String text)
-    {
-        if (StringUtils.isEmpty(text))
-        {
-            return "";
-        }
-        String trimmed = text.trim().replaceAll("\\s+", "");
-        StringBuilder sb = new StringBuilder();
-        for (int i = 0; i < trimmed.length(); i++)
-        {
-            char c = trimmed.charAt(i);
-            if (c <= 127 && Character.isLetterOrDigit(c))
-            {
-                sb.append(Character.toLowerCase(c));
-            }
-            else if (isChinese(c))
-            {
-                appendPinyin(sb, c);
-            }
-        }
-        return sb.toString();
-    }
-
-    private static void appendPinyin(StringBuilder sb, char c)
-    {
-        try
-        {
-            String[] arr = PinyinHelper.toHanyuPinyinStringArray(c, FORMAT);
-            if (arr != null && arr.length > 0 && StringUtils.isNotEmpty(arr[0]))
-            {
-                sb.append(arr[0]);
-            }
-        }
-        catch (Exception ex)
-        {
-            // skip
-        }
-    }
-
-    private static boolean isChinese(char c)
-    {
-        Character.UnicodeBlock block = Character.UnicodeBlock.of(c);
-        return block == Character.UnicodeBlock.CJK_UNIFIED_IDEOGRAPHS
-                || block == Character.UnicodeBlock.CJK_COMPATIBILITY_IDEOGRAPHS
-                || block == Character.UnicodeBlock.CJK_UNIFIED_IDEOGRAPHS_EXTENSION_A;
+        return TradingPartnerLoginNameGenerator.generate(resourceName, id);
     }
 }

baqing-admin/src/main/java/com/ruoyi/web/modules/farming/support/TechResourceRules.java

@@ -50,6 +50,14 @@ public final class TechResourceRules
 
     public static final String DEL_FLAG_DELETED = "2";
 
+    /** 专家平台角色权限字符（sys_role.role_key） */
+    public static final String ROLE_KEY_EXPERT = "exp";
+
+    public static final String MSG_ROLE_NOT_FOUND = "未配置权限字符为「%s」的角色，无法分配账号";
+
+    /**
+     * 历史常量；分配账号请使用 {@link #ROLE_KEY_EXPERT} 解析角色。
+     */
     public static final long ROLE_ID_EXPERT = 101L;
 
     public static final int MIN_NAME_LEN = 1;

baqing-admin/src/test/java/com/ruoyi/web/modules/farming/service/impl/BizTechResourceServiceImplTest.java

@@ -145,7 +145,7 @@ class BizTechResourceServiceImplTest
         when(mapper.selectBizTechResourceById(1L)).thenReturn(db);
         SysUser user = new SysUser();
         user.setUserId(100L);
-        user.setUserName("zhangzhuanjia1");
+        user.setUserName("zzj1");
         when(accountAssigner.createAccount(any(), any())).thenReturn(user);
         when(mapper.updateBizTechResource(any())).thenReturn(1);
         assertEquals(1, service.assignAccount(1L, "admin"));

baqing-admin/src/test/java/com/ruoyi/web/modules/farming/support/TechResourceAccountAssignerTest.java

@@ -0,0 +1,73 @@
+package com.ruoyi.web.modules.farming.support;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertThrows;
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.Mockito.verify;
+import static org.mockito.Mockito.when;
+
+import com.ruoyi.common.core.domain.entity.SysRole;
+import com.ruoyi.common.core.domain.entity.SysUser;
+import com.ruoyi.common.exception.ServiceException;
+import com.ruoyi.system.mapper.SysRoleMapper;
+import com.ruoyi.system.service.ISysConfigService;
+import com.ruoyi.system.service.ISysUserService;
+import com.ruoyi.web.modules.farming.TechResourceTestSamples;
+import com.ruoyi.web.modules.farming.domain.BizTechResource;
+import org.junit.jupiter.api.DisplayName;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
+import org.mockito.InjectMocks;
+import org.mockito.Mock;
+import org.mockito.junit.jupiter.MockitoExtension;
+
+@ExtendWith(MockitoExtension.class)
+@DisplayName("TechResourceAccountAssigner")
+class TechResourceAccountAssignerTest
+{
+    @Mock
+    private ISysUserService sysUserService;
+
+    @Mock
+    private ISysConfigService configService;
+
+    @Mock
+    private SysRoleMapper sysRoleMapper;
+
+    @InjectMocks
+    private TechResourceAccountAssigner assigner;
+
+    @Test
+    @DisplayName("专家绑定 exp 角色")
+    void createAccountExpert()
+    {
+        BizTechResource resource = TechResourceTestSamples.sampleExpertDraft();
+        resource.setId(12L);
+        resource.setResourceName("张专家");
+        SysRole role = new SysRole();
+        role.setRoleId(101L);
+        when(sysRoleMapper.checkRoleKeyUnique(TechResourceRules.ROLE_KEY_EXPERT)).thenReturn(role);
+        when(sysUserService.checkUserNameUnique(any())).thenReturn(true);
+        when(configService.selectConfigByKey("sys.user.initPassword")).thenReturn("123456");
+        when(sysUserService.insertUser(any())).thenAnswer(inv -> {
+            SysUser u = inv.getArgument(0);
+            u.setUserId(1L);
+            return 1;
+        });
+
+        SysUser user = assigner.createAccount(resource, "admin");
+        assertEquals("zzj12", user.getUserName());
+        verify(sysRoleMapper).checkRoleKeyUnique(TechResourceRules.ROLE_KEY_EXPERT);
+    }
+
+    @Test
+    @DisplayName("角色不存在")
+    void roleMissing()
+    {
+        BizTechResource resource = TechResourceTestSamples.sampleExpertDraft();
+        resource.setId(1L);
+        when(sysUserService.checkUserNameUnique(any())).thenReturn(true);
+        when(sysRoleMapper.checkRoleKeyUnique(TechResourceRules.ROLE_KEY_EXPERT)).thenReturn(null);
+        assertThrows(ServiceException.class, () -> assigner.createAccount(resource, "admin"));
+    }
+}

baqing-admin/src/test/java/com/ruoyi/web/modules/farming/support/TechResourceLoginNameGeneratorTest.java

@@ -1,6 +1,6 @@
 package com.ruoyi.web.modules.farming.support;
 
-import static org.junit.jupiter.api.Assertions.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
@@ -10,10 +10,9 @@ class TechResourceLoginNameGeneratorTest
 {
 
     @Test
-    @DisplayName("拼音+id")
+    @DisplayName("拼音首字母+id")
     void generate()
     {
-        String login = TechResourceLoginNameGenerator.generate("张专家", 12L);
-        assertTrue(login.endsWith("12"));
+        assertEquals("zzj12", TechResourceLoginNameGenerator.generate("张专家", 12L));
     }
 }

baqing-admin/src/test/java/com/ruoyi/web/modules/farming/support/TechResourceRulesTest.java

@@ -1,6 +1,7 @@
 package com.ruoyi.web.modules.farming.support;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertFalse;
 
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
@@ -10,9 +11,10 @@ class TechResourceRulesTest
 {
 
     @Test
-    @DisplayName("专家分配账号默认角色 101")
-    void expertRoleId()
+    @DisplayName("专家分配账号角色权限字符")
+    void expertRoleKey()
     {
-        assertEquals(101L, TechResourceRules.ROLE_ID_EXPERT);
+        assertEquals("exp", TechResourceRules.ROLE_KEY_EXPERT);
+        assertFalse(TechResourceRules.canAssignAccount(TechResourceRules.RESOURCE_TYPE_ACHIEVEMENT));
     }
 }

baqing-admin/src/test/java/com/ruoyi/web/modules/farming/support/TechResourceValidationTest.java

@@ -77,4 +77,13 @@ class TechResourceValidationTest
         achievement.setId(1L);
         assertThrows(ServiceException.class, () -> TechResourceValidation.validateForAssignAccount(achievement));
     }
+
+    @Test
+    @DisplayName("已分配不可重复")
+    void assignAccountAlready()
+    {
+        BizTechResource expert = TechResourceTestSamples.sampleExpertDraft();
+        expert.setAccountAssigned(TechResourceRules.ACCOUNT_ASSIGNED_YES);
+        assertThrows(ServiceException.class, () -> TechResourceValidation.validateForAssignAccount(expert));
+    }
 }

doc/牧业养殖科技服务/畜牧科技资源管理/畜牧科技资源管理功能需求-草稿.md

@@ -12,7 +12,7 @@
 8. 发布后不允许编辑、删除；下架后可以再次编辑、删除。
 9. 支持查看对应资源类型的表单数据。
 10. 支持逻辑删除对应资源类型的科技资源；如该科技资源已分配账号，同时删除账号。
-11. 专家库有“分配账号”操作，账号名称默认为专家姓名的拼音+该条记录的数据库Id，角色分别默认为编号为101对应的角色；分配账号只允许操作一次。
+11. 专家库有“分配账号”操作，账号名称默认为专家姓名的拼音+该条记录的数据库Id，角色分别默认为权限字符为exp对应的角色；分配账号只允许操作一次。
 12. 共享仪器库列表有“地图标点”操作。
 13. 地图标点：暂不实现。
 14. 异常返回：统一格式 code、msg、data。