Browse Source

[Fix-5726] When we used the UI page, we found some problems such as parameter validation, parameter update shows success but actually work (#5727)

* enhance the validation in UI

* enchance form verifaction

* simplify disable condition
kyoty 3 years ago
parent
commit
1f0c67bfb7

+ 1 - 5
dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/service/impl/DataSourceServiceImpl.java

@@ -119,10 +119,6 @@ public class DataSourceServiceImpl extends BaseServiceImpl implements DataSource
      * updateProcessInstance datasource
      *
      * @param loginUser login user
-     * @param name data source name
-     * @param desc data source description
-     * @param type data source type
-     * @param parameter datasource parameters
      * @param id data source id
      * @return update result code
      */
@@ -163,7 +159,7 @@ public class DataSourceServiceImpl extends BaseServiceImpl implements DataSource
 
         Date now = new Date();
 
-        dataSource.setName(dataSource.getName().trim());
+        dataSource.setName(dataSourceParam.getName().trim());
         dataSource.setNote(dataSourceParam.getNote());
         dataSource.setUserName(loginUser.getUserName());
         dataSource.setType(dataSource.getType());

+ 1 - 1
dolphinscheduler-dao/src/main/resources/org/apache/dolphinscheduler/dao/mapper/UserMapper.xml

@@ -61,7 +61,7 @@
         <include refid="baseSqlV2">
             <property name="alias" value="u"/>
         </include>
-        ,
+        ,t.tenant_code,
         case when u.queue <![CDATA[ <> ]]>  '' then u.queue else q.queue_name end as queue_name
         from t_ds_user u
         left join t_ds_tenant t on u.tenant_id=t.id

+ 1 - 1
dolphinscheduler-ui/src/js/conf/home/pages/datasource/pages/list/_source/createDataSource.vue

@@ -21,7 +21,7 @@
         <m-list-box-f>
           <template slot="name"><strong>*</strong>{{$t('Datasource')}}</template>
           <template slot="content" size="small">
-              <el-select style="width: 100%;" v-model="type">
+              <el-select style="width: 100%;" v-model="type" :disabled="this.item.id">
                 <el-option v-for="item in datasourceTypeList" :key="item.value" :value="item.value" :label="item.label">
                 </el-option>
               </el-select>

+ 1 - 1
dolphinscheduler-ui/src/js/conf/home/pages/security/pages/warningGroups/_source/createWarning.vue

@@ -109,7 +109,7 @@
           this.$message.warning(`${i18n.$t('Please enter group name')}`)
           return false
         }
-        if (this.alertInstanceIds) {
+        if (this.alertInstanceIds.length === 0) {
           this.$message.warning(`${i18n.$t('Select Alarm plugin instance')}`)
           return false
         }

+ 4 - 0
dolphinscheduler-ui/src/js/conf/home/pages/user/pages/token/_source/createToken.vue

@@ -108,6 +108,10 @@
           this.$message.warning(`${i18n.$t('Please generate token')}`)
           return false
         }
+        if (!this.expireTime) {
+          this.$message.warning(`${i18n.$t('Please Select token')}`)
+          return false
+        }
         return true
       },
       _submit () {

+ 1 - 0
dolphinscheduler-ui/src/js/module/i18n/locale/en_US.js

@@ -579,6 +579,7 @@ export default {
   'There is no data for this period of time': 'There is no data for this period of time',
   'Worker addresses cannot be empty': 'Worker addresses cannot be empty',
   'Please generate token': 'Please generate token',
+  'Please Select token': 'Please select the expiration time of token',
   'Spark Version': 'Spark Version',
   TargetDataBase: 'target database',
   TargetTable: 'target table',

+ 1 - 0
dolphinscheduler-ui/src/js/module/i18n/locale/zh_CN.js

@@ -579,6 +579,7 @@ export default {
   'There is no data for this period of time': '该时间段无数据',
   'Worker addresses cannot be empty': 'Worker地址不能为空',
   'Please generate token': '请生成Token',
+  'Please Select token': '请选择Token失效时间',
   'Spark Version': 'Spark版本',
   TargetDataBase: '目标库',
   TargetTable: '目标表',