effective_email_templates 1.0.7 → 1.0.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7f32ae5d86811f78123ceee885529f4dccd62c61031b9504400d984804af7cb1
-  data.tar.gz: a56f822ae7376f5f751721d6a26a5cfdf50c6fd2dceafe3016a0334002275bd4
+  metadata.gz: 4928f9db162ceb9b1b35fcbe367f38689be1f8977f75e46bff4e734c15f7ec90
+  data.tar.gz: 9129d4a60362284332fc2ab16685f0c325846dfe6f199cdd1e3924c974e3fb6e
 SHA512:
-  metadata.gz: f5096521a7df9bb7b3da61b121dabfdbf2b085ada6bcae15af9a0e1145b0e59d69c699a62ab9dfdf10cbe1fac444b8292b839cd833528bc07a11b5f5b6d23ed2
-  data.tar.gz: 279d24d7de7b3cedd6da782fa609acb8200dc666b615a0573f7b8bcc17af7be78f177b4a2a70b60adbf54fc5d78c2d933e88847b5bebe0f7569b916e791b0816
+  metadata.gz: 963de88d3ca0956517777a2445018ecf36bece45bf40165007403b49c347ed4a217d3d6438518318cedb25354694473639da50845cab41cd29173575192acd8c
+  data.tar.gz: 06f6f69fea13c1405bb5e93576ed5779ca412db1955b7543effa2ced254a4af7e781154df370ae97cba887dfb962f29ca424d140040213b3442388f17e640d39

data/app/datatables/effective_email_templates_datatable.rb

@@ -14,6 +14,8 @@ class EffectiveEmailTemplatesDatatable < Effective::Datatable
     col :subject
     col :body
 
+    col :content_type, visible: false
+
     actions_col partial: '/admin/email_templates/actions', partial_as: 'email_template'
   end
 
@@ -21,4 +23,3 @@ class EffectiveEmailTemplatesDatatable < Effective::Datatable
     Effective::EmailTemplate.all
   end
 end
-

data/app/helpers/effective_email_templates_helper.rb

@@ -1,11 +1,20 @@
 module EffectiveEmailTemplatesHelper
 
-  def effective_email_review_fields(form, template_name)
+  # We are given a form to essentially send an email
+  def effective_email_review_fields(form, template_name, mail = nil)
+    raise('expected a mail object') if mail && !mail.kind_of?(ActionMailer::MessageDelivery)
+    raise('expected form.object to respond to email_review') unless form.object.respond_to?(:email_review)
+
     email_review = form.object.email_review
 
     unless email_review&.template_name == template_name.to_s
       email_template = Effective::EmailTemplate.where(template_name: template_name).first!
       email_review = Effective::EmailReview.build(email_template: email_template)
+
+      if mail.present?
+        email_review.body = mail.message.body
+        email_review.subject = mail.message.subject
+      end
     end
 
     render(partial: 'effective/email_reviews/fields', locals: { email_review: email_review, form: form })

data/app/models/effective/email_review.rb

@@ -4,25 +4,32 @@ module Effective
 
     attr_accessor :email_template
     attr_accessor :template_name
+
     attr_accessor :body
+    attr_accessor :subject
+    attr_accessor :from
+    attr_accessor :cc
+    attr_accessor :bcc
 
     def self.build(attributes = {})
-      new(attributes).tap do |email_review|
-        email_review.body ||= email_review.email_template&.body
-        email_review.template_name ||= email_review.email_template&.template_name
-      end
-    end
+      email_review = new(attributes)
+      template = email_review.email_template
 
-    validates :body, presence: true
-
-    validate(if: -> { body.present? }) do
-      begin
-        Liquid::Template.parse(body)
-      rescue Liquid::SyntaxError => e
-        errors.add(:body, e.message)
+      if template.present?
+        email_review.body ||= template.body
+        email_review.subject ||= template.subject
+        email_review.from ||= template.from
+        email_review.cc ||= template.cc
+        email_review.bcc ||= template.bcc
+        email_review.template_name ||= template.template_name
       end
+
+      email_review
     end
 
+    validates :body, presence: true, liquid: true
+    validates :subject, liquid: true
+
     def email_template
       @email_template ||= Effective::EmailTemplate.where(template_name: template_name).first
     end

data/app/models/effective/email_template.rb

@@ -22,21 +22,8 @@ module Effective
       self.content_type ||= CONTENT_TYPES.first
     end
 
-    before_validation(if: -> { body.present? }) do
-      begin
-        Liquid::Template.parse(body)
-      rescue Liquid::SyntaxError => e
-        errors.add(:body, e.message)
-      end
-    end
-
-    before_validation(if: -> { subject.present? }) do
-      begin
-        Liquid::Template.parse(subject)
-      rescue Liquid::SyntaxError => e
-        errors.add(:subject, e.message)
-      end
-    end
+    validates :body, liquid: true
+    validates :subject, liquid: true
 
     validates :subject, presence: true
     validates :from, presence: true
@@ -44,6 +31,20 @@ module Effective
     validates :content_type, presence: true, inclusion: { in: CONTENT_TYPES }
     validates :template_name, presence: true
 
+    # validate(if: -> { content_type == 'text/html' && body.present? }) do
+    #   unless body.include?('<') && body.include?('>')
+    #     self.errors.add(:content_type, 'expected html tags in body')
+    #     self.errors.add(:body, 'expected html tags in body')
+    #   end
+    # end
+
+    # validate(if: -> { content_type == 'text/plain' && body.present? }) do
+    #   if body.include?('</a>') || body.include?('</p>')
+    #     self.errors.add(:content_type, 'expected no html tags in body')
+    #     self.errors.add(:body, 'expected no html tags in body')
+    #   end
+    # end
+
     def to_s
       template_name.presence || 'New Email Template'
     end
@@ -51,14 +52,14 @@ module Effective
     def render(assigns = {})
       assigns = deep_stringify_assigns(assigns)
 
-      {
+      result = {
         from: from,
-        cc: cc.presence || false,
-        bcc: bcc.presence || false,
+        cc: cc.presence,
+        bcc: bcc.presence,
         content_type: content_type,
         subject: template_subject.render(assigns),
         body: template_body.render(assigns)
-      }
+      }.compact
     end
 
     def template_variables
@@ -72,11 +73,11 @@ module Effective
     private
 
     def template_body
-      Liquid::Template.parse(body)
+      Liquid::Template.parse(body_was || body)
     end
 
     def template_subject
-      Liquid::Template.parse(subject)
+      Liquid::Template.parse(subject_was || subject)
     end
 
     def deep_stringify_assigns(assigns)

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

@@ -1,6 +1,8 @@
 = effective_form_with(model: email_template, url: email_template.persisted? ? effective_email_templates.admin_email_template_path(email_template.id) : effective_email_templates.admin_email_templates_path) do |f|
   = f.static_field :template_name, label: 'Name'
-  = f.select :content_type, Effective::EmailTemplate::CONTENT_TYPES
+
+  - 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 :cc

data/config/effective_email_templates.rb

@@ -32,4 +32,7 @@ EffectiveEmailTemplates.setup do |config|
     admin_email_templates: 'admin'
   }
 
+  # Not allowed to select text/html by default
+  config.select_content_type = false
+
 end

data/lib/effective_email_templates.rb

@@ -6,6 +6,7 @@ module EffectiveEmailTemplates
 
   mattr_accessor :email_templates_table_name
   mattr_accessor :authorization_method
+  mattr_accessor :select_content_type
   mattr_accessor :layout
 
   def self.setup

data/lib/effective_email_templates/engine.rb

@@ -2,6 +2,9 @@ module EffectiveEmailTemplates
   class Engine < ::Rails::Engine
     engine_name 'effective_email_templates'
 
+    config.autoload_paths += Dir["#{config.root}/lib/validators/"]
+    config.eager_load_paths += Dir["#{config.root}/lib/validators/"]
+
     # Set up our default configuration options.
     initializer 'effective_email_templates.defaults', before: :load_config_initializers do |app|
       eval File.read("#{config.root}/config/effective_email_templates.rb")
@@ -16,4 +19,3 @@ module EffectiveEmailTemplates
 
   end
 end
-

data/lib/effective_email_templates/importer.rb

@@ -9,16 +9,18 @@ module EffectiveEmailTemplates
     end
 
     def execute(overwrite:, quiet: false)
-      Dir[Rails.root.join('app', 'views', '**', '*.liquid')].each do |filepath|
-        name = File.basename(filepath, '.liquid')
-        email_template = Effective::EmailTemplate.find_or_initialize_by(template_name: name)
+      ActionController::Base.view_paths.map(&:path).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)
 
-        if email_template.persisted? && !overwrite
-          puts("SKIPPED #{filename(filepath)}") unless quiet
-          next
-        end
+          if email_template.persisted? && !overwrite
+            puts("SKIPPED #{filename(filepath)}") unless quiet
+            next
+          end
 
-        save(email_template, filepath, quiet: quiet)
+          save(email_template, filepath, quiet: quiet)
+        end
       end
     end
 

data/lib/effective_email_templates/version.rb

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

data/lib/validators/liquid_validator.rb

@@ -0,0 +1,15 @@
+# An ActiveRecord validator for any liquid field that you would use with effective_email_templates or otherwise
+#
+# validates :body, liquid: true
+
+class LiquidValidator < ActiveModel::EachValidator
+  def validate_each(record, attribute, value)
+    if value.present?
+      begin
+        Liquid::Template.parse(value)
+      rescue Liquid::SyntaxError => e
+        record.errors.add(attribute, e.message)
+      end
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: effective_email_templates
 version: !ruby/object:Gem::Version
-  version: 1.0.7
+  version: 1.0.12
 platform: ruby
 authors:
 - Code and Effect
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-04-17 00:00:00.000000000 Z
+date: 2021-01-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -115,11 +115,12 @@ files:
 - lib/effective_email_templates/version.rb
 - lib/generators/effective_email_templates/install_generator.rb
 - lib/tasks/effective_email_templates_tasks.rake
+- lib/validators/liquid_validator.rb
 homepage: https://github.com/code-and-effect/effective_email_templates
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -134,8 +135,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.1.2
+signing_key:
 specification_version: 4
 summary: Effective Email Templates provides an admin access to modify email templates
 test_files: []