Conversation
merge latest from fork source
…参数不符合系统要求，需要参照具体API接口说明，微信原始报文：{"errcode":40058,"errmsg":"invalid Request Parameter
…rvice.java Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…pHrServiceImpl.java Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…rvice.java Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
🤖 Augment PR SummarySummary: Updates the WeCom HR (人事助手) staff info wrapper to match recent API changes. Changes: Tweaks interface docs and extends the 🤖 Was this summary useful? React with 👍 or 👎 |
![augmentcode[bot]](https://avatars.githubusercontent.com/in/1027498?s=60&v=4){kind=small}
|
|
||
| /** | ||
| * 获取员工档案数据. | ||
| * <p> |
There was a problem hiding this comment.
This newly-added Javadoc for hr/get_staff_info appears to be attached to the updateEmployeeFieldInfo section (and 权限说明： is left blank), so the interface docs read inconsistently/misleadingly.
Severity: low
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
|
|
||
| @Override | ||
| public WxCpHrEmployeeFieldDataResp getEmployeeFieldInfo(String userid, boolean getAll, List<String> fields) throws WxErrorException { | ||
| if (userid == null || userid.trim().isEmpty()) { |
There was a problem hiding this comment.
WxCpHrService currently doesn’t declare getEmployeeFieldInfo(String, boolean, List), so this @Override overload won’t compile / won’t be reachable via the interface type.
Severity: high
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
| @Override | ||
| public WxCpHrEmployeeFieldDataResp getEmployeeFieldInfo(String userid, List<String> fields) throws WxErrorException { | ||
| return getEmployeeFieldInfo(userid,false,fields); | ||
| } |
There was a problem hiding this comment.
Defaulting the 2-arg getEmployeeFieldInfo to getAll=false means callers passing null/empty fields will send get_all=false without a field list; the official get_staff_info doc indicates the field list must be non-empty in that case, so this may start returning errors.
Severity: medium
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
|
啥也没改吗？ |
|
忽略这个PR, 你的类结构很复杂哦，和微信官方接口的对应关系搞不清楚，还是大佬自己来吧 |
2 participants
merge latest 4.8.2B