Merge pull request #11452 from umbraco/v9/feature/v8_merge_22_10_2021

Merge v8 into v9 22/10/2021

Author: bergmania

src/Umbraco.Infrastructure/Persistence/Repositories/Implement/DocumentRepository.cs

@@ -673,19 +673,6 @@ protected override void PersistUpdatedItem(IContent entity)
                     edited = true;
                 }
 
-                // To establish the new value of "edited" we compare all properties publishedValue to editedValue and look
-                // for differences.
-                //
-                // If we SaveAndPublish but the publish fails (e.g. already scheduled for release)
-                // we have lost the publishedValue on IContent (in memory vs database) so we cannot correctly make that comparison.
-                //
-                // This is a slight change to behaviour, historically a publish, followed by change & save, followed by undo change & save
-                // would change edited back to false.
-                if (!publishing && editedSnapshot)
-                {
-                    edited = true;
-                }
-
                 if (entity.ContentType.VariesByCulture())
                 {
                     // names also impact 'edited'

src/Umbraco.Web.BackOffice/Controllers/ContentTypeControllerBase.cs

@@ -12,21 +12,20 @@
 using Umbraco.Cms.Web.Common.Attributes;
 using Umbraco.Cms.Web.Common.Authorization;
 using Umbraco.Extensions;
-using Constants = Umbraco.Cms.Core.Constants;
 
 namespace Umbraco.Cms.Web.BackOffice.Controllers
 {
     /// <summary>
-    /// An API controller used for dealing with member groups
+    ///     An API controller used for dealing with member groups
     /// </summary>
     [PluginController(Constants.Web.Mvc.BackOfficeApiArea)]
     [Authorize(Policy = AuthorizationPolicies.TreeAccessMemberGroups)]
     [ParameterSwapControllerActionSelector(nameof(GetById), "id", typeof(int), typeof(Guid), typeof(Udi))]
     public class MemberGroupController : UmbracoAuthorizedJsonController
     {
+        private readonly ILocalizedTextService _localizedTextService;
         private readonly IMemberGroupService _memberGroupService;
         private readonly IUmbracoMapper _umbracoMapper;
-        private readonly ILocalizedTextService _localizedTextService;
 
         public MemberGroupController(
             IMemberGroupService memberGroupService,
@@ -35,11 +34,12 @@ public MemberGroupController(
         {
             _memberGroupService = memberGroupService ?? throw new ArgumentNullException(nameof(memberGroupService));
             _umbracoMapper = umbracoMapper ?? throw new ArgumentNullException(nameof(umbracoMapper));
-            _localizedTextService = localizedTextService ?? throw new ArgumentNullException(nameof(localizedTextService));
+            _localizedTextService =
+                localizedTextService ?? throw new ArgumentNullException(nameof(localizedTextService));
         }
 
         /// <summary>
-        /// Gets the member group json for the member group id
+        ///     Gets the member group json for the member group id
         /// </summary>
         /// <param name="id"></param>
         /// <returns></returns>
@@ -56,7 +56,7 @@ public ActionResult<MemberGroupDisplay> GetById(int id)
         }
 
         /// <summary>
-        /// Gets the member group json for the member group guid
+        ///     Gets the member group json for the member group guid
         /// </summary>
         /// <param name="id"></param>
         /// <returns></returns>
@@ -72,7 +72,7 @@ public ActionResult<MemberGroupDisplay> GetById(Guid id)
         }
 
         /// <summary>
-        /// Gets the member group json for the member group udi
+        ///     Gets the member group json for the member group udi
         /// </summary>
         /// <param name="id"></param>
         /// <returns></returns>
@@ -100,7 +100,7 @@ public IEnumerable<MemberGroupDisplay> GetByIds([FromQuery] int[] ids)
         [HttpPost]
         public IActionResult DeleteById(int id)
         {
-            var memberGroup = _memberGroupService.GetById(id);
+            IMemberGroup memberGroup = _memberGroupService.GetById(id);
             if (memberGroup == null)
             {
                 return NotFound();
@@ -112,7 +112,7 @@ public IActionResult DeleteById(int id)
 
         public IEnumerable<MemberGroupDisplay> GetAllGroups()
             => _memberGroupService.GetAll()
-                    .Select(_umbracoMapper.Map<IMemberGroup, MemberGroupDisplay>);
+                .Select(_umbracoMapper.Map<IMemberGroup, MemberGroupDisplay>);
 
         public MemberGroupDisplay GetEmpty()
         {
@@ -123,18 +123,21 @@ public MemberGroupDisplay GetEmpty()
         public bool IsMemberGroupNameUnique(int id, string oldName, string newName)
         {
             if (newName == oldName)
+            {
                 return true; // name hasn't changed
+            }
 
-            var memberGroup = _memberGroupService.GetByName(newName);
+            IMemberGroup memberGroup = _memberGroupService.GetByName(newName);
             if (memberGroup == null)
+            {
                 return true; // no member group found
+            }
 
             return memberGroup.Id == id;
         }
 
         public ActionResult<MemberGroupDisplay> PostSave(MemberGroupSave saveModel)
         {
-
             var id = int.Parse(saveModel.Id.ToString(), CultureInfo.InvariantCulture);
             IMemberGroup memberGroup = id > 0 ? _memberGroupService.GetById(id) : new MemberGroup();
             if (memberGroup == null)
@@ -157,7 +160,7 @@ public ActionResult<MemberGroupDisplay> PostSave(MemberGroupSave saveModel)
             }
             else
             {
-                var display = _umbracoMapper.Map<IMemberGroup, MemberGroupDisplay>(memberGroup);
+                MemberGroupDisplay display = _umbracoMapper.Map<IMemberGroup, MemberGroupDisplay>(memberGroup);
                 display.AddErrorNotification(
                     _localizedTextService.Localize("speechBubbles", "memberGroupNameDuplicate"),
                     string.Empty);

src/Umbraco.Web.BackOffice/Controllers/EntityController.cs

@@ -35,9 +35,6 @@
             // List of elements that can be focusable within the focus lock
             var focusableElementsSelector = '[role="button"], a[href]:not([disabled]):not(.ng-hide), button:not([disabled]):not(.ng-hide), textarea:not([disabled]):not(.ng-hide), input:not([disabled]):not(.ng-hide), select:not([disabled]):not(.ng-hide)';
 
-            // Grab the body element so we can add the tabbing class on it when needed
-            var bodyElement = document.querySelector('body');
-
             function getDomNodes(){
                 infiniteEditorsWrapper = document.querySelector('.umb-editors');
                 if(infiniteEditorsWrapper) {
@@ -47,7 +44,10 @@
 
             function getFocusableElements(targetElm) {
                 var elm = targetElm ? targetElm : target;
-                focusableElements = elm.querySelectorAll(focusableElementsSelector);
+
+                // Filter out elements that are children of parents with the .ng-hide class
+                focusableElements = [...elm.querySelectorAll(focusableElementsSelector)].filter(elm => !elm.closest('.ng-hide'));
+
                 // Set first and last focusable elements
                 firstFocusableElement = focusableElements[0];
                 lastFocusableElement = focusableElements[focusableElements.length - 1];

src/Umbraco.Web.BackOffice/Controllers/MemberGroupController.cs

@@ -118,18 +118,38 @@ Use this directive to render a ui component for selecting child items to a paren
 (function() {
     'use strict';
 
-    function ChildSelectorDirective() {
+    function ChildSelectorDirective(overlayService, localizationService) {
 
         function link(scope, el, attr, ctrl) {
 
             var eventBindings = [];
             scope.dialogModel = {};
             scope.showDialog = false;
 
-            scope.removeChild = (selectedChild, $index) => {
-               if (scope.onRemove) {
-                  scope.onRemove(selectedChild, $index);
-               }
+            scope.removeChild = (selectedChild, $index, event) => {
+               const dialog = {
+                    view: "views/components/overlays/umb-template-remove-confirm.html",
+                    layout: selectedChild,
+                    submitButtonLabelKey: "defaultdialogs_yesRemove",
+                    submitButtonStyle: "danger",
+                    submit: function () {
+                        if(scope.onRemove) {
+                            scope.onRemove(selectedChild, $index);
+                            overlayService.close();
+                        }
+                    },
+                    close: function () {
+                        overlayService.close();
+                    }
+                };
+
+                localizationService.localize("general_delete").then(value => {
+                    dialog.title = value;
+                    overlayService.open(dialog);
+                });
+
+                event.preventDefault();
+                event.stopPropagation();
             };
 
             scope.addChild = $event => {

src/Umbraco.Web.UI.Client/src/common/directives/components/forms/umbfocuslock.directive.js

@@ -214,13 +214,14 @@ When building a custom infinite editor view you can use the same components as a
          * Method to tell editors that they are begin blurred.
          */
         function blur() {
-
-            /* keyboard shortcuts will be overwritten by the new infinite editor
+            if (isEnabled === true) {
+                /* keyboard shortcuts will be overwritten by the new infinite editor
                 so we need to store the shortcuts for the current editor so they can be rebound
                 when the infinite editor closes
             */
-            unbindKeyboardShortcuts();
-            isEnabled = false;
+                unbindKeyboardShortcuts();
+                isEnabled = false;
+            }
         }
         /**
          * @ngdoc method

src/Umbraco.Web.UI.Client/src/common/directives/components/umbchildselector.directive.js

@@ -621,7 +621,6 @@
 	}
 
 	.preview-rows {
-
 		display: inline-block;
 		position: relative;
 		box-sizing: border-box;
@@ -675,9 +674,9 @@
 	}
 
 	.preview-rows.columns {
-		min-height: 18px;
+		min-height: 16px;
 		line-height: 11px;
-		padding: 1px;
+        padding: 0 1px 1px 1px;
 
         &.prevalues-rows {
             min-height: 30px;

src/Umbraco.Web.UI.Client/src/common/services/editor.service.js

@@ -0,0 +1,15 @@
+<div>
+
+    <div ng-if="model.layout" class="umb-alert umb-alert--warning mb2">
+        <localize key="contentTypeEditor_removeChildNode">You are removing the child node</localize> <strong>{{model.layout.name}}</strong>.
+    </div>
+
+    <p>
+        <localize key="contentTypeEditor_removeChildNodeWarning">
+            Removing a child node will limit the editors options to create different content types beneath a node.
+        </localize>
+    </p>
+
+    <localize key="defaultdialogs_confirmdelete">Are you sure you want to remove</localize>?
+
+</div>

src/Umbraco.Web.UI.Client/src/less/gridview.less

@@ -24,8 +24,8 @@
                 <span class="umb-child-selector__child-name">{{selectedChild.name}}</span>
             </div>
             <div class="umb-child-selector__child-actions">
-                <button type="button" class="umb-child-selector__child-remove" ng-click="removeChild(selectedChild, $index)">
-                    <umb-icon icon="icon-trash"></umb-icon>
+                <button type="button" class="umb-node-preview__action umb-node-preview__action--red umb-child-selector__child-remove" ng-click="removeChild(selectedChild, $index, $event)">
+                    <localize key="general_remove">Remove</localize>
                 </button>
             </div>
         </div>