Send content type notifications on copy (#12463)

umbraco · May 31, 2022 · 156fe59 · 156fe59
1 parent ac61f85
commit 156fe59
Showing 1 changed file with 21 additions and 3 deletions.
diff --git a/...aco.Infrastructure/Services/Implement/ContentTypeServiceBaseOfTRepositoryTItemTService.cs b/...aco.Infrastructure/Services/Implement/ContentTypeServiceBaseOfTRepositoryTItemTService.cs
@@ -730,7 +730,7 @@ public TItem Copy(TItem original, string alias, string name, TItem parent)
 
         public Attempt<OperationResult<MoveOperationStatusType, TItem>> Copy(TItem copying, int containerId)
         {
-            var evtMsgs = EventMessagesFactory.Get();
+            var eventMessages = EventMessagesFactory.Get();
 
             TItem copy;
             using (var scope = ScopeProvider.CreateScope())
@@ -765,16 +765,34 @@ public TItem Copy(TItem original, string alias, string name, TItem parent)
                     }
 
                     copy.ParentId = containerId;
+
+                    SavingNotification<TItem> savingNotification = GetSavingNotification(copy, eventMessages);
+                    if (scope.Notifications.PublishCancelable(savingNotification))
+                    {
+                        scope.Complete();
+                        return OperationResult.Attempt.Fail(MoveOperationStatusType.FailedCancelledByEvent, eventMessages, copy);
+                    }
+
                     Repository.Save(copy);
+
+                    ContentTypeChange<TItem>[] changes = ComposeContentTypeChanges(copy).ToArray();
+
+                    _eventAggregator.Publish(GetContentTypeRefreshedNotification(changes, eventMessages));
+                    scope.Notifications.Publish(GetContentTypeChangedNotification(changes, eventMessages));
+
+                    SavedNotification<TItem> savedNotification = GetSavedNotification(copy, eventMessages);
+                    savedNotification.WithStateFrom(savingNotification);
+                    scope.Notifications.Publish(savedNotification);
+
                     scope.Complete();
                 }
                 catch (DataOperationException<MoveOperationStatusType> ex)
                 {
-                    return OperationResult.Attempt.Fail<MoveOperationStatusType, TItem>(ex.Operation, evtMsgs); // causes rollback
+                    return OperationResult.Attempt.Fail<MoveOperationStatusType, TItem>(ex.Operation, eventMessages); // causes rollback
                 }
             }
 
-            return OperationResult.Attempt.Succeed(MoveOperationStatusType.Success, evtMsgs, copy);
+            return OperationResult.Attempt.Succeed(MoveOperationStatusType.Success, eventMessages, copy);
         }
 
         #endregion