HTTP Methods specific annotations

[sorenmalling]

How about we introduce HTTP specific Route annotation, making it a bit more simple

Today

#[Route('', method: 'POST')]

Tomorrow

#[Route\Get(format: 'json')]
#[Route\Post('')]

[mhsdesign]

i mean its not the symphony way: https://symfony.com/doc/current/routing.html but it sure looks cool.
Btw we can tell phpstorm which arguments to expect for the current method parameter too: neos/flow-development-collection#2961 that would be easy too ^^

[bwaidelich]

In Routes.yaml we call it "httpMethods" and allow multiple methods per route, I would suggest:

#[Route(httpMethods: ['POST'])]

[sorenmalling]

In Routes.yaml we call it "httpMethods" and allow multiple methods per route, I would suggest:

#[Route(httpMethods: ['POST'])]

We allow both array and string

https://github.com/wbsply/route-annotation/blob/main/Classes/Annotations/Route.php#L16

[sorenmalling]

i mean its not the symphony way: https://symfony.com/doc/current/routing.html but it sure looks cool. Btw we can tell phpstorm which arguments to expect for the current method parameter too: neos/flow-development-collection#2961 that would be easy too ^^

Like this (no changes, does by default)

[mhsdesign]

no but we can also tell which values are allowed ;) see what was done in neos/flow-development-collection#2961

(or maybe even phpdoc is now far enough to be able to tell this the ide?)

[lorenzulrich]

No strong opinion here, but in general it would be nice to allow more than one method.

[sorenmalling]

No strong opinion here, but in general it would be nice to allow more than one method.

@lorenzulrich Not relevant to the original goal of this "issue" :) And you can already use a array of HTTP methods

#2 (comment)

[mficzel]

I think beeing as close to the routing configuration as sensible is a good idea.

So for me #[Flow\Route(httpMethods:[“get”]…)]