プロジェクト

全般

プロフィール

Backport(バックポート) #637

opValidatorDate with date_time option don't return time correctly. (opValidatorDateをwith_timeオプション付きで使用したとき時刻が違う値になる)

Shogo Kawahara約14年前に追加. 約14年前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2010-02-13
期日:
進捗率:

100%


説明

Overview (概要)

opValidatorDateをwith_timeオプション付きで使用したとき、時刻に「フォームで入力した時刻 + 現在時刻」の値が設定されてしまいます。

Cause (原因)

配列で時刻を受け取った時、opValidatorDateの挙動が正しくありません。

122       $clean->modify('+'.(isset($value['hour']) ? intval($value['hour']) : 0).'hour');
123       $clean->modify('+'.(isset($value['second']) ? intval($value['second']) : 0).'second');

ここは、 modify()で加算するのではなく、setTime()で時刻をセットするのが正しいでしょう。

Reproduction method (再現方法)

検証コード

$v = new opValidatorDate(array('with_time' => true));

var_dump($v->clean(array('year' => 1989, 'month' => 1, 'day' => 8, 'hour' => 10, 'minute' => 2, 'second' => 10)));

結果として、時刻が正しくない結果が帰ってきます。

(上記の場合、現時時刻に 10時間と10秒が加算される)

Environment (環境)

  • OpenPNE3.5.x
  • OpenPNE3.4.x
  • OpenPNE3.2.x

関連するチケット

関連している OpenPNE 3 - Bug(バグ) #633: opValidatorDate with date_time option don't return time correctly. (opValidatorDateをwith_timeオプション付きで使用したとき時刻が違う値になる) Fixed(完了) 2010-02-13 2010-03-04

関係しているリビジョン

リビジョン 43f964ab (差分)
Youichi Kimura約14年前に追加

fixed that the opValidatorDate doesn't convert time correctly (fixes #637;BP from #633)

リビジョン 883f6404 (差分)
Shogo Kawahara約14年前に追加

added a unit test script for opValidatorDate (fixes #637;BP from #633)

履歴

#1 Youichi Kimura約14年前に更新

  • ステータスNew(新規) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

更新履歴commit:"43f964ab844ad693c56ee440409e3bf35191f32d"で適用されました。

#2 Shogo Kawahara約14年前に更新

更新履歴commit:"883f6404224d9c7a0a42532b19b9e55b3fd31462"で適用されました。

#3 Kousuke Ebihara約14年前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更

#4 Mutsumi Imamura約14年前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率50 から 100 に変更

動作テストOKです。

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