bcms_feedback_form 1.0.1 → 1.0.2

data/README.markdown

@@ -5,8 +5,9 @@ Provides a feedback form content block protected by Recaptcha. It also:
 * Allows you to specific the thank you message,
 * Gives you the ability to create a custom form,
 * Requires that you register at recaptcha.net and create an API key,
-* Uses prototype to dynamically swap out the form in-place with a thank you message, allowing this to work fine in a statically cached environment.
-* Includes the recaptcha plugin for you.
+* Uses prototype to dynamically swap out the form in-place with a thank you message, allowing this to work fine in a statically cached environment,
+* Includes the recaptcha plugin,
+* Gives you a CSV download for form submissions (as of 1.0.2).
 
 ## Installation
 

data/app/controllers/cms/feedback_forms_controller.rb

@@ -1,2 +1,40 @@
 class Cms::FeedbackFormsController < Cms::ContentBlockController
-end
+
+  def download
+    @feedback_form = FeedbackForm.find(params[:id])
+    render_csv(@feedback_form.feedback_submissions)
+  end
+
+  def render_csv(objects)
+    columns = {}
+    rows = []
+
+    # Iterate through to find all the possible columns, just in case multiple versions of a form have been submitted.
+    objects.each do |obj|
+      row = YAML.load(obj.feedback_content)
+      rows << row
+      row['user_agent'] = obj.user_agent
+      row['ip_address'] = obj.ip_address
+      row.keys.each do |column|
+        columns[column] = 1
+      end
+    end
+
+    csv_output = ''
+    sorted_columns = columns.keys.sort
+    CSV::Writer.generate(csv_output) do |csv|
+      csv << sorted_columns
+      rows.each do |row|
+        row_array = []
+        sorted_columns.each do |col|
+          row_array << row[col] || ''
+        end
+        csv << row_array
+      end
+    end
+    send_data(csv_output,
+              :type => 'application/octet-stream',
+              :filename => "feedback-submissions-#{Time.now.to_s(:number)}.csv")
+  end
+
+end

data/app/controllers/feedback_forms_controller.rb

@@ -1,21 +1,29 @@
 class FeedbackFormsController < ApplicationController
   def submit
     feedback_form = FeedbackForm.find(params[:id])
-    if feedback_form.use_recaptcha == true
-      if verify_recaptcha
-        Feedback.deliver_feedback(feedback_form, params)
-      else
-        raise
+
+    if (feedback_form.use_recaptcha == false) || verify_recaptcha
+      begin
+        FeedbackSubmission.create(
+          :feedback_form => feedback_form, 
+          :feedback_content => params[:feedback_form].to_yaml, 
+          :user_agent => request.user_agent,
+          :ip_address => request.remote_ip
+        )
+      rescue Exception => e
+        #Failed, but it's not a huge deal.
       end
-    else
       Feedback.deliver_feedback(feedback_form, params)
+    else
+      raise
     end
+
     if request.xhr?
       render :text => ((feedback_form.thank_you.blank?) ? 'Thanks!' : feedback_form.thank_you), :layout => false and return
     else
       redirect_to((params[:_redirect_to].blank?) ? '/' : params[:_redirect_to]) and return
     end
-  rescue
+  rescue Exception => e
     if request.xhr?
       render :text => 'There seems to have been an error. Please try again', :layout => false and return
     else

data/app/models/feedback_form.rb

@@ -1,5 +1,7 @@
 class FeedbackForm < ActiveRecord::Base
   acts_as_content_block
+  has_many :feedback_submissions
+
   validates_length_of :send_to, :maximum => 500
   validates_length_of :subject, :maximum => 200, :allow_blank => true
   validates_length_of :from, :maximum => 100, :allow_blank => true

data/app/models/feedback_submission.rb

@@ -0,0 +1,8 @@
+class FeedbackSubmission < ActiveRecord::Base
+  belongs_to :feedback_form
+
+  validates_length_of :feedback_content, :maximum => 256.kilobytes, :allow_blank => true
+  validates_length_of :user_agent, :maximum => 1000, :allow_blank => true
+  validates_length_of :ip_address, :maximum => 20, :allow_blank => true
+
+end

data/app/views/cms/feedback_forms/render.html.erb

@@ -1,4 +1,7 @@
 <div class="feedback-form">
+  <% if @mode == 'edit' %>
+    <div class="download-submissions"><%= @content_block.feedback_submissions.length %> submitted. <%= link_to('download as CSV', :controller => '/cms/feedback_forms', :action => :download, :id => @content_block ) %></div> 
+  <% end %>
   <%= @content_block.description %>
   <div id="feedback-form-<%= @content_block.id %>">
     <% form_remote_tag(:url => url_for(:controller => '/feedback_forms', :action => :submit, :id => @content_block.id), :update => "feedback-form-#{@content_block.id}" ) do -%>

data/db/migrate/20100315174506_create_feedback_submissions.rb

@@ -0,0 +1,18 @@
+class CreateFeedbackSubmissions < ActiveRecord::Migration
+  def self.up
+    create_table :feedback_submissions do |t|
+      t.references :feedback_form
+      t.string :feedback_content, :limit => 256.kilobytes
+      t.string :user_agent, :limit => 1000
+      t.string :ip_address, :limit => 20
+      t.timestamps
+    end
+    add_index :feedback_submissions, :feedback_form_id
+    add_index :feedback_submissions, :user_agent
+    add_index :feedback_submissions, :ip_address
+  end
+
+  def self.down
+    drop_table :feedback_submissions
+  end
+end

data/lib/bcms_feedback_form/routes.rb

@@ -2,6 +2,7 @@ module Cms::Routes
   def routes_for_bcms_feedback_form
     self.connect 'feedback_forms/:action/:id.:format', :controller => 'feedback_forms'
     namespace(:cms) do |cms|
+      cms.connect 'feedback_forms/download/:id',:controller => 'feedback_forms', :action => :download, :conditions => {:method => :get}
       cms.content_blocks :feedback_forms
     end  
   end

data/rails/init.rb

@@ -1,3 +1,4 @@
+require 'csv'
 gem_root = File.expand_path(File.join(File.dirname(__FILE__), ".."))
 Cms.add_to_rails_paths gem_root
 Cms.add_generator_paths gem_root, "db/migrate/[0-9]*_*.rb"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: bcms_feedback_form
 version: !ruby/object:Gem::Version 
-  version: 1.0.1
+  version: 1.0.2
 platform: ruby
 authors: 
 - Daniel Collis-Puro
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-03-12 00:00:00 -05:00
+date: 2010-03-15 00:00:00 -04:00
 default_executable: 
 dependencies: []
 
@@ -34,7 +34,9 @@ files:
 - app/controllers/cms/feedback_forms_controller.rb
 - app/models/feedback.rb
 - app/models/feedback_form.rb
+- app/models/feedback_submission.rb
 - db/migrate/20100312161745_create_feedback_forms.rb
+- db/migrate/20100315174506_create_feedback_submissions.rb
 - lib/bcms_feedback_form.rb
 - lib/bcms_feedback_form/routes.rb
 - rails/init.rb