developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 6 years ago

Closed 6 years ago

#1338 closed bug (fixed)

jDatetime: durationTo should use gmdate to prevent dst problem when counting days

Reported by: Julien Owned by: Julien
Priority: normal Milestone: Jelix 1.2.3
Component: jelix:utils Version: trunk
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

because we use seconds to calculate the days between 2 dates, we may have a difference due to dst (because we miss 1 hour or have 1 more hour)

patch: https://bitbucket.org/julieni/jelix-trunk-patches/src/128b1c3f30f5/jDatetime-durationto-should-use-gmdate-to-prevent-dst-problem-when-counting-days.patch

Change History (5)

comment:1 Changed 6 years ago by Julien

  • Owner set to Julien
  • Status changed from new to assigned

comment:2 Changed 6 years ago by laurentj

  • review review? deleted

tests tests tests tests :-)

comment:3 Changed 6 years ago by laurentj

  • Milestone changed from Jelix 1.3 to Jelix 1.2.3

comment:4 Changed 6 years ago by Julien

coming, coming, coming ;)

just not having enough time theses last days (/weeks/months.. ;) )

comment:5 Changed 6 years ago by laurentj

  • Resolution set to fixed
  • Status changed from assigned to closed

There are already tests.

Patch pushed https://bitbucket.org/jelix/jelix-1.2.x/changeset/8be90dfdb492 thanks.

Note: See TracTickets for help on using tickets.