Should the ServiceContainer allow the app to overload default extension classes?
まだ誰も着手していません。
評価
調査の方向性
まず、実際の Request インスタンスの選択に関与する ServiceContainer と DefaultServiceLoader のエントリーポイントを読み、再現には EditableRequest の例を使用します。アプリケーションから提供されたクラスがデフォルトのサービスクラスを拡張する場合に意図された優先順位を判断し、選択した動作を示すテストカバレッジを追加して、受信した Request が正しく解決されることを確認します。
索引モデルが issue の本文から書いたものです。
説明
Specific example: My app deals with editing HTTP Requests, so I have a class called EditableRequest that extends Gt\Http\Request. Because of this, the EditableRequest class is instantiated and passed to the DefaultServiceLoader as the actual Request, rather than the actual request that's coming in.
This is obviously a flaw in the system, because it should be fine for the App's service loader to supply classes that extend already-existing classes.
The question is, what should the default behaviour be? If there's already a class defined in the default service loader, should we ignore loading any extras that simply extend the base classes? I think that would be safest.
- 主要言語
- PHP
- スター
- 0
- フォーク
- 0
- PR マージ指標
- 30日以内にマージされた PR はありません
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
phpgt/ServiceContainer のほかの issue
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
phpgt/ServiceContainer#145 · リアクション 1 件 ·
-
good first issue help wanted
難易度 3/5 1〜2日 初心者へのやさしさ 25/100
phpgt/ServiceContainer#143 ·
phpgt/ServiceContainer の issue をすべて見る
似ている issue
-
tooling
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
UX
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
ProfessionalWiki/NeoWiki#1525 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
bug customer-reported
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
MagnaCapax/PMSS#949 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
Automattic/static-site-importer#1829 ·