effective_email_templates 1.0.13 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1ea4dc4be1510a05879a7ff2c084a17126b4d8bcaa228c693a7cb8a4093a2ffc
-  data.tar.gz: 74936efb073fc46f09e04d8543858c2a667e4737eeccc86b64882d1f5d9cf402
+  metadata.gz: 42f8567bcf6351ab1762ee602ac678e4b63e8bbde2ae4e2c2858a7eff70a2a8d
+  data.tar.gz: df20ca47c9a70a8a4867dc961da769db6f49a490fe80756e3fe9948b75dba5cc
 SHA512:
-  metadata.gz: 9cefe7ba9a4d3da5a15e4f1e7429167e7ff1d3570f2b381f5ca2c4a2f1379066748a6b0436868099badf1eb3ea178ba347f2d7889ea170b1108e14c80f0d1d61
-  data.tar.gz: d1e69f8c657c791cc2af22be8ef22fb5a0c4acecb1670727fc9e83b6e6de0ad0f3c9972f8990c3251b0b569148a23c763d64157a55a2ccb6c850bc11f969b200
+  metadata.gz: c17a239840a85a12f9768b84be0315c7d0fddf924b435809af78bad8e203b145de905d40eef06d0473a18041e5483187deb4fc4db61df01428317d7c678a9974
+  data.tar.gz: f10cfa33b6973e58b2b9e82a33c019b5ef7b8f532f23048682a6367e8e654304d61556c2a3a4e5de9114536d8a36deff4150301f68a107d462b0b3d47d1fd1c8

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,24 @@ 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.cc)
+    end
+
     col :subject
     col :body
-
     col :content_type, visible: false
 
-    actions_col partial: '/admin/email_templates/actions', partial_as: 'email_template'
+    actions_col
   end
 
   collection do

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

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.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

data/lib/effective_email_templates/version.rb

@@ -1,3 +1,3 @@
 module EffectiveEmailTemplates
-  VERSION = '1.0.13'.freeze
+  VERSION = '1.1.0'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: effective_email_templates
 version: !ruby/object:Gem::Version
-  version: 1.0.13
+  version: 1.1.0
 platform: ruby
 authors:
 - Code and Effect
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-01-29 00:00:00.000000000 Z
+date: 2021-06-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -94,13 +94,9 @@ files:
 - app/helpers/effective_email_templates_helper.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