Introducing IPageModelActivatorProvider#5712
Conversation
| namespace Microsoft.AspNetCore.Mvc.RazorPages | ||
| { | ||
| /// <summary> | ||
| /// Provides methods for creation and disposal of Razor page models. |
There was a problem hiding this comment.
The P should be capitalized because it's a proper name #JustEilonThings
| using Microsoft.AspNetCore.Mvc.Internal; | ||
| using Microsoft.Extensions.DependencyInjection; | ||
|
|
||
| namespace Microsoft.AspNetCore.Mvc.RazorPages.Internal |
There was a problem hiding this comment.
If we have a .Infrastructure namespace already parked it would be good to put this there and document it for extensibility. It was common in the past to subclass our factories.
| } | ||
|
|
||
| var factory = ActivatorUtilities.CreateFactory(modelTypeInfo, Type.EmptyTypes); | ||
| return (context) => factory(context.HttpContext.RequestServices, EmptyArray<object>.Instance); |
There was a problem hiding this comment.
It would be dope to put this part into a virtual method, then there's less boilerplate if you want to use something other than ActivatorUtilities.
There was a problem hiding this comment.
Nevermind, my brain is crazy. Ignore this comment as soon as possible
| /// the model instance. The property must have a public set method. | ||
| /// </summary> | ||
| [AttributeUsage(AttributeTargets.Property, AllowMultiple = false, Inherited = true)] | ||
| public class PageContextAttribute : Attribute |
There was a problem hiding this comment.
Should this go in .Infrastructure? Do whatever we did with ActionContextAttribute.
| /// <summary> | ||
| /// <see cref="IPageActivatorProvider"/> that uses type activation to create Pages. | ||
| /// </summary> | ||
| public class DefaultPageModelActivatorProvider : IPageModelActivatorProvider |
There was a problem hiding this comment.
We'll also need the equivalent plumbing for the DI versions of these. Please open an issue for that or do it right away 👍
looks good
3 participants
Fixes #5480