Backport(バックポート) #3463
opFormItemGenerator::generateValidator() に渡すフィールドの IsRequired や IsTrim を省略すると E_NOTICE レベルのエラーが発生する
開始日:
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 等が定義されていることを確認した上で値を使用するように修正する
関連するチケット
関係しているリビジョン
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 KOBAYASHI が10年以上前に更新
- ステータス を New(新規) から Accepted(着手) に変更
- 担当者 を Akihiro KOBAYASHI にセット
#2 Akihiro KOBAYASHI が10年以上前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
pull request:https://github.com/openpne/OpenPNE3/pull/90
#3 Youichi Kimura が10年以上前に更新
更新履歴 8b19cda75f6a95be9200525710f742c293797d85 で適用されました。
#4 Youichi Kimura が10年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
#5 Mutsumi Imamura が10年以上前に更新
- ステータス を Pending Testing(テスト待ち) から Fixed(完了) に変更
- 進捗率 を 70 から 100 に変更
./symfony test:all を実行し異常終了しないことを確認しました。
ブラウザからの動作テストも問題ありません。