プロジェクト

全般

プロフィール

Bug(バグ) #3270

sfValidatorDateで「年」に 0 を入力した場合に空値として認識される

kaoru n11年以上前に追加. 約4年前に更新.

ステータス:
New(新規)
優先度:
Normal(通常)
担当者:
-
対象バージョン:
開始日:
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の修正が必要。


関連するチケット

関連している OpenPNE 3 - Bug(バグ) #2882: opValidatorDate で「年」に 0 を入力した場合に空値として認識される Won't fix(対応せず) 2012-03-13
関連している OpenPNE 3 - Backport(バックポート) #3287: sfValidatorDateで「年」に 0 を入力した場合に空値として認識される New(新規) 2012-12-05
関連している OpenPNE 3 - Backport(バックポート) #3282: sfValidatorDateで「年」に 0 を入力した場合に空値として認識される New(新規) 2012-12-17

履歴

#1 Chiharu Nakajima11年以上前に更新

  • 対象バージョンOpenPNE 3.8.4 にセット

#2 Mutsumi Imamura11年以上前に更新

  • 対象バージョンOpenPNE 3.8.4 から OpenPNE 3.9.0-old に変更

#3 Mutsumi Imamura11年以上前に更新

  • 担当者Yuya Watanabe にセット

#4 Yuya Watanabe約11年前に更新

https://redmine.openpne.jp/issues/2882#note-12 でも書きましたが sfValidatorDate と opValidatorDate の違いを大雑把に表すと下記 2 点があります.既存の sfValidatorDate を用いている場合に下記違いが影響を与える場合はそのまま opValidatorDate を用いることができないため, opValidatorDate を用いられるように opValidatorDate の外側部分を改修するか, isEmpty 相当のもののみを入れ替えた別クラスを用いる必要があります.

sfValidatorDate と opValidatorDate の違い
  • opValidaterDate は入力にタイムスタンプを許容しない
  • opValidaterDate は empty_value に DateTime を用いる必要がある

#5 Yuya Watanabe約11年前に更新

  • 担当者 を削除 (Yuya Watanabe)

一旦担当を手放します.

#6 kaoru n約7年前に更新

  • 対象バージョンOpenPNE 3.9.0-old から OpenPNE 3.9.0 に変更

#7 isao sanoほぼ7年前に更新

再現を確認しました。

#8 kaoru n約4年前に更新

  • 対象バージョンOpenPNE 3.9.0 から OpenPNE 3.10.x に変更

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