effective_email_templates 1.0.12 → 1.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4928f9db162ceb9b1b35fcbe367f38689be1f8977f75e46bff4e734c15f7ec90
-  data.tar.gz: 9129d4a60362284332fc2ab16685f0c325846dfe6f199cdd1e3924c974e3fb6e
+  metadata.gz: 27a4c6ef0659bba64c76e45a9363fe649d2316ac988ef56bcd3fae6e87efe13f
+  data.tar.gz: effe7e9aa525fae2923b0e195ea3ec81ed492944e2cf7616f3db7af58197218d
 SHA512:
-  metadata.gz: 963de88d3ca0956517777a2445018ecf36bece45bf40165007403b49c347ed4a217d3d6438518318cedb25354694473639da50845cab41cd29173575192acd8c
-  data.tar.gz: 06f6f69fea13c1405bb5e93576ed5779ca412db1955b7543effa2ced254a4af7e781154df370ae97cba887dfb962f29ca424d140040213b3442388f17e640d39
+  metadata.gz: a31ed5dba96c21b94f282faf4023e66de6fd809e1f15ffdaf22d6179d158d0c82c01535fced61ca074d770b02ab5c9188919bca703a00ff5ae982cdf8f89d3c0
+  data.tar.gz: 8f5cf72c409cc8d4ef3a52ef12068af275963d7e1ee5b66efd96268b812df598c794eaea5fa612e8e314fe54a8a584fed7dd43216b07e7db864e9f163f21dc82

data/MIT-LICENSE

@@ -1,4 +1,4 @@
-Copyright 2020 Code and Effect Inc.
+Copyright 2021 Code and Effect Inc.
 
 Permission is hereby granted, free of charge, to any person obtaining
 a copy of this software and associated documentation files (the

data/README.md

@@ -114,67 +114,21 @@ EmailTemplatesMailer.welcome(user).deliver
 
 ## Authorization
 
-All authorization checks are handled via the config.authorization_method found in the `app/config/initializers/effective_email_templates.rb` file.
+All authorization checks are handled via the effective_resources gem found in the `config/initializers/effective_resources.rb` file.
 
-It is intended for flow through to CanCan or Pundit, but neither of those gems are required.
-
-This method is called by all controller actions with the appropriate action and resource
-
-Action will be one of [:index, :show, :new, :create, :edit, :update, :destroy]
-
-Resource will the appropriate Effective::EmailTemplate object or class
-
-The authorization method is defined in the initializer file:
-
-```ruby
-# As a Proc (with CanCan)
-config.authorization_method = Proc.new { |controller, action, resource| authorize!(action, resource) }
-```
-
-```ruby
-# As a Custom Method
-config.authorization_method = :my_authorization_method
-```
-
-and then in your application_controller.rb:
-
-```ruby
-def my_authorization_method(action, resource)
-  current_user.is?(:admin) || EffectivePunditPolicy.new(current_user, resource).send('#{action}?')
-end
-```
-
-or disabled entirely:
-
-```ruby
-config.authorization_method = false
-```
-
-If the method or proc returns false (user is not authorized) an Effective::AccessDenied exception will be raised
-
-You can rescue from this exception by adding the following to your application_controller.rb:
-
-```ruby
-rescue_from Effective::AccessDenied do |exception|
-  respond_to do |format|
-    format.html { render 'static_pages/access_denied', :status => 403 }
-    format.any { render :text => 'Access Denied', :status => 403 }
-  end
-end
-```
 
 ### Permissions
 
 To allow a user to see the admin area, using CanCan:
 
 ```ruby
-can :manage, Effective::EmailTemplate
+can [:index, :edit, :update, :destroy], Effective::EmailTemplate
 can :admin, :effective_email_templates
 ```
 
 ## License
 
-MIT License.  Copyright [Code and Effect Inc.](http://www.codeandeffect.com/)
+MIT License. Copyright [Code and Effect Inc.](http://www.codeandeffect.com/)
 
 
 ## Contributing

data/app/controllers/admin/email_templates_controller.rb

@@ -1,87 +1,19 @@
 module Admin
   class EmailTemplatesController < ApplicationController
-    before_action :authenticate_user! if respond_to?(:authenticate_user!)
+    before_action(:authenticate_user!) if defined?(Devise)
+    before_action { EffectiveResources.authorize!(self, :admin, :effective_email_templates) }
 
-    layout (EffectiveEmailTemplates.layout.kind_of?(Hash) ? EffectiveEmailTemplates.layout[:admin_email_templates] : EffectiveEmailTemplates.layout)
+    include Effective::CrudController
 
-    def index
-      @datatable = EffectiveEmailTemplatesDatatable.new
-      @page_title = 'Email Templates'
-
-      authorize_effective_email_templates!
-    end
-
-    def new
-      @email_template = Effective::EmailTemplate.new
-      @page_title = 'New Email Template'
-
-      authorize_effective_email_templates!
-    end
-
-    def create
-      @email_template = Effective::EmailTemplate.new(email_template_params)
-      @page_title = 'New Email Template'
-
-      authorize_effective_email_templates!
-
-      if @email_template.save
-        flash[:success] = 'Successfully created email template'
-        redirect_to effective_email_templates.admin_email_templates_path
-      else
-        flash.now[:danger] = 'Unable to create email template'
-        render :new
-      end
-    end
-
-    def edit
-      @email_template =
-        Effective::EmailTemplate.where(id: params[:id]).or(
-          Effective::EmailTemplate.where(template_name: params[:id])
-        ).first!
-
-      @page_title = 'Edit Email Template'
-
-      authorize_effective_email_templates!
+    if (config = EffectiveEmailTemplates.layout)
+      layout(config.kind_of?(Hash) ? config[:admin] : config)
     end
 
-    def update
-      @email_template = Effective::EmailTemplate.find(params[:id])
-      @page_title = 'Edit Email Template'
-
-      authorize_effective_email_templates!
-
-      if @email_template.update(email_template_params)
-        flash[:success] = 'Successfully updated email template'
-        redirect_to effective_email_templates.admin_email_templates_path
-      else
-        flash.now[:danger] = 'Unable to update email template'
-        render :edit
-      end
-    end
-
-    def destroy
-      @email_template = Effective::EmailTemplate.find(params[:id])
-
-      authorize_effective_email_templates!
-
-      if @email_template.destroy
-        flash[:success] = 'Successfully deleted email template'
-      else
-        flash[:danger] = 'Unable to delete email template'
-      end
-
-      redirect_to effective_email_templates.admin_email_templates_path
-    end
-
-    private
-
-    def authorize_effective_email_templates!
-      EffectiveEmailTemplates.authorize!(self, :admin, :effective_email_templates)
-      EffectiveEmailTemplates.authorize!(self, action_name.to_sym, @email_template || Effective::EmailTemplate)
-    end
+    submit :save, 'Save'
+    submit :save, 'Save and Add New', redirect: :new
 
     def email_template_params
-      params.require(:effective_email_template).permit(EffectiveEmailTemplates.permitted_params)
+      params.require(:effective_email_template).permit!
     end
 
   end

data/app/datatables/effective_email_templates_datatable.rb

@@ -8,15 +8,28 @@ class EffectiveEmailTemplatesDatatable < Effective::Datatable
     col :id, visible: false
 
     col :template_name, label: 'Name'
-    col :from
-    col :cc
-    col :bcc
+
+    col :from do |email_template|
+      html_escape(email_template.from)
+    end
+
+    col :cc do |email_template|
+      html_escape(email_template.cc)
+    end
+
+    col :bcc do |email_template|
+      html_escape(email_template.bcc)
+    end
+
     col :subject
-    col :body
+
+    col :body do |email_template|
+      simple_format(email_template.body)
+    end
 
     col :content_type, visible: false
 
-    actions_col partial: '/admin/email_templates/actions', partial_as: 'email_template'
+    actions_col
   end
 
   collection do

data/app/mailers/concerns/effective_email_templates_mailer.rb

@@ -0,0 +1,35 @@
+# HasOneEmailReview
+# Allows any model to easily review an email template and make changes to the body
+
+module EffectiveEmailTemplatesMailer
+  extend ActiveSupport::Concern
+
+  def mail(headers = {}, &block)
+    email_template = Effective::EmailTemplate.where(template_name: action_name).first!
+    assigns = (@assigns || {})
+
+    # Parse assigns. Special keys for body and subject.
+    body = assigns.delete(:body) || headers[:body] || headers['body']
+    email_template.body = body if body.present?
+
+    subject = assigns.delete(:subject) || headers[:subject] || headers['subject']
+    email_template.subject = subject if subject.present?
+
+    # Add any _url helpers
+    assigns = route_url_assigns(email_template).merge(assigns)
+
+    # Render from the template, possibly with updated body
+    rendered = email_template.render(assigns)
+
+    super(rendered.merge(headers.except(:body, :subject, 'body', 'subject')))
+  end
+
+  private
+
+  def route_url_assigns(email_template)
+    email_template.template_variables.select { |name| name.ends_with?('_url') }.inject({}) do |h, name|
+      h[name] = public_send(name) if respond_to?(name); h
+    end
+  end
+
+end

data/app/mailers/effective/email_templates_mailer.rb

@@ -1,31 +1,5 @@
 module Effective
   class EmailTemplatesMailer < ::ActionMailer::Base
-
-    def mail(headers = {}, &block)
-      email_template = Effective::EmailTemplate.where(template_name: action_name).first!
-
-      # Parse Assigns. :body is a special key
-      assigns = (@assigns || {})
-
-      if (body = assigns.delete(:body))
-        email_template.body = body
-      end
-
-      assigns = route_url_assigns(email_template).merge(assigns)
-
-      # Render from the template, possibly with updated body
-      rendered = email_template.render(assigns)
-
-      super(rendered.merge(headers))
-    end
-
-    private
-
-    def route_url_assigns(email_template)
-      email_template.template_variables.select { |name| name.ends_with?('_url') }.inject({}) do |h, name|
-        h[name] = public_send(name) if respond_to?(name); h
-      end
-    end
-
+    include EffectiveEmailTemplatesMailer
   end
 end

data/app/models/effective/email_template.rb

@@ -2,21 +2,24 @@ module Effective
   class EmailTemplate < ActiveRecord::Base
     self.table_name = EffectiveEmailTemplates.email_templates_table_name.to_s
 
+    attr_accessor :current_user
+
     log_changes if respond_to?(:log_changes)
 
     CONTENT_TYPES = ['text/plain', 'text/html']
 
-    # Attributes
-    # subject           :string
-    # from              :string
-    # cc                :string
-    # bcc               :string
-    # body              :text
-    # content_type      :string
-    #
-    # template_name     :string
-    #
-    # timestamps
+    effective_resource do
+      template_name     :string
+      content_type      :string
+
+      subject           :string
+      from              :string
+      cc                :string
+      bcc               :string
+      body              :text
+
+      timestamps
+    end
 
     before_validation do
       self.content_type ||= CONTENT_TYPES.first
@@ -26,7 +29,7 @@ module Effective
     validates :subject, liquid: true
 
     validates :subject, presence: true
-    validates :from, presence: true
+    validates :from, presence: true, email: true
     validates :body, presence: true
     validates :content_type, presence: true, inclusion: { in: CONTENT_TYPES }
     validates :template_name, presence: true
@@ -73,11 +76,11 @@ module Effective
     private
 
     def template_body
-      Liquid::Template.parse(body_was || body)
+      Liquid::Template.parse(body)
     end
 
     def template_subject
-      Liquid::Template.parse(subject_was || subject)
+      Liquid::Template.parse(subject)
     end
 
     def deep_stringify_assigns(assigns)

data/app/views/admin/email_templates/_form.html.haml

@@ -4,7 +4,7 @@
   - if EffectiveEmailTemplates.select_content_type
     = f.select :content_type, Effective::EmailTemplate::CONTENT_TYPES
 
-  = f.email_field :from, hint: 'Whom the email will be sent from'
+  = f.text_field :from, hint: 'Whom the email will be sent from'
   = f.text_field :cc
   = f.text_field :bcc
 

data/config/effective_email_templates.rb

@@ -2,37 +2,9 @@ EffectiveEmailTemplates.setup do |config|
   # Configure Database Tables
   config.email_templates_table_name = :email_templates
 
-  # Authorization Method
-  #
-  # This method is called by all controller actions with the appropriate action and resource
-  # If the method returns false, an Effective::AccessDenied Error will be raised (see README.md for complete info)
-  #
-  # Use via Proc (and with CanCan):
-  # config.authorization_method = Proc.new { |controller, action, resource| can?(action, resource) }
-  #
-  # Use via custom method:
-  # config.authorization_method = :my_authorization_method
-  #
-  # And then in your application_controller.rb:
-  #
-  # def my_authorization_method(action, resource)
-  #   current_user.is?(:admin)
-  # end
-  #
-  # Or disable the check completely:
-  # config.authorization_method = false
-  config.authorization_method = Proc.new { |controller, action, resource| authorize!(action, resource) } # CanCanCan
-
   # Layout Settings
-  # Configure the Layout per controller, or all at once
-
-  # config.layout = 'application'   # All EffectiveEmailTemplates controllers will use this layout
-  config.layout = {
-    email_templates: 'application',
-    admin_email_templates: 'admin'
-  }
+  # config.layout = { application: 'application', admin: 'admin' }
 
   # Not allowed to select text/html by default
   config.select_content_type = false
-
 end

data/lib/effective_email_templates/importer.rb

@@ -1,15 +1,19 @@
 module EffectiveEmailTemplates
   class Importer
-    def self.import(quiet: false)
-      new().execute(overwrite: false, quiet: quiet)
+    def self.import(quiet: false, paths: nil)
+      new().execute(overwrite: false, paths: paths, quiet: quiet)
     end
 
-    def self.overwrite(quiet: false)
-      new().execute(overwrite: true, quiet: quiet)
+    def self.overwrite(quiet: false, paths: nil)
+      new().execute(overwrite: true, paths: paths, quiet: quiet)
     end
 
-    def execute(overwrite:, quiet: false)
-      ActionController::Base.view_paths.map(&:path).each do |path|
+    def execute(overwrite:, paths: nil, quiet: false)
+      return false unless ActiveRecord::Base.connection.table_exists?(EffectiveEmailTemplates.email_templates_table_name)
+
+      paths ||= ActionController::Base.view_paths.map(&:path)
+
+      paths.each do |path|
         Dir[Rails.root.join(path, '**', '*_mailer/', '*.liquid')].each do |filepath|
           name = File.basename(filepath, '.liquid')
           email_template = Effective::EmailTemplate.find_or_initialize_by(template_name: name)

data/lib/effective_email_templates/version.rb

@@ -1,3 +1,3 @@
 module EffectiveEmailTemplates
-  VERSION = '1.0.12'.freeze
+  VERSION = '1.1.2'.freeze
 end

data/lib/effective_email_templates.rb

@@ -1,37 +1,18 @@
-require "liquid"
-require "effective_email_templates/engine"
-require "effective_email_templates/version"
+require 'liquid'
+require 'effective_resources'
+require 'effective_email_templates/engine'
+require 'effective_email_templates/version'
 
 module EffectiveEmailTemplates
 
-  mattr_accessor :email_templates_table_name
-  mattr_accessor :authorization_method
-  mattr_accessor :select_content_type
-  mattr_accessor :layout
-
-  def self.setup
-    yield self
-  end
-
-  def self.authorized?(controller, action, resource)
-    @_exceptions ||= [Effective::AccessDenied, (CanCan::AccessDenied if defined?(CanCan)), (Pundit::NotAuthorizedError if defined?(Pundit))].compact
-
-    return !!authorization_method unless authorization_method.respond_to?(:call)
-    controller = controller.controller if controller.respond_to?(:controller)
-
-    begin
-      !!(controller || self).instance_exec((controller || self), action, resource, &authorization_method)
-    rescue *@_exceptions
-      false
-    end
+  def self.config_keys
+    [:email_templates_table_name, :select_content_type, :layout]
   end
 
-  def self.authorize!(controller, action, resource)
-    raise Effective::AccessDenied.new('Access Denied', action, resource) unless authorized?(controller, action, resource)
-  end
+  include EffectiveGem
 
   def self.permitted_params
-    [:from, :bcc, :cc, :subject, :body, :content_type]
+    @permitted_params ||= [:from, :bcc, :cc, :subject, :body, :content_type]
   end
 
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: effective_email_templates
 version: !ruby/object:Gem::Version
-  version: 1.0.12
+  version: 1.1.2
 platform: ruby
 authors:
 - Code and Effect
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-01-28 00:00:00.000000000 Z
+date: 2022-01-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -92,15 +92,12 @@ files:
 - app/controllers/admin/email_templates_controller.rb
 - app/datatables/effective_email_templates_datatable.rb
 - app/helpers/effective_email_templates_helper.rb
+- app/mailers/concerns/effective_email_templates_mailer.rb
 - app/mailers/effective/email_templates_mailer.rb
 - app/models/concerns/has_one_email_review.rb
-- app/models/effective/access_denied.rb
 - app/models/effective/email_review.rb
 - app/models/effective/email_template.rb
-- app/views/admin/email_templates/_actions.html.haml
 - app/views/admin/email_templates/_form.html.haml
-- app/views/admin/email_templates/edit.html.haml
-- app/views/admin/email_templates/index.html.haml
 - app/views/effective/email_reviews/_fields.html.haml
 - app/views/layouts/effective_email_templates_mailer_layout.html.haml
 - config/effective_email_templates.rb

data/app/models/effective/access_denied.rb

@@ -1,17 +0,0 @@
-unless defined?(Effective::AccessDenied)
-  module Effective
-    class AccessDenied < StandardError
-      attr_reader :action, :subject
-
-      def initialize(message = nil, action = nil, subject = nil)
-        @message = message
-        @action = action
-        @subject = subject
-      end
-
-      def to_s
-        @message || I18n.t(:'unauthorized.default', :default => 'Access Denied')
-      end
-    end
-  end
-end

data/app/views/admin/email_templates/_actions.html.haml

@@ -1,5 +0,0 @@
-= dropdown(variation: :dropleft) do
-  = dropdown_link_to 'Edit', effective_email_templates.edit_admin_email_template_path(email_template)
-
-  = dropdown_link_to "Delete #{email_template}", effective_email_templates.admin_email_template_path(email_template),
-    data: { method: :delete, confirm: "Really delete #{email_template}?" }

data/app/views/admin/email_templates/edit.html.haml

@@ -1,3 +0,0 @@
-%h1.effective-admin-heading= @page_title
-
-= render partial: 'form', locals: { email_template: @email_template }

data/app/views/admin/email_templates/index.html.haml

@@ -1,3 +0,0 @@
-%h1.effective-admin-heading= @page_title
-
-= render_datatable @datatable