プロジェクト

全般

プロフィール

Backport(バックポート) #3420

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

Youichi Kimura10年以上前に追加. 10年以上前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
対象バージョン:
開始日:
2013-09-20
期日:
進捗率:

100%


説明

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 を引数として受け取る各メソッドを タイプヒンティング を使用する形に統一する


関連するチケット

関連している OpenPNE 3 - Bug(バグ) #3405: アクションの基底クラスとその派生クラスでメソッドのシグニチャが一致していない Won't fix(対応せず) 2013-09-20

関係しているリビジョン

リビジョン fb318d6f (差分)
Youichi Kimura10年以上前に追加

fix method declaration error in action classes (fixes #3420, BP from #3405)
(cherry picked from commit 0e5c3ff1759e6a35048b3eb4367dd38057c53a44)

リビジョン b50aab8b
Youichi Kimura10年以上前に追加

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

履歴

#1 Akihiro KOBAYASHI10年以上前に更新

  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者Youichi Kimura から Akihiro KOBAYASHI に変更

#2 Akihiro KOBAYASHI10年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

#4 Youichi Kimura10年以上前に更新

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

#5 Youichi Kimura10年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

#6 Mutsumi Imamura10年以上前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更

./symfony test:all を実行し異常終了しないことを確認しました。
ブラウザからの動作テストも問題ありません。

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