letter_opener_web 1.0.0 → 1.0.1

data/Guardfile

@@ -11,13 +11,4 @@ guard 'rspec' do
   watch(%r{^app/(.*)(\.erb|\.haml)$})                 { |m| "spec/#{m[1]}#{m[2]}_spec.rb" }
   watch(%r{^app/controllers/(.+)_(controller)\.rb$})  { |m| ["spec/routing/#{m[1]}_routing_spec.rb", "spec/#{m[2]}s/#{m[1]}_#{m[2]}_spec.rb", "spec/acceptance/#{m[1]}_spec.rb"] }
   watch(%r{^spec/support/(.+)\.rb$})                  { "spec" }
-  watch('config/routes.rb')                           { "spec/routing" }
-  watch('app/controllers/application_controller.rb')  { "spec/controllers" }
-
-  # Capybara features specs
-  watch(%r{^app/views/(.+)/.*\.(erb|haml)$})          { |m| "spec/features/#{m[1]}_spec.rb" }
-
-  # Turnip features and steps
-  watch(%r{^spec/acceptance/(.+)\.feature$})
-  watch(%r{^spec/acceptance/steps/(.+)_steps\.rb$})   { |m| Dir[File.join("**/#{m[1]}.feature")][0] || 'spec/acceptance' }
 end

data/README.md

@@ -7,7 +7,7 @@ browsing sent emails.
 
 First add the gem to your development environment and run the `bundle` command to install it.
 
-    gem 'letter_opener_web', '1.0.0.rc3', :group => :development
+    gem 'letter_opener_web', '1.0.0', :group => :development
 
 ## Usage
 
@@ -48,11 +48,14 @@ To prevent `rails-footnotes` from outputing debug information to your mails add
 the following to your `config/initializers/footnotes.rb`:
 
 ```ruby
+notes = Footnotes::Filter.notes
 Footnotes.setup do |config|
   config.before do |controller, filter|
-    controller.class.name =~ /LetterOpenerWeb/ ?
-      filter.notes.clear :
-      filter.notes
+    if controller.class.name =~ /LetterOpenerWeb/
+      filter.notes = []
+    else
+      filter.notes = notes
+    end
   end
 end
 ```

data/app/controllers/letter_opener_web/letters_controller.rb

@@ -14,6 +14,17 @@ module LetterOpenerWeb
       render text: text
     end
 
+    def attachment
+      letter   = Letter.find(params[:id])
+      filename = "#{params[:file]}.#{params[:format]}"
+
+      if file = letter.attachments[filename]
+        send_file(file, :filename => filename, :disposition => 'inline')
+      else
+        render :text => 'Attachment not found!', :status => 404
+      end
+    end
+
     def clear
       Letter.destroy_all
       redirect_to letters_path

data/app/models/letter_opener_web/letter.rb

@@ -44,14 +44,24 @@ module LetterOpenerWeb
         'plain'
     end
 
+    def attachments
+      @attachments ||= Dir["#{base_dir}/attachments/*"].each_with_object({}) do |file, hash|
+        hash[File.basename(file)] = File.expand_path(file)
+      end
+    end
+
     private
 
+    def base_dir
+      "#{letters_location}/#{id}"
+    end
+
     def read_file(style)
-      File.read("#{letters_location}/#{id}/#{style}.html")
+      File.read("#{base_dir}/#{style}.html")
     end
 
     def style_exists?(style)
-      File.exists?("#{letters_location}/#{id}/#{style}.html")
+      File.exists?("#{base_dir}/#{style}.html")
     end
 
     def adjust_link_targets(contents)

data/config/routes.rb

@@ -1,5 +1,6 @@
 LetterOpenerWeb::Engine.routes.draw do
-  delete 'clear'        => 'letters#clear', as: :clear_letters
-  get    '/'            => 'letters#index', as: :letters
-  get    ':id(/:style)' => 'letters#show',  as: :letter
+  delete 'clear'                 => 'letters#clear', :as => :clear_letters
+  get    '/'                     => 'letters#index', :as => :letters
+  get    ':id(/:style)'          => 'letters#show',  :as => :letter
+  get    ':id/attachments/:file' => 'letters#attachment'
 end

data/demo/application_controller.rb

@@ -6,7 +6,7 @@ class ApplicationController < ActionController::Base
   end
 
   def send_mail
-    ContactMailer.new_message(params[:email], params[:message]).deliver
+    ContactMailer.new_message(params[:email], params[:message], params[:attachment]).deliver
     redirect_to '/', :notice => 'Email sent successfully, please check letter_opener_web inbox.'
   end
 end
@@ -18,8 +18,11 @@ class ContactMailer < ActionMailer::Base
           :from          => 'no-reply@letter_opener_web.com',
           :template_path => '.'
 
-  def new_message(from, message)
+  def new_message(from, message, attachment)
     @from, @message = from, message
+    if attachment.present?
+      attachments[attachment.original_filename] = attachment.tempfile.read
+    end
     mail(:subject => 'Testing letter_opener_web', :template_name => 'new_message')
   end
 end

data/demo/index.html.erb

@@ -29,7 +29,7 @@
         <div class="span6 offset3">
 
           <div class="well">
-            <form method="post" action="/">
+            <form method="post" action="/" enctype="multipart/form-data">
               <legend>Contact form</legend>
               <div class="control-group">
                 <label class="control-label" for="email">Email</label>
@@ -37,6 +37,12 @@
                   <input type="text" class="input-block-level" name="email">
                 </div>
               </div>
+              <div class="control-group">
+                <label class="control-label" for="attachment">Attachment <em>(optional)</em></label>
+                <div class="controls">
+                  <input type='file' name='attachment' />
+                </div>
+              </div>
               <div class="control-group">
                 <label class="control-label" for="message">Message</label>
                 <div class="controls">

data/lib/letter_opener_web/version.rb

@@ -1,3 +1,3 @@
 module LetterOpenerWeb
-  VERSION = "1.0.0"
+  VERSION = "1.0.1"
 end

data/spec/controllers/letter_opener_web/letters_controller_spec.rb

@@ -47,6 +47,29 @@ describe LetterOpenerWeb::LettersController do
     end
   end
 
+  describe 'GET attachment' do
+    let(:id)              { 'an-id' }
+    let(:attachment_path) { "path/to/attachment" }
+    let(:file_name)       { 'image.jpg' }
+    let(:letter)          { mock(:letter, attachments: { file_name => attachment_path}, id: id) }
+
+    before do
+      LetterOpenerWeb::Letter.stub(find: letter)
+      controller.stub(:send_file) { controller.render :nothing => true }
+    end
+
+    it 'sends the file as an inline attachment' do
+      controller.should_receive(:send_file).with(attachment_path, :filename => file_name, :disposition => 'inline')
+      get :attachment, id: id, file: file_name.gsub(/\.\w+/, ''), format: File.extname(file_name)[1..-1]
+      response.status.should == 200
+    end
+
+    it "throws a 404 if attachment file can't be found" do
+      get :attachment, id: id, file: 'unknown', format: 'woot'
+      response.status.should == 404
+    end
+  end
+
   describe 'DELETE clear' do
     it 'removes all letters' do
       LetterOpenerWeb::Letter.should_receive(:destroy_all)

data/spec/internal/config/routes.rb

@@ -1,5 +1,6 @@
 Rails.application.routes.draw do
-  delete 'clear'        => 'letter_opener_web/letters#clear'
-  get    '/'            => 'letter_opener_web/letters#index', as: :letters
-  get    ':id(/:style)' => 'letter_opener_web/letters#show',  as: :letter
+  delete 'clear'                 => 'letter_opener_web/letters#clear'
+  get    '/'                     => 'letter_opener_web/letters#index', as: :letters
+  get    ':id(/:style)'          => 'letter_opener_web/letters#show',  as: :letter
+  get    ':id/attachments/:file' => 'letter_opener_web/letters#attachment', format: false
 end

data/spec/models/letter_opener_web/letter_spec.rb

@@ -64,6 +64,23 @@ MAIL
     end
   end
 
+  describe 'attachments' do
+    let(:file)            { 'an-image.csv' }
+    let(:attachments_dir) { "#{location}/#{id}/attachments" }
+    let(:id)              { '1111_1111' }
+
+    subject { described_class.new(id: id) }
+
+    before do
+      FileUtils.mkdir_p(attachments_dir)
+      File.open("#{attachments_dir}/#{file}", 'w') { |f| f.puts 'csv,contents' }
+    end
+
+    it 'builds a hash with file name as key and full path as value' do
+      subject.attachments.should == { file => "#{attachments_dir}/#{file}" }
+    end
+  end
+
   describe '.search' do
     let(:search_results) { described_class.search }
     let(:first_letter)   { search_results.first }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: letter_opener_web
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.1
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-02-04 00:00:00.000000000 Z
+date: 2013-02-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   prerelease: false
@@ -155,7 +155,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 497055164717147357
+      hash: -1136549867586764880
   none: false
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
@@ -164,7 +164,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 497055164717147357
+      hash: -1136549867586764880
   none: false
 requirements: []
 rubyforge_project: