[jboss-svn-commits] JBL Code SVN: r11413 - labs/jbossforums/branches/forums101P26/forums/src/main/org/jboss/portlet/forums/ui.

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

Author: unibrew
Date: 2007-04-27 09:01:42 -0400 (Fri, 27 Apr 2007)
New Revision: 11413

Modified:
   labs/jbossforums/branches/forums101P26/forums/src/main/org/jboss/portlet/forums/ui/PortalUtil.java
Log:
[JBFORUMS-198] Correcting small mistake that can be a efficency drawback.

Modified: labs/jbossforums/branches/forums101P26/forums/src/main/org/jboss/portlet/forums/ui/PortalUtil.java
===================================================================
--- labs/jbossforums/branches/forums101P26/forums/src/main/org/jboss/portlet/forums/ui/PortalUtil.java	2007-04-27 12:48:44 UTC (rev 11412)
+++ labs/jbossforums/branches/forums101P26/forums/src/main/org/jboss/portlet/forums/ui/PortalUtil.java	2007-04-27 13:01:42 UTC (rev 11413)
@@ -73,6 +73,8 @@
     private static CLLoader loader = new CLLoader("template");
 
     private static Logger log = Logger.getLogger(PortalUtil.class);
+    
+    private static SimpleDateFormat sdfForLastLoginDate;
 
     private static User user = new User()
     {
@@ -139,6 +141,8 @@
         {
             upm = temporary;
         }
+        
+        sdfForLastLoginDate = new SimpleDateFormat("EEE MMM dd HH:mm:ss zzz yyyy",Locale.ENGLISH);
     }
 
     /**
@@ -477,9 +481,12 @@
             {
                 return null;
             }
-            String property = upm.getProperty(user, User.INFO_USER_LAST_LOGIN_DATE).toString();
-            Date date =  new SimpleDateFormat("EEE MMM dd HH:mm:ss zzz yyyy",Locale.ENGLISH).parse(property);
-            return date;
+            Object property = upm.getProperty(user, User.INFO_USER_LAST_LOGIN_DATE);
+            if (property!=null)
+            {
+                Date date =  sdfForLastLoginDate.parse(property.toString());
+                return date;
+            }
         } catch (Exception e)
         {
             JSFUtil.handleException(e);