Bug(バグ) #4034
member/registerInputアクションでメンバー登録のフォームを生成する際にE_NOTICEエラーが発生する
100%
Description
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 (修正内容)¶
修正内容を記入
Subtasks
Associated revisions
move code that filtering configs by category into else block (refs #4034)
set default settings when retrieving openpne_member_config (fixes #4034)
remove checking key exists in member_config settings (refs #4034)
History
#1
Updated by Youichi Kimura almost 7 years ago
- Status changed from New(新規) to Pending Review(レビュー待ち)
- Assignee set to Youichi Kimura
- Target version set to OpenPNE 3.9.0-old
- % Done changed from 0 to 50
下記 Pull Request にて修正しました
https://github.com/openpne/OpenPNE3/pull/382
#2
Updated by Youichi Kimura over 6 years ago
- Related to Backport(バックポート) #4146: member/registerInputアクションでメンバー登録のフォームを生成する際にE_NOTICEエラーが発生する added
#3
Updated by Youichi Kimura over 6 years ago
- Related to Backport(バックポート) #4147: member/registerInputアクションでメンバー登録のフォームを生成する際にE_NOTICEエラーが発生する added
#4
Updated by kaoru n over 6 years ago
- Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)
- Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.9.0
対象バージョン変更により修正内容の確認が必要であるため差し戻します。
#6
Updated by isao sano over 5 years ago
- Status changed from Rejected(差し戻し) to Accepted(着手)
- % Done changed from 50 to 0
#7
Updated by isao sano over 5 years ago
- Status changed from Accepted(着手) to Pending Review(レビュー待ち)
- % Done changed from 0 to 50
https://github.com/openpne/OpenPNE3/pull/499 にてプルリクエストしました。
#8
Updated by Rimpei Ogawa over 5 years ago
- Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
- % Done changed from 50 to 70
#9
Updated by kaoru n almost 5 years ago
- Target version changed from OpenPNE 3.9.0 to OpenPNE-3.9.4
#10
Updated by Chiharu Nakajima almost 5 years ago
- Status changed from Pending Testing(テスト待ち) to Pending Merge(マージ待ち)
- % Done changed from 70 to 80
動作確認OKです
#11
Updated by kaoru n over 4 years ago
- Status changed from Pending Merge(マージ待ち) to Fixed(完了)
- % Done changed from 80 to 100
マージしました