プロジェクト

全般

プロフィール

Backport(バックポート) #3463

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

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

ステータス:
Fixed(完了)
優先度:
Normal(通常)
対象バージョン:
開始日:
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

関係しているリビジョン

リビジョン 8b19cda7 (差分)
Youichi Kimura10年以上前に追加

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

リビジョン b50aab8b
Youichi Kimura10年以上前に追加

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

履歴

#1 Akihiro KOBAYASHI10年以上前に更新

  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者Akihiro KOBAYASHI にセット

#2 Akihiro KOBAYASHI10年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

pull request:https://github.com/openpne/OpenPNE3/pull/90

#3 Youichi Kimura10年以上前に更新

更新履歴 8b19cda75f6a95be9200525710f742c293797d85 で適用されました。

#4 Youichi Kimura10年以上前に更新

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

#5 Mutsumi Imamura10年以上前に更新

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

./symfony test:all を実行し異常終了しないことを確認しました。
ブラウザからの動作テストも問題ありません。

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