little refactoring to move all action about ProblemDestroy in ProblemDestroy class fix #208

Cyril Mougel
1 parent 76d3d2f8
Showing 2 changed files with 16 additions and 4 deletions Show diff stats
app/controllers/errs_controller.rb
app/interactors/problem_destroy.rb
@@ -114,10 +114,8 @@ class ErrsController &lt; ApplicationController
   end
   def destroy_several
-    @selected_problems.each{|problem|
-      ProblemDestroy.new(problem).execute
-    }
-    flash[:notice] = "#{pluralize(@selected_problems.count, 'err has', 'errs have')} been deleted."
+    nb_problem_destroy = ProblemDestroy.execute(@selected_problems)
+    flash[:notice] = "#{pluralize(nb_problem_destroy, 'err has', 'errs have')} been deleted."
     redirect_to :back
   end
@@ -12,6 +12,20 @@ class ProblemDestroy
     problem.delete
   end
+  ##
+  # Destroy all problem pass in args
+  #
+  # @params [ Array[Problem] ] problems the list of problem need to be delete
+  #   can be a single Problem
+  # @return [ Integer ]
+  #   the number of problem destroy
+  #
+  def self.execute(problems)
+    Array(problems).each{ |problem|
+      ProblemDestroy.new(problem).execute
+    }.count
+  end
+
   private
   def errs_id
	@@ -12,6 +12,20 @@ class ProblemDestroy		@@ -12,6 +12,20 @@ class ProblemDestroy
12	problem.delete	12	problem.delete
13	end	13	end
14		14
		15	+ ##
		16	+ # Destroy all problem pass in args
		17	+ #
		18	+ # @params [ Array[Problem] ] problems the list of problem need to be delete
		19	+ # can be a single Problem
		20	+ # @return [ Integer ]
		21	+ # the number of problem destroy
		22	+ #
		23	+ def self.execute(problems)
		24	+ Array(problems).each{ \|problem\|
		25	+ ProblemDestroy.new(problem).execute
		26	+ }.count
		27	+ end
		28	+
15	private	29	private
16		30
17	def errs_id	31	def errs_id