Only apply validation on content update to variant cultures where the editor has permission for the culture (#18778)

* Only apply validation on content update to variant cultures where the editor has permission for the culture.

* Remove inadvertent comment updates.

* Fixed failing integration test.

Author: AndyButland

Diff for: src/Umbraco.Core/Services/ContentEditingService.cs

@@ -188,9 +188,9 @@ private async Task<IContent> EnsureOnlyAllowedFieldsAreUpdated(IContent contentW
         {
             foreach (var culture in disallowedCultures)
             {
-                    var currentValue = existingContent?.Properties.First(x => x.Alias == property.Alias)
-                        .GetValue(culture, null, false);
-                    property.SetValue(currentValue, culture, null);
+                var currentValue = existingContent?.Properties.First(x => x.Alias == property.Alias)
+                    .GetValue(culture, null, false);
+                property.SetValue(currentValue, culture, null);
             }
         }
 
@@ -279,10 +279,10 @@ public async Task<Attempt<ContentUpdateResult, ContentEditingOperationStatus>> U
         => await HandleMoveToRecycleBinAsync(key, userKey);
 
     public async Task<Attempt<IContent?, ContentEditingOperationStatus>> DeleteFromRecycleBinAsync(Guid key, Guid userKey)
-        => await HandleDeleteAsync(key, userKey,true);
+        => await HandleDeleteAsync(key, userKey, true);
 
     public async Task<Attempt<IContent?, ContentEditingOperationStatus>> DeleteAsync(Guid key, Guid userKey)
-        => await HandleDeleteAsync(key, userKey,false);
+        => await HandleDeleteAsync(key, userKey, false);
 
     public async Task<Attempt<IContent?, ContentEditingOperationStatus>> MoveAsync(Guid key, Guid? parentKey, Guid userKey)
         => await HandleMoveAsync(key, parentKey, userKey);

Diff for: src/Umbraco.Core/Services/IContentEditingService.cs

@@ -16,6 +16,9 @@ Task<Attempt<ContentValidationResult, ContentEditingOperationStatus>> ValidateCr
         => ValidateCreateAsync(createModel);
 #pragma warning restore CS0618 // Type or member is obsolete
 
+    [Obsolete("Please use the validate update method that is not obsoleted. Scheduled for removal in Umbraco 16.")]
+    Task<Attempt<ContentValidationResult, ContentEditingOperationStatus>> ValidateUpdateAsync(Guid key, ContentUpdateModel updateModel);
+
     [Obsolete("Please use the validate update method that is not obsoleted. Scheduled for removal in Umbraco 17.")]
     Task<Attempt<ContentValidationResult, ContentEditingOperationStatus>> ValidateUpdateAsync(Guid key, ValidateContentUpdateModel updateModel);