From 930e9f0f42847ba6b7602cf4daf3113e2f7ec6b4 Mon Sep 17 00:00:00 2001
From: David Wallace <david.wallace@tu-darmstadt.de>
Date: Tue, 21 May 2024 16:23:58 +0200
Subject: [PATCH] js: fix copy action element for optionsets, options and
 conditions

Signed-off-by: David Wallace <david.wallace@tu-darmstadt.de>
---
 .../assets/js/actions/elementActions.js       | 27 ++++++++++++++-----
 1 file changed, 21 insertions(+), 6 deletions(-)

diff --git a/rdmo/management/assets/js/actions/elementActions.js b/rdmo/management/assets/js/actions/elementActions.js
index 543abc9b7b..2b1e00a189 100644
--- a/rdmo/management/assets/js/actions/elementActions.js
+++ b/rdmo/management/assets/js/actions/elementActions.js
@@ -261,9 +261,14 @@ export function fetchElement(elementType, elementId, elementAction=null) {
             ConditionsApi.fetchConditions('index'),
             OptionsApi.fetchOptions('index'),
             QuestionsApi.fetchQuestions('index')
-          ]).then(([element, conditions, options, questions]) => ({
-            element, conditions, options, questions
-          }))
+          ]).then(([element, conditions, options, questions]) => {
+            if (elementAction == 'copy') {
+              delete element.questions
+            }
+            return {
+              element, conditions, options, questions
+            }
+          })
         }
         break
 
@@ -275,6 +280,7 @@ export function fetchElement(elementType, elementId, elementAction=null) {
         ]).then(([element, optionsets, conditions]) => {
           if (elementAction == 'copy') {
             delete element.optionsets
+            delete element.conditions
           }
           return {
             element, optionsets, conditions
@@ -293,9 +299,18 @@ export function fetchElement(elementType, elementId, elementAction=null) {
           QuestionsApi.fetchQuestions('index'),
           TasksApi.fetchTasks('index'),
         ]).then(([element, attributes, optionsets, options,
-                  pages, questionsets, questions, tasks]) => ({
-          element, attributes, optionsets, options, pages, questionsets, questions, tasks
-        }))
+                  pages, questionsets, questions, tasks]) => {
+           if (elementAction == 'copy') {
+            delete element.optionsets
+            delete element.pages
+            delete element.questionsets
+            delete element.questions
+            delete element.tasks
+          }
+           return {
+             element, attributes, optionsets, options, pages, questionsets, questions, tasks
+           }
+        })
         break
 
       case 'tasks':