abstract finding of object

app/controllers/application_controller.rb

@@ -5,6 +5,18 @@ class ApplicationController < ActionController::Base
   before_action :authenticate_user_from_token!, except: [:publish]
   before_action :authenticate_user!, except: [:publish] #unless Rails.env.test?
 
+
+  def set_by_options(name, field)
+    what = params[field]
+    model = name.to_s.classify.constantize
+    if params[name]
+      object =  model.find_by(field => params[name][field]) || not_found
+    else
+      object = model.find_by(field=>what) || not_found
+    end
+  end
+
+
   private
 
 

app/controllers/campaigns_controller.rb

@@ -58,12 +58,7 @@ class CampaignsController < ApplicationController
 
   private
     def set_campaign
-      listingcode = params[:listingcode]
+      @campaign = set_by_options(:campaign, :listingcode)
-      if params[:campaign]
-        @campaign = Campaign.find_by_listingcode(params[:campaign][:listingcode]) || not_found
-      else
-        @campaign = Campaign.find_by_listingcode(listingcode) || not_found
-      end
     end
 
     def campaign_params

app/controllers/videolistings_controller.rb

@@ -17,12 +17,7 @@ class VideolistingsController < ApplicationController
   private
     # Use callbacks to share common setup or constraints between actions.
     def set_videolisting
-      videocode = params[:videocode]
+      @videolisting = set_by_options(:videolisting, :videocode)
-      if params[:videolisting]
-        @videolisting =  Videolisting.find_by_videocode(params[:videolisting][:videocode]) || not_found
-      else
-        @videolisting = Videolisting.find_by_videocode(videocode) || not_found
-      end
     end
 
     # Only allow a trusted parameter "white list" through.