Project

General

Profile

Backport(バックポート) #4146

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

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

Added by Youichi Kimura almost 7 years ago. Updated almost 5 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2016-10-31
Due date:
% Done:

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 (修正内容)

修正内容を記入

Associated revisions

Revision 10528169 (diff)
Added by Youichi Kimura almost 7 years ago

move code that filtering configs by category into else block (refs #4146, BP from #4034)

(cherry picked from commit 3740ceb5c7834c1af6ecbb4b7e69b272bbf010a0)

Revision 398cf143 (diff)
Added by Youichi Kimura almost 7 years ago

set default settings when retrieving openpne_member_config (fixes #4146, BP from #4034)

(cherry picked from commit ca093ca1d27d3f366be2754f9ce597fa36f8b3aa)

Revision 2deaff86 (diff)
Added by Youichi Kimura almost 7 years ago

remove checking key exists in member_config settings (refs #4146, BP from #4034)

(cherry picked from commit 21b09efc7e86042897d31ca1096fd8f89e34efa7)

Revision ba08cbce
Added by kaoru n almost 5 years ago

Merge pull request #507 from nishizoe/t-4146

set default settings when retrieving openpne_member_config (fixes #4146, BP from #4034)

History

#1 Updated by Youichi Kimura almost 7 years ago

  • Related to Bug(バグ) #4034: member/registerInputアクションでメンバー登録のフォームを生成する際にE_NOTICEエラーが発生する added

#2 Updated by Youichi Kimura almost 7 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#4 Updated by kaoru n about 6 years ago

  • Parent task set to #4034

#5 Updated by Rimpei Ogawa over 5 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

プルリクエストがコンフリクト状態です

#6 Updated by kaoru n about 5 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

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

https://github.com/openpne/OpenPNE3/pull/443
はクローズしています。

#7 Updated by kaoru n about 5 years ago

  • Assignee changed from Youichi Kimura to kaoru n

#8 Updated by Rimpei Ogawa almost 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.6.x to OpenPNE 3.6.33

#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 almost 5 years ago

  • Status changed from Pending Merge(マージ待ち) to Fixed(完了)
  • % Done changed from 80 to 100

マージしました

Also available in: Atom PDF