georgia_mailer 0.1.3 → 0.8.0

data/app/policies/georgia/mailer/mailer_policy.rb

@@ -0,0 +1,52 @@
+module Georgia
+  module Mailer
+    class MailerPolicy < Georgia::ApplicationPolicy
+
+      def index?
+        mailer_user_permissions(:read_messages).include?(true)
+      end
+
+      def search?
+        index?
+      end
+
+      def show?
+        index?
+      end
+
+      def destroy?
+        mailer_user_permissions(:delete_messages).include?(true)
+      end
+
+      def spam?
+        mailer_user_permissions(:mark_messages_as_spam).include?(true)
+      end
+
+      def ham?
+        mailer_user_permissions(:mark_messages_as_ham).include?(true)
+      end
+
+      def print?
+        mailer_user_permissions(:print_messages).include?(true)
+      end
+
+      def destroy_all_spam?
+        mailer_user_permissions(:empty_trash).include?(true)
+      end
+
+      def resend_notification?
+        index?
+      end
+
+      private
+
+      def mailer_permissions
+        Georgia.permissions[:mailer]
+      end
+
+      def mailer_user_permissions action
+        user_permissions(mailer_permissions, action)
+      end
+    end
+  end
+end

data/app/presenters/georgia/mailer/message_actions_presenter.rb

@@ -0,0 +1,42 @@
+module Georgia
+  module Mailer
+    class MessageActionsPresenter < ::Georgia::Presenter
+
+      def initialize view_context, message
+        @message = (message.decorated? ? message.object : message)
+        super
+      end
+
+      def to_s
+        html = ActiveSupport::SafeBuffer.new
+        if @message.spam
+          html << link_to_ham# if can?(:ham, @message)
+        else
+          html << link_to_spam# if can?(:spam, @message)
+        end
+        html << link_to_resend_notification
+        html << link_to_trash# if can?(:destroy, @message)
+        html
+      end
+
+      private
+
+      def link_to_ham
+        link_to "#{icon_tag('thumbs-up')} Move to Inbox".html_safe, [:ham, @message], method: :post, class: 'btn btn-warning btn-sm'
+      end
+
+      def link_to_spam
+        link_to "#{icon_tag('exclamation-triangle')} Report spam".html_safe, [:spam, @message], method: :post, class: 'btn btn-warning btn-sm'
+      end
+
+      def link_to_resend_notification
+        link_to "#{icon_tag('envelope')} Resend notification".html_safe, [:resend_notification, @message], class: 'btn btn-warning btn-sm'
+      end
+
+      def link_to_trash
+        link_to "#{icon_tag('trash-o')} Trash".html_safe, @message, method: :delete, data: {confirm: 'Are you sure?'}, class: 'btn btn-danger btn-sm'
+      end
+
+    end
+  end
+end

data/app/searches/georgia/mailer/message_search.rb

@@ -0,0 +1,7 @@
+module Georgia
+  module Mailer
+    class MessageSearch < Georgia::SearchDefinition
+
+    end
+  end
+end

data/app/services/georgia/mailer/spam_check.rb

@@ -0,0 +1,19 @@
+module Georgia
+  module Mailer
+    class SpamCheck
+
+      def initialize message
+        @message = message
+      end
+
+      # The .spam? method comes from Rakismet. It gathers the necessary information and
+      #   and calls Akismet for a spam check. It returns true or false based on the status
+      #
+      # https://github.com/joshfrench/rakismet/blob/master/lib/rakismet/model.rb
+      def call
+        @message.spam?
+      end
+
+    end
+  end
+end

data/app/views/georgia/header/_messages.html.erb

@@ -1,3 +1,3 @@
-<% if can? :index, GeorgiaMailer::Message %>
+<% if policy(Georgia::Mailer::Message).search? %>
   <%= sidebar_navigation_link 'Inbox', georgia.search_messages_url, controller: 'messages', icon: 'envelope' %>
 <% end -%>

data/app/views/georgia/{georgia_mailer → mailer}/messages/_message.html.erb

@@ -1,4 +1,6 @@
 <tr id='<%= dom_id(message) %>' class='message' data-status="<%= message.status %>">
+  <td class='status <%= message.status %>'>
+  </td>
   <td class='checkbox-cell'>
     <%= checkboxable_tag message %>
   </td>

data/app/views/{georgia_mailer → georgia/mailer}/notifier/new_message_notification.html.erb

@@ -173,11 +173,18 @@ ul li, ol li {
         <tr>
           <td>
             <p>Hi there,</p>
-            <p>You've received a new message on Georgia CMS from <%= @message.name_or_anonymous %></p>
-            <% if @message.subject.present? %><h1><%= @message.subject %></h1><% end %>
+            <p>You've received a new message on Georgia CMS.</p>
+            <% if @message.subject.present? %>
+              <h1><%= @message.subject %></h1>
+            <% end %>
             <p><%= @message.message %></p>
+            <p>Date: <%= @message.created_at.strftime('%F %H:%M:%S') %></p>
+            <p>From: <%= @message.name_or_anonymous %></p>
+            <p>URL:  <%= @message.referrer || 'unknown' %></p>
             <% if @message.phone.present? %><p><%= @message.phone %></p><% end -%>
-            <p><a href="<%= georgia.message_path(@message, only_path: false) %>" class='btn-primary'>Have a closer look</a></p>
+              <p>
+                <a href="<%= georgia.message_path(@message, only_path: false) %>" class='btn-primary'>Have a closer look</a>
+              </p>
             <p>Thanks, have a lovely day.</p>
           </td>
         </tr>

data/app/views/georgia/messages/destroy.js.erb

@@ -1,3 +1,7 @@
+<% if @status_message and @status %>
+  new Flash('<%= @status_message %>', '<%= @status %>')
+<% end %>
+
 <% @messages.each do |m| %>
   $('tr#message_<%= m.id %>').remove()
 <% end -%>

data/app/views/georgia/messages/ham.js.erb

@@ -0,0 +1,7 @@
+<% if @status_message and @status %>
+  new Flash('<%= @status_message %>', '<%= @status %>')
+<% end %>
+
+<% @messages.each do |m| %>
+  $('tr#message_<%= m.id %>').remove();
+<% end -%>

data/app/views/georgia/messages/search.html.erb

@@ -22,42 +22,60 @@
 
 <section class="body-main">
 
-  <table class="table table-hover table-results messages js-checkboxable">
-    <thead>
-      <th colspan='3'>
-        <div class="btn-group">
-          <div class="btn btn-default dropdown-toggle" data-toggle="dropdown">
-            <%= checkboxable_all_tag :messages %>
-            <span class="caret"></span>
-          </div>
-          <ul class="dropdown-menu" role="menu">
-            <li><%= link_to 'All', '#', class: 'js-select-all' %></li>
-            <li><%= link_to 'None', '#', class: 'js-select-none' %></li>
-            <li><%= link_to 'Spam', '#', class: 'js-select-spam' %></li>
-          </ul>
-        </div>
-        <%= link_to "#{icon_tag('thumbs-up')} ham".html_safe, '#', class: 'btn btn-default btn-xs js-ham disabled' %>
-        <%= link_to "#{icon_tag('thumbs-down')} spam".html_safe, '#', class: 'btn btn-default btn-xs js-spam disabled' %>
-        <%= link_to "#{icon_tag('trash-o')} trash".html_safe, '#', class: 'btn btn-default btn-xs js-delete disabled' %>
-        <%#= link_to '#', class: 'btn btn-warning' do %>
-          <%#= icon_tag('tags') %> <%#= icon_tag('caret-down') %>
-        <%# end -%>
-      </th>
-      <th colspan='3'>
-        <%= pagination_tag(@search) %>
-      </th>
-    </thead>
-    <tbody>
-      <% if @messages and @messages.any? %>
-        <%= render @messages %>
-      <% else %>
-      <tr>
-        <td colspan="6">
-          There are currently no messages.
-        </td>
-      </tr>
-      <% end %>
-    </tbody>
-  </table>
+  <div class="row">
+    <div class="col-md-1">
+      <div class="inbox-categories">
+        <%= link_to 'Inbox', search_messages_path, class: "inbox-category-link#{' active' unless params[:spam].present?}" %>
+        <%= link_to 'Spam', search_messages_path(spam: 1), class: "inbox-category-link#{' active' if params[:spam] == '1'}" %>
+        <%= link_to 'All', search_messages_path(spam: 0), class: "inbox-category-link#{' active' if params[:spam] == '0'}" %>
+      </div>
+    </div>
+    <div class="col-md-11">
+      <table class="table table-hover table-results messages js-checkboxable">
+        <thead>
+          <tr>
+            <th class='status'></th>
+            <th class='checkbox-cell'>
+              <%= checkboxable_all_tag :messages %>
+            </th>
+            <th class='actions'>
+              <% if params[:spam] == '1'  %>
+                <%= link_to "#{icon_tag('thumbs-up')} Move to Inbox".html_safe, '#', class: 'btn btn-default btn-xs js-ham disabled' %>
+              <% else %>
+                <%= link_to "#{icon_tag('thumbs-down')} Report spam".html_safe, '#', class: 'btn btn-default btn-xs js-spam disabled' %>
+              <% end %>
+              <%= link_to "#{icon_tag('trash-o')} Delete".html_safe, '#', class: 'btn btn-default btn-xs js-delete disabled' %>
+            </th>
+            <th colspan='3'>
+              <%= pagination_tag(@search) %>
+            </th>
+          </tr>
+          <% if params[:spam] == '1' and @messages and @messages.any? %>
+            <tr class='active'>
+              <th colspan='7' class='messages-destroy-spam'>
+                <%= link_to 'delete all spam messages now', destroy_all_spam_messages_path %>
+              </th>
+            </tr>
+          <% end %>
+        </thead>
+        <tbody>
+          <% if @messages and @messages.any? %>
+            <%= render @messages %>
+          <% else %>
+            <tr>
+              <td colspan="7">
+                <%# FIXME: Should work with I18n %>
+                <% if params[:spam] == '1' %>
+                  Hooray, no spam here!
+                <% else %>
+                  Your inbox is empty.
+                <% end %>
+              </td>
+            </tr>
+          <% end %>
+        </tbody>
+      </table>
+    </div>
+  </div>
 
 </section>

data/app/views/georgia/messages/show.html.erb

@@ -7,7 +7,9 @@
   <div class="header-gutter">
     <%= link_to_back search_messages_path %>
   </div>
-  <div class="header-title"></div>
+  <div class="header-title">
+    <%= message_actions_list(@message) %>
+  </div>
   <div class="header-actions">
   </div>
 </header>
@@ -16,13 +18,11 @@
   <h2><%= @message.subject %></h2>
   <div class="message-info">
     <div class="btn-group pull-right hidden-print">
-      <%= link_to "#{icon_tag('reply')} Reply".html_safe, "mailto:#{@message.email}", class: 'btn btn-warning btn-sm', target: '_blank' %>
-      <button type="button" class="btn btn-warning btn-sm dropdown-toggle" data-toggle="dropdown">
-        <span class="caret"></span>
-        <span class="sr-only">Toggle Dropdown</span>
-      </button>
-      <ul class="dropdown-menu" role="menu">
-        <%= message_actions_list(@message) %>
+      <ul class="nav">
+        <li>
+          <%= link_to "#{icon_tag('print')} Print".html_safe, "javascript:window.print()", class: 'btn btn-info btn-sm', target: '_blank' %></li>
+        <li>
+          <%= link_to "#{icon_tag('reply')} Reply".html_safe, "mailto:#{@message.email}", class: 'btn btn-warning btn-sm', target: '_blank' %></li>
       </ul>
     </div>
     <strong><%= @message.name_or_anonymous %></strong>

data/app/views/georgia/messages/spam.js.erb

@@ -0,0 +1,7 @@
+<% if @status_message and @status %>
+  new Flash('<%= @status_message %>', '<%= @status %>')
+<% end %>
+
+<% @messages.each do |m| %>
+  $('tr#message_<%= m.id %>').remove()
+<% end -%>

data/app/views/public_activity/georgia_mailer_message/_create.html.erb

@@ -0,0 +1,17 @@
+<% if activity.trackable and !activity.trackable.spam %>
+  <div id='activity_<%= activity.id %>' class="activity media">
+    <%= image_tag(avatar_url(activity.trackable.email), alt: '', class: 'media-object pull-left') %>
+    <div class="media-body">
+      <h4 class="media-heading">
+        <%= activity.trackable.name %> posted a new message <em><%= link_to activity.trackable.subject, activity.trackable %></em>
+      </h4>
+      <p>
+        <small>
+          <time class='text-muted js-time-ago' datetime="<%= activity.created_at %>">
+            About <%= time_ago_in_words(activity.created_at) %> ago
+          </time>
+        </small>
+      </p>
+    </div>
+  </div>
+<% end -%>

data/app/workers/georgia/mailer/destroy_all_spam_worker.rb

@@ -0,0 +1,18 @@
+module Georgia
+  module Mailer
+    class DestroyAllSpamWorker
+      include SuckerPunch::Job
+
+      def perform
+        ActiveRecord::Base.connection_pool.with_connection do
+          begin
+            Message.destroy_all(spam: true)
+          rescue ActiveRecord::RecordNotFound
+            Rails.logger.info "Message with ID #{message_id} was destroy before it could be processed"
+          end
+        end
+      end
+
+    end
+  end
+end

data/app/workers/georgia/mailer/email_delivery_worker.rb

@@ -0,0 +1,21 @@
+module Georgia
+  module Mailer
+    class EmailDeliveryWorker
+      include SuckerPunch::Job
+
+      def perform(message_id)
+        ActiveRecord::Base.connection_pool.with_connection do
+          message = Message.find(message_id)
+          unless message.spam or !Georgia::Mailer.turn_on_email_notifications
+            Notifier.new_message_notification(message).deliver_now
+          end
+        end
+      end
+
+      def later(sec, message_id)
+        after(sec) { perform(message_id) }
+      end
+
+    end
+  end
+end

data/app/workers/georgia/mailer/spam_worker.rb

@@ -0,0 +1,20 @@
+module Georgia
+  module Mailer
+    class SpamWorker
+      include SuckerPunch::Job
+
+      def perform(message_id)
+        ActiveRecord::Base.connection_pool.with_connection do
+          begin
+            message = Message.find(message_id)
+            is_spam = SpamCheck.new(message).call
+            message.update_attributes(spam: is_spam, verified_at: Time.zone.now)
+          rescue ActiveRecord::RecordNotFound
+            Rails.logger.info "Message with ID #{message_id} was destroy before it could be processed"
+          end
+        end
+      end
+
+    end
+  end
+end

data/config/routes.rb

@@ -1,4 +1,4 @@
-GeorgiaMailer::Engine.routes.draw do
+Georgia::Mailer::Engine.routes.draw do
 
   resources :messages, only: :create
 
@@ -12,9 +12,9 @@ Georgia::Engine.routes.draw do
       get :destroy_all_spam
     end
     member do
-      get :spam
-      get :ham
       get :resend_notification
+      post :spam
+      post :ham
     end
   end
 

data/lib/generators/georgia/mailer/install/install_generator.rb

@@ -0,0 +1,24 @@
+module Georgia
+  module Mailer
+    module Generators
+      class InstallGenerator < ::Rails::Generators::Base
+
+        def mount_engine
+          route "mount Georgia::Mailer::Engine => '/mailer'"
+        end
+
+        def run_migrations
+          rake "railties:install:migrations"
+          rake "db:migrate"
+        end
+
+        def create_index
+          Georgia::Mailer::Message.__elasticsearch__.indices.delete! Georgia::Mailer::Message.index_name rescue nil
+          Georgia::Mailer::Message.__elasticsearch__.create_index! force: true
+          Georgia::Mailer::Message.import
+        end
+
+      end
+    end
+  end
+end

data/lib/georgia/mailer/engine.rb

@@ -0,0 +1,12 @@
+module Georgia
+  module Mailer
+    class Engine < ::Rails::Engine
+      isolate_namespace Georgia::Mailer
+
+      require 'georgia'
+      require 'rakismet'
+      require 'sucker_punch'
+
+    end
+  end
+end