Remove tenant permission provider

framework/src/Volo.Abp.Authorization/Volo/Abp/Authorization/Permissions/PermissionDefinition.cs

@@ -24,11 +24,6 @@ namespace Volo.Abp.Authorization.Permissions
         /// </summary>
         public List<string> Providers { get; } //TODO: Rename to AllowedProviders?
 
-        /// <summary>
-        /// Used to mark this permission as also a feature of the application.
-        /// </summary>
-        public bool IsFeature { get; set; }
-
         public ILocalizableString DisplayName
         {
             get => _displayName;
@@ -60,12 +55,10 @@ namespace Volo.Abp.Authorization.Permissions
 
         protected internal PermissionDefinition(
             [NotNull] string name, 
-            ILocalizableString displayName = null,
+            ILocalizableString displayName = null)
-            bool isFeature = false)
         {
             Name = Check.NotNull(name, nameof(name));
             DisplayName = displayName ?? new FixedLocalizableString(name);
-            IsFeature = isFeature;
 
             Properties = new Dictionary<string, object>();
             Providers = new List<string>();
@@ -74,10 +67,9 @@ namespace Volo.Abp.Authorization.Permissions
 
         public virtual PermissionDefinition AddChild(
             [NotNull] string name, 
-            ILocalizableString displayName = null,
+            ILocalizableString displayName = null)
-            bool isFeature = false)
         {
-            var child = new PermissionDefinition(name, displayName, isFeature)
+            var child = new PermissionDefinition(name, displayName)
             {
                 Parent = this
             };

framework/src/Volo.Abp.Authorization/Volo/Abp/Authorization/Permissions/PermissionGroupDefinition.cs

@@ -50,10 +50,9 @@ namespace Volo.Abp.Authorization.Permissions
 
         public virtual PermissionDefinition AddPermission(
             string name, 
-            ILocalizableString displayName = null,
+            ILocalizableString displayName = null)
-            bool isFeature = false)
         {
-            var permission = new PermissionDefinition(name, displayName, isFeature);
+            var permission = new PermissionDefinition(name, displayName);
 
             _permissions.Add(permission);
 

framework/src/Volo.Abp.MultiTenancy.Abstractions/Volo/Abp/MultiTenancy/AbpMultiTenancyAbstractionsModule.cs

@@ -1,5 +1,4 @@
 using Volo.Abp.Authorization;
-using Volo.Abp.Authorization.Permissions;
 using Volo.Abp.Data;
 using Volo.Abp.Modularity;
 
@@ -11,12 +10,6 @@ namespace Volo.Abp.MultiTenancy
         )]
     public class AbpMultiTenancyAbstractionsModule : AbpModule //TODO: Rename to AbpMultiTenancyModule?
     {
-        public override void ConfigureServices(ServiceConfigurationContext context)
+
-        {
-            Configure<PermissionOptions>(options =>
-            {
-                options.ValueProviders.Add<TenantPermissionValueProvider>();
-            });
-        }
     }
 }

framework/src/Volo.Abp.MultiTenancy.Abstractions/Volo/Abp/MultiTenancy/TenantPermissionValueProvider.cs

@@ -1,45 +0,0 @@
-using System.Threading.Tasks;
-using Volo.Abp.Authorization.Permissions;
-using Volo.Abp.Security.Claims;
-
-namespace Volo.Abp.MultiTenancy
-{
-    public class TenantPermissionValueProvider : PermissionValueProvider
-    {
-        public const string ProviderName = "Tenant";
-
-        public override string Name => ProviderName;
-
-        protected ICurrentTenant CurrentTenant { get; }
-
-        public TenantPermissionValueProvider(
-            IPermissionStore permissionStore,
-            ICurrentTenant currentTenant)
-            : base(permissionStore)
-        {
-            CurrentTenant = currentTenant;
-        }
-
-        public override async Task<PermissionGrantResult> CheckAsync(PermissionValueCheckContext context)
-        {
-            if (!context.Permission.IsFeature)
-            {
-                return PermissionGrantResult.Undefined;
-            }
-
-            var tenantId = context.Principal?.FindFirst(AbpClaimTypes.TenantId)?.Value;
-
-            if (tenantId == null)
-            {
-                return PermissionGrantResult.Undefined;
-            }
-
-            using (CurrentTenant.Change(null))
-            {
-                return await PermissionStore.IsGrantedAsync(context.Permission.Name, Name, tenantId)
-                    ? PermissionGrantResult.Granted
-                    : PermissionGrantResult.Undefined;
-            }
-        }
-    }
-}