You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@roller.apache.org by ag...@apache.org on 2005/07/14 23:21:38 UTC

svn commit: r219101 - /incubator/roller/trunk/src/org/roller/business/ThemeManagerImpl.java

Author: agilliland
Date: Thu Jul 14 14:21:35 2005
New Revision: 219101

URL: http://svn.apache.org/viewcvs?rev=219101&view=rev
Log:
bug fix for null pointer which handles case when no themes are found on disk.

Modified:
    incubator/roller/trunk/src/org/roller/business/ThemeManagerImpl.java

Modified: incubator/roller/trunk/src/org/roller/business/ThemeManagerImpl.java
URL: http://svn.apache.org/viewcvs/incubator/roller/trunk/src/org/roller/business/ThemeManagerImpl.java?rev=219101&r1=219100&r2=219101&view=diff
==============================================================================
--- incubator/roller/trunk/src/org/roller/business/ThemeManagerImpl.java (original)
+++ incubator/roller/trunk/src/org/roller/business/ThemeManagerImpl.java Thu Jul 14 14:21:35 2005
@@ -48,8 +48,9 @@
         // rather than be lazy we are going to load all themes from
         // the disk preemptively during initialization and cache them
         mLogger.debug("Initializing ThemeManagerImpl");
-        mLogger.info("Loading all themes from disk ... ");
+        
         this.themes = this.loadAllThemesFromDisk();
+        mLogger.info("Loaded "+this.themes.size()+" themes from disk.");
     }
     
     
@@ -187,6 +188,9 @@
             }
         };
         String[] themenames = themesdir.list(filter);
+        
+        if(themenames == null)
+            themenames = new String[0];
         
         // now go through each theme and read all it's templates
         Theme theme = null;