操作
Bug（バグ） #3270
未完了
sfValidatorDateで「年」に 0 を入力した場合に空値として認識される
開始日:
2012-12-05
期日:
進捗率:
0%
予定工数:
3.6 で発生するか:
Yes (はい)
3.8 で発生するか:
Yes (はい)
説明
https://redmine.openpne.jp/issues/2882の修正時に発見。
現象¶
コミュニティイベントの作成時、「募集期日」の「年」に 0 と入力された状態（「月」「日」は入力しない）で送信しても「正しくありません。」と表示されずにイベントの作成が完了する。
原因¶
入力チェックに使用しているクラスsfValidatorDate内で入力値のEmptyチェックを行っています(sfValidatorDateクラスのisEmptyメソッド)が、与えられた引数がarrayの場合に「array_filter」という関数を使用しています。
この関数は(http://php.net/manual/ja/function.array-filter.phpより)
コールバック関数が与えられなかった場合、 input のエントリの中で FALSE と等しいもの (boolean への変換 を参照ください) がすべて削除されます。
ということで、年に入力された0がfalse扱いされ削除されます。
その後Emptyチェックを行うため、年に0のみが入力された場合、Emptyと判定され必須項目ではないためエラーチェックに引っかかりません。
修正内容¶
sfValidatorDateでなく、opValidatorDateを使用するようにする。
但し、#2882の修正が必要。
Yuya Watanabe さんがほぼ12年前に更新
https://redmine.openpne.jp/issues/2882#note-12 でも書きましたが sfValidatorDate と opValidatorDate の違いを大雑把に表すと下記 2 点があります．既存の sfValidatorDate を用いている場合に下記違いが影響を与える場合はそのまま opValidatorDate を用いることができないため， opValidatorDate を用いられるように opValidatorDate の外側部分を改修するか， isEmpty 相当のもののみを入れ替えた別クラスを用いる必要があります．
sfValidatorDate と opValidatorDate の違い- opValidaterDate は入力にタイムスタンプを許容しない
- opValidaterDate は empty_value に DateTime を用いる必要がある
操作