[GROOVY-3326] Obscure method getTimeZone(java.util.Date) in org.codehaus.groovy.runtime.TimeCategory

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.6-rc-2
• Fix Version/s: 2.1.0-rc-2, 2.1.0
• Component/s: groovy-jdk
• Labels: breaking
• Assignee: Paul King
• Reporter: Alexander Veit
• Created: 01/Feb/09 11:41
• Updated: 18/Jan/13 16:06
• Resolved: 17/Jan/13 16:45

Description

TimeCategory#getTimeZone(Date) pretends to return a TimeZone that would be somehow associated with the given Date.

• A java.util.Date does not carry any time zone information as long as it's deprecated functionality is not used an it's toString() method is not called.
• A java.util.Calendar's time zone is not affected by calling #setTime(Date).

So the current implementation of TimeCategory#getTimeZone(Date) is effectively equivalent to

public static TimeZone getTimeZone(Date self) {
  return TimeZone.getDefault();
}

The result does not depend on the passed Date, and the method should probably be deprecated.