[exo-jcr-commits] exo-jcr SVN: r5669 - jcr/trunk/exo.jcr.component.ext/src/main/java/org/exoplatform/services/jcr/ext/backup/impl.

[do-not-reply at jboss.org]

Author: andrew.plotnikov
Date: 2012-02-20 09:02:22 -0500 (Mon, 20 Feb 2012)
New Revision: 5669

Modified:
   jcr/trunk/exo.jcr.component.ext/src/main/java/org/exoplatform/services/jcr/ext/backup/impl/BackupManagerImpl.java
Log:
EXOJCR-1767: Fixed BackupManager throws NPE in case of wrong configuration of backup logs folder

Modified: jcr/trunk/exo.jcr.component.ext/src/main/java/org/exoplatform/services/jcr/ext/backup/impl/BackupManagerImpl.java
===================================================================
--- jcr/trunk/exo.jcr.component.ext/src/main/java/org/exoplatform/services/jcr/ext/backup/impl/BackupManagerImpl.java	2012-02-20 12:00:00 UTC (rev 5668)
+++ jcr/trunk/exo.jcr.component.ext/src/main/java/org/exoplatform/services/jcr/ext/backup/impl/BackupManagerImpl.java	2012-02-20 14:02:22 UTC (rev 5669)
@@ -897,6 +897,10 @@
     */
    public void start()
    {
+      if (!PrivilegedFileHelper.exists(tempDir))
+      {
+         throw new IllegalStateException("Directory " + tempDir.getAbsolutePath() + " not found. Please create it.");
+      }
 
       //remove if exists all old jcrrestorewi*.tmp files.
       File[] files = PrivilegedFileHelper.listFiles(tempDir, new JcrRestoreWiFilter());
@@ -935,6 +939,12 @@
          readParamsFromFile();
       }
 
+      if (!PrivilegedFileHelper.exists(logsDirectory))
+      {
+         throw new IllegalStateException("Directory " + logsDirectory.getAbsolutePath()
+            + " not found. Please create it.");
+      }
+
       // scan for task files
       File[] tasks = PrivilegedFileHelper.listFiles(this.logsDirectory, new TaskFilter());
       for (File task : tasks)