Bug(バグ) #3584
完了フォームタイプが日付のプロフィール項目の最小値に now を設定し、プロフィールに当日日付を設定するとエラーになる
100%
説明
初めてです
3.8.10を使用。プロフィールに項目追加し、フォームタイプを日付にした時、最小値、最大値のいづれかまたは両方をnowにすると、SNS会員が当日の日付を入れて、送信ボタンを押すとエラーになって、プロフィールを更新できない。3.8.9の時は、エラーにならなかった。
kaoru n さんがほぼ2年前に更新
https://github.com/openpne/OpenPNE3/blob/master/lib/validator/opValidatorDate.class.php#L54-L70
if ($this->hasOption('max'))
{
$max = new DateTime($this->getOption('max'));
if ($max && $clean->format('U') > $max->format('U'))
{
throw new sfValidatorError($this, 'max', array('max' => $max->format($this->getOption('date_format_range_error'))));
}
}
if ($this->hasOption('min'))
{
$min = new DateTime($this->getOption('min'));
if ($min && $clean->format('U') < $min->format('U'))
{
throw new sfValidatorError($this, 'min', array('min' => $min->format($this->getOption('date_format_range_error'))));
}
}
$min, $max ともに now を設定した場合、new DateTime() の引数にそのまま now が設定されるため、時刻も比較対象とされる。
例えば now = '2023-02-07 18:18:18' だった場合、プロフィール編集で '2023/02/07' を入力すると入力値は '2023-02-07 00:00:00' として扱われるため、最小値との比較でエラーになる。
kaoru n さんがほぼ2年前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
https://github.com/openpne/OpenPNE3/pull/685
にてプルリクエストしました
Rimpei Ogawa さんが1年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Rejected(差し戻し) に変更
with_time オプションを設定している際は日付だけでなく時刻の情報も扱う想定になっていて、
max/min オプションについてもこれに沿うのが自然な挙動だと思います。
参考:ユニットテストの with_time オプションの箇所
https://github.com/openpne/OpenPNE3/blob/master/test/unit/validator/opValidatorDateTest.php#L132
OpenPNE本体には with_time オプションを使っている箇所はなかったのですが、
時刻の情報を消して比較するのは with_time がないときだけにする方がよいかなと思いました。
Rimpei Ogawa さんが1年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更