Fix errors in JobsController - warvox - VoIP based wardialing tool, forked from rapid7/warvox.
(DIR) Log
(DIR) Files
(DIR) Refs
(DIR) README
---
(DIR) commit a2c220956e092cd5bad321f827e7d3663276c312
(DIR) parent d7e775889c384144700de870bc8479a7e3b6233c
(HTM) Author: Chee Yeo <ckyeo.1@gmail.com>
Date: Fri, 26 Aug 2016 12:54:53 +0100
Fix errors in JobsController
Replace old AR count which don't take conditions anymore
Add logic to prevent nil errors in purge_calls method.
Diffstat:
M app/controllers/jobs_controller.rb | 17 ++++++++++-------
1 file changed, 10 insertions(+), 7 deletions(-)
---
(DIR) diff --git a/app/controllers/jobs_controller.rb b/app/controllers/jobs_controller.rb
@@ -40,16 +40,16 @@ class JobsController < ApplicationController
@job = Job.find(params[:id])
@call_results = {
- :Timeout => @job.calls.count(:conditions => { :answered => false }),
- :Busy => @job.calls.count(:conditions => { :busy => true }),
- :Answered => @job.calls.count(:conditions => { :answered => true }),
+ :Timeout => @job.calls.where(:answered => false ).count,
+ :Busy => @job.calls.where(:busy => true).count,
+ :Answered => @job.calls.where(:answered => true).count,
}
sort_by = params[:sort_by] || 'number'
sort_dir = params[:sort_dir] || 'asc'
@results = []
- @results_total_count = @job.calls.count()
+ @results_total_count = @job.calls.count
if request.format.json?
if params[:iDisplayLength] == '-1'
@@ -144,9 +144,12 @@ class JobsController < ApplicationController
end
def purge_calls
- Call.delete_all(:id => params[:result_ids])
- CallMedium.delete_all(:call_id => params[:result_ids])
- flash[:notice] = "Purged #{params[:result_ids].length} calls"
+ unless params[:result_ids].blank?
+ Call.delete_all(:id => params[:result_ids])
+ CallMedium.delete_all(:call_id => params[:result_ids])
+ flash[:notice] = "Purged #{params[:result_ids].length} calls"
+ end
+
if params[:id]
@job = Job.find(params[:id])
redirect_to view_results_path(@job.project_id, @job.id)