Updated package manifest filter to set "PackageId" property in Umbrac…

…o 12

src/Limbo.Umbraco.YouTube/Manifests/YouTubeManifestFilter.cs

@@ -1,4 +1,5 @@
 using System.Collections.Generic;
+using System.Reflection;
 using Umbraco.Cms.Core.Manifest;
 
 namespace Limbo.Umbraco.YouTube.Manifests {
@@ -8,7 +9,9 @@ public class YouTubeManifestFilter : IManifestFilter {
 
         /// <inheritdoc />
         public void Filter(List<PackageManifest> manifests) {
-            manifests.Add(new PackageManifest {
+
+            // Initialize a new manifest filter for this package
+            PackageManifest manifest = new() {
                 AllowPackageTelemetry = true,
                 PackageName = YouTubePackage.Name,
                 Version = YouTubePackage.InformationalVersion,
@@ -20,7 +23,21 @@ public class YouTubeManifestFilter : IManifestFilter {
                 Stylesheets = new [] {
                     $"/App_Plugins/{YouTubePackage.Alias}/Styles/Default.css"
                 }
-            });
+            };
+
+            // The "PackageId" property isn't available prior to Umbraco 12, and since the package is build against
+            // Umbraco 10, we need to use reflection for setting the property value for Umbraco 12+. Ideally this
+            // shouldn't fail, but we might at least add a try/catch to be sure
+            try {
+                PropertyInfo? property = manifest.GetType().GetProperty("PackageId");
+                property?.SetValue(manifest, YouTubePackage.Alias);
+            } catch {
+                // We don't really care about the exception
+            }
+
+            // Append the manifest
+            manifests.Add(manifest);
+
         }
 
     }