Revise action name normalization.

src/Volo.Abp.AspNetCore.Mvc/Volo/Abp/AspNetCore/Mvc/AbpAppServiceConvention.cs

@@ -214,7 +214,7 @@ namespace Volo.Abp.AspNetCore.Mvc
             return HttpMethodHelper.GetConventionalVerbForMethodName(action.ActionName);
         }
 
-        protected virtual void NormalizeSelectorRoutes(string rootPath, string controllerName, ActionModel action, AbpControllerAssemblySetting configuration)
+        protected virtual void NormalizeSelectorRoutes(string rootPath, string controllerName, ActionModel action, [CanBeNull] AbpControllerAssemblySetting configuration)
         {
             foreach (var selector in action.Selectors)
             {
@@ -238,7 +238,7 @@ namespace Volo.Abp.AspNetCore.Mvc
             return _options.AppServiceControllers.ControllerAssemblySettings.GetSettingOrNull(controllerType);
         }
 
-        protected virtual AttributeRouteModel CreateAbpServiceAttributeRouteModel(string rootPath, string controllerName, ActionModel action, string httpMethod, AbpControllerAssemblySetting configuration)
+        protected virtual AttributeRouteModel CreateAbpServiceAttributeRouteModel(string rootPath, string controllerName, ActionModel action, string httpMethod, [CanBeNull] AbpControllerAssemblySetting configuration)
         {
             return new AttributeRouteModel(
                 new RouteAttribute(
@@ -247,7 +247,7 @@ namespace Volo.Abp.AspNetCore.Mvc
             );
         }
 
-        protected virtual string CalculateRouteTemplate(string rootPath, string controllerName, ActionModel action, string httpMethod, AbpControllerAssemblySetting configuration)
+        protected virtual string CalculateRouteTemplate(string rootPath, string controllerName, ActionModel action, string httpMethod, [CanBeNull] AbpControllerAssemblySetting configuration)
         {
             var controllerNameInUrl = NormalizeUrlControllerName(rootPath, controllerName, action, httpMethod, configuration);
 
@@ -260,7 +260,7 @@ namespace Volo.Abp.AspNetCore.Mvc
             }
 
             //Add action name if needed
-            var actionNameInUrl = NormalizeUrlActionName(rootPath, controllerName, action, httpMethod);
+            var actionNameInUrl = NormalizeUrlActionName(rootPath, controllerName, action, httpMethod, configuration);
             if (!actionNameInUrl.IsNullOrEmpty())
             {
                 url += $"/{actionNameInUrl.ToCamelCase()}";
@@ -276,16 +276,26 @@ namespace Volo.Abp.AspNetCore.Mvc
             return url;
         }
 
-        protected virtual string NormalizeUrlActionName(string rootPath, string controllerName, ActionModel action, string httpMethod)
+        protected virtual string NormalizeUrlActionName(string rootPath, string controllerName, ActionModel action, string httpMethod, [CanBeNull] AbpControllerAssemblySetting configuration)
         {
-            var context = new UrlActionNameNormalizerContext(rootPath, controllerName, action, httpMethod, action.ActionName);
+            var actionNameInUrl = HttpMethodHelper
+                .RemoveHttpMethodPrefix(action.ActionName, httpMethod)
+                .RemovePostFix("Async");
 
-            foreach (var normalizer in _options.AppServiceControllers.UrlActionNameNormalizers)
+            if (configuration?.UrlActionNameNormalizer == null)
             {
-                normalizer.Normalize(context);
+                return actionNameInUrl;
             }
 
-            return context.ActionNameInUrl;
+            return configuration.UrlActionNameNormalizer(
+                new UrlActionNameNormalizerContext(
+                    rootPath,
+                    controllerName,
+                    action,
+                    actionNameInUrl,
+                    httpMethod
+                )
+            );
         }
 
         protected virtual string NormalizeUrlControllerName(string rootPath, string controllerName, ActionModel action, string httpMethod, [CanBeNull] AbpControllerAssemblySetting configuration)

src/Volo.Abp.AspNetCore.Mvc/Volo/Abp/AspNetCore/Mvc/AbpControllerAssemblySetting.cs

@@ -22,6 +22,9 @@ namespace Volo.Abp.AspNetCore.Mvc
         [CanBeNull]
         public Func<UrlControllerNameNormalizerContext, string> UrlControllerNameNormalizer { get; set; }
 
+        [CanBeNull]
+        public Func<UrlActionNameNormalizerContext, string> UrlActionNameNormalizer { get; set; }
+
         public AbpControllerAssemblySetting([NotNull] Assembly assembly, [NotNull] string rootPath)
         {
             Check.NotNull(assembly, rootPath);

src/Volo.Abp.AspNetCore.Mvc/Volo/Abp/AspNetCore/Mvc/AbpControllerAssemblySettingBuilder.cs

@@ -29,5 +29,11 @@ namespace Volo.Abp.AspNetCore.Mvc
             _setting.UrlControllerNameNormalizer = normalizer;
             return this;
         }
+
+        public AbpControllerAssemblySettingBuilder NormalizeActionNameInUrl(Func<UrlActionNameNormalizerContext, string> normalizer)
+        {
+            _setting.UrlActionNameNormalizer = normalizer;
+            return this;
+        }
     }
 }

src/Volo.Abp.AspNetCore.Mvc/Volo/Abp/AspNetCore/Mvc/AppServiceControllerOptions.cs

@@ -12,8 +12,6 @@ namespace Volo.Abp.AspNetCore.Mvc
 
         public List<Type> FormBodyBindingIgnoredTypes { get; }
         
-        public List<IUrlActionNameNormalizer> UrlActionNameNormalizers { get; }
-
         public AppServiceControllerOptions()
         {
             ControllerAssemblySettings = new ControllerAssemblySettingList();
@@ -22,11 +20,6 @@ namespace Volo.Abp.AspNetCore.Mvc
             {
                 typeof(IFormFile)
             };
-
-            UrlActionNameNormalizers = new List<IUrlActionNameNormalizer>
-            {
-                new DefaultUrlActionNameNormalizer()
-            };
         }
 
         public AbpControllerAssemblySettingBuilder CreateFor(Assembly assembly, string rootPath = ModuleApiDescriptionModel.DefaultRootPath)

src/Volo.Abp.AspNetCore.Mvc/Volo/Abp/AspNetCore/Mvc/DefaultUrlActionNameNormalizer.cs

@@ -1,20 +0,0 @@
-using System;
-using Volo.Abp.Http;
-
-namespace Volo.Abp.AspNetCore.Mvc
-{
-    public class DefaultUrlActionNameNormalizer : IUrlActionNameNormalizer
-    {
-        public void Normalize(UrlActionNameNormalizerContext context)
-        {
-            if (context.ActionNameInUrl.IsNullOrEmpty())
-            {
-                return;
-            }
-
-            context.ActionNameInUrl = HttpMethodHelper
-                .RemoveHttpMethodPrefix(context.ActionNameInUrl, context.HttpMethod)
-                .RemovePostFix("Async");
-        }
-    }
-}

src/Volo.Abp.AspNetCore.Mvc/Volo/Abp/AspNetCore/Mvc/IUrlActionNameNormalizer.cs

@@ -1,7 +0,0 @@
-namespace Volo.Abp.AspNetCore.Mvc
-{
-    public interface IUrlActionNameNormalizer
-    {
-        void Normalize(UrlActionNameNormalizerContext context);
-    }
-}

src/Volo.Abp.AspNetCore.Mvc/Volo/Abp/AspNetCore/Mvc/UrlActionNameNormalizerContext.cs

@@ -10,18 +10,17 @@ namespace Volo.Abp.AspNetCore.Mvc
 
         public ActionModel Action { get; }
 
-        public string HttpMethod { get; }
+        public string ActionNameInUrl { get; }
 
-        public string ActionNameInUrl { get; set; }
+        public string HttpMethod { get; }
 
-        public UrlActionNameNormalizerContext(string rootPath, string controllerName, ActionModel action, string httpMethod, string actionNameInUrl)
+        public UrlActionNameNormalizerContext(string rootPath, string controllerName, ActionModel action, string actionNameInUrl, string httpMethod)
         {
             RootPath = rootPath;
             ControllerName = controllerName;
             Action = action;
-            HttpMethod = httpMethod;
-
             ActionNameInUrl = actionNameInUrl;
+            HttpMethod = httpMethod;
         }
     }
 }

test/Volo.Abp.AspNetCore.Mvc.Tests/Volo/Abp/AspNetCore/App/AbpAspNetCoreMvcTestModule.cs

@@ -1,4 +1,5 @@
-using Microsoft.AspNetCore.Builder;
+using System;
+using Microsoft.AspNetCore.Builder;
 using Microsoft.Extensions.DependencyInjection;
 using Volo.Abp.AspNetCore.Modularity;
 using Volo.Abp.AspNetCore.Mvc;
@@ -24,8 +25,15 @@ namespace Volo.Abp.AspNetCore.App
 
             services.Configure<AbpAspNetCoreMvcOptions>(options =>
             {
-                options.AppServiceControllers.CreateFor(typeof(TestAppModule).Assembly);
-                options.AppServiceControllers.UrlActionNameNormalizers.Add(new PhoneBookUrlActionNameNormalizer());
+                options
+                    .AppServiceControllers
+                    .CreateFor(typeof(TestAppModule).Assembly)
+                    .NormalizeActionNameInUrl(
+                        context =>
+                            string.Equals(context.ActionNameInUrl, "phone", StringComparison.OrdinalIgnoreCase)
+                                ? "phones"
+                                : context.ActionNameInUrl
+                    );
             });
 
             services.AddAssemblyOf<AbpAspNetCoreMvcTestModule>();

test/Volo.Abp.AspNetCore.Mvc.Tests/Volo/Abp/AspNetCore/App/PhoneBookUrlActionNameNormalizer.cs

@@ -1,17 +0,0 @@
-using System;
-using Volo.Abp.AspNetCore.Mvc;
-
-namespace Volo.Abp.AspNetCore.App
-{
-    public class PhoneBookUrlActionNameNormalizer : IUrlActionNameNormalizer
-    {
-        public void Normalize(UrlActionNameNormalizerContext context)
-        {
-            if (string.Equals(context.ActionNameInUrl, "phone", StringComparison.OrdinalIgnoreCase))
-            {
-                context.ActionNameInUrl = "phones";
-                return;
-            }
-        }
-    }
-}