Bug(バグ) #3405
Task(タスク) #3403: OpenPNE の既存のテストコードが正常に動作するように修正を行う
アクションの基底クラスとその派生クラスでメソッドのシグニチャが一致していない
0%
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
fix method declaration error in action classes (fixes #3405)
History
#1 Updated by Youichi Kimura about 11 years ago
このバグにより 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))
#2 Updated by Youichi Kimura about 11 years ago
- Subject changed from opMemberActionクラスとその派生クラスでメソッドのシグニチャが一致していない to アクションの基底クラスとその派生クラスでメソッドのシグニチャが一致していない
- Description updated (diff)
#3 Updated by Youichi Kimura about 11 years ago
- Status changed from Accepted(着手) to Pending Review(レビュー待ち)
- % Done changed from 0 to 50
更新履歴 0e5c3ff1759e6a35048b3eb4367dd38057c53a44 で適用されました。
#4 Updated by Youichi Kimura about 11 years ago
pc_backend でも利用されるクラスについては opWebRequest ではなく sfWebRequest に統一しました
#5 Updated by Shinichi Urabe about 11 years ago
- Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
- % Done changed from 50 to 70
レビューOKです
#7 Updated by isao sano over 7 years ago
- Status changed from Pending Testing(テスト待ち) to Won't fix(対応せず)
- % Done changed from 70 to 0
OpenPNE 3.8.9 にて対応済みであったため、対応せずとします。