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;