Afform - Fix saving and editing entity blocks

Fixes dev/core#3120 Regression caused by civicrm#21218 Also fixes undefined variable errors when editing a block.
colemanw · Mar 17, 2022 · 1c2bf8e · 1c2bf8e
1 parent 64e1d54
commit 1c2bf8e
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 7 deletions.
diff --git a/ext/afform/admin/ang/afGuiEditor/afGuiEditor.component.js b/ext/afform/admin/ang/afGuiEditor/afGuiEditor.component.js
@@ -193,7 +193,7 @@
       };
 
       this.getEntityDefn = function(entity) {
-        if (entity.type === 'Contact' && entity.data.contact_type) {
+        if (entity.type === 'Contact' && entity.data && entity.data.contact_type) {
           return editor.meta.entities[entity.data.contact_type];
         }
         return editor.meta.entities[entity.type];
@@ -203,12 +203,15 @@
       this.scrollToEntity = function(entityName) {
         var $canvas = $('#afGuiEditor-canvas-body'),
           $entity = $('.af-gui-container-type-fieldset[data-entity="' + entityName + '"]').first(),
+          scrollValue, maxScroll;
+        if ($entity.length) {
           // Scrolltop value needed to place entity's fieldset at top of canvas
-          scrollValue = $canvas.scrollTop() + ($entity.offset().top - $canvas.offset().top),
+          scrollValue = $canvas.scrollTop() + ($entity.offset().top - $canvas.offset().top);
           // Maximum possible scrollTop (height minus contents height, adjusting for padding)
           maxScroll = $('#afGuiEditor-canvas-body > *').height() - $canvas.height() + 20;
-        // Exceeding the maximum scrollTop breaks the animation so keep it under the limit
-        $canvas.animate({scrollTop: scrollValue > maxScroll ? maxScroll : scrollValue}, 500);
+          // Exceeding the maximum scrollTop breaks the animation so keep it under the limit
+          $canvas.animate({scrollTop: scrollValue > maxScroll ? maxScroll : scrollValue}, 500);
+        }
       };
 
       this.getAfform = function() {

diff --git a/ext/afform/admin/ang/afGuiEditor/elements/afGuiContainer.component.js b/ext/afform/admin/ang/afGuiEditor/elements/afGuiContainer.component.js
@@ -242,15 +242,14 @@
           layout: ctrl.node['#children']
         };
         if (ctrl.join) {
-          model.join = ctrl.join;
+          model.join_entity = ctrl.join;
         }
         if ($scope.block && $scope.block.original) {
           model.title = afGui.meta.blocks[$scope.block.original].title;
           model.name = afGui.meta.blocks[$scope.block.original].name;
-          model.block = afGui.meta.blocks[$scope.block.original].block;
         }
         else {
-          model.block = ctrl.getFieldEntityType();
+          model.entity_type = ctrl.getFieldEntityType();
         }
         dialogService.open('saveBlockDialog', '~/afGuiEditor/saveBlock.html', model, options)
           .then(function(block) {