Add move repository feature. (It should be tested!)

Author: root

Diff for: app/controllers/repository_git_extras_controller.rb

@@ -18,6 +18,7 @@ def update
   end
 
 
+<<<<<<< HEAD
   def sort_urls
     @git_extra = @repository.extra
     if request.post?
@@ -29,4 +30,28 @@ def sort_urls
     end
   end
 
+
+  def move
+    @projects = Project.all
+    @projects.delete(@project)
+    @move_repository_form = RepositoryMover.new()
+    if request.post?
+      @move_repository_form = RepositoryMover.new(params[:repository_mover].merge(repository_id: @repository.id))
+      if @move_repository_form.valid?
+        project = @move_repository_form.project
+        @repository.update_attribute(:project_id, project.id)
+        GitoliteAccessor.move_repository(@repository)
+        redirect_to settings_project_path(project, tab: 'repositories')
+      end
+    end
+  end
+
+
+  private
+
+
+    def set_git_extra
+      @git_extra = @repository.extra
+    end
+
 end

Diff for: app/models/repository_mover.rb

@@ -0,0 +1,48 @@
+class RepositoryMover
+  include ActiveModel::Validations
+  include ActiveModel::Conversion
+  extend ActiveModel::Naming
+
+  attr_accessor :repository_id, :project_id
+
+  validates_presence_of :repository_id, :project_id
+  validate :repository_identifier
+  validate :repository_uniqueness
+
+
+  def initialize(attributes = {})
+    attributes.each do |name, value|
+      send("#{name}=", value)
+    end
+  end
+
+
+  def project
+    Project.find_by_id(project_id)
+  end
+
+
+  def repository
+    Repository.find_by_id(repository_id)
+  end
+
+
+  def persisted?
+    false
+  end
+
+
+  private
+
+
+    def repository_identifier
+      errors.add(:base, :identifier_empty) if (repository.identifier.nil? || repository.identifier.blank?)
+    end
+
+
+    def repository_uniqueness
+      new_repo = project.repositories.find_by_identifier(repository.identifier)
+      errors.add(:base, :identifier_taken) if !new_repo.nil?
+    end
+
+end

Diff for: app/services/gitolite_accessor.rb

@@ -34,6 +34,12 @@ def update_repository(repository, opts = {})
     end
 
 
+    def move_repository(repository, opts = {})
+      logger.info("User '#{User.current.login}' has moved repository '#{repository.gitolite_repository_name}'")
+      resync_gitolite(:move_repository, repository.id, opts)
+    end
+
+
     def destroy_repository(repository)
       logger.info("User '#{User.current.login}' has removed repository '#{repository.gitolite_repository_name}'")
       resync_gitolite(:delete_repository, repository.data_for_destruction)

Diff for: app/views/projects/settings/_repositories.html.haml

@@ -44,6 +44,9 @@
               = link_to l(:button_edit), edit_repository_path(repository),
                                          class: 'icon icon-edit'
 
+              = link_to l(:button_move), move_repository_git_extras_path(repository),
+                                         class: 'icon icon-move'
+
               = delete_link repository_path(repository)
 
 - else

Diff for: app/views/repository_git_extras/move.html.haml

@@ -0,0 +1,14 @@
+%h2= l(:label_move_repository, repo_name: @repository.gitolite_repository_name)
+
+.box
+
+  = labelled_form_for :repository_mover, @move_repository_form,
+                      url:  move_repository_git_extras_path(@repository),
+                      html: { method: :post, class: 'tabular', data: { confirm: l(:text_are_you_sure) } } do |f|
+
+    .flash-messages= error_messages_for 'move_repository_form'
+
+    %p= f.select :project_id, options_from_collection_for_select(@projects, :id, :name), prompt: 'Select a project'
+    %p
+      = f.submit l(:button_move)
+      = link_to l(:button_cancel), settings_project_path(@project, tab: 'repositories')

Diff for: assets/images/move.png

@@ -5,6 +5,7 @@ PLUGIN ICONS
 
 .icon-git-logo { background-image: url('../images/git_icon.png'); padding-right: 5px; }
 .icon-clone { background-image: url('../images/clone.png'); }
+.icon-move { background-image: url('../images/move.png'); }
 
 .icon-git { padding-right: 15px; }
 .icon-git-disabled { opacity: 0.5; }

Diff for: assets/stylesheets/application.css

@@ -0,0 +1,11 @@
+fr:
+  label_move_repository: "Déplacement du dépôt : '%{repo_name}'"
+
+  activemodel:
+    errors:
+      models:
+        repository_mover:
+          attributes:
+            base:
+              identifier_empty: Ne peut déplacer un dépôt avec un identifiant vide
+              identifier_taken: Identifiant de dépôt déjà utilisé par le nouveau projet

Diff for: config/locales/models/repository_mover/fr.yml

@@ -12,6 +12,9 @@
 
   resource  :git_extras, controller: 'repository_git_extras', only: [:update] do
     match 'sort_urls', via: [:get, :post]
+    member do
+      match 'move', via: [:get, :post]
+    end
   end
 
   resource  :git_notifications, controller: 'repository_git_notifications'

Diff for: config/routes.rb

@@ -69,6 +69,21 @@ def delete_repository
       end
 
 
+      def move_repository
+        if repository = Repository.find_by_id(object_id)
+          admin.transaction do
+            RedmineGitHosting::GitoliteHandlers::RepositoryMover.new(repository, gitolite_config, action).call
+            gitolite_admin_repo_commit("#{repository.gitolite_repository_name}")
+          end
+
+          # Fetch changeset
+          repository.fetch_changesets
+        else
+          logger.error("#{action} : repository does not exist anymore, object is nil, exit !")
+        end
+      end
+
+
       private