Resolved #1026: Align AbpValidationActionFilter with MethodInvocationValidator.

framework/src/Volo.Abp.AspNetCore.Mvc/Volo/Abp/AspNetCore/Mvc/Validation/AbpValidationActionFilter.cs

@@ -26,11 +26,8 @@ namespace Volo.Abp.AspNetCore.Mvc.Validation
                 return;
             }
 
-            using (AbpCrossCuttingConcerns.Applying(context.Controller, AbpCrossCuttingConcerns.Validation))
-            {
-                _validator.Validate(context.ModelState);
-                await next();
-            }
+            _validator.Validate(context.ModelState);
+            await next();
         }
     }
 }

framework/src/Volo.Abp.Authorization/Volo/Abp/Authorization/AuthorizationInterceptor.cs

@@ -1,5 +1,4 @@
 using System.Threading.Tasks;
-using Volo.Abp.Aspects;
 using Volo.Abp.DependencyInjection;
 using Volo.Abp.DynamicProxy;
 using Volo.Abp.Threading;
@@ -17,24 +16,12 @@ namespace Volo.Abp.Authorization
 
         public override void Intercept(IAbpMethodInvocation invocation)
         {
-            if (AbpCrossCuttingConcerns.IsApplied(invocation.TargetObject, AbpCrossCuttingConcerns.Authorization))
-            {
-                invocation.Proceed();
-                return;
-            }
-
             AsyncHelper.RunSync(() => AuthorizeAsync(invocation));
             invocation.Proceed();
         }
 
         public override async Task InterceptAsync(IAbpMethodInvocation invocation)
         {
-            if (AbpCrossCuttingConcerns.IsApplied(invocation.TargetObject, AbpCrossCuttingConcerns.Authorization))
-            {
-                await invocation.ProceedAsync();
-                return;
-            }
-
             await AuthorizeAsync(invocation);
             await invocation.ProceedAsync();
         }

framework/src/Volo.Abp.Core/Volo/Abp/Aspects/AbpCrossCuttingConcerns.cs

@@ -9,9 +9,7 @@ namespace Volo.Abp.Aspects
         //TODO: Move these constants to their own assemblies!
 
         public const string Auditing = "AbpAuditing";
-        public const string Validation = "AbpValidation";
         public const string UnitOfWork = "AbpUnitOfWork";
-        public const string Authorization = "AbpAuthorization";
         public const string FeatureChecking = "AbpFeatureChecking";
 
         public static void AddApplied(object obj, params string[] concerns)

framework/src/Volo.Abp.Validation/Volo/Abp/Validation/MethodInvocationValidator.cs

@@ -96,7 +96,13 @@ namespace Volo.Abp.Validation
                              parameterInfo.IsOut ||
                              TypeHelper.IsPrimitiveExtended(parameterInfo.ParameterType, includeEnums: true);
 
-            context.Errors.AddRange(_objectValidator.GetErrors(parameterValue, parameterInfo.Name, allowNulls));
+            context.Errors.AddRange(
+                _objectValidator.GetErrors(
+                    parameterValue,
+                    parameterInfo.Name,
+                    allowNulls
+                )
+            );
         }
     }
 }

framework/src/Volo.Abp.Validation/Volo/Abp/Validation/ObjectValidator.cs

@@ -56,7 +56,8 @@ namespace Volo.Abp.Validation
             {
                 foreach (var contributorType in Options.ObjectValidationContributors)
                 {
-                    var contributor = (IObjectValidationContributor) scope.ServiceProvider.GetRequiredService(contributorType);
+                    var contributor = (IObjectValidationContributor) 
+                        scope.ServiceProvider.GetRequiredService(contributorType);
                     contributor.AddErrors(context);
                 }
             }

framework/src/Volo.Abp.Validation/Volo/Abp/Validation/ValidationInterceptor.cs

@@ -1,5 +1,4 @@
 using System.Threading.Tasks;
-using Volo.Abp.Aspects;
 using Volo.Abp.DependencyInjection;
 using Volo.Abp.DynamicProxy;
 
@@ -16,27 +15,13 @@ namespace Volo.Abp.Validation
 
         public override void Intercept(IAbpMethodInvocation invocation)
         {
-            if (AbpCrossCuttingConcerns.IsApplied(invocation.TargetObject, AbpCrossCuttingConcerns.Validation))
-            {
-                invocation.Proceed();
-                return;
-            }
-
             Validate(invocation);
-
             invocation.Proceed();
         }
 
         public override async Task InterceptAsync(IAbpMethodInvocation invocation)
         {
-            if (AbpCrossCuttingConcerns.IsApplied(invocation.TargetObject, AbpCrossCuttingConcerns.Validation))
-            {
-                await invocation.ProceedAsync();
-                return;
-            }
-
             Validate(invocation);
-
             await invocation.ProceedAsync();
         }