Bug（バグ） #3428
Task（タスク） #3403: OpenPNE の既存のテストコードが正常に動作するように修正を行う
opFormItemGenerator::generateValidator() に渡すフィールドの IsRequired や IsTrim を省略すると E_NOTICE レベルのエラーが発生する
開始日:
2013-10-07
期日:
進捗率:
0%
3.6 で発生するか:
Unknown (未調査)
3.8 で発生するか:
Unknown (未調査)
説明
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 等が定義されていることを確認した上で値を使用するように修正する
関連するチケット
関係しているリビジョン
make IsRequired and IsTrim optional in opFormItemGenerator (fixes #3428)
履歴
#1
Youichi Kimura が10年以上前に更新
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
更新履歴 0962d48b8fbfa2da9be0d65764a569bf9ccfd1e7 で適用されました。
#2
Shinichi Urabe が10年以上前に更新
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更
レビューOKです
今まで、trim, required が未定義のため null となっていたものに false が渡されることになりますが、 sfValidatorBase::clean() や opValidatorString::clean() の処理を確認する限り、動作は問題ないと考えます。
#3
誠二 天重 が9年以上前に更新
- 次のチケットが重複 Bug（バグ） #3167: IsTrim の undefined Index notice を追加
#4
Chiharu Nakajima が9年以上前に更新
- 優先度 を Normal（通常） から Low（低め） に変更
#5
Chiharu Nakajima が9年以上前に更新
このチケットの内容は下記の項目に該当する不具合であるため、一旦優先度を下げます。
・未対応でもユーザーに影響が少ない不具合（回避可能な不具合である等）
