Skip to content

Comments

[3.x] Allow container bindings for http classes (laravel arch preset)#1332

Closed
ol1s wants to merge 1 commit intopestphp:3.xfrom
ol1s:allow-container-bindings-for-http
Closed

[3.x] Allow container bindings for http classes (laravel arch preset)#1332
ol1s wants to merge 1 commit intopestphp:3.xfrom
ol1s:allow-container-bindings-for-http

Conversation

@ol1s
Copy link

@ol1s ol1s commented Dec 24, 2024

What:

  • Bug Fix

Description:

Current Laravel arch preset only allows App\Http classes to be used within App\Http.

This makes sense MOST of the time.

But feels reasonable to accept that container bindings for controllers/middleware MAY be manually registered in service providers eg. primitives https://laravel.com/docs/11.x/container#binding-primitives

This means App\Http also needs to be usable in App\Providers.

@ol1s ol1s closed this by deleting the head repository Jul 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant