diff --git a/core/core-frontend/src/utils/utils.ts b/core/core-frontend/src/utils/utils.ts
index e67cef41ee..ceb6eb0921 100644
--- a/core/core-frontend/src/utils/utils.ts
+++ b/core/core-frontend/src/utils/utils.ts
@@ -245,7 +245,7 @@ export const isFreeFolder = (node, flag) => {
   const freeRootId = (Number(oid) + flag).toString()
   let cNode = node
   while (cNode) {
-    const data = node.data
+    const data = cNode.data
     const id = data['id']
     if (id === freeRootId) {
       return true
diff --git a/core/core-frontend/src/views/common/DeResourceTree.vue b/core/core-frontend/src/views/common/DeResourceTree.vue
index e657ef120b..b2223ce75f 100644
--- a/core/core-frontend/src/views/common/DeResourceTree.vue
+++ b/core/core-frontend/src/views/common/DeResourceTree.vue
@@ -541,14 +541,14 @@ const sortTypeChange = sortType => {
   state.curSortType = sortType
 }
 
-const proxyHandleDrop = (arg1, arg2, arg3) => {
-  const flagArray = ['dashboard', 'datav', 'dataset', 'datasource']
+const proxyAllowDrop = (arg1, arg2) => {
+  const flagArray = ['dashboard', 'dataV', 'dataset', 'datasource']
   const flag = flagArray.findIndex(item => item === curCanvasType.value)
   if (flag < 0 || !isFreeFolder(arg2, flag + 1)) {
-    handleDrop(arg1, arg2, arg3)
-    return
+    return allowDrop(arg1, arg2)
   }
   ElMessage.warning(t('free.save_error'))
+  return false
 }
 
 watch(filterText, val => {
@@ -697,8 +697,8 @@ defineExpose({
         @node-collapse="nodeCollapse"
         @node-click="nodeClick"
         @node-drag-start="handleDragStart"
-        :allow-drop="allowDrop"
-        @node-drop="proxyHandleDrop"
+        :allow-drop="proxyAllowDrop"
+        @node-drop="handleDrop"
         draggable
       >
         <template #default="{ node, data }">
diff --git a/core/core-frontend/src/views/visualized/data/dataset/index.vue b/core/core-frontend/src/views/visualized/data/dataset/index.vue
index 6e334f24fb..b041a8360b 100644
--- a/core/core-frontend/src/views/visualized/data/dataset/index.vue
+++ b/core/core-frontend/src/views/visualized/data/dataset/index.vue
@@ -783,14 +783,15 @@ const getMenuList = (val: boolean) => {
         }
       ].concat(menuList)
 }
-const proxyHandleDrop = (arg1, arg2, arg3) => {
-  const flagArray = ['dashboard', 'datav', 'dataset', 'datasource']
+
+const proxyAllowDrop = (arg1, arg2) => {
+  const flagArray = ['dashboard', 'dataV', 'dataset', 'datasource']
   const flag = flagArray.findIndex(item => item === 'dataset')
   if (flag < 0 || !isFreeFolder(arg2, flag + 1)) {
-    handleDrop(arg1, arg2, arg3)
-    return
+    return allowDrop(arg1, arg2)
   }
   ElMessage.warning(t('free.save_error'))
+  return false
 }
 </script>
 
@@ -901,8 +902,8 @@ const proxyHandleDrop = (arg1, arg2, arg3) => {
             expand-on-click-node
             highlight-current
             @node-drag-start="handleDragStart"
-            :allow-drop="allowDrop"
-            @node-drop="proxyHandleDrop"
+            :allow-drop="proxyAllowDrop"
+            @node-drop="handleDrop"
             draggable
             @node-expand="nodeExpand"
             @node-collapse="nodeCollapse"
diff --git a/core/core-frontend/src/views/visualized/data/datasource/index.vue b/core/core-frontend/src/views/visualized/data/datasource/index.vue
index 36719ca0c7..369165a53a 100644
--- a/core/core-frontend/src/views/visualized/data/datasource/index.vue
+++ b/core/core-frontend/src/views/visualized/data/datasource/index.vue
@@ -1009,14 +1009,15 @@ const loadInit = () => {
     state.curSortType = historyTreeSort
   }
 }
-const proxyHandleDrop = (arg1, arg2, arg3) => {
-  const flagArray = ['dashboard', 'datav', 'dataset', 'datasource']
+
+const proxyAllowDrop = (arg1, arg2) => {
+  const flagArray = ['dashboard', 'dataV', 'dataset', 'datasource']
   const flag = flagArray.findIndex(item => item === 'datasource')
   if (flag < 0 || !isFreeFolder(arg2, flag + 1)) {
-    handleDrop(arg1, arg2, arg3)
-    return
+    return allowDrop(arg1, arg2)
   }
   ElMessage.warning(t('free.save_error'))
+  return false
 }
 onMounted(() => {
   const dsId = wsCache.get('ds-info-id') || route.params.id
@@ -1160,8 +1161,8 @@ const getMenuList = (val: boolean) => {
             :data="state.datasourceTree"
             :props="defaultProps"
             @node-drag-start="handleDragStart"
-            :allow-drop="allowDrop"
-            @node-drop="proxyHandleDrop"
+            :allow-drop="proxyAllowDrop"
+            @node-drop="handleDrop"
             draggable
             @node-click="handleNodeClick"
           >