Question/Poll: Should we bump to v3 for per route and per group middleware? about route

[hannesvdvreken]

That's it? Then I'd lean more towards my first proposal: deprecate JsonStrategy, and make a new JsonResponseStrategy, tag 2.1.0 and be done with it.

[hannesvdvreken]

Is it perhaps possible to make a different strategy and deprecate the old one? Then you'd be fine with a new minor.

I'm OK with a v3 as well, though.

[hannesvdvreken]

If it's a small change, could you provide a branch to showcase the changes and discuss further based on that?

[philipobenito]

The discussion is going to dictate where I go with the code.

The breaking change would be here https://github.com/thephpleague/route/blob/master/src/Strategy/JsonStrategy.php#L19 where I would be adding the response object as the second parameter on this call.

[philipobenito]

And just have the original strategy ignore the new middleware functionality?

[hannesvdvreken]

Maybe, does that seem plausible to you?

[philipobenito]

Yeah, I think that works for me.

RequestResponseStrategy & JsonResponseStrategy - can add middleware and will invoke them.

JsonStrategy & ParamStrategy - throw exception when dispatching on this strategy and a middleware is found?

JsonStrategy - deprecated and will be removed in version 3.0.0?

[philipobenito]

Maybe actually have it as JsonRequestResponseStrategy, that way all strategies going forward that are suffixed RequestResponseStrategy can be used with middleware and we don't necessarily need to deprecate the JsonStrategy?

That way we can continue to introduce strategies and it has a format. As in SomeStrangeFormatRequestResponseStrategy etc down the line.

[hannesvdvreken]

Deprecating JsonStrategy isn't necessary, no. But you'd want that to avoid confusion why there are 2 json related strategy classes, and if you want to remove it in v3.

[philipobenito]

Well there would be value in keeping both, the difference would just have to be explained thoroughly.

JsonRequestResponseStrategy you would be expected to manipulate the response and return it.

JsonStrategy would attempt to build a response from what you return (array, json string etc).

[CMCDragonkai]

Route 2.x is not compatible with things like League\Relay right?

[hannesvdvreken]

@CMCDragonkai I guess you meant https://github.com/relayphp/Relay.Relay ? League\Relay doesn't exist, afaik.

if you have a League\Route\RouteCollection object, you can use [$routeCollection, 'dispatch'] (which is a PHP callable) to pass to Relay as a middleware.

[CMCDragonkai]

Yes that's what I meant. Oh that's nice. Hopefully this can be clearer in the docs.