diff --git a/app/controllers/campaigns_controller.rb b/app/controllers/campaigns_controller.rb index 6072fc6..c838612 100644 --- a/app/controllers/campaigns_controller.rb +++ b/app/controllers/campaigns_controller.rb @@ -22,7 +22,6 @@ class CampaignsController < ApplicationController def edit @action = campaign_path(@campaign.clientid) - end @@ -54,7 +53,8 @@ class CampaignsController < ApplicationController private def set_campaign - @campaign = Campaign.find_by_clientid(params[:clientid]) || not_found + clientid = params[:clientid].to_s.gsub(/\D/, '').to_i + @campaign = Campaign.find_by_clientid(clientid) || Campaign.find_by_clientid(params[:campaign][:clientid]) || not_found end def campaign_params @@ -73,7 +73,6 @@ class CampaignsController < ApplicationController raise ActionController::RoutingError.new("Not Found by clientid") end - def set_api_key @BETTER_VIDEO_API_KEY = ENV['BETTER_VIDEO_API_KEY'] @BETTER_VIDEO_API_KEY ||= "propel test key" diff --git a/app/models/campaign.rb b/app/models/campaign.rb index 43a5400..63e367c 100644 --- a/app/models/campaign.rb +++ b/app/models/campaign.rb @@ -6,6 +6,8 @@ class Campaign < ActiveRecord::Base validates :listingcode, presence: true, uniqueness: true validates :advertisername, presence: true validates :websiteurl, presence: true + validates :description, presence: true + validates :productsandservices, presence: true after_create :send_to_better_video def send_to_better_video diff --git a/app/models/videolisting.rb b/app/models/videolisting.rb index 9cdc036..1f47c4d 100644 --- a/app/models/videolisting.rb +++ b/app/models/videolisting.rb @@ -2,6 +2,9 @@ class Videolisting < ActiveRecord::Base belongs_to :campaign validates :videocode, presence: true, uniqueness: true validates :producttypeid, presence: true + validates :voiceoverselection, presence: true + validates :musicselection, presence: true + def asseturls_array self.asseturls.split(',').collect { |url| { asseturl: url.strip } } diff --git a/app/views/campaigns/_videolisting.html.erb b/app/views/campaigns/_videolisting.html.erb new file mode 100644 index 0000000..ee59b16 --- /dev/null +++ b/app/views/campaigns/_videolisting.html.erb @@ -0,0 +1,38 @@ +
+ videocode + <%= videolisting.videocode %> +
++ producttypeid + <%= videolisting.producttypeid %> +
++ voiceoverselection + <%= videolisting.voiceoverselection %> +
++ musicselection + <%= videolisting.musicselection %> +
++ clientprovidedscript + <%= videolisting.clientprovidedscript %> +
++ keywords + <%= videolisting.keywords %> +
++ focus + <%= videolisting.focus %> +
++ notes + <%= videolisting.notes %> +
++ asseturls + <%= videolisting.asseturls %> +
+- Salesrep: - <%= @campaign.salesrep %> -
- -- Salesrepemail: - <%= @campaign.salesrepemail %> -
Businessphone: @@ -76,6 +67,12 @@
+ Description: + <%= @campaign.description %> +
+ + +Contactphone: <%= @campaign.contactphone %>
@@ -87,53 +84,12 @@- videocode - <%= vl.videocode %> -
-- producttypeid - <%= vl.producttypeid %> -
-- remoteassetsarchive - <%= vl.remoteassetsarchive %> -
-- voiceoverselection - <%= vl.voiceoverselection %> -
-- musicselection - <%= vl.musicselection %> -
-- clientprovidedscript - <%= vl.clientprovidedscript %> -
-- keywords - <%= vl.keywords %> -
-- focus - <%= vl.focus %> -
-- notes - <%= vl.notes %> -
-- asseturls - <%= vl.asseturls %> -
-