プロジェクト

全般

プロフィール

Bug(バグ) #3405

完了

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

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

Youichi Kimura さんが11年以上前に追加. 7年以上前に更新.

ステータス:
Won't fix(対応せず)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
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 を引数として受け取る各メソッドを タイプヒンティング を使用する形に統一する


関連するチケット 2 (0件未完了2件完了)

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

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

操作

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です

isao sano さんが7年以上前に更新

  • ステータスPending Testing(テスト待ち) から Won't fix(対応せず) に変更
  • 進捗率70 から 0 に変更

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

他の形式にエクスポート: Atom PDF