Track content object on MetaTagsNotifications (#194)

Co-authored-by: Ian Leeder <ileeder@intuitiveit.com.au>
patrickdemooij9 · Jun 1, 2023 · 6b3d856 · 6b3d856
1 parent da0c5e1
commit 6b3d856
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 2 deletions.
diff --git a/src/SeoToolkit.Umbraco.MetaFields.Core/Notifications/AfterMetaTagsNotification.cs b/src/SeoToolkit.Umbraco.MetaFields.Core/Notifications/AfterMetaTagsNotification.cs
@@ -1,4 +1,6 @@
 using SeoToolkit.Umbraco.MetaFields.Core.Models.SeoService;
+using System;
+using Umbraco.Cms.Core.Models.PublishedContent;
 using Umbraco.Cms.Core.Notifications;
 
 namespace SeoToolkit.Umbraco.MetaFields.Core.Notifications
@@ -7,11 +9,20 @@ public class AfterMetaTagsNotification : INotification
     {
         public string ContentTypeAlias { get; }
         public MetaTagsModel MetaTags { get; }
+        public IPublishedContent Content { get; set; }
 
+        [Obsolete("This constructor is deprecated and will be removed in the next major release.")]
         public AfterMetaTagsNotification(string contentTypeAlias, MetaTagsModel metaTags)
         {
             ContentTypeAlias = contentTypeAlias;
             MetaTags = metaTags;
         }
+
+        public AfterMetaTagsNotification(IPublishedContent content, MetaTagsModel metaTags)
+        {
+            Content = content;
+            ContentTypeAlias = content.ContentType.Alias;
+            MetaTags = metaTags;
+        }
     }
 }
diff --git a/src/SeoToolkit.Umbraco.MetaFields.Core/Notifications/BeforeMetaTagsNotification.cs b/src/SeoToolkit.Umbraco.MetaFields.Core/Notifications/BeforeMetaTagsNotification.cs
@@ -1,4 +1,6 @@
 using SeoToolkit.Umbraco.MetaFields.Core.Models.SeoService;
+using System;
+using Umbraco.Cms.Core.Models.PublishedContent;
 using Umbraco.Cms.Core.Notifications;
 
 namespace SeoToolkit.Umbraco.MetaFields.Core.Notifications
@@ -7,11 +9,20 @@ public class BeforeMetaTagsNotification : INotification
     {
         public string ContentTypeAlias { get; }
         public MetaTagsModel MetaTags { get; }
+        public IPublishedContent Content { get; set; }
 
+        [Obsolete("This constructor is deprecated and will be removed in the next major release.")]
         public BeforeMetaTagsNotification(string contentTypeAlias, MetaTagsModel metaTags)
         {
             ContentTypeAlias = contentTypeAlias;
             MetaTags = metaTags;
         }
+
+        public BeforeMetaTagsNotification(IPublishedContent content, MetaTagsModel metaTags)
+        {
+            Content = content;
+            ContentTypeAlias = content.ContentType.Alias;
+            MetaTags = metaTags;
+        }
     }
 }
diff --git a/src/SeoToolkit.Umbraco.MetaFields.Core/Providers/DefaultMetaTagsProvider.cs b/src/SeoToolkit.Umbraco.MetaFields.Core/Providers/DefaultMetaTagsProvider.cs
@@ -57,7 +57,7 @@ public MetaTagsModel Get(IPublishedContent content, bool includeUserValues)
 
                 //Make sure that the fields are set, otherwise the values cannot be set!
                 var metaTags = new MetaTagsModel(allFields.ToDictionary(it => it, it => (object)null));
-                _eventAggregator.Publish(new BeforeMetaTagsNotification(content.ContentType.Alias, metaTags));
+                _eventAggregator.Publish(new BeforeMetaTagsNotification(content, metaTags));
 
                 var settings = _documentTypeSettingsService.Get(content.ContentType.Id);
                 if (_seoSettingsService.IsEnabled(content.ContentType.Id) != true)
@@ -111,7 +111,7 @@ public MetaTagsModel Get(IPublishedContent content, bool includeUserValues)
                     metaTags.SetValue(fieldValue.Field.Alias, fieldValue.Value);
                 }
 
-                _eventAggregator.Publish(new AfterMetaTagsNotification(content.ContentType.Alias, metaTags));
+                _eventAggregator.Publish(new AfterMetaTagsNotification(content, metaTags));
 
                 return metaTags;
             }