プロジェクト

全般

プロフィール

Bug(バグ) #4034

member/registerInputアクションでメンバー登録のフォームを生成する際にE_NOTICEエラーが発生する

Youichi Kimura7年以上前に追加. 約5年前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2016-10-31
期日:
進捗率:

100%

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

説明

Overview (概要)

新規メンバー登録で使用される member/registerInput の画面で下記のような E_NOTICE レベルのエラーが発生する(pc_frontend_dev.php を使用するとエラー出力を確認できる)

Notice: Undefined index: IsRegist in /home/upsilon/git/openpne3/master/lib/form/doctrine/MemberConfigForm.class.php on line 51

Causes (原因)

MemberConfigForm::generateConfigWidgets() メソッドで下記のように $value['IsRegist'] を参照しているが、member_config.yml で定義された設定の中には IsRegist が省略されている項目も存在するため未定義なインデックスの参照としてエラーが発生する場合がある。

source:lib/form/doctrine/MemberConfigForm.class.php@ac9a15cd#L51:

    foreach ($this->memberConfigSettings as $key => $value) {
      if ($this->isNew && $value['IsRegist'] || !$this->isNew && $value['IsConfig']) {
        $this->setMemberConfigWidget($key);
      }   
    }

類似の不具合として過去に修正された #3428 がある。

Way to fix (修正内容)

修正内容を記入


子チケット

Backport(バックポート) #4146: member/registerInputアクションでメンバー登録のフォームを生成する際にE_NOTICEエラーが発生するFixed(完了)kaoru n

Backport(バックポート) #4147: member/registerInputアクションでメンバー登録のフォームを生成する際にE_NOTICEエラーが発生するFixed(完了)kaoru n

関係しているリビジョン

リビジョン 4353ecfc (差分)
isao sano約6年前に追加

move code that filtering configs by category into else block (refs #4034)

リビジョン c3ac632d (差分)
Youichi Kimura約6年前に追加

set default settings when retrieving openpne_member_config (fixes #4034)

リビジョン cb7cd186 (差分)
Youichi Kimura約6年前に追加

remove checking key exists in member_config settings (refs #4034)

リビジョン 33cb4f83
kaoru n約5年前に追加

Merge pull request #499 from isaosano/t-4034

set default settings when retrieving openpne_member_config (fixes #4034)

履歴

#1 Youichi Kimura7年以上前に更新

  • ステータスNew(新規) から Pending Review(レビュー待ち) に変更
  • 担当者Youichi Kimura にセット
  • 対象バージョンOpenPNE 3.9.0-old にセット
  • 進捗率0 から 50 に変更

下記 Pull Request にて修正しました
https://github.com/openpne/OpenPNE3/pull/382

#2 Youichi Kimura約7年前に更新

  • 関連している Backport(バックポート) #4146: member/registerInputアクションでメンバー登録のフォームを生成する際にE_NOTICEエラーが発生する を追加

#3 Youichi Kimura約7年前に更新

  • 関連している Backport(バックポート) #4147: member/registerInputアクションでメンバー登録のフォームを生成する際にE_NOTICEエラーが発生する を追加

#4 kaoru nほぼ7年前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更
  • 対象バージョンOpenPNE 3.9.0-old から OpenPNE 3.9.0 に変更

対象バージョン変更により修正内容の確認が必要であるため差し戻します。

#6 isao sano約6年前に更新

  • ステータスRejected(差し戻し) から Accepted(着手) に変更
  • 進捗率50 から 0 に変更

#7 isao sano約6年前に更新

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

https://github.com/openpne/OpenPNE3/pull/499 にてプルリクエストしました。

#8 Rimpei Ogawa約6年前に更新

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

#9 kaoru n5年以上前に更新

  • 対象バージョンOpenPNE 3.9.0 から OpenPNE-3.9.4 に変更

#10 Chiharu Nakajima5年以上前に更新

  • ステータスPending Testing(テスト待ち) から Pending Merge(マージ待ち) に変更
  • 進捗率70 から 80 に変更

動作確認OKです

#11 kaoru n約5年前に更新

  • ステータスPending Merge(マージ待ち) から Fixed(完了) に変更
  • 進捗率80 から 100 に変更

マージしました

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