A category with assigned issue can now be deleted. 2 options are proposed:

* remove assignments (issues are set to 'no category')
* reassign issues to another category (if at least an other category exists)
If no issue is assigned to the category, it's deleted silently.

git-svn-id: http://redmine.rubyforge.org/svn/trunk@733 e93f8b46-1217-0410-a6f0-8f06a7374b81

diff --git a/app/controllers/issue_categories_controller.rb b/app/controllers/issue_categories_controller.rb
index 29a3f02afb4f46d2f7a4be7b44dcda490353f16a..2c1c6657b98c073dca7ab38c6ecf636dba9f7b38 100644 (file)
--- a/app/controllers/issue_categories_controller.rb
+++ b/app/controllers/issue_categories_controller.rb
@@ -18,6 +18,8 @@
 class IssueCategoriesController < ApplicationController
   layout 'base'
   before_filter :find_project, :authorize
+  
+  verify :method => :post, :only => :destroy
 
   def edit
     if request.post? and @category.update_attributes(params[:category])
@@ -27,11 +29,17 @@ class IssueCategoriesController < ApplicationController
   end
 
   def destroy
-    @category.destroy
-    redirect_to :controller => 'projects', :action => 'settings', :tab => 'categories', :id => @project
-  rescue
-    flash[:error] = "Categorie can't be deleted"
-    redirect_to :controller => 'projects', :action => 'settings', :tab => 'categories', :id => @project
+    @issue_count = @category.issues.size
+    if @issue_count == 0
+      # No issue assigned to this category
+      @category.destroy
+      redirect_to :controller => 'projects', :action => 'settings', :id => @project, :tab => 'categories'
+    elsif params[:todo]
+      reassign_to = @project.issue_categories.find_by_id(params[:reassign_to_id]) if params[:todo] == 'reassign'
+      @category.destroy(reassign_to)
+      redirect_to :controller => 'projects', :action => 'settings', :id => @project, :tab => 'categories'
+    end
+    @categories = @project.issue_categories - [@category]
   end
 
 private

diff --git a/app/models/issue_category.rb b/app/models/issue_category.rb
index fb2c099a185a6f1a84ad2047cc0546119d284101..1abcf4c32c4fa4b07dad11c1cb88f9927e2a8486 100644 (file)
--- a/app/models/issue_category.rb
+++ b/app/models/issue_category.rb
@@ -16,15 +16,21 @@
 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
 
 class IssueCategory < ActiveRecord::Base
-  before_destroy :check_integrity  
   belongs_to :project
   belongs_to :assigned_to, :class_name => 'User', :foreign_key => 'assigned_to_id'
-
+  has_many :issues, :foreign_key => 'category_id', :dependent => :nullify
+  
   validates_presence_of :name
   validates_uniqueness_of :name, :scope => [:project_id]
   
-private
-  def check_integrity
-    raise "Can't delete category" if Issue.find(:first, :conditions => ["category_id=?", self.id])
+  alias :destroy_without_reassign :destroy
+  
+  # Destroy the category
+  # If a category is specified, issues are reassigned to this category
+  def destroy(reassign_to = nil)
+    if reassign_to && reassign_to.is_a?(IssueCategory) && reassign_to.project == self.project
+      Issue.update_all("category_id = #{reassign_to.id}", "category_id = #{id}")
+    end
+    destroy_without_reassign
   end
 end

diff --git a/app/views/issue_categories/destroy.rhtml b/app/views/issue_categories/destroy.rhtml
new file mode 100644 (file)
index 0000000..a563736
--- /dev/null
+++ b/app/views/issue_categories/destroy.rhtml
@@ -0,0 +1,15 @@
+<h2><%=l(:label_issue_category)%>: <%=h @category.name %></h2>
+
+<% form_tag({}) do %>
+<div class="box">
+<p><strong><%= l(:text_issue_category_destroy_question, @issue_count) %></strong></p>
+<p><%= radio_button_tag 'todo', 'nullify', true %> <%= l(:text_issue_category_destroy_assignments) %><br />
+<% if @categories.size > 0 %>
+<%= radio_button_tag 'todo', 'reassign', false %> <%= l(:text_issue_category_reassign_to) %>:
+<%= select_tag 'reassign_to_id', options_from_collection_for_select(@categories, 'id', 'name') %></p>
+<% end %>
+</div>
+
+<%= submit_tag l(:button_apply) %>
+<%= link_to l(:button_cancel), :controller => 'projects', :action => 'settings', :id => @project, :tab => 'categories' %>
+<% end %>

diff --git a/app/views/projects/settings/_issue_categories.rhtml b/app/views/projects/settings/_issue_categories.rhtml
index bad330e20e42e67cd1ebb04b293f7e531baf1263..c1e77e9cc78f0ba7f3887370fec86d6dd2931610 100644 (file)
--- a/app/views/projects/settings/_issue_categories.rhtml
+++ b/app/views/projects/settings/_issue_categories.rhtml
@@ -12,7 +12,7 @@
     <td><%=h(category.name) %></td>
     <td><%=h(category.assigned_to.name) if category.assigned_to %></td>
     <td align="center"><small><%= link_to_if_authorized l(:button_edit), { :controller => 'issue_categories', :action => 'edit', :id => category }, :class => 'icon icon-edit' %></small></td>
-    <td align="center"><small><%= link_to_if_authorized l(:button_delete), {:controller => 'issue_categories', :action => 'destroy', :id => category}, :confirm => l(:text_are_you_sure), :method => :post, :class => 'icon icon-del' %></small></td>
+    <td align="center"><small><%= link_to_if_authorized l(:button_delete), {:controller => 'issue_categories', :action => 'destroy', :id => category}, :method => :post, :class => 'icon icon-del' %></small></td>
        </tr>
        <% end %>
 <% end %>

diff --git a/lang/bg.yml b/lang/bg.yml
index c3ce4f12954b8cf7f046729c5cadc44baab6bb3a..c518f8f9eabf43afcd1b630e42a227ab67723385 100644 (file)
--- a/lang/bg.yml
+++ b/lang/bg.yml
@@ -479,6 +479,9 @@ text_issues_ref_in_commit_messages: Отбелязване и приключва
 text_issue_added: Публикувана е нова задача с номер %s.
 text_issue_updated: Задача %s е обновена.
 text_wiki_destroy_confirmation: Are you sure you want to delete this wiki and all its content ?
+text_issue_category_destroy_question: Some issues (%d) are assigned to this category. What do you want to do ?
+text_issue_category_destroy_assignments: Remove category assignments
+text_issue_category_reassign_to: Reassing issues to this category
 
 default_role_manager: Мениджър
 default_role_developper: Разработчик

diff --git a/lang/de.yml b/lang/de.yml
index 7e7441dd8ecbc462d38fffcbfd77a87c483bc8e7..88cdbf52143fe589804ced9961f658d1864034a2 100644 (file)
--- a/lang/de.yml
+++ b/lang/de.yml
@@ -479,6 +479,9 @@ text_issues_ref_in_commit_messages: Referencing and fixing issues in commit mess
 text_issue_added: Ticket %s wurde erstellt.
 text_issue_updated: Ticket %s wurde aktualisiert.
 text_wiki_destroy_confirmation: Are you sure you want to delete this wiki and all its content ?
+text_issue_category_destroy_question: Some issues (%d) are assigned to this category. What do you want to do ?
+text_issue_category_destroy_assignments: Remove category assignments
+text_issue_category_reassign_to: Reassing issues to this category
 
 default_role_manager: Manager
 default_role_developper: Developer

diff --git a/lang/en.yml b/lang/en.yml
index f89cfae7ed764f0e1a0473cc20af78353b026442..546570694b3a8c3aaa30d980d6f47731879bbb46 100644 (file)
--- a/lang/en.yml
+++ b/lang/en.yml
@@ -479,6 +479,9 @@ text_issues_ref_in_commit_messages: Referencing and fixing issues in commit mess
 text_issue_added: Issue %s has been reported.
 text_issue_updated: Issue %s has been updated.
 text_wiki_destroy_confirmation: Are you sure you want to delete this wiki and all its content ?
+text_issue_category_destroy_question: Some issues (%d) are assigned to this category. What do you want to do ?
+text_issue_category_destroy_assignments: Remove category assignments
+text_issue_category_reassign_to: Reassing issues to this category
 
 default_role_manager: Manager
 default_role_developper: Developer

diff --git a/lang/es.yml b/lang/es.yml
index 9deecd9ad73d6daec9dba0b6c19fd8f7c8bf207c..ad05632af5d25a257907b995d7013a0b98e1f411 100644 (file)
--- a/lang/es.yml
+++ b/lang/es.yml
@@ -479,6 +479,9 @@ text_issues_ref_in_commit_messages: Referencing and fixing issues in commit mess
 text_issue_added: Issue %s has been reported.
 text_issue_updated: Issue %s has been updated.
 text_wiki_destroy_confirmation: Are you sure you want to delete this wiki and all its content ?
+text_issue_category_destroy_question: Some issues (%d) are assigned to this category. What do you want to do ?
+text_issue_category_destroy_assignments: Remove category assignments
+text_issue_category_reassign_to: Reassing issues to this category
 
 default_role_manager: Manager
 default_role_developper: Desarrollador

diff --git a/lang/fr.yml b/lang/fr.yml
index 852b04f955c871c5b9d8ee7b093076a9f62eb99e..0aaf41c4b9de8ad0980228aa481b6a2ba7d113ae 100644 (file)
--- a/lang/fr.yml
+++ b/lang/fr.yml
@@ -479,6 +479,9 @@ text_issues_ref_in_commit_messages: Référencement et résolution des demandes
 text_issue_added: La demande %s a été soumise.
 text_issue_updated: La demande %s a été mise à jour.
 text_wiki_destroy_confirmation: Etes-vous sûr de vouloir supprimer ce wiki et tout son contenu ?
+text_issue_category_destroy_question: Des demandes (%d) sont affectées à cette catégories. Que voulez-vous faire ?
+text_issue_category_destroy_assignments: N'affecter les demandes à aucune autre catégorie
+text_issue_category_reassign_to: Réaffecter les demandes à cette catégorie
 
 default_role_manager: Manager
 default_role_developper: Développeur

diff --git a/lang/it.yml b/lang/it.yml
index acaeb65b7e664bc5071c889fa16d1e8d3257a1f2..49e352abe9228c96327cdf280c278143d3dfdeed 100644 (file)
--- a/lang/it.yml
+++ b/lang/it.yml
@@ -479,6 +479,9 @@ text_issues_ref_in_commit_messages: Referencing and fixing issues in commit mess
 text_issue_added: "E' stata segnalata l'anomalia %s."
 text_issue_updated: "L'anomalia %s e' stata aggiornata."
 text_wiki_destroy_confirmation: Are you sure you want to delete this wiki and all its content ?
+text_issue_category_destroy_question: Some issues (%d) are assigned to this category. What do you want to do ?
+text_issue_category_destroy_assignments: Remove category assignments
+text_issue_category_reassign_to: Reassing issues to this category
 
 default_role_manager: Manager
 default_role_developper: Sviluppatore

diff --git a/lang/ja.yml b/lang/ja.yml
index 8134cefa27298772f592249183fe0d582cd0426b..a59ccaf7e0947835563bce2331847b381c218099 100644 (file)
--- a/lang/ja.yml
+++ b/lang/ja.yml
@@ -480,6 +480,9 @@ text_issues_ref_in_commit_messages: コミットメッセージ内で問題の
 text_issue_added: 問題 %s が報告されました。
 text_issue_updated: 問題 %s が更新されました。
 text_wiki_destroy_confirmation: Are you sure you want to delete this wiki and all its content ?
+text_issue_category_destroy_question: Some issues (%d) are assigned to this category. What do you want to do ?
+text_issue_category_destroy_assignments: Remove category assignments
+text_issue_category_reassign_to: Reassing issues to this category
 
 default_role_manager: 管理者
 default_role_developper: 開発者

diff --git a/lang/nl.yml b/lang/nl.yml
index 75c2996ace2f35107d324278092ad503dc8a5b5e..4112339090faba3bb210f185d021b450c72b73b2 100644 (file)
--- a/lang/nl.yml
+++ b/lang/nl.yml
@@ -479,6 +479,9 @@ text_issues_ref_in_commit_messages: Opzoeken en aanpassen van issues in commit b
 text_issue_added: Issue %s is gerapporteerd.
 text_issue_updated: Issue %s is gewijzigd.
 text_wiki_destroy_confirmation: Are you sure you want to delete this wiki and all its content ?
+text_issue_category_destroy_question: Some issues (%d) are assigned to this category. What do you want to do ?
+text_issue_category_destroy_assignments: Remove category assignments
+text_issue_category_reassign_to: Reassing issues to this category
 
 default_role_manager: Manager
 default_role_developper: Ontwikkelaar

diff --git a/lang/pt-br.yml b/lang/pt-br.yml
index dd61d3ed32265666797fa5c0cd39d8cca0a12888..229abe833c683308c233997ca2a44fc14ee066c5 100644 (file)
--- a/lang/pt-br.yml
+++ b/lang/pt-br.yml
@@ -479,6 +479,9 @@ text_issues_ref_in_commit_messages: Referencing and fixing issues in commit mess
 text_issue_added: Tarefa %s foi incluída.\r
 text_issue_updated: Tarefa %s foi alterada.\r
 text_wiki_destroy_confirmation: Are you sure you want to delete this wiki and all its content ?\r
+text_issue_category_destroy_question: Some issues (%d) are assigned to this category. What do you want to do ?\r
+text_issue_category_destroy_assignments: Remove category assignments\r
+text_issue_category_reassign_to: Reassing issues to this category\r
 \r
 default_role_manager: Analista de Negocio ou Gerente de Projeto\r
 default_role_developper: Desenvolvedor\r

diff --git a/lang/pt.yml b/lang/pt.yml
index fad533d3707e4640984cdeaaaa38be3b2363c358..7d7cedaf264bda6eb9f3054b9cc0c297070f70f5 100644 (file)
--- a/lang/pt.yml
+++ b/lang/pt.yml
@@ -479,6 +479,9 @@ text_issues_ref_in_commit_messages: Referenciando e arrumando tarefas nas mensag
 text_issue_added: Tarefa %s foi incluída.
 text_issue_updated: Tarefa %s foi alterada.
 text_wiki_destroy_confirmation: Are you sure you want to delete this wiki and all its content ?
+text_issue_category_destroy_question: Some issues (%d) are assigned to this category. What do you want to do ?
+text_issue_category_destroy_assignments: Remove category assignments
+text_issue_category_reassign_to: Reassing issues to this category
 
 default_role_manager: Analista de Negócio ou Gerente de Projeto
 default_role_developper: Desenvolvedor

diff --git a/lang/sv.yml b/lang/sv.yml
index 7244f86ea1d06248d4383351b4098b7842faba29..2196799dde516f32b06fc8dec23c0ea78ebff041 100644 (file)
--- a/lang/sv.yml
+++ b/lang/sv.yml
@@ -479,6 +479,9 @@ text_issues_ref_in_commit_messages: Referencing and fixing issues in commit mess
 text_issue_added: Brist %s har rapporterats.
 text_issue_updated: Brist %s har uppdaterats.
 text_wiki_destroy_confirmation: Are you sure you want to delete this wiki and all its content ?
+text_issue_category_destroy_question: Some issues (%d) are assigned to this category. What do you want to do ?
+text_issue_category_destroy_assignments: Remove category assignments
+text_issue_category_reassign_to: Reassing issues to this category
 
 default_role_manager: Förvaltare
 default_role_developper: Utvecklare

diff --git a/lang/zh.yml b/lang/zh.yml
index bdce4ebbe86f99c342a2d64c4a6bc1c34b5771ba..deb9ee41b65cd1476e9b815c22dda6bbbb824bff 100644 (file)
--- a/lang/zh.yml
+++ b/lang/zh.yml
@@ -481,6 +481,9 @@ text_issues_ref_in_commit_messages: Referencing and fixing issues in commit mess
 text_issue_added: %s ѱ
 text_issue_updated: %s Ѹ
 text_wiki_destroy_confirmation: Are you sure you want to delete this wiki and all its content ?
+text_issue_category_destroy_question: Some issues (%d) are assigned to this category. What do you want to do ?
+text_issue_category_destroy_assignments: Remove category assignments
+text_issue_category_reassign_to: Reassing issues to this category
 
 default_role_manager: 管理员
 default_role_developper: 开发人员

diff --git a/test/unit/issue_category_test.rb b/test/unit/issue_category_test.rb
new file mode 100644 (file)
index 0000000..a6edb3c
--- /dev/null
+++ b/test/unit/issue_category_test.rb
@@ -0,0 +1,41 @@
+# redMine - project management software
+# Copyright (C) 2006-2007  Jean-Philippe Lang
+#
+# This program is free software; you can redistribute it and/or
+# modify it under the terms of the GNU General Public License
+# as published by the Free Software Foundation; either version 2
+# of the License, or (at your option) any later version.
+# 
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+# 
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write to the Free Software
+# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
+
+require File.dirname(__FILE__) + '/../test_helper'
+
+class IssueCategoryTest < Test::Unit::TestCase
+  fixtures :issue_categories, :issues
+
+  def setup
+    @category = IssueCategory.find(1)
+  end
+  
+  def test_destroy
+    issue = @category.issues.first
+    @category.destroy
+    # Make sure the category was nullified on the issue
+    assert_nil issue.reload.category
+  end
+  
+  def test_destroy_with_reassign
+    issue = @category.issues.first
+    reassign_to = IssueCategory.find(2)
+    @category.destroy(reassign_to)
+    # Make sure the issue was reassigned
+    assert_equal reassign_to, issue.reload.category
+  end
+end