Project

General

Profile

Bug(バグ) #633

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

Added by Youichi Kimura over 9 years ago. Updated over 9 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2010-02-13
Due date:
2010-03-04
% Done:

100%

3.6 で発生するか:
Unknown (未調査)
3.8 で発生するか:
Unknown (未調査)

Description

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
  • OpenPNE3.0.x

Related issues

Related to OpenPNE 3 - Backport(バックポート) #638: opValidatorDate with date_time option don't return time correctly. (opValidatorDateをwith_timeオプション付きで使用したとき時刻が違う値になる) Fixed(完了) 2010-02-13
Related to OpenPNE 3 - Backport(バックポート) #637: opValidatorDate with date_time option don't return time correctly. (opValidatorDateをwith_timeオプション付きで使用したとき時刻が違う値になる) Fixed(完了) 2010-02-13
Related to OpenPNE 3 - Bug(バグ) #754: opValidatorDate with max or min throw exception if the value is not array (opValidatorDateで配列を利用しないときにmax/minを設定しても例外を投げない) Fixed(完了) 2010-02-25 2010-03-04
Related to OpenPNE 3 - Backport(バックポート) #757: opValidatorDate with date_time option don't return time correctly. (opValidatorDateをwith_timeオプション付きで使用したとき時刻が違う値になる) Fixed(完了) 2010-02-13
Related to OpenPNE 3 - Bug(バグ) #1937: 日付のプロフィール項目で、管理画面で指定した上限の日が弾かれてしまう Fixed(完了)

Associated revisions

Revision c4757fb1 (diff)
Added by Youichi Kimura over 9 years ago

fixed that the opValidatorDate doesn't convert time correctly (fixes #633)

Revision d2636f1a (diff)
Added by Shogo Kawahara over 9 years ago

added a unit test script for opValidatorDate (fixes #633)

Revision 9448de5f (diff)
Added by Shogo Kawahara over 9 years ago

fixed unit test script for opValidatorDateTest (refs #633)

History

#1 Updated by Shogo Kawahara over 9 years ago

  • Status changed from New(新規) to Pending Fixing(修正待ち)
  • Assignee set to Shogo Kawahara

#2 Updated by Shogo Kawahara over 9 years ago

  • Target version set to OpenPNE 3.5.0

#3 Updated by Shogo Kawahara over 9 years ago

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

#4 Updated by Youichi Kimura over 9 years ago

  • Status changed from Pending Fixing(修正待ち) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#5 Updated by Shogo Kawahara over 9 years ago

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

#6 Updated by Kousuke Ebihara over 9 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)

#7 Updated by Mutsumi Imamura over 9 years ago

  • Due date set to 2010-03-04
  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 50 to 100

動作テストしました。問題ありません。

Also available in: Atom PDF