[ http://jira.jboss.com/jira/browse/JBPM-1098?page=all ] Ronald van Kuijk updated JBPM-1098: ----------------------------------- Fix Version/s: jBPM jPDL 3.2.3 Priority: Blocker (was: Minor) 2008 is a leapyear, so we'll soon have a problem with February 29 -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ http://jira.jboss.com/jira/browse/JBPM-1098?page=comments#action_12392165 ] Alejandro Guizar commented on JBPM-1098: ---------------------------------------- java.util.Calendar has a method to add an amount of time to a given field of the calendar. Addition is aware of calendar rules, including leap years. Now, if you just want to know whether a year is a leap, you can use method GregorianCalendar.isLeapYear(int year). -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ http://jira.jboss.com/jira/browse/JBPM-1098?page=all ] Ronald van Kuijk reassigned JBPM-1098: -------------------------------------- Assignee: Ronald van Kuijk (was: Tom Baeyens) -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ http://jira.jboss.com/jira/browse/JBPM-1098?page=comments#action_12399275 ] Alejandro Guizar commented on JBPM-1098: ---------------------------------------- Ronald, some of our customers have expressed concerns about this issue, so I want to clarify the real nature of this problem. Please review the following description and tell me if I'm missing something. Durations in jBPM, as implemented by org.jbpm.calendar.Duration, are always treated as milliseconds regardless of the unit of time that occurs in the process definition. Quoting the aforementioned class, we have: SECOND = 1000; MINUTE = 60*SECOND; HOUR = 60*MINUTE; DAY = 24*HOUR; WEEK = 7*DAY; MONTH = 30*DAY; YEAR = 365*DAY; The problem is, not all months and years have the same length. Leap years are 366 days. Months have between 28 and 31 days. Therefore, to accurately add months or years to a time instant, the original unit of time must be preserved in order to determine how many days to add, in relation to the month or year of the time instant. Nevertheless, durations consisting of seconds, minutes, hours, days or weeks are NOT affected, because these units always have the same length. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ http://jira.jboss.com/jira/browse/JBPM-1098?page=comments#action_12399687 ] Ronald van Kuijk commented on JBPM-1098: ---------------------------------------- Alex, (sorry for the delayed response, I have not been following Jira issues the last week.) Yes you are right. That is what I encountered as well. You proposal of keeping the original unit of time might make adding business month (currently fixed at 20 days) work better as well. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira

[ http://jira.jboss.com/jira/browse/JBPM-1098?page=all ] Alejandro Guizar resolved JBPM-1098. ------------------------------------ Resolution: Done Fix applied to all relevant branches. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira