Update error for "generic ViewComponents" to explain what a valid view component is

[NTaylorMullen]

Given the following example:

public class FooViewComponent<T>
{
    public string Invoke(T value)
    {
        return value.ToString();
    }
}

Trying to run this like so:

@{
    Write(await Component.InvokeAsync<FooViewComponent<string>>(new
    {
        value = "Hello World"
    }));
}

Results in the following:

[NTaylorMullen]

I'm not entirely sure if we want to support generic ViewComponents due to how you can invoke a ViewComponent. IE await Component.InvokeAsync("VC Name", params...).

At worst we should at least create a more readable error.

[Eilon]

After discussion with @NTaylorMullen @dougbu and @rynowak we decided to just update the error message to be more like:

A view component named '<type>' could not be found. View components must be X, Y, and Z.

Where X, Y, and Z are an indication of the criteria for being a view component, e.g.:

• Public non-generic concrete type
• Implement IFoo interface
• Whatever else

(I don't recall the exact rules.)

[iscifoni]

Hi,

from the code ( https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewComponents/ViewComponentConventions.cs#L76 ), I think the rules is :

• is class
• is public
• not is abstract
• not contains generic parameter
• not is defined NonViewComponentAttribute
• is defined ViewComponentAttribute
• class name end with "ViewComponent"

So the message could be :

"A view component named '<type>' could not be found. View components must be a Public non-generic concrete type, not contains generic parameters, use ViewComponentAttribute or class name end with 'ViewComponent' ".