Project

General

Profile

Bug(バグ) #3405

Task(タスク) #3403: OpenPNE の既存のテストコードが正常に動作するように修正を行う

アクションの基底クラスとその派生クラスでメソッドのシグニチャが一致していない

Added by Youichi Kimura about 6 years ago. Updated over 2 years ago.

Status:
Won't fix(対応せず)
Priority:
Normal(通常)
Target version:
Start date:
2013-09-20
Due date:
% Done:

0%

3.6 で発生するか:
Unknown (未調査)
3.8 で発生するか:
Unknown (未調査)

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

Related to OpenPNE 3 - Backport(バックポート) #3419: アクションの基底クラスとその派生クラスでメソッドのシグニチャが一致していない Fixed(完了) 2013-09-20
Related to OpenPNE 3 - Backport(バックポート) #3420: アクションの基底クラスとその派生クラスでメソッドのシグニチャが一致していない Fixed(完了) 2013-09-20

Associated revisions

Revision 0e5c3ff1 (diff)
Added by Youichi Kimura about 6 years ago

fix method declaration error in action classes (fixes #3405)

History

#1 Updated by Youichi Kimura about 6 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 6 years ago

  • Subject changed from opMemberActionクラスとその派生クラスでメソッドのシグニチャが一致していない to アクションの基底クラスとその派生クラスでメソッドのシグニチャが一致していない
  • Description updated (diff)

#3 Updated by Youichi Kimura about 6 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

更新履歴 0e5c3ff1759e6a35048b3eb4367dd38057c53a44 で適用されました。

#4 Updated by Youichi Kimura about 6 years ago

pc_backend でも利用されるクラスについては opWebRequest ではなく sfWebRequest に統一しました

#5 Updated by Shinichi Urabe about 6 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

レビューOKです

#7 Updated by isao sano over 2 years ago

  • Status changed from Pending Testing(テスト待ち) to Won't fix(対応せず)
  • % Done changed from 70 to 0

OpenPNE 3.8.9 にて対応済みであったため、対応せずとします。

Also available in: Atom PDF