[
https://issues.apache.org/jira/browse/GROOVY-7964?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15573385#comment-15573385
]
Los Pejos edited comment on GROOVY-7964 at 10/13/16 10:20 PM:
--------------------------------------------------------------
Well, I run this script, and on the same OS with the same jvm and timezone java
versions (ChronoUnit and JodaTime) has no errors, while Groovy TimeCategory
has. Thus I suppose TimeCategory is buggy. Currently as a workaround I moved
from TimeCategory to ChronoUnit all my projects code.
was (Author: lospejos):
Well, I run this script, and on the same OS with the same jvm and timezone java
versions (ChronoUnit and JodaTime) has no errors, while Groovy TimeCategory
has. Thus I suppose TineCategory is buggy. Currently as a workaround I moved
from TimeCategory to ChronoUnit all my projects code.
> TimeCategory incorrectly counts days difference between two dates
> -----------------------------------------------------------------
>
> Key: GROOVY-7964
> URL: https://issues.apache.org/jira/browse/GROOVY-7964
> Project: Groovy
> Issue Type: Bug
> Components: groovy-runtime
> Affects Versions: 2.4.7
> Environment: Windows 10 x86_64, Oracle Java 1.8.0.102 x86_64
> Reporter: Los Pejos
> Labels: timecategory
> Attachments: DatesDiff.groovy
>
>
> TimeCategory incorrectly counts days difference between two dates.
> Exhibit (Groovy code):
> {code}
> #!/usr/bin/env groovy
> @Grab(group='joda-time', module='joda-time', version='2.9.4+')
> import java.io.BufferedReader
> import java.io.IOException
> import java.io.InputStreamReader
> import java.time.LocalDate
> import java.time.format.DateTimeFormatter
> import java.time.temporal.ChronoUnit
> import groovy.time.TimeCategory
> String date1 = '2016-03-22'
> String date2 = '2016-03-29'
> String fmt = 'yyyy-MM-dd'
> // Java 8 ChronoUnit:
> DateTimeFormatter formatter = DateTimeFormatter.ofPattern(fmt)
> //LocalDate firstDate = LocalDate.parse(date1, formatter)
> //LocalDate secondDate = LocalDate.parse(date2, formatter)
> long days = ChronoUnit.DAYS.between(
> LocalDate.parse(date1, formatter),
> LocalDate.parse(date2, formatter)
> )
> println "Java 8 ChronoUnit: Dates between $date1 and $date2 : $days"
> // Groovy TimeCategory/Date:
> use (TimeCategory) {
> def d = (Date.parse(fmt, date2) - Date.parse(fmt, date1)).days
> println "Groovy TimeCategory/Date: Dates between $date1 and $date2 : $d"
> }
> // Joda Time:
> // LocalDate fromDateFields(Date date)
> // Date.parse('yyyy-MM-dd', dlf.@dldpf.toString())
> days = org.joda.time.Days.daysBetween(
> org.joda.time.LocalDate.fromDateFields(Date.parse(fmt,date1)),
> org.joda.time.LocalDate.fromDateFields(Date.parse(fmt,date2))
> ).getDays()
> println "Joda Time: Dates between $date1 and $date2 : $days"
> {code}
> Current Output:
> {noformat}
> $ groovy DatesDiff.groovy
> Java 8 ChronoUnit: Dates between 2016-03-22 and 2016-03-29 : 7
> Groovy TimeCategory/Date: Dates between 2016-03-22 and 2016-03-29 : 6
> Joda Time: Dates between 2016-03-22 and 2016-03-29 : 7
> {noformat}
> Expected Output:
> {noformat}
> $ groovy DatesDiff.groovy
> Java 8 ChronoUnit: Dates between 2016-03-22 and 2016-03-29 : 7
> Groovy TimeCategory/Date: Dates between 2016-03-22 and 2016-03-29 : 7
> Joda Time: Dates between 2016-03-22 and 2016-03-29 : 7
> {noformat}
> Resume: notice the difference in "Groovy TimeCategory/Date" line.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
