Add deletion of repos, taken from pull request dergachev#19

Author: Markus M. May

app/models/repository/git_remote.rb

@@ -10,14 +10,7 @@ class Repository::GitRemote < Repository::Git
   PATH_PREFIX = PLUGIN_ROOT + "/repos/"
 
   before_validation :initialize_clone
-
-  # TODO: figure out how to do this safely (if at all)
-  # before_deletion :rm_removed_repo
-  # def rm_removed_repo
-  #   if Repository.find_all_by_url(repo.url).length <= 1
-  #     system "rm -Rf #{self.clone_path}"
-  #   end
-  # end
+  before_destroy :remove_unused_repos
 
   def extra_clone_url
     return nil unless extra_info
@@ -85,6 +78,16 @@ def initialize_clone
     errors.add :extra_clone_url, err if err
   end
 
+  ## Deletes repository directory if it's inside plugin directory (i.e. belongs to plugin)
+  ## and this repo is not used by other repositories
+  def remove_unused_repos
+    inside_plugin_bundle = self.clone_path.include? PATH_PREFIX
+    nobody_else_need_it = Repository.where(url: self.relative_url).count <= 1
+    if inside_plugin_bundle && nobody_else_need_it
+      system "rm -Rf #{self.clone_path}"
+    end
+  end
+
   # equality check ignoring trailing whitespace and slashes
   def two_remotes_equal(a,b)
     a.chomp.gsub(/\/$/,'') == b.chomp.gsub(/\/$/,'')