Project

General

Profile

Backport(バックポート) #3437

opFormItemGenerator::generateValidator() に渡すフィールドの IsRequired や IsTrim を省略すると E_NOTICE レベルのエラーが発生する

Added by Mutsumi Imamura almost 6 years ago. Updated almost 6 years ago.

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

100%


Description

Overview (現象)

source:lib/config/config/sns_config.yml などの opFormItemGenerator で自動生成するフィールドの定義では、しばしば IsRequired や IsTrim が省略されているが、opFormItemGenerator::generateValidator() の下記のコードはそれらの省略を考慮されておらず E_NOTICE レベルのエラーを発生させてしまう。

  public static function generateValidator($field, $choices = array())
  {     
    $field = self::arrayKeyCamelize($field);
    $option = array('required' => $field['IsRequired'], 'trim' => $field['IsTrim']);

Way to fix (修正内容)

isset() を使用し IsRequired 等が定義されていることを確認した上で値を使用するように修正する


Related issues

Related to OpenPNE 3 - Bug(バグ) #3428: opFormItemGenerator::generateValidator() に渡すフィールドの IsRequired や IsTrim を省略すると E_NOTICE レベルのエラーが発生する Won't fix(対応せず) 2013-10-07

Associated revisions

Revision 3775fd95 (diff)
Added by Youichi Kimura almost 6 years ago

make IsRequired and IsTrim optional in opFormItemGenerator (fixes #3437, BP from #3428)

(cherry picked from commit 0962d48b8fbfa2da9be0d65764a569bf9ccfd1e7)

History

#1 Updated by Mutsumi Imamura almost 6 years ago

  • Tracker changed from Bug(バグ) to Backport(バックポート)

#2 Updated by Youichi Kimura almost 6 years ago

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

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

#3 Updated by Shinichi Urabe almost 6 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

レビューOKです

#4 Updated by Mutsumi Imamura almost 6 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

./symfony test:all を実行し異常終了しないことを確認しました。

Also available in: Atom PDF