##// END OF EJS Templates
Rails4: replace deprecated find_all_by_* at WorkflowsController#copy...
Toshi MARUYAMA -
r12308:ee49bdd81bc8
parent child
Show More
@@ -88,7 +88,6 class WorkflowsController < ApplicationController
88 end
88 end
89
89
90 def copy
90 def copy
91
92 if params[:source_tracker_id].blank? || params[:source_tracker_id] == 'any'
91 if params[:source_tracker_id].blank? || params[:source_tracker_id] == 'any'
93 @source_tracker = nil
92 @source_tracker = nil
94 else
93 else
@@ -99,10 +98,10 class WorkflowsController < ApplicationController
99 else
98 else
100 @source_role = Role.find_by_id(params[:source_role_id].to_i)
99 @source_role = Role.find_by_id(params[:source_role_id].to_i)
101 end
100 end
102
101 @target_trackers = params[:target_tracker_ids].blank? ?
103 @target_trackers = params[:target_tracker_ids].blank? ? nil : Tracker.find_all_by_id(params[:target_tracker_ids])
102 nil : Tracker.where(:id => params[:target_tracker_ids]).all
104 @target_roles = params[:target_role_ids].blank? ? nil : Role.find_all_by_id(params[:target_role_ids])
103 @target_roles = params[:target_role_ids].blank? ?
105
104 nil : Role.where(:id => params[:target_role_ids])
106 if request.post?
105 if request.post?
107 if params[:source_tracker_id].blank? || params[:source_role_id].blank? || (@source_tracker.nil? && @source_role.nil?)
106 if params[:source_tracker_id].blank? || params[:source_role_id].blank? || (@source_tracker.nil? && @source_role.nil?)
108 flash.now[:error] = l(:error_workflow_copy_source)
107 flash.now[:error] = l(:error_workflow_copy_source)
General Comments 0
You need to be logged in to leave comments. Login now