From 2d46810bb2a7b53ad5a667807a0fc55560f4f4c7 Mon Sep 17 00:00:00 2001
From: Sebastian Sdorra <sebastian.sdorra@cloudogu.com>
Date: Wed, 30 May 2012 08:50:50 +0200
Subject: [PATCH] cleanup empty directories during repository delete

---
 .../AbstractSimpleRepositoryHandler.java      | 43 +++++++++++++++++++
 1 file changed, 43 insertions(+)

diff --git a/scm-core/src/main/java/sonia/scm/repository/AbstractSimpleRepositoryHandler.java b/scm-core/src/main/java/sonia/scm/repository/AbstractSimpleRepositoryHandler.java
index 159a14facd..20710a20ad 100644
--- a/scm-core/src/main/java/sonia/scm/repository/AbstractSimpleRepositoryHandler.java
+++ b/scm-core/src/main/java/sonia/scm/repository/AbstractSimpleRepositoryHandler.java
@@ -181,6 +181,8 @@ public abstract class AbstractSimpleRepositoryHandler<C extends SimpleRepository
     if (directory.exists())
     {
       fileSystem.destroy(directory);
+      cleanupEmptyDirectories(config.getRepositoryDirectory(),
+                              directory.getParentFile());
     }
     else if (logger.isWarnEnabled())
     {
@@ -436,6 +438,47 @@ public abstract class AbstractSimpleRepositoryHandler<C extends SimpleRepository
     }
   }
 
+  /**
+   * Method description
+   *
+   *
+   * @param baseDirectory
+   * @param directory
+   */
+  private void cleanupEmptyDirectories(File baseDirectory, File directory)
+  {
+    if (IOUtil.isChild(baseDirectory, directory))
+    {
+      if (IOUtil.isEmpty(directory))
+      {
+
+        // TODO use filesystem
+        if (directory.delete())
+        {
+          if (logger.isInfoEnabled())
+          {
+            logger.info("successfully deleted directory {}", directory);
+          }
+
+          cleanupEmptyDirectories(baseDirectory, directory.getParentFile());
+        }
+        else if (logger.isWarnEnabled())
+        {
+          logger.warn("could not delete directory {}", directory);
+        }
+      }
+      else if (logger.isDebugEnabled())
+      {
+        logger.debug("could not remove non empty directory {}", directory);
+      }
+    }
+    else if (logger.isWarnEnabled())
+    {
+      logger.warn("directory {} is not a child of {}", directory,
+                  baseDirectory);
+    }
+  }
+
   //~--- fields ---------------------------------------------------------------
 
   /** Field description */