プロジェクト

全般

プロフィール

Backport（バックポート） #3437

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

Mutsumi Imamura が10年以上前に追加. 10年以上前に更新.

ステータス:
Fixed（完了）
優先度:
Normal（通常）
担当者:
Youichi Kimura
対象バージョン:
OpenPNE 3.6.12
開始日:
2013-10-07
期日:
進捗率:

100%

説明

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 等が定義されていることを確認した上で値を使用するように修正する

関連するチケット

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

関係しているリビジョン

リビジョン 3775fd95 (差分)
Youichi Kimura が10年以上前に追加

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

(cherry picked from commit 0962d48b8fbfa2da9be0d65764a569bf9ccfd1e7)

履歴

#1 Mutsumi Imamura が10年以上前に更新

  • トラッカー を Bug（バグ） から Backport（バックポート） に変更

#2 Youichi Kimura が10年以上前に更新

  • ステータス を New（新規） から Pending Review（レビュー待ち） に変更
  • 進捗率 を 0 から 50 に変更

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

#3 Shinichi Urabe が10年以上前に更新

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

レビューOKです

#4 Mutsumi Imamura が10年以上前に更新

  • ステータス を Pending Testing（テスト待ち） から Fixed（完了） に変更
  • 進捗率 を 70 から 100 に変更

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

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