Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

AmbiguousActionException: Multiple actions matched. The following actions matched route data and had all constraints satisfied

I am creating a website using ASP.NET Core MVC. When I click on an action I get this error:

AmbiguousActionException: Multiple actions matched. The following actions matched route data and had all constraints satisfied:

Web.Controllers.ChangeEventsController.Create (Web)
Web.Controllers.ProductsController.CreateChangeEvent (Web)

This is how I defined my action in the index.cshtmlm for my ProductsController:

<a asp-controller="ChangeEvents" asp-action="Create" asp-route-id="@item.Id">Create Change Event</a>

Here is my routing:

        app.UseMvc(routes =>
        {
            routes.MapRoute(
                name: "default",
                template: "{controller=Home}/{action=Index}/{id?}");
        });

Here is how I defined the actions:

// ChangeEventsController
[HttpGet("{id}")]
public IActionResult Create(Guid id)

// ProductsController
[HttpGet("{id}")]
public IActionResult CreateChangeEvent(Guid id)

What have I done wrong?

Update

Thanks @MegaTron for your response, however I would like to know why I can't have the same action path for different controllers. I feel like the solution you proposed won't scale well if I have many controllers that each create entities.

like image 238
Zeus82 Avatar asked Nov 24 '16 23:11

Zeus82


3 Answers

Try:

// ChangeEventsController
[HttpGet("Create/{id}")]
public IActionResult Create(Guid id)

// ProductsController
[HttpGet("CreateChangeEvent/{id}")]
public IActionResult CreateChangeEvent(Guid id)
like image 172
Roman Marusyk Avatar answered Nov 03 '22 15:11

Roman Marusyk


While the most up-voted answer does solve the issue, as mentioned by @B12Toaster it would violate the rules of REST. With my answer I will try to solve the problem while remaining RESTful.


TLDR: Add the Name property to your HTTP verb attribute (GET or otherwise)

In order to get both GET to work in both controllers do this:

// ChangeEventsController
[HttpGet(Name = "Get an event")]
[Route("{id}")]
public IActionResult Create(Guid id)

// ProductsController
[HttpGet(Name = "Get a product")]
[Route("{id}")]
public IActionResult CreateChangeEvent(Guid id)

This answer explains why you can't have two paths with the same name on two different controllers in Web API. You can implement the solution discussed in the answer to avoid this problem, or you can use ServiceStack which I personally would recommend.


Long answer: Explaining how to be RESTful within Web API

First: let's focus on the controller names. The controller names should be plural and nouns only. That would result in these two controllers:

  • Events: Instead of ChangeEvents. The change can happen within a PUT, not as controller name.
  • Products

Explanation on RESTful naming standards


Second: The endpoints within a controller should be named as CRUD operations in respect to RESTful standards.

  • POST
  • GET
  • PUT
  • DELETE
  • PATCH: Optional

This is instead of Create and CreateChangeEvent. This helps you locate which verbs you're invoking. There is no need for custom naming for the operations, as there shouldn't be too many in the first place to begin with in each controller.


Third: Your routes should not have custom names for each. Again, sticking to our method names, they should be CRUD operations only.

In this case:

// EventsController
[HttpGet(Name = "Get an event")]
[Route("events/{id}")]
public IActionResult Get(Guid id)

// ProductsController
[HttpGet(Name = "Get a product")]
[Route("products/{id}")]
public IActionResult Get(Guid id)

This would result in:

  • GET for /events/{id}
  • GET for /products/{id}

Last: For GET HTTP calls, you should send your input via query rather than body. Only PUT/POST/PATCH should send a representation via body. This is part of the Roy Fieldings constraints in REST. If you want to know further, look here and here.

You can do this by adding the [FromQuery] attribute before each of the parameters.

// EventsController
[HttpGet(Name = "Get an event")]
[Route("events/{id}")]
public IActionResult Get([FromQuery] Guid id)

// ProductsController
[HttpGet(Name = "Get a product")]
[Route("products/{id}")]
public IActionResult Get([FromQuery] Guid id)

I hope this would be helpful to future readers.

like image 36
AzzamAziz Avatar answered Nov 03 '22 15:11

AzzamAziz


If you want to use default routing , follow blew instrument:

  1. Remove [Route("[controller]")] from top of 'ChangeEvents' controller (if exists).
  2. Remove routing pattern from HttpGet

summery, try this :

// ChangeEventsController
[HttpGet]
public IActionResult Create(Guid id)

// ProductsController
[HttpGet]
public IActionResult CreateChangeEvent(Guid id)
like image 2
D.L.MAN Avatar answered Nov 03 '22 15:11

D.L.MAN