操作
Bug（バグ） #3405
完了
Task（タスク） #3403: OpenPNE の既存のテストコードが正常に動作するように修正を行う
アクションの基底クラスとその派生クラスでメソッドのシグニチャが一致していない
開始日:
2013-09-20
期日:
進捗率:
0%
予定工数:
3.6 で発生するか:
Unknown (未調査)
3.8 で発生するか:
Unknown (未調査)
説明
Overview (現象)¶
opMemberAction::executeLogin() 等のメソッドで、下記のように継承先クラスのメソッドのシグニチャが異なるために E_STRICT エラーが発生している。
apps/pc_frontend/modules/member/actions/actions.class.php の executeLogin():
public function executeLogin(opWebRequest $request)
apps/mobile_frontend/modules/member/actions/actions.class.php の executeLogin():
public function executeLogin($request)
lib/action/opMemberAction.class.php の executeLogin():
public function executeLogin($request)
Way to fix (修正内容)¶
opWebRequest を引数として受け取る各メソッドを タイプヒンティング を使用する形に統一する
Youichi Kimura さんが11年以上前に更新
このバグにより test/functional/pc_frontend/memberActionsTest.php を実行したときに下記のようなエラーが発生する。(#3403 の note-7 より抜粋)
RuntimeException: PHP sent a strict error at /home/upsilon/git/openpne3/master/apps/pc_frontend/modules/member/actions/actions.class.php line 19 (Declaration of memberActions::executeLogin() should be compatible with opMemberAction::executeLogin($request))
Youichi Kimura さんが11年以上前に更新
- 題名 を opMemberActionクラスとその派生クラスでメソッドのシグニチャが一致していない から アクションの基底クラスとその派生クラスでメソッドのシグニチャが一致していない に変更
- 説明 を更新 (差分)
Youichi Kimura さんが11年以上前に更新
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
更新履歴 0e5c3ff1759e6a35048b3eb4367dd38057c53a44 で適用されました。
Youichi Kimura さんが11年以上前に更新
pc_backend でも利用されるクラスについては opWebRequest ではなく sfWebRequest に統一しました
Shinichi Urabe さんが約11年前に更新
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更
レビューOKです
操作