Backport（バックポート） #3463: opFormItemGenerator::generateValidator() に渡すフィールドの IsRequired や IsTrim を省略すると E_NOTICE レベルのエラーが発生する - OpenPNE 3 - OpenPNE Issue Tracking System

Backport（バックポート） #3463

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

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

Status:

Fixed（完了）

Priority:

Normal（通常）

Assignee:

Akihiro KOBAYASHI

Target version:

OpenPNE 3.8.9

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

Associated revisions

Revision 8b19cda7 (diff)
Added by Youichi Kimura over 6 years ago

make IsRequired and IsTrim optional in opFormItemGenerator (fixes #3463,
BP from #3428)
(cherry picked from commit 0962d48b8fbfa2da9be0d65764a569bf9ccfd1e7)

Revision b50aab8b
Added by Youichi Kimura over 6 years ago

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