From cfa1f2bcfb2973f0e532cef6aa249aa68b4da94f Mon Sep 17 00:00:00 2001
From: Kenn Jacobsen <kja@vertica.dk>
Date: Mon, 14 Oct 2019 17:57:11 +0200
Subject: [PATCH] Don't allow deleting property groups with locked properties
 (#6567)

---
 .../directives/components/umbgroupsbuilder.directive.js       | 4 ++++
 .../src/views/components/umb-groups-builder.html              | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/Umbraco.Web.UI.Client/src/common/directives/components/umbgroupsbuilder.directive.js b/src/Umbraco.Web.UI.Client/src/common/directives/components/umbgroupsbuilder.directive.js
index d1f35f8e03..32d3e83dac 100644
--- a/src/Umbraco.Web.UI.Client/src/common/directives/components/umbgroupsbuilder.directive.js
+++ b/src/Umbraco.Web.UI.Client/src/common/directives/components/umbgroupsbuilder.directive.js
@@ -418,6 +418,10 @@
 
       };
 
+      scope.canRemoveGroup = function(group){
+        return _.find(group.properties, function(property) { return property.locked === true; }) == null;
+      }
+
       scope.removeGroup = function(groupIndex) {
         scope.model.groups.splice(groupIndex, 1);
       };
diff --git a/src/Umbraco.Web.UI.Client/src/views/components/umb-groups-builder.html b/src/Umbraco.Web.UI.Client/src/views/components/umb-groups-builder.html
index 67ff493930..1f4eb44c56 100644
--- a/src/Umbraco.Web.UI.Client/src/views/components/umb-groups-builder.html
+++ b/src/Umbraco.Web.UI.Client/src/views/components/umb-groups-builder.html
@@ -98,7 +98,7 @@
                         </div>
                     </ng-form>
 
-                    <div class="umb-group-builder__group-remove" ng-if="!sortingMode">
+                    <div class="umb-group-builder__group-remove" ng-if="!sortingMode && canRemoveGroup(tab)">
                         <i class="icon-trash" ng-click="togglePrompt(tab)"></i>
                         <umb-confirm-action
                             ng-if="tab.deletePrompt"