radiant-database_mailer-extension 1.0.1

data/public/stylesheets/sass/admin/database_mailer.sass

@@ -0,0 +1,235 @@
+.clearfix:after
+  :content "."
+  :display block
+  :height 0
+  :clear both
+  :visibility hidden
+.clearfix
+  :display inline-block
+.clearfix
+  :display block
+
+.grey
+  :background-color #9E9E9E
+  :height 21px
+  :margin -20px -20px 0 -20px
+
+table.results
+  :width 100%
+  a
+    &.sort_asc
+      background: transparent url(/images/admin/metadata_toggle.png) no-repeat scroll 95% 5px
+      padding-right: 15px
+
+    &.sort_desc
+      background: transparent url(/images/admin/metadata_toggle.png) no-repeat scroll 95% -23px
+      padding-right: 15px
+  thead
+    th
+      :background-color #7E7E7E
+      :font-weight normal
+      :padding 2px 4px
+      :text-shadow 1px 1px 0 #666666
+      :color white
+      :font-size 90%
+      &.created_at
+        :width 150px
+      &.url, &.exported
+        :width 200px
+      &.actions
+        :width 100px
+      a
+        :text-shadow 1px 1px 0 #666666
+        :color white
+        :font-size 90%
+  tbody
+    tr:hover
+      :background-color #ecf4ff
+    td
+      :border-bottom 1px solid #F0F0F0
+      :padding 8px 4px
+      :text-align left
+      :vertical-align top
+      &.actions a
+        :color black
+        :text-decoration none
+        &:hover
+          :text-decoration underline
+  tfoot
+    th
+      :background-color #7E7E7E
+      :font-weight normal
+      :padding 2px 4px
+      :text-shadow 1px 1px 0 #666666
+      :color white
+      :font-size 90%
+      :vertical-align middle
+      a
+        :text-shadow 1px 1px 0 #666666
+        :color white
+        :font-size 90%
+
+.pagination
+  :text-align center
+  :float right
+  :margin-right 20px
+  a
+    :text-shadow 1px 1px 0 #666666
+    :color white
+    :font-size 90%
+    :padding .2em .3em
+    :text-decoration none
+    &:hover, &:focus
+      :text-decoration underline
+  span
+    :padding .2em .3em
+    &.disabled
+      :color #CCC
+    &.current
+      :font-weight bold
+      :background-color #cac8c0
+      :display inline-block
+      :padding 3px 8px
+      :-webkit-border-radius 12px
+      :-moz-border-radius 12px
+      
+#filtering
+  position: relative
+  :height 35px
+  a.expand
+    :-moz-border-radius-bottomleft 6px
+    :-moz-border-radius-bottomright 6px
+    :-webkit-border-bottom-left-radius 6px
+    :-webkit-border-bottom-right-radius 6px
+    :text-shadow 1px 1px 0 #666666
+    :outline none
+    :z-index 100
+    :padding 3px 20px 3px 10px
+    :color #000
+    :position absolute
+    :background #7E7E7E url(/images/admin/metadata_toggle.png) no-repeat 92% 8px
+    :cursor pointer
+    :color white
+    :text-decoration none
+    :font-size 80%
+  .expanded a.expand
+    :-moz-border-radius-topleft 6px
+    :-moz-border-radius-topright 6px
+    :-moz-border-radius-bottomleft 0
+    :-moz-border-radius-bottomright 0
+    :-webkit-border-top-left-radius 6px
+    :-webkit-border-top-right-radius 6px
+    :-webkit-border-bottom-left-radius 0
+    :-webkit-border-bottom-right-radius 0
+    :background #7E7E7E url(/images/admin/metadata_toggle.png) no-repeat 92% -20px
+
+#x_expander
+  right: 0
+
+#form, #x_form
+  background-color: #7E7E7E
+  z-index: 10
+  position: absolute
+  top: 22px
+  width: 350px
+  :-moz-border-radius 6px
+  :-webkit-border-radius 6px
+  :border 1px solid #999999
+  border-top-left-radius: 6px
+  border-top-right-radius: 6px
+  :color white
+  input[type="text"]
+    :width 210px
+  a.close
+    :background transparent url('/images/admin/tab_close.png') no-repeat left top
+    :text-indent -9999px
+    :display block
+    :width 16px
+    :height 16px
+    :float right
+    :margin 0 5px 5px 0
+
+#form
+  :-webkit-border-top-left-radius 0
+  :-moz-border-radius-topleft 0
+  a
+    :color white
+    :text-decoration none
+    &:hover
+      :text-decoration underline
+      
+#x_form
+  right: 0
+  :-moz-border-radius-topright 0
+  :-webkit-border-top-right-radius 0
+  :padding 5px 0 0 5px
+  
+table.search
+  :width 340px
+  :margin-left 5px
+  tr
+    :border-top 1px solid #999999
+    &:first-child
+      :border none
+  th, td
+    :padding 5px
+    :vertical-align middle
+  th
+    :width 100px
+    label
+      :color white
+      :font-size 80%
+  td
+    :width 220px
+  
+table.export
+  :width 340px
+  :margin-left 5px
+  tr
+    :border-top 1px solid #999999
+    &:first-child
+      :border none
+  th
+    :padding 3px 0 3px 5px
+  td
+    :padding 3px 5px 3px 0
+
+#current_filters
+  margin-left: 100px
+  padding-top: 3px
+  position: absolute
+  width: 600px
+
+  a
+    color: #000
+
+#show
+  a.back
+    :color red
+    :font-size 80%
+    :text-decoration none
+    &:hover
+      :text-decoration underline
+  .page
+    :min-height 350px
+  table.result
+    :width 96%
+    :margin-left 2%
+    tr
+      :border-bottom 1px solid #ccc
+    th
+      :font-weight bold
+    th, td
+      :padding 5px 10px
+      :vertical-align top
+  table.attachments
+    :width 700px
+    tr
+      :border-bottom 1px solid #ccc
+    th
+      :font-weight bold
+    th, td
+      :padding 5px 10px
+      :vertical-align top
+    a
+      :color black

data/radiant-database_mailer-extension.gemspec

@@ -0,0 +1,26 @@
+# -*- encoding: utf-8 -*-
+$:.push File.expand_path("../lib", __FILE__)
+require "radiant-database_mailer-extension"
+
+Gem::Specification.new do |s|
+  s.name        = "radiant-database_mailer-extension"
+  s.version     = RadiantDatabaseMailerExtension::VERSION
+  s.platform    = Gem::Platform::RUBY
+  s.authors     = RadiantDatabaseMailerExtension::AUTHORS
+  s.email       = RadiantDatabaseMailerExtension::EMAIL
+  s.homepage    = RadiantDatabaseMailerExtension::URL
+  s.summary     = RadiantDatabaseMailerExtension::SUMMARY
+  s.description = RadiantDatabaseMailerExtension::DESCRIPTION
+
+  ignores = if File.exist?('.gitignore')
+    File.read('.gitignore').split("\n").inject([]) {|a,p| a + Dir[p] }
+  else
+    []
+  end
+  s.files         = Dir['**/*'] - ignores
+  s.test_files    = Dir['test/**/*','spec/**/*','features/**/*'] - ignores
+  # s.executables   = Dir['bin/*'] - ignores
+  s.require_paths = ["lib"]
+  
+  s.add_dependency 'radiant-mailer-extension', '~> 1.0.3'
+end

data/spec/controllers/form_datas_controller_spec.rb

@@ -0,0 +1,272 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe Admin::FormDatasController do
+  dataset :users
+  
+  before :each do
+    login_as :designer
+  end
+  
+  describe  'handling GET index' do
+  
+    before do
+      @form_datas = (1..10).map {|i| mock_model(FormData) }
+      @urls = (1..4).map {|i| mock_model(FormData)}
+      
+      FormData.stub!(:form_paginate).and_return(@form_datas)
+      FormData.stub!(:find_all_group_by_url).and_return(@urls)
+      @list_params = mock("list_params")
+      controller.stub!(:filter_by_params)
+    end
+  
+    def do_get
+      get :index
+    end
+  
+    it "should be succesful" do
+      do_get
+      response.should be_success
+    end
+  
+    it "should render index template" do
+      do_get
+      response.should render_template('index')
+    end
+  
+    it "should parse list_params" do
+      controller.should_receive(:filter_by_params).with(FormData::FILTER_COLUMNS)
+      do_get
+    end
+  
+    it "should find all saved items with list_params" do
+      controller.should_receive(:list_params).and_return(@list_params)
+      FormData.should_receive(:form_paginate).with(@list_params).and_return(@form_datas)
+      do_get
+    end
+  
+    it "should assign the found saved items for the view" do
+      do_get
+      assigns[:saved_items].should == @form_datas
+    end
+    
+    it "should assign the found urls for the view" do
+      do_get
+      assigns[:urls].should == @urls
+    end
+  end
+  
+  describe "handling GET show" do
+    before(:each) do
+      @form_data = mock_model(FormData)
+      FormData.stub!(:find).and_return(@form_data)
+    end
+    
+    def do_get
+      get :show, :id => @form_data.id
+    end
+    
+    it "is successful" do
+      do_get
+      response.should be_success
+    end
+    
+    it "finds the corresponding form_data and assigns it to the view" do
+      FormData.should_receive(:find).with(@form_data.id.to_s).and_return(@form_data)
+      do_get
+      assigns[:form_data].should == @form_data
+    end
+    
+  end
+
+  describe 'handling GET index.csv' do
+    before do
+      @list_params = mock("list_params")
+      controller.stub!(:list_params).and_return(@list_params)
+      controller.stub!(:filter_by_params)
+      
+      t = Time.now
+      @time = mock("time", :to_s => 'time', :to_f => t.to_f, :to_i => t.to_i, :strftime => 'time')
+      Time.stub!(:now).and_return(@time)
+
+      FormData.stub!(:export_csv).and_return('csv')
+      controller.stub!(:send_data)
+      
+      @request.accept = "text/csv" #=> request JS
+    end
+    
+    def do_get(options={})
+      get :index, options, :format => 'csv'
+    end
+    
+    it "is succesful" do
+      do_get
+      response.should be_redirect
+    end    
+    
+    it "parses list_params" do
+      controller.should_receive(:filter_by_params).with(FormData::FILTER_COLUMNS)
+      do_get
+    end
+    
+    it "passes the selected_export_columns" do
+      FormData.should_receive(:export_csv).with(@list_params, %w(name url), @time, false).and_return('csv')
+      do_get({:export_name => "name", :export_url => "url"})
+    end
+    
+    it "passes the include_all columns" do
+      FormData.should_receive(:export_csv).with(@list_params, [], @time, true).and_return('csv')
+      do_get({:include_all => "yes"})
+    end
+    
+    it "sends data as csv string" do
+      FormData.should_receive(:export_csv).and_return('csv_string')
+      controller.should_receive(:send_data).with("csv_string", {:type=>"text/csv", :filename=>"export_#{@time.to_s}.csv", :disposition=>"attachment"})
+      do_get
+    end
+  end
+  
+  describe "handling GET index.xls" do
+    before do
+      @list_params = mock("list_params")
+      controller.stub!(:list_params).and_return(@list_params)
+      controller.stub!(:filter_by_params)
+      
+      t = Time.now
+      @time = mock("time", :to_s => 'time', :to_f => t.to_f, :to_i => t.to_i, :strftime => 'time')
+      Time.stub!(:now).and_return(@time)
+
+      FormData.stub!(:export_xls).and_return('xls')
+      controller.stub!(:send_file)
+    end
+    
+    def do_get(options={})
+      get :index, options.merge(:format => 'xls')
+    end
+    
+    it "is succesful" do
+      do_get
+      response.should be_success
+    end
+    
+    it "parses list_params" do
+      controller.should_receive(:filter_by_params).with(FormData::FILTER_COLUMNS)
+      do_get
+    end
+    
+    it "passes the selected_export_columns" do
+      FormData.should_receive(:export_xls).with(@list_params, %w(name url), @time, false).and_return('xls')
+      do_get({:export_name => "name", :export_url => "url"})
+    end
+    
+    it "passes the include_all columns" do
+      FormData.should_receive(:export_xls).with(@list_params, [], @time, true).and_return('xls')
+      do_get({:include_all => "yes"})
+    end
+    
+    it "sends data as csv string" do
+      FormData.should_receive(:export_xls).and_return('xls_file')
+      controller.should_receive(:send_file).with("xls_file", {:type=>"application/vnd.ms-excel", :filename=>"form_data_#{@time.to_s}.xls", :disposition=>"attachment"})
+      do_get
+    end
+    
+  end
+
+  describe "parsing list_params" do
+    
+    def do_get(options={})
+      get :index, options
+    end
+  
+    def filter_by_params(args=[])
+      @controller.send(:filter_by_params, args)
+    end
+    
+    def list_params
+      @controller.send(:list_params)
+    end
+    
+    def set_cookie(key, value)
+      # request.cookies[key] = CGI::Cookie.new('name' => key, 'value' => value)
+      request.cookies[key] = value
+    end
+  
+    it "should have default set of params" do
+      do_get
+      filter_by_params
+      [:page, :sort_by, :sort_order].each {|p| response.cookies.keys.should include(p.to_s)}
+    end
+    
+    it "should take arbitrary params" do
+      do_get(:name => 'Blah', :test => 10)
+      filter_by_params([:name, :test])
+      [:name, :test].each {|p| response.cookies.keys.should include(p.to_s)}
+    end
+        
+    it "should load list_params from cookies by default" do
+      set_cookie('page', '98')
+      do_get
+      filter_by_params
+      list_params[:page].should == '98'
+    end
+    
+    it "should prefer request params over cookies" do
+      set_cookie('page', '98')
+      do_get(:page => '99')
+      filter_by_params
+      list_params[:page].should == '99'
+    end
+    
+    it "should update cookies with new values" do
+      set_cookie('page', '98')
+      do_get(:page => '99')
+      filter_by_params
+      response.cookies['page'].should == '99'
+    end
+    
+    it "should reset list_params when params[:reset] == 1" do
+      set_cookie('page', '98')
+      do_get(:reset => 1)
+      filter_by_params
+      response.cookies['page'].should == "1"
+    end
+    it "should set params[:page] if loading from cookies (required for will_paginate to work)" do
+      set_cookie('page', '98')
+      do_get
+      filter_by_params
+      params[:page].should == '98'
+    end
+  end
+
+  describe "handling DELETE destroy" do
+    
+    before do
+      @form_data = mock_model(FormData)
+      FormData.stub!(:find).and_return(@form_data)
+      @form_data.stub!(:destroy)
+    end
+    
+    def do_delete
+      delete :destroy, :id => @form_data.id
+    end
+    
+    it "finds the coresponding record" do
+      FormData.should_receive(:find).with(@form_data.id.to_s).and_return(@form_data)
+      do_delete
+    end
+    
+    it "destroys the record" do
+      @form_data.should_receive(:destroy)
+      do_delete
+    end
+    
+    it "assigns flash message" do
+      do_delete
+      flash[:notice].should == "Record deleted!"
+    end
+    
+    it "redirects on success" do
+      do_delete
+      response.should be_redirect
+    end
+  end
+end

data/spec/controllers/mail_controller_spec.rb

@@ -0,0 +1,40 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe MailController do
+  dataset :users, :mailers
+  
+  before do
+    login_as :designer
+    @form_data = mock_model(FormData)
+    FormData.stub!(:create)
+  end
+  
+  def do_post(options={})
+    post :create, {:page_id => page_id(:contact), :mailer => {}}.merge(options)
+  end
+  
+  it "redirects to /contact-success on success" do
+    do_post
+    response.should redirect_to('/contact/thank-you')
+  end
+  
+  it "creates a new FormData from mailer params" do
+    FormData.should_receive(:create).and_return(@form_data)
+    do_post
+  end
+  
+  it "passes the page url to the new form data" do
+    FormData.should_receive(:create).with("url" => "/contact/", "blob" => "").and_return(@form_data)
+    do_post
+  end
+  
+  it "passes mailer params to the new for data" do
+    FormData.should_receive(:create).with("name" => "piki", "url" => "/contact/", "blob" => "").and_return(@form_data)
+    do_post(:mailer => {"name" => 'piki'})
+  end
+  
+  it "does not create FormData when save_to_database config is false" do
+    FormData.should_not_receive(:create)
+    do_post(:page_id => page_id(:contact_no_save))
+  end
+end

data/spec/helpers/form_datas_helper_spec.rb

@@ -0,0 +1,5 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe Admin::FormDatasHelper do
+  helper_name 'admin/form_datas'
+end

data/spec/models/form_data_asset_spec.rb

@@ -0,0 +1,11 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe FormDataAsset do
+  before(:each) do
+    @form_data_asset = FormDataAsset.new
+  end
+
+  it "should be valid" do
+    @form_data_asset.should be_valid
+  end
+end