Commit 4c890014 by Thanh Hung Pham

Fix comment - create scope to find all jobs by city,company

parent 12e304a8
...@@ -49,7 +49,7 @@ class JobsController < ApplicationController ...@@ -49,7 +49,7 @@ class JobsController < ApplicationController
def show def show
if params[:city_id].present? if params[:city_id].present?
@job_count = Job.where(city_id: params[:city_id]).count @job_count = Job.all_jobs_in_city_id(params[:city_id]).count
@jobs = Job.where(city_id: params[:city_id]).page params[:page] @jobs = Job.where(city_id: params[:city_id]).page params[:page]
@search_conditions = "All jobs in #{City.find(params[:city_id]).name}" @search_conditions = "All jobs in #{City.find(params[:city_id]).name}"
end end
...@@ -62,7 +62,7 @@ class JobsController < ApplicationController ...@@ -62,7 +62,7 @@ class JobsController < ApplicationController
end end
if params[:company_id].present? if params[:company_id].present?
@job_count = Job.where(company_id: params[:company_id]).count @job_count = Job.all_jobs_in_company_id(params[:company_id]).count
@jobs = Job.where(company_id: params[:company_id]).page params[:page] @jobs = Job.where(company_id: params[:company_id]).page params[:page]
@search_conditions = "All jobs in #{Company.find(params[:company_id]).name}" @search_conditions = "All jobs in #{Company.find(params[:company_id]).name}"
end end
......
...@@ -9,4 +9,7 @@ class Job < ApplicationRecord ...@@ -9,4 +9,7 @@ class Job < ApplicationRecord
paginates_per 20 paginates_per 20
validates :name, presence: true, length: { maximum: 200 } validates :name, presence: true, length: { maximum: 200 }
scope :all_jobs_in_city_id, ->(city_id) { where('city_id = ?', city_id) }
scope :all_jobs_in_company_id, ->(company_id) { where('company_id = ?', company_id) }
end end
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment