Add node duplication

Author: bram2w

backend/src/baserow/contrib/automation/nodes/actions.py

@@ -264,7 +264,9 @@ def undo(
         action_to_undo: Action,
     ):
         # Trash the duplicated node.
-        AutomationNodeService().delete_node(user, params.duplicated_node_id)
+        AutomationNodeService().delete_node(
+            user, params.duplicated_node_id, ignore_user_for_signal=True
+        )
 
     @classmethod
     def redo(

backend/src/baserow/contrib/automation/nodes/service.py

@@ -241,9 +241,7 @@ def update_node(
         )
 
     def delete_node(
-        self,
-        user: AbstractUser,
-        node_id: int,
+        self, user: AbstractUser | None, node_id: int, ignore_user_for_signal=False
     ) -> AutomationNode:
         """
         Deletes the specified automation node.
@@ -268,7 +266,7 @@ def delete_node(
         node.get_type().before_delete(node.specific)
 
         TrashHandler.trash(
-            user,
+            user if not ignore_user_for_signal else None,
             automation.workspace,
             automation,
             node,
@@ -278,7 +276,7 @@ def delete_node(
             self,
             workflow=workflow,
             node_id=node.id,
-            user=user,
+            user=user if not ignore_user_for_signal else None,
         )
 
         return node

web-frontend/modules/automation/components/workflow/WorkflowEdge.vue

@@ -50,6 +50,7 @@
       @remove-node="emit('remove-node', $event)"
       @replace-node="emit('replace-node', $event)"
       @move-node="emit('move-node', $event)"
+      @duplicate-node="emit('duplicate-node', $event)"
     />
   </div>
 </template>
@@ -90,7 +91,14 @@ const props = defineProps({
   },
 })
 
-const emit = defineEmits(['add-node', 'select-node', 'move-node'])
+const emit = defineEmits([
+  'add-node',
+  'select-node',
+  'move-node',
+  'remove-node',
+  'replace-node',
+  'duplicate-node',
+])
 
 const store = useStore()
 const workflow = inject('workflow')

web-frontend/modules/automation/components/workflow/WorkflowEditor.vue

@@ -29,6 +29,7 @@
         @replace-node="emit('replace-node', $event)"
         @select-node="emit('input', $event.id)"
         @move-node="emit('move-node', $event)"
+        @duplicate-node="emit('duplicate-node', $event)"
       />
       <template v-else>
         <div class="workflow-editor__trigger-selector" @scroll.stop>
@@ -72,7 +73,13 @@ const props = defineProps({
 })
 
 const vueFlowEdges = []
-const emit = defineEmits(['add-node', 'remove-node', 'input', 'move-node'])
+const emit = defineEmits([
+  'add-node',
+  'remove-node',
+  'input',
+  'move-node',
+  'duplicate-node',
+])
 
 const { onPaneClick } = useVueFlow()
 

web-frontend/modules/automation/components/workflow/WorkflowNode.vue

@@ -14,6 +14,7 @@
       @select-node="emit('select-node', $event)"
       @remove-node="emit('remove-node', $event)"
       @replace-node="emit('replace-node', $event)"
+      @duplicate-node="emit('duplicate-node', $event)"
     />
     <div v-if="nodeType.isContainer" class="workflow-node__children">
       <div ref="children" class="workflow-node__children-wrapper">
@@ -67,6 +68,7 @@
         @remove-node="emit('remove-node', $event)"
         @replace-node="emit('replace-node', $event)"
         @move-node="emit('move-node', $event)"
+        @duplicate-node="emit('duplicate-node', $event)"
       />
     </div>
   </div>
@@ -111,6 +113,7 @@ const emit = defineEmits([
   'remove-node',
   'replace-node',
   'move-node',
+  'duplicate-node',
 ])
 
 const { app } = useContext()

web-frontend/modules/automation/components/workflow/WorkflowNodeContent.vue

@@ -75,7 +75,17 @@
             {{ $t('workflowNode.moreReplace') }}
           </a>
         </li>
-        <li class="context__menu-item">
+        <li v-if="canBeDuplicated" class="context__menu-item">
+          <a
+            role="button"
+            class="context__menu-item-link"
+            @click="emit('duplicate-node', node.id)"
+          >
+            <i class="context__menu-item-icon iconoir-copy"></i>
+            {{ $t('workflowNode.moreDuplicate') }}
+          </a>
+        </li>
+        <li class="context__menu-item context__menu-item--with-separator">
           <a
             :key="getDeleteErrorMessage"
             v-tooltip="getDeleteErrorMessage || null"
@@ -131,7 +141,12 @@ const props = defineProps({
   },
 })
 
-const emit = defineEmits(['remove-node', 'replace-node', 'select-node'])
+const emit = defineEmits([
+  'remove-node',
+  'replace-node',
+  'select-node',
+  'duplicate-node',
+])
 
 const isDragging = ref(false)
 
@@ -277,4 +292,11 @@ const getDataBeforeLabel = computed(() => {
     output,
   })
 })
+
+const canBeDuplicated = computed(() => {
+  return nodeType.value.isDuplicable({
+    workflow: workflow.value,
+    node: props.node,
+  })
+})
 </script>

web-frontend/modules/automation/locales/en.json

@@ -132,7 +132,9 @@
     "beforeLabelRepeat": "For each item",
     "moreEdit": "Edit",
     "moreReplace": "Replace",
-    "nodeOptions": "Node options"
+    "nodeOptions": "Node options",
+    "displayLabelDebug": "ID: {id} | Prev: {previousNodeId} | {outputUid}",
+    "moreDuplicate": "Duplicate"
   },
   "workflowAddNode": {
     "displayTitle": "Create automation node"

web-frontend/modules/automation/nodeTypeMixins.js

@@ -20,6 +20,10 @@ export const TriggerNodeTypeMixin = (Base) =>
     getDeleteErrorMessage({ workflow, node }) {
       return this.app.i18n.t('nodeType.triggerDeletionError')
     }
+
+    isDuplicable({ workflow, node }) {
+      return false
+    }
   }
 
 export const ActionNodeTypeMixin = (Base) =>

web-frontend/modules/automation/nodeTypes.js

@@ -158,6 +158,13 @@ export class NodeType extends Registerable {
     return Boolean(this.getDeleteErrorMessage({ workflow, node }))
   }
 
+  /**
+   * Returns whether the given node can be duplicated.
+   */
+  isDuplicable({ workflow, node }) {
+    return true
+  }
+
   /**
    * Returns the error message we should show when a node cannot be deleted.
    * By default, this method is empty, but can be overridden by the node type
@@ -655,6 +662,10 @@ export class CoreIteratorNodeType extends containerNodeTypeMixin(
     }
     return ''
   }
+
+  isDuplicable({ workflow, node }) {
+    return false
+  }
 }
 
 export class CoreSMTPEmailNodeType extends ActionNodeTypeMixin(NodeType) {
@@ -805,6 +816,10 @@ export class CoreRouterNodeType extends ActionNodeTypeMixin(
       },
     ]
   }
+
+  isDuplicable({ workflow, node }) {
+    return false
+  }
 }
 
 export class AIAgentActionNodeType extends ActionNodeTypeMixin(NodeType) {

web-frontend/modules/automation/pages/automationWorkflow.vue

@@ -27,6 +27,7 @@
             @remove-node="handleRemoveNode"
             @replace-node="handleReplaceNode"
             @move-node="handleMoveNode"
+            @duplicate-node="handleDuplicateNode"
           />
         </client-only>
       </div>
@@ -265,6 +266,12 @@ export default {
         notifyIf(err, 'automation')
       }
     },
+    async handleDuplicateNode(nodeId) {
+      await this.$store.dispatch('automationWorkflowNode/duplicate', {
+        workflow: this.workflow,
+        nodeId,
+      })
+    },
   },
 }
 </script>