Examples don't match text, in routing chapter, section "Adding HTTP Method Requirements"

[toddkaufmann]

or maybe I don't understand something.

The text says
"Despite the fact that these two routes have identical paths (/contact), the first route will match only GET requests and the second route will match only POST requests."

but the examples (annotations,yaml,xml,php) have "/news" and "/contact" and does not match the description.

Okay here in 2.2:
http://symfony.com/doc/2.2/book/routing.html#adding-http-method-requirements

It looks like something happened when annotations were added. I'm not familiar enough with the docs and where the example code block gets pulled in from.

[xabbuh]

That's a good catch @toddkaufmann. We changed the code examples in #5201, but apparently forgot to update the text too.Do you like to create a pull request for this?

[toddkaufmann]

I can't update the text because I don't understand the intent.
To me the previous example was clear so I'd just revert to that.

I scanned through the discussion on this issue, but not being versed enough in Symfony best practices I couldn't explain the use case & implementation properly.

[bicpi]

Hi everyone,
I've read trough the affected text, examples and discussions.

I think it's essential for understanding this feature to have the exact same URL pattern for two different actions. The routing should only differ in the HTTP method, so you can clearly get the idea behind this. It was clear in the previous example using /contact only, but I see the objection that it's not a best practice. My proposal would be to show a piece of an API, let's say for the URL pattern /api/events. GET lists the events, POST creates an event. The /contact form with [GET, POST] could be used as an additional example how to restrict a route to multiple methods.

What do you think?

[xabbuh]

Wouldn't this then become to lengthy and complicated? I bet beginners would sadly just stop reading when it comes to APIs here. Imo rewording the paragraph would be enough to show how you can use methods requirements.

[bicpi]

I understand your concerns, but is there a good use case for Adding HTTP Method Requirements for beginners then? I mean, does restricting contactFormAction() to GET and POST makes sense as the default HTTP method is ANY and it will just work without setting a method restriction. From a beginner's perspective without API knowledge there's no PUT or PATCH etc.

[xabbuh]

I am actually not sure. Maybe this should simply be moved to the cookbook explained with the API example.

[Saphyel]

I agree with bicpi an example of PUT would be nice

[ycklsr]

Hi there,
I just added a PR (before xabbuh kindly informs me that this issue is still being discussed)
At this stage of the course, no reference of any kind has been made regarding APIs, so I don't think it would be a great idea to unnecessarily complexify this part. We should keep in mind that the original intent in this routing section was to show how easily we can segregate HTTP methods and their subsequent routing even if the routes share the same path.