githubjeka
Repos
137
Followers
141

A plugin for PHPStorm that brings modules to the PHP language

46
2

[WIP] PHP Time library

0
0

Events

started
Created at 1 week ago
pull request opened
Fix notice: Array to string conversion
Created at 1 week ago

Fix notice: Array to string conversion

Created at 1 week ago
Created at 1 week ago
Created at 1 week ago
FATAL: Creating new database failed.

@sebheitzmann

Same on 4.2 To fix it see your log to more details

For example

There error Error: Data directory /var/lib/postgresql/14/main must not be owned by root

before createdb: error: connection to server on socket "/var/run/postgresql/.s.PGSQL.5432" failed: No such file or directory error.

Created at 2 weeks ago
Created at 2 weeks ago
delete branch
githubjeka delete branch patch-1
Created at 1 month ago
pull request closed
Reordered $resolvers

Now https://github.com/PHP-DI/Slim-Bridge/blob/master/src/Bridge.php#L44

         $resolvers = [
            // Inject parameters by name first
            new AssociativeArrayResolver(),
            // Then inject services by type-hints for those that weren't resolved
            new TypeHintContainerResolver($container),
            // Then fall back on parameters default values for optional route parameters
            new DefaultValueResolver(),
        ];

The code

$app->post('/', [MyController::class, 'run'])

final class MyController
{
    public function run(\App\Model\RpcRequest $request): Response
    {
        // ...
    }
}

throws

TypeError
Argument 1 passed to App\MyController::run() must be an instance of \App\Model\RpcRequest, instance of Slim\Psr7\Request given

This looks more like a bug than consistent behavior, even though it has been documented. It is possible that this will lead to broken backward compatibility :/

Created at 1 month ago
issue comment
Reordered $resolvers

Thanks for the clarification and your time.

Created at 1 month ago
issue comment
Reordered $resolvers

App\Model\RpcRequest could be a valid dto that has been checked and set in container on middleware level.

Created at 1 month ago
issue comment
Reordered $resolvers

App\Model\RpcRequest can be a valid dto that has been set in container as stateless objects the middleware.

Created at 1 month ago
issue comment
Reordered $resolvers

Yea, it shouldn't be in a container. Bad example.

Created at 1 month ago
Created at 1 month ago
pull request opened
Reordered $resolvers

Now https://github.com/PHP-DI/Slim-Bridge/blob/master/src/Bridge.php#L44

         $resolvers = [
            // Inject parameters by name first
            new AssociativeArrayResolver(),
            // Then inject services by type-hints for those that weren't resolved
            new TypeHintContainerResolver($container),
            // Then fall back on parameters default values for optional route parameters
            new DefaultValueResolver(),
        ];

The code

$app->post('/', [MyController::class, 'run'])

final class MyController
{
    public function run(\App\Model\RpcRequest $request): Response
    {
        // ...
    }
}

throws

TypeError
Argument 1 passed to App\MyController::run() must be an instance of \App\Model\RpcRequest, instance of Slim\Psr7\Request given

This looks more like a bug than consistent behavior, even though it has been documented. It is possible that this will lead to broken backward compatibility :/

Created at 1 month ago

Reordered $resolvers

To fix strict type of arguments

Created at 1 month ago
Created at 1 month ago
delete branch
githubjeka delete branch 4.x-route
Created at 1 month ago
Created at 1 month ago
Created at 1 month ago
started
Created at 2 months ago
Created at 2 months ago
Created at 2 months ago
Created at 2 months ago
started
Created at 2 months ago

test: Fix tests

Created at 2 months ago
pull request opened
feat: Added ability to override RouteGroupInterface

Added methods RouteCollector::createGroup and createProxy

Created at 2 months ago
create branch
githubjeka create branch 4.x-route
Created at 2 months ago
Created at 2 months ago
started
Created at 2 months ago