]> source.dussan.org Git - redmine.git/commitdiff
Let the user choose when deleting issues with reported hours (closes #734, #71):
authorJean-Philippe Lang <jp_lang@yahoo.fr>
Fri, 29 Feb 2008 22:54:07 +0000 (22:54 +0000)
committerJean-Philippe Lang <jp_lang@yahoo.fr>
Fri, 29 Feb 2008 22:54:07 +0000 (22:54 +0000)
* to delete the hours
* to assign the hours to the project
* to reassign the hours to another issue

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

29 files changed:
app/controllers/issues_controller.rb
app/models/issue.rb
app/views/issue_categories/destroy.rhtml
app/views/issues/destroy.rhtml [new file with mode: 0644]
lang/bg.yml
lang/cs.yml
lang/de.yml
lang/en.yml
lang/es.yml
lang/fi.yml
lang/fr.yml
lang/he.yml
lang/it.yml
lang/ja.yml
lang/ko.yml
lang/lt.yml
lang/nl.yml
lang/pl.yml
lang/pt-br.yml
lang/pt.yml
lang/ro.yml
lang/ru.yml
lang/sr.yml
lang/sv.yml
lang/uk.yml
lang/zh-tw.yml
lang/zh.yml
test/functional/issues_controller_test.rb
test/unit/issue_test.rb

index e03afb252d2abd064af794c044a0a7f1b829bb86..53627db928b08c7784faf4ea757606332858e5d6 100644 (file)
@@ -263,6 +263,26 @@ class IssuesController < ApplicationController
   end
   
   def destroy
+    @hours = TimeEntry.sum(:hours, :conditions => ['issue_id IN (?)', @issues]).to_f
+    if @hours > 0
+      case params[:todo]
+      when 'destroy'
+        # nothing to do
+      when 'nullify'
+        TimeEntry.update_all('issue_id = NULL', ['issue_id IN (?)', @issues])
+      when 'reassign'
+        reassign_to = @project.issues.find_by_id(params[:reassign_to_id])
+        if reassign_to.nil?
+          flash.now[:error] = l(:error_issue_not_found_in_project)
+          return
+        else
+          TimeEntry.update_all("issue_id = #{reassign_to.id}", ['issue_id IN (?)', @issues])
+        end
+      else
+        # display the destroy form
+        return
+      end
+    end
     @issues.each(&:destroy)
     redirect_to :action => 'index', :project_id => @project
   end
index 9d7f6eaaf89cfd42a890e4ae0dfa014c50ae66e2..1d25a46043140db2284fe3784ea037f2be364f57 100644 (file)
@@ -27,7 +27,7 @@ class Issue < ActiveRecord::Base
 
   has_many :journals, :as => :journalized, :dependent => :destroy
   has_many :attachments, :as => :container, :dependent => :destroy
-  has_many :time_entries, :dependent => :nullify
+  has_many :time_entries, :dependent => :delete_all
   has_many :custom_values, :dependent => :delete_all, :as => :customized
   has_many :custom_fields, :through => :custom_values
   has_and_belongs_to_many :changesets, :order => "revision ASC"
index a563736e22a8bed8729022e0a158f5b78ee892f0..2b61810e75eb1e98f23bd8b45cd6eb24a44cb909 100644 (file)
@@ -3,9 +3,9 @@
 <% 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 />
+<p><label><%= radio_button_tag 'todo', 'nullify', true %> <%= l(:text_issue_category_destroy_assignments) %></label><br />
 <% if @categories.size > 0 %>
-<%= radio_button_tag 'todo', 'reassign', false %> <%= l(:text_issue_category_reassign_to) %>:
+<label><%= radio_button_tag 'todo', 'reassign', false %> <%= l(:text_issue_category_reassign_to) %></label>:
 <%= select_tag 'reassign_to_id', options_from_collection_for_select(@categories, 'id', 'name') %></p>
 <% end %>
 </div>
diff --git a/app/views/issues/destroy.rhtml b/app/views/issues/destroy.rhtml
new file mode 100644 (file)
index 0000000..2f3c036
--- /dev/null
@@ -0,0 +1,15 @@
+<h2><%= l(:confirmation) %></h2>
+
+<% form_tag do %>
+<%= @issues.collect {|i| hidden_field_tag 'ids[]', i.id } %>
+<div class="box">
+<p><strong><%= l(:text_destroy_time_entries_question, @hours) %></strong></p>
+<p>
+<label><%= radio_button_tag 'todo', 'destroy', true %> <%= l(:text_destroy_time_entries) %></label><br />
+<label><%= radio_button_tag 'todo', 'nullify', false %> <%= l(:text_assign_time_entries_to_project) %></label><br />
+<label><%= radio_button_tag 'todo', 'reassign', false, :onchange => 'if (this.checked) { $("reassign_to_id").focus(); }' %> <%= l(:text_reassign_time_entries) %></label>
+<%= text_field_tag 'reassign_to_id', params[:reassign_to_id], :size => 6, :onfocus => '$("todo_reassign").checked=true;' %>
+</p>
+</div>
+<%= submit_tag l(:button_apply) %>
+<% end %>
index 64fff5ae9b35947dffcec4474058fcdcf93891d5..12d5dfa5f75efa4a4fd8a81227e4ae80abcbfc6c 100644 (file)
@@ -602,3 +602,8 @@ label_yesterday: yesterday
 label_last_month: last month
 label_add_another_file: Add another file
 label_optional_description: Optional description
+text_destroy_time_entries_question: %.02f hours were reported on the issues you are about to delete. What do you want to do ?
+error_issue_not_found_in_project: 'The issue was not found or does not belong to this project'
+text_assign_time_entries_to_project: Assign reported hours to the project
+text_destroy_time_entries: Delete reported hours
+text_reassign_time_entries: 'Reassign reported hours to this issue:'
index 6de5ee01b124545d2491f925c2e738297846baab..56b77d87b90d8a94e10b144bfc7f326c4ed30b61 100644 (file)
@@ -602,3 +602,8 @@ label_yesterday: yesterday
 label_last_month: last month
 label_add_another_file: Add another file
 label_optional_description: Optional description
+text_destroy_time_entries_question: %.02f hours were reported on the issues you are about to delete. What do you want to do ?
+error_issue_not_found_in_project: 'The issue was not found or does not belong to this project'
+text_assign_time_entries_to_project: Assign reported hours to the project
+text_destroy_time_entries: Delete reported hours
+text_reassign_time_entries: 'Reassign reported hours to this issue:'
index f2b1c2023d3c138e7a51be936941b89f22e5290b..9397cf8922ef52433f4b7d925dcf288a1603a6fb 100644 (file)
@@ -602,3 +602,8 @@ label_yesterday: yesterday
 label_last_month: last month
 label_add_another_file: Add another file
 label_optional_description: Optional description
+text_destroy_time_entries_question: %.02f hours were reported on the issues you are about to delete. What do you want to do ?
+error_issue_not_found_in_project: 'The issue was not found or does not belong to this project'
+text_assign_time_entries_to_project: Assign reported hours to the project
+text_destroy_time_entries: Delete reported hours
+text_reassign_time_entries: 'Reassign reported hours to this issue:'
index b06977035290f31d0e5997638accb25e63b25d8e..7ed2c87754a7d18a02876b7fecb1444a70daa9b7 100644 (file)
@@ -80,6 +80,7 @@ notice_default_data_loaded: Default configuration successfully loaded.
 error_can_t_load_default_data: "Default configuration could not be loaded: %s"
 error_scm_not_found: "Entry and/or revision doesn't exist in the repository."
 error_scm_command_failed: "An error occurred when trying to access the repository: %s"
+error_issue_not_found_in_project: 'The issue was not found or does not belong to this project'
 
 mail_subject_lost_password: Your Redmine password
 mail_body_lost_password: 'To change your Redmine password, click on the following link:'
@@ -577,6 +578,10 @@ text_select_project_modules: 'Select modules to enable for this project:'
 text_default_administrator_account_changed: Default administrator account changed
 text_file_repository_writable: File repository writable
 text_rmagick_available: RMagick available (optional)
+text_destroy_time_entries_question: %.02f hours were reported on the issues you are about to delete. What do you want to do ?
+text_destroy_time_entries: Delete reported hours
+text_assign_time_entries_to_project: Assign reported hours to the project
+text_reassign_time_entries: 'Reassign reported hours to this issue:'
 
 default_role_manager: Manager
 default_role_developper: Developer
index 9c72af18d6b67dda3338e3d3a659f158485224b0..9032d3599e42b1e2437d1b1c5b78dba3911af7cc 100644 (file)
@@ -605,3 +605,8 @@ label_yesterday: yesterday
 label_last_month: last month
 label_add_another_file: Add another file
 label_optional_description: Optional description
+text_destroy_time_entries_question: %.02f hours were reported on the issues you are about to delete. What do you want to do ?
+error_issue_not_found_in_project: 'The issue was not found or does not belong to this project'
+text_assign_time_entries_to_project: Assign reported hours to the project
+text_destroy_time_entries: Delete reported hours
+text_reassign_time_entries: 'Reassign reported hours to this issue:'
index 350e5ba44118576c36a37ad74fc55109e48cdf58..dbcc3763c72aa0ce121cf1e8ce3e07100bd014c9 100644 (file)
@@ -606,3 +606,8 @@ label_yesterday: yesterday
 label_last_month: last month
 label_add_another_file: Add another file
 label_optional_description: Optional description
+text_destroy_time_entries_question: %.02f hours were reported on the issues you are about to delete. What do you want to do ?
+error_issue_not_found_in_project: 'The issue was not found or does not belong to this project'
+text_assign_time_entries_to_project: Assign reported hours to the project
+text_destroy_time_entries: Delete reported hours
+text_reassign_time_entries: 'Reassign reported hours to this issue:'
index c2baa3b7ece17180e5bff0fbb07269e9dc884dea..accd5c1bfc2dd4b4315fabe3d31ead154bf5eb70 100644 (file)
@@ -80,6 +80,7 @@ notice_default_data_loaded: Paramétrage par défaut chargé avec succès.
 error_can_t_load_default_data: "Une erreur s'est produite lors du chargement du paramétrage: %s"
 error_scm_not_found: "L'entrée et/ou la révision demandée n'existe pas dans le dépôt."
 error_scm_command_failed: "Une erreur s'est produite lors de l'accès au dépôt: %s"
+error_issue_not_found_in_project: "La demande n'existe pas ou n'appartient pas à ce projet"
 
 mail_subject_lost_password: Votre mot de passe redMine
 mail_body_lost_password: 'Pour changer votre mot de passe Redmine, cliquez sur le lien suivant:'
@@ -565,7 +566,7 @@ text_issues_ref_in_commit_messages: Référencement et résolution des demandes
 text_issue_added: La demande %s a été soumise par %s.
 text_issue_updated: La demande %s a été mise à jour par %s.
 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_question: %d demandes 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
 text_user_mail_option: "Pour les projets non sélectionnés, vous recevrez seulement des notifications pour ce que vous surveillez ou à quoi vous participez (exemple: demandes dont vous êtes l'auteur ou la personne assignée)."
@@ -577,6 +578,10 @@ text_select_project_modules: 'Selectionner les modules à activer pour ce projec
 text_default_administrator_account_changed: Compte administrateur par défaut changé
 text_file_repository_writable: Répertoire de stockage des fichiers accessible en écriture
 text_rmagick_available: Bibliothèque RMagick présente (optionnelle)
+text_destroy_time_entries_question: %.02f heures ont été enregistrées sur les demandes à supprimer. Que voulez-vous faire ?
+text_destroy_time_entries: Supprimer les heures
+text_assign_time_entries_to_project: Reporter les heures sur le projet
+text_reassign_time_entries: 'Reporter les heures sur cette demande:'
 
 default_role_manager: Manager
 default_role_developper: Développeur
index 680364714696fbf50b2c687407429eecd7cb29f4..a43f1c02235b61f1005046e00aa2331ecd7bda4b 100644 (file)
@@ -602,3 +602,8 @@ label_yesterday: yesterday
 label_last_month: last month
 label_add_another_file: Add another file
 label_optional_description: Optional description
+text_destroy_time_entries_question: %.02f hours were reported on the issues you are about to delete. What do you want to do ?
+error_issue_not_found_in_project: 'The issue was not found or does not belong to this project'
+text_assign_time_entries_to_project: Assign reported hours to the project
+text_destroy_time_entries: Delete reported hours
+text_reassign_time_entries: 'Reassign reported hours to this issue:'
index b2eb1c9c6719b1cf21dca9c25d5a75cd145ae7fd..80ffc5f6c66e68e3763b8967f27dc63ef6bbbca5 100644 (file)
@@ -602,3 +602,8 @@ label_yesterday: yesterday
 label_last_month: last month
 label_add_another_file: Add another file
 label_optional_description: Optional description
+text_destroy_time_entries_question: %.02f hours were reported on the issues you are about to delete. What do you want to do ?
+error_issue_not_found_in_project: 'The issue was not found or does not belong to this project'
+text_assign_time_entries_to_project: Assign reported hours to the project
+text_destroy_time_entries: Delete reported hours
+text_reassign_time_entries: 'Reassign reported hours to this issue:'
index 2a742072d4e0ecf13907b4af161f9bf933dc80bf..d652f4b196c763e375594e1795cdc009d4cd6bdd 100644 (file)
@@ -603,3 +603,8 @@ label_yesterday: yesterday
 label_last_month: last month
 label_add_another_file: Add another file
 label_optional_description: Optional description
+text_destroy_time_entries_question: %.02f hours were reported on the issues you are about to delete. What do you want to do ?
+error_issue_not_found_in_project: 'The issue was not found or does not belong to this project'
+text_assign_time_entries_to_project: Assign reported hours to the project
+text_destroy_time_entries: Delete reported hours
+text_reassign_time_entries: 'Reassign reported hours to this issue:'
index cce1480d857e8ce8cd5631a1a802dc80507f7b16..12fa4b8fb165a9dd222ed72fc6770ca03be61bba 100644 (file)
@@ -602,3 +602,8 @@ label_yesterday: yesterday
 label_last_month: last month
 label_add_another_file: Add another file
 label_optional_description: Optional description
+text_destroy_time_entries_question: %.02f hours were reported on the issues you are about to delete. What do you want to do ?
+error_issue_not_found_in_project: 'The issue was not found or does not belong to this project'
+text_assign_time_entries_to_project: Assign reported hours to the project
+text_destroy_time_entries: Delete reported hours
+text_reassign_time_entries: 'Reassign reported hours to this issue:'
index 7701280a0b3421f7383d27743cbfe164b8856d83..5d26c7ea9cb31b22b0c9d1c618dee9315823fd18 100644 (file)
@@ -603,3 +603,8 @@ label_yesterday: yesterday
 label_last_month: last month
 label_add_another_file: Add another file
 label_optional_description: Optional description
+text_destroy_time_entries_question: %.02f hours were reported on the issues you are about to delete. What do you want to do ?
+error_issue_not_found_in_project: 'The issue was not found or does not belong to this project'
+text_assign_time_entries_to_project: Assign reported hours to the project
+text_destroy_time_entries: Delete reported hours
+text_reassign_time_entries: 'Reassign reported hours to this issue:'
index a0a5305e49b13955c62604358b0e1570687cd1a5..12563f2e4da32e1a4ffcd05b1001631b8e289ea3 100644 (file)
@@ -603,3 +603,8 @@ label_yesterday: yesterday
 label_last_month: last month
 label_add_another_file: Add another file
 label_optional_description: Optional description
+text_destroy_time_entries_question: %.02f hours were reported on the issues you are about to delete. What do you want to do ?
+error_issue_not_found_in_project: 'The issue was not found or does not belong to this project'
+text_assign_time_entries_to_project: Assign reported hours to the project
+text_destroy_time_entries: Delete reported hours
+text_reassign_time_entries: 'Reassign reported hours to this issue:'
index 6de13aea4325931c60b7bb6204429777c8cb3938..575c8262aae673646bf33bb8278633afba9d3881 100644 (file)
@@ -602,3 +602,8 @@ label_yesterday: yesterday
 label_last_month: last month
 label_add_another_file: Add another file
 label_optional_description: Optional description
+text_destroy_time_entries_question: %.02f hours were reported on the issues you are about to delete. What do you want to do ?
+error_issue_not_found_in_project: 'The issue was not found or does not belong to this project'
+text_assign_time_entries_to_project: Assign reported hours to the project
+text_destroy_time_entries: Delete reported hours
+text_reassign_time_entries: 'Reassign reported hours to this issue:'
index 83b640a4317f21c24728a3ffcd8b07c9fd3f0a45..d128617dad2f60109471480881ab49d596c4c0b4 100644 (file)
@@ -602,3 +602,8 @@ label_yesterday: yesterday
 label_last_month: last month\r
 label_add_another_file: Add another file\r
 label_optional_description: Optional description\r
+text_destroy_time_entries_question: %.02f hours were reported on the issues you are about to delete. What do you want to do ?\r
+error_issue_not_found_in_project: 'The issue was not found or does not belong to this project'\r
+text_assign_time_entries_to_project: Assign reported hours to the project\r
+text_destroy_time_entries: Delete reported hours\r
+text_reassign_time_entries: 'Reassign reported hours to this issue:'\r
index e100d153e287d796f83e200de8c243318630b818..33b76892b8a0d65660d109e1fc0879489b42ad80 100644 (file)
@@ -602,3 +602,8 @@ label_yesterday: yesterday
 label_last_month: last month
 label_add_another_file: Add another file
 label_optional_description: Optional description
+text_destroy_time_entries_question: %.02f hours were reported on the issues you are about to delete. What do you want to do ?
+error_issue_not_found_in_project: 'The issue was not found or does not belong to this project'
+text_assign_time_entries_to_project: Assign reported hours to the project
+text_destroy_time_entries: Delete reported hours
+text_reassign_time_entries: 'Reassign reported hours to this issue:'
index 142a2a84f6d5e6ebd68453ca639c01e69a89e9f4..b8b737e429c9df7796d78b91799f7dc4f94b1506 100644 (file)
@@ -602,3 +602,8 @@ label_yesterday: yesterday
 label_last_month: last month
 label_add_another_file: Add another file
 label_optional_description: Optional description
+text_destroy_time_entries_question: %.02f hours were reported on the issues you are about to delete. What do you want to do ?
+error_issue_not_found_in_project: 'The issue was not found or does not belong to this project'
+text_assign_time_entries_to_project: Assign reported hours to the project
+text_destroy_time_entries: Delete reported hours
+text_reassign_time_entries: 'Reassign reported hours to this issue:'
index 83e63115fca52591a2bc7aa842f6db3c74481a39..93fbd3b747537e844dd2672bcb51d12bdf03420c 100644 (file)
@@ -606,3 +606,8 @@ label_yesterday: yesterday
 label_last_month: last month
 label_add_another_file: Add another file
 label_optional_description: Optional description
+text_destroy_time_entries_question: %.02f hours were reported on the issues you are about to delete. What do you want to do ?
+error_issue_not_found_in_project: 'The issue was not found or does not belong to this project'
+text_assign_time_entries_to_project: Assign reported hours to the project
+text_destroy_time_entries: Delete reported hours
+text_reassign_time_entries: 'Reassign reported hours to this issue:'
index 2cd92607fbe0edb492f7bbe0a78c595b973843bd..16e870b7670f65252ae4af2a047318fb909ca49e 100644 (file)
@@ -603,3 +603,8 @@ label_yesterday: yesterday
 label_last_month: last month
 label_add_another_file: Add another file
 label_optional_description: Optional description
+text_destroy_time_entries_question: %.02f hours were reported on the issues you are about to delete. What do you want to do ?
+error_issue_not_found_in_project: 'The issue was not found or does not belong to this project'
+text_assign_time_entries_to_project: Assign reported hours to the project
+text_destroy_time_entries: Delete reported hours
+text_reassign_time_entries: 'Reassign reported hours to this issue:'
index a9b30ae2f5c669cce745243d19472d58687ac96a..3375a6850007ca0d6954ee49499ce8d825b2d9b9 100644 (file)
@@ -603,3 +603,8 @@ label_yesterday: yesterday
 label_last_month: last month
 label_add_another_file: Add another file
 label_optional_description: Optional description
+text_destroy_time_entries_question: %.02f hours were reported on the issues you are about to delete. What do you want to do ?
+error_issue_not_found_in_project: 'The issue was not found or does not belong to this project'
+text_assign_time_entries_to_project: Assign reported hours to the project
+text_destroy_time_entries: Delete reported hours
+text_reassign_time_entries: 'Reassign reported hours to this issue:'
index 0be41cf04b8f5191433dac532213bb9199e85d3b..2fd3589f52cbea927f4b0f5a3fb0134296bd85b4 100644 (file)
@@ -604,3 +604,8 @@ label_yesterday: yesterday
 label_last_month: last month
 label_add_another_file: Add another file
 label_optional_description: Optional description
+text_destroy_time_entries_question: %.02f hours were reported on the issues you are about to delete. What do you want to do ?
+error_issue_not_found_in_project: 'The issue was not found or does not belong to this project'
+text_assign_time_entries_to_project: Assign reported hours to the project
+text_destroy_time_entries: Delete reported hours
+text_reassign_time_entries: 'Reassign reported hours to this issue:'
index c63e6c3bf3221e260d4e639752d3d4d77a30e679..070aa290cbdfa168c4a3b7bb2103b6d7dae03526 100644 (file)
@@ -602,3 +602,8 @@ label_yesterday: yesterday
 label_last_month: last month
 label_add_another_file: Add another file
 label_optional_description: Optional description
+text_destroy_time_entries_question: %.02f hours were reported on the issues you are about to delete. What do you want to do ?
+error_issue_not_found_in_project: 'The issue was not found or does not belong to this project'
+text_assign_time_entries_to_project: Assign reported hours to the project
+text_destroy_time_entries: Delete reported hours
+text_reassign_time_entries: 'Reassign reported hours to this issue:'
index a50d8be488ac0a17cd98bcfc0f7cfb7828c0cc83..6b2e64abd8fde3ed1556b0c5b6d303bd4820462f 100644 (file)
@@ -605,3 +605,8 @@ label_yesterday: yesterday
 label_last_month: last month
 label_add_another_file: Add another file
 label_optional_description: Optional description
+text_destroy_time_entries_question: %.02f hours were reported on the issues you are about to delete. What do you want to do ?
+error_issue_not_found_in_project: 'The issue was not found or does not belong to this project'
+text_assign_time_entries_to_project: Assign reported hours to the project
+text_destroy_time_entries: Delete reported hours
+text_reassign_time_entries: 'Reassign reported hours to this issue:'
index 34b030d989359bcf850f09bcc47da89554d24152..da8118668330505197b7c99ee08b9a11faff8ec5 100644 (file)
@@ -37,7 +37,8 @@ class IssuesControllerTest < Test::Unit::TestCase
            :workflows,
            :custom_fields,
            :custom_values,
-           :custom_fields_trackers
+           :custom_fields_trackers,
+           :time_entries
   
   def setup
     @controller = IssuesController.new
@@ -428,13 +429,48 @@ class IssuesControllerTest < Test::Unit::TestCase
                                              :class => 'icon-del disabled' }
   end
   
-  def test_destroy
+  def test_destroy_issue_with_no_time_entries
     @request.session[:user_id] = 2
-    post :destroy, :id => 1
+    post :destroy, :id => 3
     assert_redirected_to 'projects/ecookbook/issues'
-    assert_nil Issue.find_by_id(1)
+    assert_nil Issue.find_by_id(3)
   end
 
+  def test_destroy_issues_with_time_entries
+    @request.session[:user_id] = 2
+    post :destroy, :ids => [1, 3]
+    assert_response :success
+    assert_template 'destroy'
+    assert_not_nil assigns(:hours)
+    assert Issue.find_by_id(1) && Issue.find_by_id(3)
+  end
+
+  def test_destroy_issues_and_destroy_time_entries
+    @request.session[:user_id] = 2
+    post :destroy, :ids => [1, 3], :todo => 'destroy'
+    assert_redirected_to 'projects/ecookbook/issues'
+    assert !(Issue.find_by_id(1) || Issue.find_by_id(3))
+    assert_nil TimeEntry.find_by_id([1, 2])
+  end
+
+  def test_destroy_issues_and_assign_time_entries_to_project
+    @request.session[:user_id] = 2
+    post :destroy, :ids => [1, 3], :todo => 'nullify'
+    assert_redirected_to 'projects/ecookbook/issues'
+    assert !(Issue.find_by_id(1) || Issue.find_by_id(3))
+    assert_nil TimeEntry.find(1).issue_id
+    assert_nil TimeEntry.find(2).issue_id
+  end
+  
+  def test_destroy_issues_and_reassign_time_entries_to_another_issue
+    @request.session[:user_id] = 2
+    post :destroy, :ids => [1, 3], :todo => 'reassign', :reassign_to_id => 2
+    assert_redirected_to 'projects/ecookbook/issues'
+    assert !(Issue.find_by_id(1) || Issue.find_by_id(3))
+    assert_equal 2, TimeEntry.find(1).issue_id
+    assert_equal 2, TimeEntry.find(2).issue_id
+  end
+  
   def test_destroy_attachment
     issue = Issue.find(3)
     a = issue.attachments.size
index da91dd02c41b0da07e1af1d68632d9d36aabe53c..7712b764e37df3cfae7a63288d9389db4ec0bc8a 100644 (file)
@@ -70,4 +70,10 @@ class IssueTest < Test::Unit::TestCase
     # Make sure time entries were move to the target project
     assert_equal 2, issue.time_entries.first.project_id
   end
+  
+  def test_issue_destroy
+    Issue.find(1).destroy
+    assert_nil Issue.find_by_id(1)
+    assert_nil TimeEntry.find_by_issue_id(1)
+  end
 end