effective_posts 2.0.4 → 2.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5b14ecbfb59bf639f5a0950fac43873d6cf2ca7c30aa1a1c69d71250bebf769d
-  data.tar.gz: 901c1ceca8dfa718866150dab53c7e5d76b59aa29fa9bd3003b81b1b1cd153af
+  metadata.gz: 1ceb1f23b04e83df1d9e2f1392dc541e9f80631bd503b95fd787fd0a41608e06
+  data.tar.gz: 6171ca2f46ba3c5124b804fc06ddb6b27b405be2a2669fed7fc19fae095af9a5
 SHA512:
-  metadata.gz: 4543b3fcfc8068dd7eeb13002d6d793b200c74065efa960e13ce86f787eb3d2635b4d0e33a359227c5da41d4f9eea9438a147603a0329105b2efff08407d390d
-  data.tar.gz: d51323feaf466cdda3dff70816929228f8875df491af0c7054db1fb682e72816f4910773f79e0f9ceb6a4bef4cfba19259ad33f9d2dc309d459b2d963cb6f67f
+  metadata.gz: 1565992dbf8ab2e6b25ab0772bdcbc05193fb42591091cc40a05f847515e5d18416f14aa6c42afbb3afce19d4eb0fb7d9fe63514ca03c6ae0d4d22a3f4519a91
+  data.tar.gz: 9425f6f6c6ea39f60d89392dfbcf3e50d132e25da4b670d5e09851c88538eb89140e63b4d6741fb98143a8fbf21f458857f99fd6e0caac2b91b2f788949f23ed

data/app/assets/config/effective_posts_manifest.js

@@ -0,0 +1,2 @@
+//= link_directory ../javascripts .js
+//= link_directory ../stylesheets .css

data/app/controllers/admin/posts_controller.rb

@@ -5,10 +5,6 @@ module Admin
 
     include Effective::CrudController
 
-    if (config = EffectivePosts.layout)
-      layout(config.kind_of?(Hash) ? config[:admin] : config)
-    end
-
     submit :save, 'Save'
     submit :save, 'Save and Add New', redirect: :new
     submit :save, 'Save and View', redirect: -> { effective_posts.post_path(resource) }

data/app/controllers/effective/posts_controller.rb

@@ -7,10 +7,6 @@ module Effective
 
     include Effective::CrudController
 
-    if (config = EffectivePosts.layout)
-      layout(config.kind_of?(Hash) ? (config[:application] || config[:posts]) : config)
-    end
-
     def index
       @posts ||= Effective::Post.posts(
         user: current_user,
@@ -79,7 +75,7 @@ module Effective
         flash.now[:success] = 'Successfully submitted post'
 
         if EffectivePosts.submissions_require_approval
-          @post.send_post_submitted_to_admin!
+          @post.send_post_submitted!
         end
 
         render :submitted
@@ -104,12 +100,12 @@ module Effective
 
       EffectiveResources.authorize!(self, :update, @post)
 
-      if @post.update_attributes(post_params)
+      if @post.update(post_params)
         @page_title ||= 'Post Submitted'
         flash.now[:success] = 'Successfully re-submitted post'
 
         if EffectivePosts.submissions_require_approval && draft_was != true
-          @post.send_post_submitted_to_admin!
+          @post.send_post_submitted!
         end
 
         render :submitted

data/app/mailers/effective/posts_mailer.rb

@@ -1,35 +1,18 @@
 module Effective
-  class PostsMailer < ActionMailer::Base
-    helper EffectivePostsHelper
-
-    layout EffectivePosts.mailer[:layout].presence || 'effective_posts_mailer_layout'
+  class PostsMailer < EffectivePosts.parent_mailer_class
+    include EffectiveMailer
 
-    def post_submitted_to_admin(post_param)
-      @post = (post_param.kind_of?(Effective::Post) ? post_param : Effective::Post.find(post_param))
-
-      mail(
-        to: EffectivePosts.mailer[:admin_email],
-        from: EffectivePosts.mailer[:default_from],
-        subject: subject_for_post_submitted_to_admin(@post),
-        tenant: (Tenant.current if defined?(Tenant))
-      )
-    end
-
-    private
+    helper EffectivePostsHelper
 
-    def subject_for_post_submitted_to_admin(post)
-      string_or_callable = EffectivePosts.mailer[:subject_for_post_submitted_to_admin]
+    def post_submitted(resource, opts = {})
+      raise('expected an Effective::Post') unless resource.kind_of?(Effective::Post)
 
-      if string_or_callable.respond_to?(:call) # This is a Proc or a function, not a string
-        string_or_callable = self.instance_exec(post, &string_or_callable)
-      end
+      @post = resource
+      subject = subject_for(__method__, 'Post Submitted', resource, opts)
+      headers = headers_for(resource, opts)
 
-      prefix_subject(string_or_callable.presence || "A new post has been submitted that needs approval")
+      mail(to: mailer_admin, subject: subject, **headers)
     end
 
-    def prefix_subject(text)
-      prefix = (EffectivePosts.mailer[:subject_prefix].to_s rescue '')
-      prefix.present? ? (prefix.chomp(' ') + ' ' + text) : text
-    end
   end
 end

data/app/models/effective/post.rb

@@ -106,9 +106,8 @@ module Effective
       (regions.to_a.sum { |region| (region.content || '').scan(/\w+/).size } / reading_speed).seconds
     end
 
-    def send_post_submitted_to_admin!
-      deliver_method = EffectivePosts.mailer[:deliver_method] || EffectiveResources.deliver_method
-      Effective::PostsMailer.post_submitted_to_admin(to_param).send(deliver_method)
+    def send_post_submitted!
+      EffectivePosts.send_email(:post_submitted, self)
     end
 
     # Returns a duplicated post object, or throws an exception

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

@@ -7,7 +7,10 @@
     = f.text_field :slug, required: f.object.persisted?,
       hint: "The slug controls this post's internet address. Be careful, changing the slug will break links that other websites may have to the old address.<br>#{('This post is currently reachable via ' + link_to(current_url.gsub(f.object.slug, '<strong>' + f.object.slug + '</strong>').html_safe, current_url)) if current_url && f.object.slug.present? }".html_safe
 
-  = f.rich_text_area :excerpt, hint: 'Will be used for the post excerpt on index pages.'
+  - if defined?(EffectiveArticleEditor)
+    = f.article_editor :excerpt, hint: 'Will be used for the post excerpt on index pages.'
+  - else
+    = f.rich_text_area :excerpt, hint: 'Will be used for the post excerpt on index pages.'
 
   = f.text_field :description, hint: 'The content of the post meta tags.', maxlength: 150
 
@@ -19,7 +22,10 @@
   - if EffectivePosts.use_active_storage
     = f.file_field :image, hint: 'An image for your post'
 
-  = f.rich_text_area :body, hint: 'The main body of your post'
+  - if defined?(EffectiveArticleEditor)
+    = f.article_editor :body, hint: 'The main body of your post'
+  - else
+    = f.rich_text_area :body, hint: 'The main body of your post'
 
   = render partial: '/effective/posts/additional_fields', locals: { post: post, form: f, f: f }
 

data/app/views/effective/posts/_form.html.haml

@@ -8,12 +8,16 @@
   - if Array(EffectivePosts.categories).length > 1
     = f.select :category, EffectivePosts.categories
   - else
-    = f.hidden_field :category, (EffectivePosts.categories.first || 'posts')
+    = f.hidden_field :category, value: (EffectivePosts.categories.first || 'posts')
 
   - if EffectivePosts.use_active_storage
     = f.file_field :image, hint: 'An image for your post'
 
-  = f.rich_text_area :excerpt, hint: 'Will be used for the post excerpt on index pages.'
+  - if defined?(EffectiveArticleEditor)
+    = f.article_editor :excerpt, hint: 'Will be used for the post excerpt on index pages.'
+  - else
+    = f.rich_text_area :excerpt, hint: 'Will be used for the post excerpt on index pages.'
+
   = f.text_field :description, hint: 'The content of the post meta tags.', maxlength: 150
 
   = render partial: '/effective/posts/additional_fields', locals: { post: post, form: f, f: f }
@@ -22,6 +26,9 @@
 
   = f.check_box :draft, hint: 'Save this post as a draft. It will not be accessible on the website.'
 
-  = f.rich_text_area :body, hint: 'The main body of your post'
+  - if defined?(EffectiveArticleEditor)
+    = f.article_editor :body, hint: 'The main body of your post'
+  - else
+    = f.rich_text_area :body, hint: 'The main body of your post'
 
   = f.submit 'Save'

data/app/views/effective/posts/submitted.html.haml

@@ -7,4 +7,4 @@
 
 %hr
 
-= render file: '/effective/posts/show'
+= render template: 'effective/posts/show'

data/app/views/effective/posts_mailer/{post_submitted_to_admin.html.haml → post_submitted.html.haml}

@@ -1,6 +1,6 @@
 %p A new post has been submitted and needs your approval before it may be displayed on the website:
 
-%p= link_to 'Approve Post', effective_posts.admin_approve_post_path(@post)
+%p= link_to 'Approve Post', effective_posts.approve_admin_post_url(@post)
 
 %hr
 
@@ -19,4 +19,4 @@
 
 %hr
 
-%p= link_to 'Approve Post', effective_posts.admin_approve_post_path(@post)
+%p= link_to 'Approve Post', effective_posts.approve_admin_post_url(@post)

data/config/effective_posts.rb

@@ -3,7 +3,7 @@ EffectivePosts.setup do |config|
 
   # Every post must belong to one or more category.
   # Don't use the category :posts
-  config.categories = [:news, :events]
+  config.categories = [:news]
 
   # Create top level routes for each category
   # Should each of the above categories have a top level route created for it
@@ -49,31 +49,20 @@ EffectivePosts.setup do |config|
   config.submissions_require_approval = true
 
   # The Thank you message when they submit a post
-  config.submissions_note = "News & Event submitted! A confirmation email has been sent to the website owner. When approved, your submission will appear on the website."
+  config.submissions_note = "Post submitted! A confirmation email has been sent to the website owner. When approved, your submission will appear on the website."
 
   # Mailer Settings
-  # effective_posts will send the admin an email when a post is submitted
-  # For all the emails, the same :subject_prefix will be prefixed.  Leave as nil / empty string if you don't want any prefix
+  # Please see config/initializers/effective_resources.rb for default effective_* gem mailer settings
   #
-  # The subject_for_post_submitted_to_admin can be one of:
-  # - nil / empty string to use the built in defaults
-  # - A string with the full subject line for this email
-  # - A Proc to create the subject line based on the email
-  # In all three of these cases, the subject_prefix will still be used.
-
-  # The Procs are the same for admin & buyer receipt, the seller Proc is different
-  # subject_for_post_submitted_to_admin: Proc.new { |post| "Post needs approval"}
-
-  config.mailer = {
-    subject_prefix: '[example]',
-    subject_for_post_submitted_to_admin: '',
-
-    layout: 'effective_posts_mailer_layout',
-
-    default_from: 'info@example.com',
-    admin_email: 'admin@example.com',
-
-    deliver_method: :deliver_now
-  }
-
+  # Configure the class responsible to send e-mails.
+  # config.mailer = 'Effective::PostsMailer'
+  #
+  # Override effective_resource mailer defaults
+  #
+  # config.parent_mailer = nil      # The parent class responsible for sending emails
+  # config.deliver_method = nil     # The deliver method, deliver_later or deliver_now
+  # config.mailer_layout = nil      # Default mailer layout
+  # config.mailer_sender = nil      # Default From value
+  # config.mailer_admin = nil       # Default To value for Admin correspondence
+  # config.mailer_subject = nil     # Proc.new method used to customize Subject
 end

data/lib/effective_posts/engine.rb

@@ -4,8 +4,10 @@ module EffectivePosts
 
     # Include Helpers to base application
     initializer 'effective_posts.action_controller' do |app|
-      ActiveSupport.on_load :action_controller_base do
-        helper EffectivePostsHelper
+      app.config.to_prepare do
+        ActiveSupport.on_load :action_controller_base do
+          helper EffectivePostsHelper
+        end
       end
     end
 

data/lib/effective_posts/version.rb

@@ -1,3 +1,3 @@
 module EffectivePosts
-  VERSION = '2.0.4'.freeze
+  VERSION = '2.1.0'.freeze
 end

data/lib/effective_posts.rb

@@ -7,17 +7,21 @@ module EffectivePosts
   def self.config_keys
     [
       :posts_table_name, :layout, :categories,
+      :mailer, :parent_mailer, :deliver_method, :mailer_layout, :mailer_sender, :mailer_admin, :mailer_subject,
       :use_category_routes, :use_blog_routes,
       :use_effective_roles, :use_active_storage,
       :per_page, :post_meta_author,
       :submissions_enabled, :submissions_require_current_user,
-      :submissions_require_approval, :submissions_note,
-      :mailer
+      :submissions_require_approval, :submissions_note
     ]
   end
 
   include EffectiveGem
 
+  def self.mailer_class
+    mailer&.constantize || Effective::PostsMailer
+  end
+
   def self.permitted_params
     @permitted_params ||= [
       :title, :excerpt, :description, :draft, :category, :slug, :published_at, :body, :tags, :extra,

data/lib/generators/templates/effective_posts_mailer_preview.rb

@@ -3,8 +3,8 @@
 # to see a preview of the following 3 emails:
 
 class EffectivePostsMailerPreview < ActionMailer::Preview
-  def post_submitted_to_admin
-    Effective::PostsMailer.post_submitted_to_admin(build_preview_post)
+  def post_submitted
+    EffectivePosts.mailer_class.post_submitted(build_preview_post)
   end
 
   protected

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: effective_posts
 version: !ruby/object:Gem::Version
-  version: 2.0.4
+  version: 2.1.0
 platform: ruby
 authors:
 - Code and Effect
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-01-03 00:00:00.000000000 Z
+date: 2022-08-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -90,6 +90,7 @@ extra_rdoc_files: []
 files:
 - MIT-LICENSE
 - README.md
+- app/assets/config/effective_posts_manifest.js
 - app/assets/javascripts/effective_posts.js
 - app/assets/javascripts/effective_posts/additional_fields.js.coffee
 - app/assets/stylesheets/effective_posts.scss
@@ -118,8 +119,7 @@ files:
 - app/views/effective/posts/new.html.haml
 - app/views/effective/posts/show.html.haml
 - app/views/effective/posts/submitted.html.haml
-- app/views/effective/posts_mailer/post_submitted_to_admin.html.haml
-- app/views/layouts/effective_posts_mailer_layout.html.haml
+- app/views/effective/posts_mailer/post_submitted.html.haml
 - config/effective_posts.rb
 - config/routes.rb
 - db/migrate/01_create_effective_posts.rb.erb

data/app/views/layouts/effective_posts_mailer_layout.html.haml

@@ -1,7 +0,0 @@
-!!!
-%html{style: 'background: #fff;'}
-  %head
-    %meta{:content => 'text/html; charset=UTF-8', 'http-equiv' => 'Content-Type'}
-
-  %body{style: 'background: #fff;'}
-    = yield