Add support for specifying filters on page models.

Fixes #6334

src/Microsoft.AspNetCore.Mvc.Core/ApplicationModels/ActionModel.cs

@@ -120,7 +120,7 @@ public ActionModel(ActionModel other)
 
         MemberInfo ICommonModel.MemberInfo => ActionMethod;
 
-        string ICommonModel.Name => ActionName;
+        string ICommonModel.MethodName => ActionName;
 
         public IList<SelectorModel> Selectors { get; }
     }

src/Microsoft.AspNetCore.Mvc.Core/ApplicationModels/ControllerModel.cs

@@ -87,7 +87,7 @@ public ControllerModel(ControllerModel other)
 
         MemberInfo ICommonModel.MemberInfo => ControllerType;
 
-        string ICommonModel.Name => ControllerName;
+        string ICommonModel.MethodName => ControllerName;
 
         public string ControllerName { get; set; }
 

src/Microsoft.AspNetCore.Mvc.Core/ApplicationModels/ICommonModel.cs

@@ -10,6 +10,6 @@ public interface ICommonModel : IPropertyModel
     {
         IReadOnlyList<object> Attributes { get; }
         MemberInfo MemberInfo { get; }
-        string Name { get; }
+        string MethodName { get; }
     }
 }

src/Microsoft.AspNetCore.Mvc.Core/ApplicationModels/ParameterModel.cs

@@ -54,7 +54,7 @@ public ParameterModel(ParameterModel other)
 
         MemberInfo ICommonModel.MemberInfo => ParameterInfo.Member;
 
-        string ICommonModel.Name => ParameterName;
+        string ICommonModel.MethodName => ParameterName;
 
         public ParameterInfo ParameterInfo { get; }
 

src/Microsoft.AspNetCore.Mvc.Core/ApplicationModels/PropertyModel.cs

@@ -72,7 +72,7 @@ public PropertyModel(PropertyModel other)
 
         MemberInfo ICommonModel.MemberInfo => PropertyInfo;
 
-        string ICommonModel.Name => PropertyName;
+        string ICommonModel.MethodName => PropertyName;
 
         /// <summary>
         /// Gets or sets the <see cref="BindingInfo"/> associated with this model.

src/Microsoft.AspNetCore.Mvc.RazorPages/ApplicationModels/IPageRouteModelConvention.cs

@@ -0,0 +1,17 @@
+// Copyright (c) .NET Foundation. All rights reserved.
+// Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
+
+namespace Microsoft.AspNetCore.Mvc.ApplicationModels
+{
+    /// <summary>
+    /// Allows customization of the of the <see cref="PageRouteModel"/>.
+    /// </summary>
+    public interface IPageRouteModelConvention
+    {
+        /// <summary>
+        /// Called to apply the convention to the <see cref="PageRouteModel"/>.
+        /// </summary>
+        /// <param name="model">The <see cref="PageRouteModel"/>.</param>
+        void Apply(PageRouteModel model);
+    }
+}

src/Microsoft.AspNetCore.Mvc.RazorPages/ApplicationModels/IPageRouteModelProvider.cs

@@ -0,0 +1,44 @@
+// Copyright (c) .NET Foundation. All rights reserved.
+// Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
+
+namespace Microsoft.AspNetCore.Mvc.ApplicationModels
+{
+    /// <summary>
+    /// Builds or modifies an <see cref="PageRouteModelProviderContext"/> for Razor Page routing.
+    /// </summary>
+    public interface IPageRouteModelProvider
+    {
+        /// <summary>
+        /// Gets the order value for determining the order of execution of providers. Providers execute in
+        /// ascending numeric value of the <see cref="Order"/> property.
+        /// </summary>
+        /// <remarks>
+        /// <para>
+        /// Providers are executed in an ordering determined by an ascending sort of the <see cref="Order"/> property.
+        /// A provider with a lower numeric value of <see cref="Order"/> will have its
+        /// <see cref="OnProvidersExecuting"/> called before that of a provider with a higher numeric value of
+        /// <see cref="Order"/>. The <see cref="OnProvidersExecuted"/> method is called in the reverse ordering after
+        /// all calls to <see cref="OnProvidersExecuting"/>. A provider with a lower numeric value of
+        /// <see cref="Order"/> will have its <see cref="OnProvidersExecuted"/> method called after that of a provider
+        /// with a higher numeric value of <see cref="Order"/>.
+        /// </para>
+        /// <para>
+        /// If two providers have the same numeric value of <see cref="Order"/>, then their relative execution order
+        /// is undefined.
+        /// </para>
+        /// </remarks>
+        int Order { get; }
+
+        /// <summary>
+        /// Executed for the first pass of building <see cref="PageRouteModel"/> instances. See <see cref="Order"/>.
+        /// </summary>
+        /// <param name="context">The <see cref="PageRouteModelProviderContext"/>.</param>
+        void OnProvidersExecuting(PageRouteModelProviderContext context);
+
+        /// <summary>
+        /// Executed for the second pass of building <see cref="PageRouteModel"/> instances. See <see cref="Order"/>.
+        /// </summary>
+        /// <param name="context">The <see cref="PageRouteModelProviderContext"/>.</param>
+        void OnProvidersExecuted(PageRouteModelProviderContext context);
+    }
+}

src/Microsoft.AspNetCore.Mvc.RazorPages/ApplicationModels/PageApplicationModel.cs

@@ -4,7 +4,10 @@
 using System;
 using System.Collections.Generic;
 using System.Linq;
+using System.Reflection;
 using Microsoft.AspNetCore.Mvc.Filters;
+using Microsoft.AspNetCore.Mvc.RazorPages;
+using Microsoft.Extensions.Internal;
 
 namespace Microsoft.AspNetCore.Mvc.ApplicationModels
 {
@@ -16,26 +19,21 @@ public class PageApplicationModel
         /// <summary>
         /// Initializes a new instance of <see cref="PageApplicationModel"/>.
         /// </summary>
-        /// <param name="relativePath">The application relative path of the page.</param>
-        /// <param name="viewEnginePath">The path relative to the base path for page discovery.</param>
-        public PageApplicationModel(string relativePath, string viewEnginePath)
+        public PageApplicationModel(
+            PageActionDescriptor actionDescriptor,
+            TypeInfo handlerType,
+            IReadOnlyList<object> handlerAttributes)
         {
-            if (relativePath == null)
-            {
-                throw new ArgumentNullException(nameof(relativePath));
-            }
-
-            if (viewEnginePath == null)
-            {
-                throw new ArgumentNullException(nameof(viewEnginePath));
-            }
-
-            RelativePath = relativePath;
-            ViewEnginePath = viewEnginePath;
+            ActionDescriptor = actionDescriptor ?? throw new ArgumentNullException(nameof(actionDescriptor));
+            HandlerType = handlerType;
 
             Filters = new List<IFilterMetadata>();
-            Properties = new Dictionary<object, object>();
-            Selectors = new List<SelectorModel>();
+            Properties = new CopyOnWriteDictionary<object, object>(
+                actionDescriptor.Properties, 
+                EqualityComparer<object>.Default);
+            HandlerMethods = new List<PageHandlerModel>();
+            HandlerProperties = new List<PagePropertyModel>();
+            HandlerTypeAttributes = handlerAttributes;
         }
 
         /// <summary>
@@ -49,24 +47,38 @@ public PageApplicationModel(PageApplicationModel other)
                 throw new ArgumentNullException(nameof(other));
             }
 
-            RelativePath = other.RelativePath;
-            ViewEnginePath = other.ViewEnginePath;
+            ActionDescriptor = other.ActionDescriptor;
+            HandlerType = other.HandlerType;
+            PageType = other.PageType;
+            ModelType = other.ModelType;
 
             Filters = new List<IFilterMetadata>(other.Filters);
             Properties = new Dictionary<object, object>(other.Properties);
 
-            Selectors = new List<SelectorModel>(other.Selectors.Select(m => new SelectorModel(m)));
+            HandlerMethods = new List<PageHandlerModel>(other.HandlerMethods.Select(m => new PageHandlerModel(m)));
+            HandlerProperties  = new List<PagePropertyModel>(other.HandlerProperties.Select(p => new PagePropertyModel(p)));
+            HandlerTypeAttributes = other.HandlerTypeAttributes;
         }
 
+        /// <summary>
+        /// Gets the <see cref="PageActionDescriptor"/>.
+        /// </summary>
+        public PageActionDescriptor ActionDescriptor { get; }
+
         /// <summary>
         /// Gets the application root relative path for the page.
         /// </summary>
-        public string RelativePath { get; }
+        public string RelativePath => ActionDescriptor.RelativePath;
 
         /// <summary>
         /// Gets the path relative to the base path for page discovery.
         /// </summary>
-        public string ViewEnginePath { get; }
+        public string ViewEnginePath => ActionDescriptor.ViewEnginePath;
+
+        /// <summary>
+        /// Gets the route template for the page.
+        /// </summary>
+        public string RouteTemplate => ActionDescriptor.AttributeRouteInfo?.Template;
 
         /// <summary>
         /// Gets the applicable <see cref="IFilterMetadata"/> instances.
@@ -79,8 +91,33 @@ public PageApplicationModel(PageApplicationModel other)
         public IDictionary<object, object> Properties { get; }
 
         /// <summary>
-        /// Gets the <see cref="SelectorModel"/> instances.
+        /// Gets or sets the <see cref="TypeInfo"/> of the Razor page.
+        /// </summary>
+        public TypeInfo PageType { get; set; }
+
+        /// <summary>
+        /// Gets or sets the <see cref="TypeInfo"/> of the Razor page model.
+        /// </summary>
+        public TypeInfo ModelType { get; set; }
+
+        /// <summary>
+        /// Gets the <see cref="TypeInfo"/> of the handler.
+        /// </summary>
+        public TypeInfo HandlerType { get; }
+
+        /// <summary>
+        /// Gets the sequence of attributes declared on <see cref="HandlerType"/>.
+        /// </summary>
+        public IReadOnlyList<object> HandlerTypeAttributes { get; }
+
+        /// <summary>
+        /// Gets the sequence of <see cref="PageHandlerModel"/> instances.
+        /// </summary>
+        public IList<PageHandlerModel> HandlerMethods { get; }
+
+        /// <summary>
+        /// Gets the sequence of <see cref="PagePropertyModel"/> instances on <see cref="PageHandlerModel"/>.
         /// </summary>
-        public IList<SelectorModel> Selectors { get; }
+        public IList<PagePropertyModel> HandlerProperties { get; }
     }
 }

src/Microsoft.AspNetCore.Mvc.RazorPages/ApplicationModels/PageApplicationModelProviderContext.cs

@@ -1,7 +1,8 @@
 // Copyright (c) .NET Foundation. All rights reserved.
 // Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
 
-using System.Collections.Generic;
+using System.Reflection;
+using Microsoft.AspNetCore.Mvc.RazorPages;
 
 namespace Microsoft.AspNetCore.Mvc.ApplicationModels
 {
@@ -10,9 +11,25 @@ namespace Microsoft.AspNetCore.Mvc.ApplicationModels
     /// </summary>
     public class PageApplicationModelProviderContext
     {
+        public PageApplicationModelProviderContext(PageActionDescriptor descriptor, TypeInfo pageTypeInfo)
+        {
+            ActionDescriptor = descriptor;
+            PageType = pageTypeInfo;
+        }
+
+        /// <summary>
+        /// Gets the <see cref="PageActionDescriptor"/>.
+        /// </summary>
+        public PageActionDescriptor ActionDescriptor { get; }
+
+        /// <summary>
+        /// Gets the page <see cref="TypeInfo"/>.
+        /// </summary>
+        public TypeInfo PageType { get; }
+
         /// <summary>
-        /// Gets the <see cref="PageApplicationModel"/> instances.
+        /// Gets or sets the <see cref="ApplicationModels.PageApplicationModel"/>.
         /// </summary>
-        public IList<PageApplicationModel> Results { get; } = new List<PageApplicationModel>();
+        public PageApplicationModel PageApplicationModel { get; set; }
     }
 }

src/Microsoft.AspNetCore.Mvc.RazorPages/ApplicationModels/PageHandlerModel.cs

@@ -0,0 +1,98 @@
+// Copyright (c) .NET Foundation. All rights reserved.
+// Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
+
+using System;
+using System.Collections.Generic;
+using System.Diagnostics;
+using System.Linq;
+using System.Reflection;
+
+namespace Microsoft.AspNetCore.Mvc.ApplicationModels
+{
+    /// <summary>
+    /// Represents a handler in a <see cref="PageApplicationModel"/>.
+    /// </summary>
+    [DebuggerDisplay("PageHandlerModel: Name={" + nameof(PageHandlerModel.MethodName) + "}")]
+    public class PageHandlerModel : ICommonModel
+    {
+        /// <summary>
+        /// Creates a new <see cref="PageHandlerModel"/>.
+        /// </summary>
+        /// <param name="handlerMethod">The <see cref="System.Reflection.MethodInfo"/> for the handler.</param>
+        /// <param name="attributes">Any attributes annotated on the handler method.</param>
+        public PageHandlerModel(
+            MethodInfo handlerMethod,
+            IReadOnlyList<object> attributes)
+        {
+            MethodInfo = handlerMethod ?? throw new ArgumentNullException(nameof(handlerMethod));
+            Attributes = attributes ?? throw new ArgumentNullException(nameof(attributes));
+
+            Parameters = new List<PageParameterModel>();
+            Properties = new Dictionary<object, object>();
+        }
+
+        /// <summary>
+        /// Creats a new instance of <see cref="PageHandlerModel"/> from a given <see cref="PageHandlerModel"/>.
+        /// </summary>
+        /// <param name="other">The <see cref="PageHandlerModel"/> which needs to be copied.</param>
+        public PageHandlerModel(PageHandlerModel other)
+        {
+            if (other == null)
+            {
+                throw new ArgumentNullException(nameof(other));
+            }
+
+            MethodInfo = other.MethodInfo;
+            HandlerName = other.HandlerName;
+            HttpMethod = other.HttpMethod;
+            MethodName = other.MethodName;
+
+            Page = other.Page;
+
+            // These are just metadata, safe to create new collections
+            Attributes = new List<object>(other.Attributes);
+            Properties = new Dictionary<object, object>(other.Properties);
+
+            // Make a deep copy of other 'model' types.
+            Parameters = new List<PageParameterModel>(other.Parameters.Select(p => new PageParameterModel(p) { Handler = this }));
+        }
+
+        /// <summary>
+        /// Gets the <see cref="System.Reflection.MethodInfo"/> for the handler.
+        /// </summary>
+        public MethodInfo MethodInfo { get; }
+
+        /// <summary>
+        /// Gets or sets the HTTP method supported by this handler.
+        /// </summary>
+        public string HttpMethod { get; set; }
+
+        /// <summary>
+        /// Gets or sets the handler method name.
+        /// </summary>
+        public string HandlerName { get; set; }
+
+        /// <summary>
+        /// Gets or sets a descriptive name for the handler.
+        /// </summary>
+        public string MethodName { get; set; }
+
+        /// <summary>
+        /// Gets the seqeunce of <see cref="PageParameterModel"/> instances.
+        /// </summary>
+        public IList<PageParameterModel> Parameters { get; }
+
+        /// <summary>
+        /// Gets or sets the <see cref="PageApplicationModel"/>.
+        /// </summary>
+        public PageApplicationModel Page { get; set; }
+
+        /// <inheritdoc />
+        public IReadOnlyList<object> Attributes { get; }
+
+        /// <inheritdoc />
+        public IDictionary<object, object> Properties { get; }
+
+        MemberInfo ICommonModel.MemberInfo => MethodInfo;
+    }
+}