Skip to content

4.x - Route InvocationStrategy Setter#2634

Merged
akrabat merged 5 commits into
slimphp:4.xfrom
l0gicgate:4.x-RouteInvocationStrategySetter
Apr 20, 2019
Merged

4.x - Route InvocationStrategy Setter#2634
akrabat merged 5 commits into
slimphp:4.xfrom
l0gicgate:4.x-RouteInvocationStrategySetter

Conversation

@l0gicgate

@l0gicgate l0gicgate commented Apr 17, 2019

Copy link
Copy Markdown
Member

This is pull 3 out of 7 to complete the goals set in #2604

This PR adds the capability to set a Route's invocation strategy per route.

Closes #2597

Example usage:

use Slim\App;
use Slim\Handlers\Strategies\RequestResponse;

$app = new App(...);

$invocationStrategy = new RequestResponse();
$app
    ->get('/', function () {...})
    ->setInvocationStrategy(...);

@l0gicgate l0gicgate added this to the 4.0 milestone Apr 17, 2019
@l0gicgate
l0gicgate requested a review from akrabat April 17, 2019 19:50
@coveralls

coveralls commented Apr 17, 2019

Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.004%) to 98.516% when pulling 2cf746c on l0gicgate:4.x-RouteInvocationStrategySetter into 8e07064 on slimphp:4.x.

Comment thread tests/Routing/RouteTest.php
@l0gicgate
l0gicgate requested a review from akrabat April 19, 2019 22:03
@akrabat
akrabat merged commit 2cf746c into slimphp:4.x Apr 20, 2019
@l0gicgate l0gicgate mentioned this pull request Apr 20, 2019
7 tasks
@l0gicgate
l0gicgate deleted the 4.x-RouteInvocationStrategySetter branch April 20, 2019 18:08
@l0gicgate l0gicgate mentioned this pull request Aug 1, 2019
chriscct7 pushed a commit to awesomemotive/aioseo-slim-v3-php-8.1 that referenced this pull request Sep 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants