From 5f6249346074b0e71eb741bc70e61120cae714c1 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?J=C3=A9r=C3=A9mie=20Pardou?=
 <571533+jrmi@users.noreply.github.com>
Date: Mon, 30 Mar 2026 08:55:00 +0200
Subject: [PATCH] fix: automation template replacing editor content and other
 template bugs (#5049)

---
 .../enterprise/builder/elementTypes.spec.js   | 11 ++++++
 .../components/AutomationHeader.vue           | 21 +++--------
 .../components/workflow/WorkflowTemplate.vue  | 17 +++------
 .../automation/pages/automationWorkflow.vue   |  4 +--
 .../builder/components/page/PagePreview.vue   |  2 +-
 .../components/page/PageTemplateContent.vue   |  3 ++
 .../components/template/TemplateModal.vue     |  2 +-
 .../core/components/templateModal.spec.js     | 36 +++++++++++++++++++
 8 files changed, 62 insertions(+), 34 deletions(-)
 create mode 100644 enterprise/web-frontend/test/unit/enterprise/builder/elementTypes.spec.js
 create mode 100644 web-frontend/test/unit/core/components/templateModal.spec.js

diff --git a/enterprise/web-frontend/test/unit/enterprise/builder/elementTypes.spec.js b/enterprise/web-frontend/test/unit/enterprise/builder/elementTypes.spec.js
new file mode 100644
index 0000000000..ade6d34ab2
--- /dev/null
+++ b/enterprise/web-frontend/test/unit/enterprise/builder/elementTypes.spec.js
@@ -0,0 +1,11 @@
+describe('Enterprise builder element types', () => {
+  test('file input deactivation checks tolerate a missing workspace', () => {
+    const testApp = useNuxtApp()
+    const elementType = testApp.$registry.get('element', 'input_file')
+
+    expect(elementType.isDeactivatedReason({ workspace: undefined })).toBeNull()
+    expect(elementType.getDeactivatedClickModal({ workspace: undefined })).toBe(
+      null
+    )
+  })
+})
diff --git a/web-frontend/modules/automation/components/AutomationHeader.vue b/web-frontend/modules/automation/components/AutomationHeader.vue
index 6bc45b5770..ada8ca005a 100644
--- a/web-frontend/modules/automation/components/AutomationHeader.vue
+++ b/web-frontend/modules/automation/components/AutomationHeader.vue
@@ -114,8 +114,8 @@
             testRunEnabled
               ? $t('automationHeader.stopTestRun')
               : $t('automationHeader.startTestRun')
-          }}</Button
-        >
+          }}
+        </Button>
         <Button
           data-highlight="automation-publish"
           :loading="isPublishing"
@@ -159,21 +159,11 @@ export default defineComponent({
     const store = useStore()
     const app = useNuxtApp()
     const isDev = inject('isDev')
+    const workflow = inject('workflow')
 
     const debug = ref(false)
     const isPublishing = ref(false)
 
-    const workflow = inject('workflow')
-
-    const selectedWorkflow = computed(() => {
-      if (!props.automation) return null
-      try {
-        return store.getters['automationWorkflow/getSelected']
-      } catch {
-        return null
-      }
-    })
-
     const testRunDisabled = computed(() => {
       if (!workflow.value?.graph) {
         return true
@@ -207,12 +197,12 @@ export default defineComponent({
     })
 
     const publishedOn = computed(() => {
-      if (!selectedWorkflow.value?.published_on || isPublishing.value) {
+      if (!workflow.value?.published_on || isPublishing.value) {
         return null
       }
 
       return moment
-        .utc(selectedWorkflow.value.published_on)
+        .utc(workflow.value.published_on)
         .tz(getUserTimeZone())
         .format('MMM D, YYYY HH:mm:ss')
     })
@@ -313,7 +303,6 @@ export default defineComponent({
       isPublishing,
       isPaused,
       isDisabled,
-      selectedWorkflow,
       workflow,
       activeSidePanel,
       testRunDisabled,
diff --git a/web-frontend/modules/automation/components/workflow/WorkflowTemplate.vue b/web-frontend/modules/automation/components/workflow/WorkflowTemplate.vue
index 8b3b79bcb2..2267be6702 100644
--- a/web-frontend/modules/automation/components/workflow/WorkflowTemplate.vue
+++ b/web-frontend/modules/automation/components/workflow/WorkflowTemplate.vue
@@ -34,30 +34,21 @@ export default {
   },
   watch: {
     'pageValue.workflow.id': {
-      handler() {
+      handler(newValue) {
         this.loadData()
       },
       immediate: true,
     },
   },
-  unmounted() {
-    // Restore the current application to the selected application if any
-    this.$store.dispatch('userSourceUser/setCurrentApplication', {
-      application: this.$store.getters['application/getSelected'],
-    })
-  },
   methods: {
     async loadData() {
       this.loading = true
 
       try {
         const automation = this.pageValue.automation
-        const workflow = await this.$store.dispatch(
-          'automationWorkflow/selectById',
-          {
-            automation,
-            workflowId: this.pageValue.workflow.id,
-          }
+        const workflow = this.$store.getters['automationWorkflow/getById'](
+          automation,
+          this.pageValue.workflow.id
         )
 
         const automationApplicationType = this.$registry.get(
diff --git a/web-frontend/modules/automation/pages/automationWorkflow.vue b/web-frontend/modules/automation/pages/automationWorkflow.vue
index 9b1e3471b8..678cde5fd9 100644
--- a/web-frontend/modules/automation/pages/automationWorkflow.vue
+++ b/web-frontend/modules/automation/pages/automationWorkflow.vue
@@ -119,9 +119,7 @@ if (error.value) {
 // Computed properties from async data
 const automation = computed(() => pageData.value?.automation ?? null)
 const workspace = computed(() => pageData.value?.workspace ?? null)
-const workflow = computed(
-  () => $store.getters['automationWorkflow/getSelected']
-)
+const workflow = computed(() => pageData.value?.workflow ?? null)
 
 function onRouteChange(from) {
   const currentAutomation = $store.getters['application/get'](
diff --git a/web-frontend/modules/builder/components/page/PagePreview.vue b/web-frontend/modules/builder/components/page/PagePreview.vue
index eea42658e4..6d708060b9 100644
--- a/web-frontend/modules/builder/components/page/PagePreview.vue
+++ b/web-frontend/modules/builder/components/page/PagePreview.vue
@@ -315,7 +315,7 @@ export default {
       })
     },
     elementSelectedId(newValue) {
-      if (newValue) {
+      if (newValue && this.$refs.previewScaled) {
         this.$refs.previewScaled.focus()
       }
     },
diff --git a/web-frontend/modules/builder/components/page/PageTemplateContent.vue b/web-frontend/modules/builder/components/page/PageTemplateContent.vue
index 1959b29231..3bac94d7a4 100644
--- a/web-frontend/modules/builder/components/page/PageTemplateContent.vue
+++ b/web-frontend/modules/builder/components/page/PageTemplateContent.vue
@@ -36,7 +36,9 @@ export default {
       mode,
       formulaComponent: ApplicationBuilderFormulaInput,
       applicationContext: {
+        workspace: this.workspace,
         builder: this.builder,
+        page: this.currentPage,
         mode,
       },
     }
@@ -65,6 +67,7 @@ export default {
   computed: {
     applicationContext() {
       return {
+        workspace: this.workspace,
         builder: this.builder,
         page: this.currentPage,
         mode,
diff --git a/web-frontend/modules/core/components/template/TemplateModal.vue b/web-frontend/modules/core/components/template/TemplateModal.vue
index 5c3d7e8087..968ffabc07 100644
--- a/web-frontend/modules/core/components/template/TemplateModal.vue
+++ b/web-frontend/modules/core/components/template/TemplateModal.vue
@@ -1,5 +1,5 @@
 <template>
-  <Modal ref="modal" :full-screen="true" :close-button="false">
+  <Modal ref="modal" :full-screen="true" :close-button="false" keep-content>
     <div v-if="loading" class="loading-absolute-center"></div>
     <template v-else>
       <TemplateHeader
diff --git a/web-frontend/test/unit/core/components/templateModal.spec.js b/web-frontend/test/unit/core/components/templateModal.spec.js
new file mode 100644
index 0000000000..975761d04f
--- /dev/null
+++ b/web-frontend/test/unit/core/components/templateModal.spec.js
@@ -0,0 +1,36 @@
+import { defineComponent } from 'vue'
+import { shallowMount } from '@vue/test-utils'
+
+import TemplateModal from '@baserow/modules/core/components/template/TemplateModal'
+
+describe('TemplateModal', () => {
+  const mountComponent = () => {
+    return shallowMount(TemplateModal, {
+      propsData: {
+        workspace: {
+          id: 1,
+        },
+      },
+      global: {
+        stubs: {
+          Modal: defineComponent({
+            name: 'Modal',
+            props: ['keepContent', 'fullScreen', 'closeButton'],
+            template: '<div><slot /></div>',
+          }),
+          TemplateHeader: true,
+          TemplateCategories: true,
+          TemplatePreview: true,
+        },
+      },
+    })
+  }
+
+  it('keeps the root modal content mounted while closed', () => {
+    const wrapper = mountComponent()
+
+    expect(wrapper.findComponent({ name: 'Modal' }).props('keepContent')).toBe(
+      ''
+    )
+  })
+})