fix review 20200312 2

app/controllers/jobs_controller.rb

@@ -5,22 +5,14 @@ class JobsController < ApplicationController
 
 
   def index
   def index
     @job_count = Job.count
     @job_count = Job.count
-
-    if params[:page].to_i > 0
-      pagin = params[:page].to_i
-    else
-      pagin = 1
-    end
+    pagin = params[:page].to_i > 0 ? params[:page].to_i : 1
     @pagin_job = Job.page(pagin).per(Settings.page)
     @pagin_job = Job.page(pagin).per(Settings.page)
   end
   end
 
 
   def search
   def search
-    if params[:search].nil?
-      return redirect_to root_path, alert: "Empty field!"
-    else
-      @search_job = Job.where("job_name LIKE ?","%#{params[:search]}%")
-      @pagin_job = @search_job.page(params[:page]).per(Settings.page)
-    end
-    render :index
+    return redirect_to root_path, alert: "Empty field!" if params[:search].nil?
+
+    @search_job = Job.where("job_name LIKE ?","%#{params[:search]}%")
+    @pagin_job = @search_job.page(params[:page]).per(Settings.page)
   end
   end
 end
 end

app/views/jobs/index.html.erb

@@ -16,7 +16,7 @@
     <label> Total Jobs: <%= Job.count %></label>
     <label> Total Jobs: <%= Job.count %></label>
   </div>
   </div>
 
 
-    <%= render partial: "jobs" %>
+    <%= render partial: "jobs"%>
 
 
 </div>
 </div>