Bugfix: GH-309 Update Parents after adding connection

Wunderbyte-GmbH · Jan 15, 2025 · 5cf36e8 · 5cf36e8
1 parent 3e6dccc
commit 5cf36e8
Show file tree

Hide file tree

Showing 6 changed files with 14 additions and 9 deletions.
diff --git a/amd/build/app-lazy.min.js b/amd/build/app-lazy.min.js
diff --git a/amd/build/app-lazy.min.js.map b/amd/build/app-lazy.min.js.map
diff --git a/amd/src/app-lazy.js b/amd/src/app-lazy.js
diff --git a/...tion/conditions/parent_node_completed.php → ...itions/disabled/parent_node_completed.php b/...tion/conditions/parent_node_completed.php → ...itions/disabled/parent_node_completed.php
diff --git a/vue3/components/flowchart/LearningPath.vue b/vue3/components/flowchart/LearningPath.vue
@@ -134,6 +134,7 @@ import ExpandNodeEdit from '../nodes/ExpandNodeEdit.vue'
 import onNodeClick from '../../composables/flowHelper/onNodeClick'
 import { debounce } from 'lodash';
 import onWheel from '../../composables/flowHelper/onWheel'
+import recalculateParentChild from '../../composables/recalculateParentChild'
 
 // Load Store and Router
 const store = useStore()
@@ -282,7 +283,7 @@ function onDragOver(event) {
 }
 
 // Adjust and add edges if connection was made
-function handleConnect(params) {
+async function handleConnect(params) {
 if (params.source !== store.state.startnode) {
  // Swap source and target positions
  params.target = params.source;
@@ -291,6 +292,10 @@ if (params.source !== store.state.startnode) {
 
 addEdges(addCustomEdge( params.target, params.source));
 emit('add-edge', addCustomEdge( params.target, params.source));
+
+props.learningpath.json.tree =
+recalculateParentChild(props.learningpath.json.tree, 'parentCourse', 'childCourse', 'starting_node')
+await store.dispatch('saveLearningpath', props.learningpath);
 }
 
 function handleSaveEdit(params){

diff --git a/vue3/composables/conditions/addAutoRestrictions.js b/vue3/composables/conditions/addAutoRestrictions.js
@@ -44,11 +44,11 @@ function createRestriction (node_id, parent_node_id, store) {
       {
         "childCondition": [],
         "data": {
-          "description": store.state.strings.course_description_condition_parent_node_completed,
-          "description_before": store.state.strings.course_restricition_before_condition_parent_node_completed,
+          "description": store.state.strings.course_description_condition_parent_courses,
+          "description_before": store.state.strings.course_restricition_before_condition_parent_courses,
           "id": 150,
-          "label": "parent_node_completed",
-          "name": store.state.strings.course_name_condition_parent_node_completed,
+          "label": "parent_courses",
+          "name": store.state.strings.course_name_condition_parent_courses,
           "node_id": 'condition_1',
           "value": {
             "node_id": parent_node_id
@@ -73,7 +73,7 @@ function createRestriction (node_id, parent_node_id, store) {
         "data": {
           "childCondition": 'condition_1',
           "visibility": true,
-          "feedback_before": store.state.strings.course_restricition_before_condition_parent_node_completed,
+          "feedback_before": store.state.strings.course_restricition_before_condition_parent_courses,
           "feedback_before_checkmark": true,
         },
         "draggable": false,