You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by ad...@apache.org on 2008/08/06 23:13:09 UTC
svn commit: r683414 -
/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/SetCalendar.java
Author: adrianc
Date: Wed Aug 6 14:13:09 2008
New Revision: 683414
URL: http://svn.apache.org/viewvc?rev=683414&view=rev
Log:
Minilang set-calendar improvement - supply default local and time zone when they aren't available.
There was a problem with the set-calendar operation when it was called from the job scheduler - a NPE was thrown. This commit fixes that.
Modified:
ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/SetCalendar.java
Modified: ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/SetCalendar.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/SetCalendar.java?rev=683414&r1=683413&r2=683414&view=diff
==============================================================================
--- ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/SetCalendar.java (original)
+++ ofbiz/trunk/framework/minilang/src/org/ofbiz/minilang/method/envops/SetCalendar.java Wed Aug 6 14:13:09 2008
@@ -121,9 +121,15 @@
if (locale == null) {
locale = methodContext.getLocale();
}
+ if (locale == null) {
+ locale = Locale.getDefault();
+ }
if (timeZone == null) {
timeZone = methodContext.getTimeZone();
}
+ if (timeZone == null) {
+ timeZone = TimeZone.getDefault();
+ }
fromStamp = (Timestamp) ObjectType.simpleTypeConvert(newValue, "Timestamp", UtilDateTime.DATE_TIME_FORMAT, timeZone, locale, true);
} catch (Exception e) {
// Catching all exceptions - even potential ClassCastException