fix datasetLabel must be non-empty bug

WeBankBlockchain · Nov 6, 2024 · 15d4423 · 15d4423
1 parent 3503508
commit 15d4423
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/...taset/src/main/java/com/webank/wedpr/components/dataset/controller/DatasetController.java b/...taset/src/main/java/com/webank/wedpr/components/dataset/controller/DatasetController.java
@@ -400,14 +400,14 @@ public WeDPRResponse updateDatasetList(
 
             for (UpdateDatasetRequest updateDatasetRequest : updateDatasetRequestList) {
                 String datasetTitle = updateDatasetRequest.getDatasetTitle();
-                String datasetLabel = updateDatasetRequest.getDatasetLabel();
+                // String datasetLabel = updateDatasetRequest.getDatasetLabel();
                 String datasetDesc = updateDatasetRequest.getDatasetDesc();
                 String datasetId = updateDatasetRequest.getDatasetId();
                 String approvalChain = updateDatasetRequest.getApprovalChain();
 
                 Common.requireNonEmpty("datasetId", datasetId);
                 Common.requireNonEmpty("datasetTitle", datasetTitle);
-                Common.requireNonEmpty("datasetLabel", datasetLabel);
+                // Common.requireNonEmpty("datasetLabel", datasetLabel);
                 Common.requireNonEmpty("datasetDesc", datasetDesc);
 
                 Integer datasetVisibility = updateDatasetRequest.getDatasetVisibility();

diff --git a/wedpr-components/db-mapper/dataset/src/main/resources/mapper/DatasetMapper.xml b/wedpr-components/db-mapper/dataset/src/main/resources/mapper/DatasetMapper.xml
@@ -249,7 +249,9 @@
             wedpr_dataset
         SET
             dataset_title = #{datasetTitle},
+        <if test="datasetLabel != null and datasetLabel.trim() != ''">
             dataset_label = #{datasetLabel},
+        </if>
             dataset_desc = #{datasetDesc},
             visibility = #{visibility},
             visibility_details = #{visibilityDetails},