You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@roller.apache.org by gm...@apache.org on 2014/07/09 13:36:37 UTC
svn commit: r1609108 - in
/roller/trunk/app/src/main/java/org/apache/roller/weblogger:
business/startup/WebloggerStartup.java ui/core/RollerContext.java
Author: gmazza
Date: Wed Jul 9 11:36:37 2014
New Revision: 1609108
URL: http://svn.apache.org/r1609108
Log:
ROL-1948 NPE check when roller.war never got expanded.
Modified:
roller/trunk/app/src/main/java/org/apache/roller/weblogger/business/startup/WebloggerStartup.java
roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/core/RollerContext.java
Modified: roller/trunk/app/src/main/java/org/apache/roller/weblogger/business/startup/WebloggerStartup.java
URL: http://svn.apache.org/viewvc/roller/trunk/app/src/main/java/org/apache/roller/weblogger/business/startup/WebloggerStartup.java?rev=1609108&r1=1609107&r2=1609108&view=diff
==============================================================================
--- roller/trunk/app/src/main/java/org/apache/roller/weblogger/business/startup/WebloggerStartup.java (original)
+++ roller/trunk/app/src/main/java/org/apache/roller/weblogger/business/startup/WebloggerStartup.java Wed Jul 9 11:36:37 2014
@@ -176,13 +176,8 @@ public final class WebloggerStartup {
try {
mailProvider = new MailProvider();
} catch(StartupException ex) {
- if (LOG.isDebugEnabled()) {
- LOG.debug("Failed to setup mail provider, continuing anyways.\n"
- + "Reason: " + ex.getMessage(), ex);
- } else {
- LOG.warn("Failed to setup mail provider, continuing anyways.\n"
- + "Reason: " + ex.getMessage());
- }
+ LOG.warn("Failed to setup mail provider, continuing anyways.\n"
+ + "Reason: " + ex.getMessage());
}
// now we need to deal with database install/upgrade logic
Modified: roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/core/RollerContext.java
URL: http://svn.apache.org/viewvc/roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/core/RollerContext.java?rev=1609108&r1=1609107&r2=1609108&view=diff
==============================================================================
--- roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/core/RollerContext.java (original)
+++ roller/trunk/app/src/main/java/org/apache/roller/weblogger/ui/core/RollerContext.java Wed Jul 9 11:36:37 2014
@@ -104,6 +104,10 @@ public class RollerContext extends Conte
// get the *real* path to <context>/resources
String ctxPath = servletContext.getRealPath("/");
+ if (ctxPath == null) {
+ log.fatal("Roller requires an exploded WAR file to run.");
+ return;
+ }
if (!ctxPath.endsWith(File.separator)) {
ctxPath += File.separator + "resources";
} else {