Cleanup exist code

src/Acl.Net.Core.Database/AclDbContext.cs

@@ -56,21 +56,21 @@
 /// <typeparam name="TUser">The type of the user entity.</typeparam>
 /// <typeparam name="TRole">The type of the role entity.</typeparam>
 /// <typeparam name="TResource">The type of the resource entity.</typeparam>
 public abstract class AclDbContext<TKey, TUser, TRole, TResource> : DbContext
     where TKey : IEquatable<TKey>
     where TUser : User<TKey>
     where TRole : Role<TKey>
     where TResource : Resource<TKey>
 {
-    private readonly IInitialDataSeeder<TKey, TRole> seeder;
+    private readonly IInitialDataSeeder<TKey, TRole> _seeder;
 
     /// <summary>
     /// Initializes a new instance of the <see cref="AclDbContext{TKey, TUser, TRole, TResource}"/> class with the provided data seeder.
     /// </summary>
     /// <param name="seeder">The initial data seeder.</param>
     protected AclDbContext(IInitialDataSeeder<TKey, TRole> seeder)
     {
-        this.seeder = seeder;
+        _seeder = seeder;
     }
 
     /// <summary>
@@ -80,7 +80,7 @@
     /// <param name="seeder">The initial data seeder.</param>
     protected AclDbContext(DbContextOptions options, IInitialDataSeeder<TKey, TRole> seeder) : base(options)
     {
-        this.seeder = seeder;
+        _seeder = seeder;
     }
 
     /// <summary>
@@ -122,7 +122,7 @@
 
             entity.HasMany<TResource>().WithOne().HasForeignKey(res => res.RoleId).IsRequired();
 
-            entity.HasData(seeder.SeedAdminRole(), seeder.SeedUserRole());
+            entity.HasData(_seeder.SeedAdminRole(), _seeder.SeedUserRole());
         });
 
         modelBuilder.Entity<TResource>(entity =>

src/Acl.Net.Core.Database/Entities/Resource.cs

@@ -3,7 +3,7 @@
 /// <summary>
 /// Represents a specific resource in the system.
 /// </summary>
-public class Resource : Resource<int> { }
+public class Resource : Resource<int>;
 
 /// <summary>
 /// Represents a specific resource in the system, identified by a generic key type.

src/Acl.Net.Core.Database/Entities/Role.cs

@@ -3,7 +3,7 @@
 /// <summary>
 /// Represents a role within the Access Control List (ACL) system.
 /// </summary>
-public class Role : Role<int> { }
+public class Role : Role<int>;
 
 /// <summary>
 /// Represents a role within the Access Control List (ACL) system, with a specific key type.

src/Acl.Net.Core.Database/Entities/User.cs

@@ -3,7 +3,7 @@
 /// <summary>
 /// Represents a user in the system.
 /// </summary>
-public class User : User<int> { }
+public class User : User<int>;
 
 /// <summary>
 /// Represents a user in the system, identified by a generic key type.

src/Acl.Net.Core.Managers/AclManager.cs

@@ -112,15 +112,15 @@
 /// <typeparam name="TUser">The type of the user, which must be a derived type of <see cref="User{TKey}"/>.</typeparam>
 /// <typeparam name="TRole">The type of the role, which must be a derived type of <see cref="Role{TKey}"/></typeparam>
 /// <typeparam name="TResource">The type of the resource, which must be a derived type of <see cref="Resource{TKey}"/></typeparam>
 public class AclManager<TKey, TUser, TRole, TResource> : IAclManager<TKey, TUser, TResource>
     where TKey : IEquatable<TKey>
     where TUser : User<TKey>, new()
     where TRole : Role<TKey>
     where TResource : Resource<TKey>
 {
-    protected readonly IInitialDataSeeder<TKey, TRole> initialDataSeeder;
-    protected readonly IUserManager<TKey, TUser, TRole> userManager;
-    protected readonly IResourceManager<TKey, TUser, TResource> resourceManager;
+    protected readonly IInitialDataSeeder<TKey, TRole> InitialDataSeeder;
+    protected readonly IUserManager<TKey, TUser, TRole> UserManager;
+    protected readonly IResourceManager<TKey, TUser, TResource> ResourceManager;
 
     /// <summary>
     /// Initializes a new instance of the <see cref="AclManager{TKey,TUser,TRole,TResource}"/> class
@@ -134,9 +134,9 @@
         AclDbContext<TKey, TUser, TRole, TResource> context
     )
     {
-        this.initialDataSeeder = initialDataSeeder;
-        userManager = new UserManager<TKey, TUser, TRole, TResource>(context);
-        resourceManager = new ResourceManager<TKey, TUser, TRole, TResource>(context, initialDataSeeder);
+        InitialDataSeeder = initialDataSeeder;
+        UserManager = new UserManager<TKey, TUser, TRole, TResource>(context);
+        ResourceManager = new ResourceManager<TKey, TUser, TRole, TResource>(context, initialDataSeeder);
     }
 
     /// <summary>
@@ -152,9 +152,9 @@
         IResourceManager<TKey, TUser, TResource> resourceManager
     )
     {
-        this.initialDataSeeder = initialDataSeeder;
-        this.userManager = userManager;
-        this.resourceManager = resourceManager;
+        InitialDataSeeder = initialDataSeeder;
+        UserManager = userManager;
+        ResourceManager = resourceManager;
     }
 
     /// <summary>
@@ -168,8 +168,8 @@
     /// </exception>
     public virtual bool IsPermitted(string userName, string resourceName)
     {
-        var user = userManager.UserProcessing(userName, initialDataSeeder.SeedUserRole());
-        return resourceManager.IsPermitted(user, resourceName);
+        var user = UserManager.UserProcessing(userName, InitialDataSeeder.SeedUserRole());
+        return ResourceManager.IsPermitted(user, resourceName);
     }
 
     /// <summary>
@@ -183,7 +183,7 @@
     /// </exception>
     public virtual bool IsPermitted(TUser user, string resourceName)
     {
-        return resourceManager.IsPermitted(user, resourceName);
+        return ResourceManager.IsPermitted(user, resourceName);
     }
 
     /// <summary>
@@ -194,8 +194,8 @@
     /// <returns><see langword="true"/> if the user is permitted to access the resource; otherwise, <see langword="false"/>.</returns>
     public virtual bool IsPermitted(string userName, TResource resource)
     {
-        var user = userManager.UserProcessing(userName, initialDataSeeder.SeedUserRole());
-        return resourceManager.IsPermitted(user, resource);
+        var user = UserManager.UserProcessing(userName, InitialDataSeeder.SeedUserRole());
+        return ResourceManager.IsPermitted(user, resource);
     }
 
     /// <summary>
@@ -206,7 +206,7 @@
     /// <returns><see langword="true"/> if the user is permitted to access the resource; otherwise, <see langword="false"/>.</returns>
     public virtual bool IsPermitted(TUser user, TResource resource)
     {
-        return resourceManager.IsPermitted(user, resource);
+        return ResourceManager.IsPermitted(user, resource);
     }
 
     /// <summary>
@@ -221,8 +221,8 @@
     /// </exception>
     public virtual IEnumerable<TResource> IsPermitted(string userName, IEnumerable<string> resourceNames)
     {
-        var user = userManager.UserProcessing(userName, initialDataSeeder.SeedUserRole());
-        return resourceManager.IsPermitted(user, resourceNames);
+        var user = UserManager.UserProcessing(userName, InitialDataSeeder.SeedUserRole());
+        return ResourceManager.IsPermitted(user, resourceNames);
     }
 
     /// <summary>
@@ -237,8 +237,8 @@
     /// </exception>
     public virtual async Task<bool> IsPermittedAsync(string userName, string resourceName)
     {
-        var user = await userManager.UserProcessingAsync(userName, initialDataSeeder.SeedUserRole());
-        return await resourceManager.IsPermittedAsync(user, resourceName);
+        var user = await UserManager.UserProcessingAsync(userName, InitialDataSeeder.SeedUserRole());
+        return await ResourceManager.IsPermittedAsync(user, resourceName);
     }
 
     /// <summary>
@@ -253,7 +253,7 @@
     /// </exception>
     public virtual async Task<bool> IsPermittedAsync(TUser user, string resourceName)
     {
-        return await resourceManager.IsPermittedAsync(user, resourceName);
+        return await ResourceManager.IsPermittedAsync(user, resourceName);
     }
 
     /// <summary>
@@ -265,8 +265,8 @@
     /// The task result contains <see langword="true"/> if the user is permitted to access the resource; otherwise, <see langword="false"/>.</returns>
     public virtual async Task<bool> IsPermittedAsync(string userName, TResource resource)
     {
-        var user = await userManager.UserProcessingAsync(userName, initialDataSeeder.SeedUserRole());
-        return await resourceManager.IsPermittedAsync(user, resource);
+        var user = await UserManager.UserProcessingAsync(userName, InitialDataSeeder.SeedUserRole());
+        return await ResourceManager.IsPermittedAsync(user, resource);
     }
 
     /// <summary>
@@ -278,7 +278,7 @@
     /// The task result contains <see langword="true"/> if the user is permitted to access the resource; otherwise, <see langword="false"/>.</returns>
     public virtual async Task<bool> IsPermittedAsync(TUser user, TResource resource)
     {
-        return await resourceManager.IsPermittedAsync(user, resource);
+        return await ResourceManager.IsPermittedAsync(user, resource);
     }
 
     /// <summary>
@@ -294,7 +294,7 @@
     /// </exception>
     public virtual async Task<IEnumerable<TResource>> IsPermittedAsync(string userName, IEnumerable<string> resourceNames)
     {
-        var user = await userManager.UserProcessingAsync(userName, initialDataSeeder.SeedUserRole());
-        return await resourceManager.IsPermittedAsync(user, resourceNames);
+        var user = await UserManager.UserProcessingAsync(userName, InitialDataSeeder.SeedUserRole());
+        return await ResourceManager.IsPermittedAsync(user, resourceNames);
     }
 }

src/Acl.Net.Core.Managers/Exceptions/ResourceNotFoundException.cs

@@ -1,11 +1,8 @@
-using System.Runtime.Serialization;
-
-namespace Acl.Net.Core.Managers.Exceptions;
+namespace Acl.Net.Core.Managers.Exceptions;
 
 /// <summary>
 /// Represents an exception that is thrown when a specific resource is not found.
 /// </summary>
-[Serializable]
 public class ResourceNotFoundException : Exception
 {
     /// <summary>
@@ -15,13 +12,4 @@ public class ResourceNotFoundException : Exception
     public ResourceNotFoundException(string resourceName)
         : base($"Resource with name '{resourceName}' not found.")
     { }
-
-    /// <summary>
-    /// Initializes a new instance of the <see cref="ResourceNotFoundException"/> class with serialized data.
-    /// </summary>
-    /// <param name="info">The <see cref="SerializationInfo"/> that holds the serialized object data about the exception being thrown.</param>
-    /// <param name="context">The <see cref="StreamingContext"/> that contains contextual information about the source or destination.</param>
-    protected ResourceNotFoundException(SerializationInfo info, StreamingContext context)
-        : base(info, context)
-    { }
 }

src/Acl.Net.Core.Managers/IAclManager.cs

@@ -5,18 +5,14 @@
 /// <summary>
 /// Defines the basic contract for Access Control List (ACL) management.
 /// </summary>
-public interface IAclManager : IAclManager<int>
-{
-}
+public interface IAclManager : IAclManager<int>;
 
 /// <summary>
 /// Defines the contract for Access Control List (ACL) management with support for a specific key type.
 /// </summary>
 /// <typeparam name="TKey">The type of key used to identify users and resources.</typeparam>
 public interface IAclManager<TKey> : IAclManager<TKey, User<TKey>, Resource<TKey>>
-    where TKey : IEquatable<TKey>
-{
-}
+    where TKey : IEquatable<TKey>;
 
 /// <summary>
 /// Defines the contract for Access Control List (ACL) management with support for specific key, user, and resource types.
@@ -24,7 +20,7 @@
 /// <typeparam name="TKey">The type of key used to identify users and resources.</typeparam>
 /// <typeparam name="TUser">The type representing a user.</typeparam>
 /// <typeparam name="TResource">The type representing a resource.</typeparam>
 public interface IAclManager<TKey, in TUser, TResource>
     where TKey : IEquatable<TKey>
     where TUser : User<TKey>
     where TResource : Resource<TKey>

src/Acl.Net.Core.Managers/IResourceManager.cs

@@ -5,18 +5,14 @@
 /// <summary>
 /// Manages resource-related operations with a specific integer key type.
 /// </summary>
-public interface IResourceManager : IResourceManager<int>
-{
-}
+public interface IResourceManager : IResourceManager<int>;
 
 /// <summary>
 /// Manages resource-related operations with a specific key type.
 /// </summary>
 /// <typeparam name="TKey">The type of the key.</typeparam>
 public interface IResourceManager<TKey> : IResourceManager<TKey, User<TKey>, Resource<TKey>>
-    where TKey : IEquatable<TKey>
-{
-}
+    where TKey : IEquatable<TKey>;
 
 /// <summary>
 /// Manages resource-related operations with specific user and resource types.
@@ -24,7 +20,7 @@
 /// <typeparam name="TKey">The type of the key.</typeparam>
 /// <typeparam name="TUser">The type of the user.</typeparam>
 /// <typeparam name="TResource">The type of the resource.</typeparam>
 public interface IResourceManager<TKey, in TUser, TResource>
     where TKey : IEquatable<TKey>
     where TUser : User<TKey>
     where TResource : Resource<TKey>

src/Acl.Net.Core.Managers/IUserManager.cs

@@ -5,18 +5,14 @@
 /// <summary>
 /// Defines the contract for managing user-related operations using integer keys.
 /// </summary>
-public interface IUserManager : IUserManager<int>
-{
-}
+public interface IUserManager : IUserManager<int>;
 
 /// <summary>
 /// Defines the contract for managing user-related operations with a specific key type.
 /// </summary>
 /// <typeparam name="TKey">The type of the key.</typeparam>
 public interface IUserManager<TKey> : IUserManager<TKey, User<TKey>, Role<TKey>>
-    where TKey : IEquatable<TKey>
-{
-}
+    where TKey : IEquatable<TKey>;
 
 /// <summary>
 /// Defines the contract for managing user-related operations with specific user and role types.
@@ -24,7 +20,7 @@
 /// <typeparam name="TKey">The type of the key.</typeparam>
 /// <typeparam name="TUser">The type of the user.</typeparam>
 /// <typeparam name="TRole">The type of the role.</typeparam>
 public interface IUserManager<TKey, TUser, in TRole>
     where TKey : IEquatable<TKey>
     where TUser : User<TKey>
     where TRole : Role<TKey>

src/Acl.Net.Core.Managers/ResourceManager.cs

@@ -46,14 +46,14 @@
 /// <typeparam name="TUser">The type of the user.</typeparam>
 /// <typeparam name="TRole">The type of the role.</typeparam>
 /// <typeparam name="TResource">The type of the resource.</typeparam>
 public class ResourceManager<TKey, TUser, TRole, TResource> : IResourceManager<TKey, TUser, TResource>
     where TKey : IEquatable<TKey>
     where TUser : User<TKey>, new()
     where TRole : Role<TKey>
     where TResource : Resource<TKey>
 {
-    protected readonly AclDbContext<TKey, TUser, TRole, TResource> context;
-    protected readonly IInitialDataSeeder<TKey, TRole> initialDataSeeder;
+    protected readonly AclDbContext<TKey, TUser, TRole, TResource> Context;
+    protected readonly IInitialDataSeeder<TKey, TRole> InitialDataSeeder;
 
     /// <summary>
     /// Initializes a new instance of the <see cref="ResourceManager{TKey, TUser, TRole, TResource}"/> class.
@@ -65,8 +65,8 @@
         IInitialDataSeeder<TKey, TRole> initialDataSeeder
     )
     {
-        this.context = context;
-        this.initialDataSeeder = initialDataSeeder;
+        Context = context;
+        InitialDataSeeder = initialDataSeeder;
     }
 
     /// <summary>
@@ -92,8 +92,8 @@
     /// <returns><see langword="true"/> if the user is permitted to access the resource; otherwise, <see langword="false"/>.</returns>
     public virtual bool IsPermitted(TUser user, TResource resource)
     {
-        return user.RoleId.Equals(initialDataSeeder.SeedAdminRole().Id) ||
-            context.Resources.Any(r => r.RoleId.Equals(user.RoleId) && r.Id.Equals(resource.Id));
+        return user.RoleId.Equals(InitialDataSeeder.SeedAdminRole().Id) ||
+            Context.Resources.Any(r => r.RoleId.Equals(user.RoleId) && r.Id.Equals(resource.Id));
     }
 
     /// <summary>
@@ -146,8 +146,8 @@
     /// The task result contains <see langword="true"/> if the user is permitted to access the resource; otherwise, <see langword="false"/>.</returns>
     public virtual async Task<bool> IsPermittedAsync(TUser user, TResource resource)
     {
-        return user.RoleId.Equals(initialDataSeeder.SeedAdminRole().Id) ||
-            await context.Resources.AnyAsync(r => r.RoleId.Equals(user.RoleId) && r.Id.Equals(resource.Id));
+        return user.RoleId.Equals(InitialDataSeeder.SeedAdminRole().Id) ||
+            await Context.Resources.AnyAsync(r => r.RoleId.Equals(user.RoleId) && r.Id.Equals(resource.Id));
     }
 
     /// <summary>
@@ -204,7 +204,7 @@
     /// </exception>
     public virtual TResource GetResourceByName(string resourceName)
     {
-        return context.Resources.FirstOrDefault(r => r.Name == resourceName)
+        return Context.Resources.FirstOrDefault(r => r.Name == resourceName)
             ?? throw new ResourceNotFoundException(resourceName);
     }
 
@@ -219,7 +219,7 @@
     /// </exception>
     public virtual async Task<TResource> GetResourceByNameAsync(string resourceName)
     {
-        return await context.Resources.FirstOrDefaultAsync(r => r.Name == resourceName)
+        return await Context.Resources.FirstOrDefaultAsync(r => r.Name == resourceName)
             ?? throw new ResourceNotFoundException(resourceName);
     }
 }