Member types: Prepare container support

src/Umbraco.Core/Models/EntityContainer.cs

@@ -12,6 +12,7 @@ public sealed class EntityContainer : TreeEntityBase, IUmbracoEntity
         { Constants.ObjectTypes.DataType, Constants.ObjectTypes.DataTypeContainer },
         { Constants.ObjectTypes.DocumentType, Constants.ObjectTypes.DocumentTypeContainer },
         { Constants.ObjectTypes.MediaType, Constants.ObjectTypes.MediaTypeContainer },
+        { Constants.ObjectTypes.MemberType, Constants.ObjectTypes.MemberTypeContainer },
         { Constants.ObjectTypes.DocumentBlueprint, Constants.ObjectTypes.DocumentBlueprintContainer },
     };
 

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

@@ -32,8 +32,11 @@ public EntityContainerRepository(
     {
         Guid[] allowedContainers =
         {
-            Constants.ObjectTypes.DocumentTypeContainer, Constants.ObjectTypes.MediaTypeContainer,
-            Constants.ObjectTypes.DataTypeContainer, Constants.ObjectTypes.DocumentBlueprintContainer,
+            Constants.ObjectTypes.DataTypeContainer,
+            Constants.ObjectTypes.DocumentTypeContainer,
+            Constants.ObjectTypes.MediaTypeContainer,
+            Constants.ObjectTypes.MemberTypeContainer,
+            Constants.ObjectTypes.DocumentBlueprintContainer,
         };
         NodeObjectTypeId = containerObjectType;
         if (allowedContainers.Contains(NodeObjectTypeId) == false)

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

@@ -1,36 +1,26 @@
-using Umbraco.Cms.Core.Models;
+using Microsoft.Extensions.Logging;
+using Umbraco.Cms.Core;
+using Umbraco.Cms.Core.Cache;
 using Umbraco.Cms.Core.Persistence.Repositories;
+using Umbraco.Cms.Infrastructure.Scoping;
 
-namespace Umbraco.Cms.Infrastructure.Persistence.Repositories.Implement
+namespace Umbraco.Cms.Infrastructure.Persistence.Repositories.Implement;
+
+internal sealed class MemberTypeContainerRepository : EntityContainerRepository, IMemberTypeContainerRepository
 {
-    /// <summary>
-    /// A no-op implementation of <see cref="IMemberTypeContainerRepository"/>, as containers aren't supported for members.
-    /// </summary>
-    /// <remarks>
-    /// Introduced to avoid inconsistencies with nullability of dependencies for type repositories for content, media and members.
-    /// </remarks>
-    internal sealed class MemberTypeContainerRepository : IMemberTypeContainerRepository
+    public MemberTypeContainerRepository(
+        IScopeAccessor scopeAccessor,
+        AppCaches cache,
+        ILogger<MemberTypeContainerRepository> logger,
+        IRepositoryCacheVersionService repositoryCacheVersionService,
+        ICacheSyncService cacheSyncService)
+        : base(
+            scopeAccessor,
+            cache,
+            logger,
+            Constants.ObjectTypes.MemberTypeContainer,
+            repositoryCacheVersionService,
+            cacheSyncService)
     {
-        public void Delete(EntityContainer entity)
-        {
-        }
-
-        public bool Exists(int id) => false;
-
-        public EntityContainer? Get(Guid id) => null;
-
-        public IEnumerable<EntityContainer> Get(string name, int level) => Enumerable.Empty<EntityContainer>();
-
-        public bool HasDuplicateName(Guid parentKey, string name) => false;
-
-        public bool HasDuplicateName(int parentId, string name) => false;
-
-        public EntityContainer? Get(int id) => null;
-
-        public IEnumerable<EntityContainer> GetMany(params int[]? ids) => Enumerable.Empty<EntityContainer>();
-
-        public void Save(EntityContainer entity)
-        {
-        }
     }
 }