Backport（バックポート） #3420
アクションの基底クラスとその派生クラスでメソッドのシグニチャが一致していない
Start date:
2013-09-20
Due date:
% Done:
100%
Description
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 を引数として受け取る各メソッドを タイプヒンティング を使用する形に統一する
Related issues
Associated revisions
Merge branches 't-3418', 't-3420', 't-3422', 't-3459', 't-3460', 't-3461', 't-3462', 't-3463', 't-3465' and 't-3471' into stable-3.8.x
History
#1
Updated by Akihiro KOBAYASHI about 10 years ago
Updated by - Status changed from New（新規） to Accepted（着手）
- Assignee changed from Youichi Kimura to Akihiro KOBAYASHI
#2
Updated by Akihiro KOBAYASHI about 10 years ago
- Status changed from Accepted（着手） to Pending Review（レビュー待ち）
- % Done changed from 0 to 50
#3
Updated by Akihiro KOBAYASHI about 10 years ago
pull request: https://github.com/openpne/OpenPNE3/pull/82
#4
Updated by Youichi Kimura about 10 years ago
Updated by 更新履歴 fb318d6f79fa3858b0ee53b9f2db1b609c45c29a で適用されました。
#5
Updated by Youichi Kimura about 10 years ago
- Status changed from Pending Review（レビュー待ち） to Pending Testing（テスト待ち）
- % Done changed from 50 to 70
#6
Updated by Mutsumi Imamura about 10 years ago
Updated by - Status changed from Pending Testing（テスト待ち） to Fixed（完了）
- % Done changed from 70 to 100
./symfony test:all を実行し異常終了しないことを確認しました。
ブラウザからの動作テストも問題ありません。