letter_bomb 0.0.1 → 0.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 9c67ffe4932cf6ab7ad4afb2b846bbaac8750fa8
-  data.tar.gz: 0d2ba7da06ca5b5fec46b5dbec6224fb864890bd
+  metadata.gz: ca88daac8b46e5920ae83fb0df5232d37deab93d
+  data.tar.gz: e01a4819e6c2717b49db1a82f7c34590b29d5804
 SHA512:
-  metadata.gz: 2a178785f941f88ab127a964e421f6dac918484aaaa4b0e870ead025748ed1baa54ba62eccce731e81026bcd76d2699216dd5d532fca261be0b63b4d022848c9
-  data.tar.gz: 62665b33af2ecbe0b38dd0f3002c0b0efe8e2b5d633f744fd859fa1801e5d77c0117e58209e7ba4c8909c95b477f448d5d0dc29d632112f4783fb750ad268eb9
+  metadata.gz: df503fcfd3a2406b40c15dc87780c24b8ae38c6c0ae575cf48fb0389ca9b086f1b3995240f000651a38defc82f5f40f19ecac5dcaec39f7b138dd61fdc01c2ea
+  data.tar.gz: b063d82333e4db87cd7560265921d64e3bfb3046b021f9d607b5ccd83de8e41b9e1a9d865ba5967bd6e9213efe484536046efbef8fb06d89e5e6ba87fb95c071

data/README.md

@@ -25,7 +25,8 @@ mount LetterBomb::Engine, at: '/letter_bomb'
 
 and hit `/letter_bomb` for a list of previews.
 
-Previews can be defined anywhere within `app/mailers`.
+Previews can be defined anywhere within `app/mailers` suffixed with `preview`, i.e. `app/mailers/user_mailer_preview.rb` or `app/mailers/user_mailer/preview.rb`.
+Preview method names are arbitrary so long as they return a `Mail` object.
 
 ```ruby
 class UserMailer::Preview < LetterBomb::Preview
@@ -53,8 +54,6 @@ class UserMailer::Preview < LetterBomb::Preview
 end
 ```
 
-Preview class names and methods are arbitrary so long as they return a `Mail` object.
-
 Alternatives
 ------------
 

data/app/controllers/letter_bomb/mailers_controller.rb

@@ -1,13 +1,39 @@
 module LetterBomb
   class MailersController < ApplicationController
+
     def index
       @mailers = LetterBomb::Preview.previews
     end
 
     def show
-      mailer_class = params[:mailer_class]
-      mailer_action = params[:mailer_action]
-      @mail = mailer_class.constantize.preview_action(mailer_action)
+      klass = params[:mailer_class]
+      @action = params[:mailer_action]
+      @mail = klass.constantize.preview_action(@action)
+
+      respond_to do |format|
+        format.html
+        format.text { render formats: [:html], content_type: 'text/html' }
+      end
     end
+
+    private
+
+    def body_part
+      return @mail unless @mail.multipart?
+
+      content_type = Rack::Mime.mime_type(".#{params[:format]}")
+      if @mail.respond_to?(:all_parts)
+        @mail.all_parts.find { |part| part.content_type.match(content_type) } || @mail.parts.first
+      else
+        @mail.parts.find { |part| part.content_type.match(content_type) } || @mail.parts.first
+      end
+    end
+    helper_method :body_part
+
+    def html_template?
+      params[:format] == 'html'
+    end
+    helper_method :html_template?
+
   end
 end

data/app/views/letter_bomb/mailers/index.html.erb

@@ -1,11 +1,11 @@
-<h1>Mailers</h1>
+<h1>Mailer Previews</h1>
 
 <% @mailers.each do |mailer| %>
   <p><%= mailer.to_s %></p>
 
   <ul>
   <% mailer.mailer_actions.each do |action| %>
-    <li><%= link_to action, mailer_path(mailer.to_s, action) %></li>
+    <li><%= link_to action, mailer_path(mailer.to_s, action, format: :html) %></li>
   <% end %>
   </ul>
 

data/app/views/letter_bomb/mailers/show.html.erb

@@ -75,14 +75,21 @@
 
     <dt>To:</dt>
     <dd><%= @mail.to %></dd>
-
-    <dt>Content-Type:</dt>
-    <dd><%= @mail.content_type %></dd>
   </dl>
+
+  <% if @mail.multipart? %>
+    <p class="alternate">
+      <% if html_template? %>
+        <a href="<%= @action %>.txt">View plain text version</a>
+      <% else %>
+        <a href="<%= @action %>.html">View HTML version</a>
+      <% end %>
+    </p>
+  <% end %>
 </div>
 
-<% if @mail.content_type && @mail.content_type.match(/text\/html/) %>
-  <%= @mail.body.to_s.html_safe %>
+<% if html_template? %>
+  <%= body_part.body.to_s.html_safe %>
 <% else %>
-  <pre id="message_body"><%= @mail.body %></pre>
+  <pre id="message_body"><%= body_part.body %></pre>
 <% end %>

data/lib/letter_bomb/engine.rb

@@ -1,17 +1,5 @@
 module LetterBomb
   class Engine < ::Rails::Engine
     isolate_namespace LetterBomb
-
-    initializer 'letterbomb.mailers.autoload' do |app|
-      # TODO this is sloppy. the reason behind it is:
-      # 1) to generate the list of previews, we need all subclasses.
-      #    rails doesn't load them up front, so we need to do that manually.
-      #
-      # 2) because we load the mailers and previews up-front, we have to manually tell
-      #    rails to reload each per request.
-      ActionDispatch::Callbacks.before do
-        Dir[File.join(app.config.root, "/app/mailers/**/*.rb")].each { |file| load(file) }
-      end
-    end
   end
 end

data/lib/letter_bomb/preview.rb

@@ -2,11 +2,15 @@ module LetterBomb
   class Preview
 
     def self.previews
-      subclasses
+      load_dir = Rails.application.root.join('app/mailers').to_s
+      Dir.glob(load_dir + '/**/*preview.rb').map do |filename|
+        # app/mailers/module/class.rb => module/class.rb => module/class => Module::Class
+        lchomp(filename, load_dir).chomp('.rb').camelize.constantize
+      end
     end
 
     def self.mailer_actions
-      new.public_methods(false).map(&:to_s).sort
+      public_instance_methods(false).map(&:to_s).sort
     end
 
     def self.preview_action(mailer_action)
@@ -29,5 +33,9 @@ module LetterBomb
       end
     end
 
+    def self.lchomp(base, arg)
+      base.to_s.reverse.chomp(arg.to_s.reverse).reverse
+    end
+
   end
 end

data/lib/letter_bomb/version.rb

@@ -1,3 +1,3 @@
 module LetterBomb
-  VERSION = "0.0.1"
+  VERSION = '0.0.2'
 end

data/spec/dummy/app/mailers/foo_mailer_preview.rb

@@ -0,0 +1,12 @@
+class FooMailerPreview < LetterBomb::Preview
+  def good
+    User.create!(name: 'bob')
+    'result of good'
+  end
+
+  def bad; end
+
+  private
+
+  def ugly; end
+end

data/spec/dummy/app/mailers/user_mailer/preview.rb

@@ -0,0 +1,6 @@
+class UserMailer::Preview < LetterBomb::Preview
+  def welcome
+    user = User.create!(name: 'john')
+    UserMailer.welcome(user.name)
+  end
+end

data/spec/dummy/app/mailers/user_mailer.rb

@@ -4,18 +4,6 @@ class UserMailer < ActionMailer::Base
 
   def welcome(var)
     @var = var
-    mail(subject: 'welcome to the show!') 
-  end
-end
-
-class UserMailer::Preview < LetterBomb::Preview
-  def welcome
-    user = User.create!(name: 'john')
-    UserMailer.welcome(user.name)
-  end
-end
-
-class UserMailer::PreviewAgaian < LetterBomb::Preview
-  def foo
+    mail(subject: 'welcome to the show!')
   end
 end

data/spec/dummy/app/views/user_mailer/welcome.html.erb

@@ -1,3 +1,3 @@
 <h1>welcome!</h1>
 
-<p>a mailer var: <%= @var %></p>
+<p style="color: red">a mailer var: <%= @var %></p>

data/spec/dummy/app/views/user_mailer/welcome.text.erb

@@ -0,0 +1,3 @@
+welcome!
+
+a mailer var: <%= @var %>