This is an automated email from the ASF dual-hosted git repository.
zihaoxiang pushed a commit to branch dev
in repository https://gitbox.apache.org/repos/asf/dolphinscheduler.git
The following commit(s) were added to refs/heads/dev by this push:
new f9db86559d [Fix] fix dingtalk at user Params (#16379)
f9db86559d is described below
commit f9db86559da282c0ad8ba48ced5196d905df2bcc
Author: zuo <[email protected]>
AuthorDate: Mon Aug 12 10:42:42 2024 +0800
[Fix] fix dingtalk at user Params (#16379)
---
.../plugin/alert/dingtalk/DingTalkAlertChannelFactory.java | 4 ++--
.../plugin/alert/dingtalk/DingTalkParamsConstants.java | 4 ++--
.../plugin/alert/dingtalk/DingTalkSender.java | 12 ++++++------
dolphinscheduler-ui/src/locales/en_US/security.ts | 2 +-
dolphinscheduler-ui/src/locales/zh_CN/security.ts | 2 +-
5 files changed, 12 insertions(+), 12 deletions(-)
diff --git
a/dolphinscheduler-alert/dolphinscheduler-alert-plugins/dolphinscheduler-alert-dingtalk/src/main/java/org/apache/dolphinscheduler/plugin/alert/dingtalk/DingTalkAlertChannelFactory.java
b/dolphinscheduler-alert/dolphinscheduler-alert-plugins/dolphinscheduler-alert-dingtalk/src/main/java/org/apache/dolphinscheduler/plugin/alert/dingtalk/DingTalkAlertChannelFactory.java
index 4bf8b59d64..85a03abf25 100644
---
a/dolphinscheduler-alert/dolphinscheduler-alert-plugins/dolphinscheduler-alert-dingtalk/src/main/java/org/apache/dolphinscheduler/plugin/alert/dingtalk/DingTalkAlertChannelFactory.java
+++
b/dolphinscheduler-alert/dolphinscheduler-alert-plugins/dolphinscheduler-alert-dingtalk/src/main/java/org/apache/dolphinscheduler/plugin/alert/dingtalk/DingTalkAlertChannelFactory.java
@@ -87,7 +87,7 @@ public final class DingTalkAlertChannelFactory implements
AlertChannelFactory {
.setRequired(false)
.build())
.build();
- InputParam atUserIdsParam = InputParam
+ InputParam atDingtalkIdsParam = InputParam
.newBuilder(DingTalkParamsConstants.NAME_DING_TALK_AT_USERIDS,
DingTalkParamsConstants.DING_TALK_AT_USERIDS)
.addValidate(Validate.newBuilder()
@@ -141,7 +141,7 @@ public final class DingTalkAlertChannelFactory implements
AlertChannelFactory {
.setType("password")
.build();
- return Arrays.asList(webHookParam, keywordParam, secretParam,
msgTypeParam, atMobilesParam, atUserIdsParam,
+ return Arrays.asList(webHookParam, keywordParam, secretParam,
msgTypeParam, atMobilesParam, atDingtalkIdsParam,
isAtAll, isEnableProxy, proxyParam, portParam, userParam,
passwordParam);
}
diff --git
a/dolphinscheduler-alert/dolphinscheduler-alert-plugins/dolphinscheduler-alert-dingtalk/src/main/java/org/apache/dolphinscheduler/plugin/alert/dingtalk/DingTalkParamsConstants.java
b/dolphinscheduler-alert/dolphinscheduler-alert-plugins/dolphinscheduler-alert-dingtalk/src/main/java/org/apache/dolphinscheduler/plugin/alert/dingtalk/DingTalkParamsConstants.java
index d68a1bde52..bf658cb562 100644
---
a/dolphinscheduler-alert/dolphinscheduler-alert-plugins/dolphinscheduler-alert-dingtalk/src/main/java/org/apache/dolphinscheduler/plugin/alert/dingtalk/DingTalkParamsConstants.java
+++
b/dolphinscheduler-alert/dolphinscheduler-alert-plugins/dolphinscheduler-alert-dingtalk/src/main/java/org/apache/dolphinscheduler/plugin/alert/dingtalk/DingTalkParamsConstants.java
@@ -40,8 +40,8 @@ public final class DingTalkParamsConstants {
static final String DING_TALK_AT_MOBILES = "$t('atMobiles')";
static final String NAME_DING_TALK_AT_MOBILES = "AtMobiles";
- static final String DING_TALK_AT_USERIDS = "$t('atUserIds')";
- static final String NAME_DING_TALK_AT_USERIDS = "AtUserIds";
+ static final String DING_TALK_AT_USERIDS = "$t('atDingtalkIds')";
+ static final String NAME_DING_TALK_AT_USERIDS = "AtDingtalkIds";
static final String DING_TALK_AT_ALL = "$t('isAtAll')";
static final String NAME_DING_TALK_AT_ALL = "IsAtAll";
diff --git
a/dolphinscheduler-alert/dolphinscheduler-alert-plugins/dolphinscheduler-alert-dingtalk/src/main/java/org/apache/dolphinscheduler/plugin/alert/dingtalk/DingTalkSender.java
b/dolphinscheduler-alert/dolphinscheduler-alert-plugins/dolphinscheduler-alert-dingtalk/src/main/java/org/apache/dolphinscheduler/plugin/alert/dingtalk/DingTalkSender.java
index 527e38cf77..467b74f0e8 100644
---
a/dolphinscheduler-alert/dolphinscheduler-alert-plugins/dolphinscheduler-alert-dingtalk/src/main/java/org/apache/dolphinscheduler/plugin/alert/dingtalk/DingTalkSender.java
+++
b/dolphinscheduler-alert/dolphinscheduler-alert-plugins/dolphinscheduler-alert-dingtalk/src/main/java/org/apache/dolphinscheduler/plugin/alert/dingtalk/DingTalkSender.java
@@ -67,7 +67,7 @@ public final class DingTalkSender {
private String msgType;
private final String atMobiles;
- private final String atUserIds;
+ private final String atDingtalkIds;
private final Boolean atAll;
private final Boolean enableProxy;
@@ -87,7 +87,7 @@ public final class DingTalkSender {
msgType = config.get(DingTalkParamsConstants.NAME_DING_TALK_MSG_TYPE);
atMobiles =
config.get(DingTalkParamsConstants.NAME_DING_TALK_AT_MOBILES);
- atUserIds =
config.get(DingTalkParamsConstants.NAME_DING_TALK_AT_USERIDS);
+ atDingtalkIds =
config.get(DingTalkParamsConstants.NAME_DING_TALK_AT_USERIDS);
atAll =
Boolean.valueOf(config.get(DingTalkParamsConstants.NAME_DING_TALK_AT_ALL));
enableProxy =
Boolean.valueOf(config.get(DingTalkParamsConstants.NAME_DING_TALK_PROXY_ENABLE));
@@ -271,8 +271,8 @@ public final class DingTalkSender {
builder.append(" ");
});
}
- if (org.apache.commons.lang3.StringUtils.isNotBlank(atUserIds)) {
- Arrays.stream(atUserIds.split(",")).forEach(value -> {
+ if (org.apache.commons.lang3.StringUtils.isNotBlank(atDingtalkIds)) {
+ Arrays.stream(atDingtalkIds.split(",")).forEach(value -> {
builder.append("@");
builder.append(value);
builder.append(" ");
@@ -297,12 +297,12 @@ public final class DingTalkSender {
org.apache.commons.lang3.StringUtils.isNotBlank(atMobiles) ?
atMobiles.split(",")
: new String[0];
String[] atUserArray =
- org.apache.commons.lang3.StringUtils.isNotBlank(atUserIds) ?
atUserIds.split(",")
+ org.apache.commons.lang3.StringUtils.isNotBlank(atDingtalkIds)
? atDingtalkIds.split(",")
: new String[0];
boolean isAtAll = Objects.isNull(atAll) ? false : atAll;
at.put("atMobiles", atMobileArray);
- at.put("atUserIds", atUserArray);
+ at.put("atDingtalkIds", atUserArray);
at.put("isAtAll", isAtAll);
items.put("at", at);
diff --git a/dolphinscheduler-ui/src/locales/en_US/security.ts
b/dolphinscheduler-ui/src/locales/en_US/security.ts
index 6a3dcf03dc..f43adb839e 100644
--- a/dolphinscheduler-ui/src/locales/en_US/security.ts
+++ b/dolphinscheduler-ui/src/locales/en_US/security.ts
@@ -263,7 +263,7 @@ export default {
AtSomeoneInRoom: 'At Someone In Room',
Destination: 'Destination',
AtMobiles: 'At User Mobiles',
- AtUserIds: 'At User Ids',
+ AtDingtalkIds: 'At User Ids',
MsgType: 'Msg Type',
// eslint-disable-next-line quotes
IsAtAll: "{'@'}All",
diff --git a/dolphinscheduler-ui/src/locales/zh_CN/security.ts
b/dolphinscheduler-ui/src/locales/zh_CN/security.ts
index 6ee469db60..463b18f33f 100644
--- a/dolphinscheduler-ui/src/locales/zh_CN/security.ts
+++ b/dolphinscheduler-ui/src/locales/zh_CN/security.ts
@@ -262,7 +262,7 @@ export default {
// eslint-disable-next-line quotes
AtMobiles: "被{'@'}人的手机号",
// eslint-disable-next-line quotes
- AtUserIds: "被{'@'}人的用户ID",
+ AtDingtalkIds: "被{'@'}人的用户ID",
MsgType: '消息类型',
// eslint-disable-next-line quotes
IsAtAll: "{'@'}所有人",