You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by jd...@apache.org on 2008/12/19 19:14:06 UTC

svn commit: r728086 - /wicket/trunk/wicket-datetime/src/main/java/org/apache/wicket/extensions/yui/calendar/DatePicker.java

Author: jdonnerstag
Date: Fri Dec 19 10:14:06 2008
New Revision: 728086

URL: http://svn.apache.org/viewvc?rev=728086&view=rev
Log:
wicket-1988: SimpleDateFormat is not thread safe. Do not use static final

Modified:
    wicket/trunk/wicket-datetime/src/main/java/org/apache/wicket/extensions/yui/calendar/DatePicker.java

Modified: wicket/trunk/wicket-datetime/src/main/java/org/apache/wicket/extensions/yui/calendar/DatePicker.java
URL: http://svn.apache.org/viewvc/wicket/trunk/wicket-datetime/src/main/java/org/apache/wicket/extensions/yui/calendar/DatePicker.java?rev=728086&r1=728085&r2=728086&view=diff
==============================================================================
--- wicket/trunk/wicket-datetime/src/main/java/org/apache/wicket/extensions/yui/calendar/DatePicker.java (original)
+++ wicket/trunk/wicket-datetime/src/main/java/org/apache/wicket/extensions/yui/calendar/DatePicker.java Fri Dec 19 10:14:06 2008
@@ -85,7 +85,6 @@
 		{
 			// ignore
 		}
-
 	}
 
 	/**
@@ -111,13 +110,15 @@
 	 * Format to be used when configuring YUI calendar. Can be used when using the
 	 * "selected" property.
 	 */
-	public static final DateFormat FORMAT_DATE = new SimpleDateFormat("MM/dd/yyyy");
+	// See wicket-1988: SimpleDateFormat is not thread safe. Do not use static final
+	public final DateFormat FORMAT_DATE = new SimpleDateFormat("MM/dd/yyyy");
 
 	/**
 	 * For specifying which page (month/year) to show in the calendar, use this format for the date.
 	 * This is to be used together with the property "pagedate"
 	 */
-	public static final DateFormat FORMAT_PAGEDATE = new SimpleDateFormat("MM/yyyy");
+	// See wicket-1988: SimpleDateFormat is not thread safe. Do not use static final
+	public final DateFormat FORMAT_PAGEDATE = new SimpleDateFormat("MM/yyyy");
 
 	private static final ResourceReference YUI = new JavascriptResourceReference(YuiLib.class, "");