using gem draper

app/controllers/jobs_controller.rb

@@ -2,4 +2,8 @@ class JobsController < ApplicationController
   def index
   def index
     @jobs = Job.page(params[:page]).per(Settings.job.per_page)
     @jobs = Job.page(params[:page]).per(Settings.job.per_page)
   end
   end
+
+  def show
+    @job = Job.find(params[:id])
+  end
 end
 end

app/controllers/tops_controller.rb

 class TopsController < ApplicationController
 class TopsController < ApplicationController
   def index
   def index
-    @vn_cities = City.city_order.take(Settings.top.city.limit)
-    @industries = Industry.industry_order.take(Settings.top.city.limit)
+    @vn_cities = City.city_order
+    @industries = Industry.industry_order
     @jobs = Job.latest_job
     @jobs = Job.latest_job
   end
   end
 end
 end

app/decorators/job_decorator.rb

@@ -3,10 +3,10 @@ class JobDecorator < Draper::Decorator
   decorates_association :city
   decorates_association :city
 
 
   def city_name
   def city_name
-    object&.cities&.first&.name
+    object.cities&.first&.name
   end
   end
 
 
   def display_short_des
   def display_short_des
-    object.short_des.truncate(250)
+    object.short_des&.truncate(250)
   end
   end
 end
 end

app/models/city.rb

@@ -25,6 +25,6 @@ class City < ApplicationRecord
   end
   end
 
 
   def self.city_order
   def self.city_order
-    @city_order ||= all.sort_by(&:job_count).reverse
+    @city_order ||= all.sort_by(&:job_count).reverse.take(Settings.top.city.limit)
   end
   end
 end
 end

app/models/industry.rb

@@ -21,6 +21,6 @@ class Industry < ApplicationRecord
   end
   end
 
 
   def self.industry_order
   def self.industry_order
-    @industry_order ||= all.sort_by(&:job_count).reverse
+    @industry_order ||= all.sort_by(&:job_count).reverse.take(Settings.top.city.limit)
   end
   end
 end
 end

config/routes.rb

@@ -2,7 +2,7 @@ Rails.application.routes.draw do
   resources :companies, only: :index
   resources :companies, only: :index
   resources :industries, only: :index
   resources :industries, only: :index
   resources :cities, only: :index
   resources :cities, only: :index
-  resources :jobs, only: :index
+  resources :jobs, only: [:index, :show]
   devise_for :users
   devise_for :users
   root "tops#index"
   root "tops#index"
   namespace :users do
   namespace :users do