Removing of sites, spots and campaigns

.gitignore

@@ -3,4 +3,5 @@
 config/database.yml
 db/test.sqlite
 schema/*.sql
-public/advertisements
+public/advertisements
+coverage

TODO.markdown

@@ -3,7 +3,6 @@
 * Track clicks to ads
 * Build out dashboard page
 * Support default campaigns
-* Remove campaigns / spots / sites
 * List out inactive / default campaigns
 
 ## Version 1.0
@@ -13,4 +12,5 @@
 
 ## Version 1.1
 
+* Estimate impression per campaign / spot / site based on historical data
 * Add in calendar-based saturation views

app/controllers/campaigns.rb

@@ -34,4 +34,12 @@ def edit
     render
   end
 
+  def destroy
+    if request.method == :post
+      @campaign = Campaign.find(params[:id])
+      @campaign.destroy
+    end
+    redirect url(:action => "show", :site_id => @campaign.spot.site.id)
+  end
+
 end

app/controllers/sites.rb

@@ -22,4 +22,20 @@ def edit
     render
   end
 
+  def destroy
+    @site = Site.find(params[:id])
+    if request.method == :post
+      @site.destroy
+    end
+    redirect url(:action => "index")
+  end
+
+  def destroy_spot
+    @spot = Spot.find(params[:id])
+    if request.method == :post
+      @spot.destroy
+    end
+    redirect url(:action => "edit", :id => @spot.site.id)
+  end
+
 end

app/helpers/global_helpers.rb

@@ -24,5 +24,9 @@ def format_for_javascript(body)
       body.gsub(/\'/, "'").gsub(/\n/, "")
     end
 
+    def destroy_button(url, text)
+      "<form action=\"#{url}\" method=\"post\" class=\"destroy\"><button type=\"submit\" class=\"destroy\">#{text}</button></form>"
+    end
+
   end
 end

app/models/campaign.rb

@@ -1,7 +1,7 @@
 class Campaign < ActiveRecord::Base
 
   belongs_to :spot
-  has_many :impressions
+  has_many :impressions, :dependent => :destroy
 
   validates_presence_of :name, :starts_on, :ends_on, :spot_id
 

app/models/site.rb

@@ -1,6 +1,6 @@
 class Site < ActiveRecord::Base
 
-  has_many :spots
+  has_many :spots, :dependent => :destroy
 
   validates_presence_of :name, :url
   validates_uniqueness_of :url

app/models/spot.rb

@@ -1,7 +1,7 @@
 class Spot < ActiveRecord::Base
 
   belongs_to :site
-  has_many :campaigns
+  has_many :campaigns, :dependent => :destroy
   has_many :active_campaigns, :class_name => "Campaign", :conditions => ["starts_on < ? AND ends_on > ? AND active = ?", Time.now.to_formatted_s(:db), Time.now.to_formatted_s(:db), true]
 
   validates_presence_of :name

app/views/campaigns/edit.html.erb

@@ -5,6 +5,9 @@
     <h2>Edit details</h2>
     <%= error_messages_for @campaign %>
     <%= partial :form %>
-    <div class="actions"><%= submit_button "Save" %></div>
+    <div class="actions">
+      <%= submit_button "Save" %>
+    </div>
   </div>
-<% end %>
+<% end %>
+<%= destroy_button url(:action => "destroy", :id => @campaign.id), "Remove this campaign" %>

app/views/sites/edit.html.erb

@@ -15,6 +15,7 @@
     <div class="actions"><%= submit_button "Edit site details" %></div>
   <% end %>
 </div>
+<%= destroy_button url(:action => "destroy", :id => @site.id), "Remove this site" %>
 
 <h2>Site Spots</h2>
 <% if @site.spots.empty? %>
@@ -27,6 +28,7 @@
       <p class="meta"><%= spot.campaigns.count %> / <%= spot.campaign_limit %> campaigns filled</p>
       <p class="action"><a href="#" class="codetoggler" toggle="code-<%= spot.id %>">Show Code</a></p>
       <textarea id="code-<%= spot.id %>" class="code-insertion"><%= partial :insertion_code, :spot => spot %></textarea>
+      <%= destroy_button url(:action => "destroy_spot", :id => spot.id), "Remove" %>
     </li>
     <% end %>
   </ul>

public/javascripts/application.js

@@ -53,6 +53,12 @@ var rules = {
       }
       (new Event(e)).stop();
     });
+  },
+
+  'form.destroy': function(element){
+    element.onsubmit = function(){
+      return confirm("Are you sure you want to remove this item?")
+    }
   }
 }
 

public/stylesheets/master.css

@@ -214,6 +214,18 @@ input[type=submit], button[type=submit]{
 input[type=submit]:hover, button[type=submit]:hover{
   background-color:#333;
 }
+button.destroy{
+  position:relative;
+  top:-48px;
+  left:10px;
+  background-color:#cc0000;
+}
+button.destroy:hover{
+  background-color:#990000;
+}
+html>body*#wrap button.destroy{
+  top:-45px;
+}
 
 p.dateresults{
   margin-left:-20px;
@@ -296,6 +308,7 @@ ul.full-width-spots{
   margin-left:0;
 }
 ul.spots li{
+  position:relative;
   list-style-type:none;
   float:left;
   margin:0 0 10px 10px;
@@ -329,4 +342,21 @@ ul.full-width-spots li{
 .spots p.action{
   margin:-22px 0 0 0;
   float:right;
+}
+
+.spots button.destroy{
+  position:absolute;
+  top:10px;
+  right:10px;
+  left:auto;
+  font-size:10px;
+  padding:2px 5px;
+  opacity:0.5;
+  background-color:#333;
+}
+.spots button.destroy:hover{
+  opacity:1;
+}
+html>body*#wrap button.destroy{
+  top:10px;
 }