[jboss-cvs] JBossAS SVN: r106224 - projects/specs/trunk/jboss-ejb-api_3.1_spec/src/main/java/javax/ejb.

[jboss-cvs-commits at lists.jboss.org]

Author: jaikiran
Date: 2010-06-22 12:03:03 -0400 (Tue, 22 Jun 2010)
New Revision: 106224

Modified:
   projects/specs/trunk/jboss-ejb-api_3.1_spec/src/main/java/javax/ejb/ScheduleExpression.java
Log:
JBEE-50 Fixed ScheduleExpression.getTimezone() to return null for default ScheduleExpression

Modified: projects/specs/trunk/jboss-ejb-api_3.1_spec/src/main/java/javax/ejb/ScheduleExpression.java
===================================================================
--- projects/specs/trunk/jboss-ejb-api_3.1_spec/src/main/java/javax/ejb/ScheduleExpression.java	2010-06-22 15:29:42 UTC (rev 106223)
+++ projects/specs/trunk/jboss-ejb-api_3.1_spec/src/main/java/javax/ejb/ScheduleExpression.java	2010-06-22 16:03:03 UTC (rev 106224)
@@ -154,6 +154,14 @@
    
    public String getTimezone()
    {
+      // As per EJB3.1 spec:
+      // [97] Note that annotation java.lang.String attributes use the empty string "" as a default, 
+      // so the expression @Schedule(timezone="", ...) will result in a null value from the 
+      // corresponding ScheduleExpression.getTimezone() method.
+      if (timezone != null && timezone.isEmpty())
+      {
+         return null;
+      }
       return timezone;
    }