publify_core 9.0.0 → 9.2.2

data/app/models/article/factory.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class Article::Factory
   attr_reader :blog, :user
 
@@ -10,13 +12,14 @@ class Article::Factory
     blog.articles.build.tap do |art|
       art.allow_comments = blog.default_allow_comments
       art.allow_pings = blog.default_allow_pings
-      art.text_filter = user.default_text_filter
-      art.state = 'draft'
+      art.text_filter_name = default_text_filter
+      art.state = "draft"
     end
   end
 
   def get_or_build_from(id)
     return blog.articles.find(id) if id.present?
+
     default
   end
 
@@ -31,10 +34,10 @@ class Article::Factory
   end
 
   def extract_params(path, format)
-    specs = format.split('/')
-    specs.delete('')
-    parts = path.split('/')
-    parts.delete('')
+    specs = format.split("/")
+    specs.delete("")
+    parts = path.split("/")
+    parts.delete("")
 
     return if parts.length != specs.length
 
@@ -45,6 +48,7 @@ class Article::Factory
         after_format = Regexp.last_match[3]
         item =~ /^#{before_format}(.*)#{after_format}$/
         break unless Regexp.last_match
+
         value = Regexp.last_match[1]
         result[format_string.to_sym] = value
       elsif spec != item
@@ -53,4 +57,8 @@ class Article::Factory
       result
     end
   end
+
+  def default_text_filter
+    user.text_filter_name || blog.text_filter
+  end
 end

data/app/models/blog.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # The Blog class represents the one and only blog.  It stores most
 # configuration settings and is linked to most of the assorted content
 # classes via has_many.
@@ -5,7 +7,7 @@
 # Once upon a time, there were plans to make publify handle multiple blogs,
 # but it never happened and publify is now firmly single-blog.
 #
-class Blog < ActiveRecord::Base
+class Blog < ApplicationRecord
   include ConfigManager
   include Rails.application.routes.url_helpers
 
@@ -13,122 +15,126 @@ class Blog < ActiveRecord::Base
   has_many :articles
   has_many :feedback, through: :articles
 
-  has_many :published_articles, ->() { published }, class_name: 'Article'
+  has_many :published_articles, ->() { published }, class_name: "Article"
 
   has_many :pages
   has_many :tags
   has_many :notes
 
   has_many :redirects
-  has_many :sidebars, ->() { order('active_position ASC') }
+  has_many :sidebars, ->() { order("active_position ASC") }
 
   attr_accessor :custom_permalink
 
-  default_scope -> { order('id') }
+  default_scope -> { order("id") }
 
   validates :blog_name, presence: true
 
   serialize :settings, Hash
 
   # Description
-  setting :blog_name, :string, 'My Shiny Weblog!'
-  setting :blog_subtitle, :string, ''
-  setting :geourl_location, :string, ''
-  setting :canonical_server_url, :string, '' # Deprecated
-  setting :lang, :string, 'en_US'
-  setting :title_prefix, :integer, 0 # Deprecated but needed for a migration
+  setting :blog_name, :string, "My Shiny Weblog!"
+  setting :blog_subtitle, :string, ""
+  setting :geourl_location, :string, ""
+  setting :canonical_server_url, :string, "" # Deprecated
+  setting :lang, :string, "en_US"
 
   # Spam
   setting :sp_global, :boolean, false
   setting :sp_article_auto_close, :integer, 0
   setting :sp_url_limit, :integer, 0
-  setting :sp_akismet_key, :string, ''
+  setting :sp_akismet_key, :string, ""
   setting :use_recaptcha, :boolean, false
 
   # Mostly Behaviour
-  setting :text_filter, :string, 'markdown smartypants'
-  setting :comment_text_filter, :string, 'markdown smartypants'
+  setting :text_filter, :string, "markdown smartypants"
+  setting :comment_text_filter, :string, "markdown smartypants"
   setting :limit_article_display, :integer, 10
   setting :limit_archives_display, :integer, 20
   setting :limit_rss_display, :integer, 10
   setting :default_allow_pings, :boolean, false
   setting :default_allow_comments, :boolean, true
   setting :default_moderate_comments, :boolean, false
-  setting :link_to_author, :boolean, false
-  setting :show_extended_on_rss, :boolean, true # deprecated but still needed for backward compatibility
+  # deprecated but still needed for backward compatibility
+  setting :show_extended_on_rss, :boolean, true
   setting :hide_extended_on_rss, :boolean, false
-  setting :theme, :string, 'plain'
-  setting :plugin_avatar, :string, ''
+  setting :theme, :string, "plain"
+  setting :plugin_avatar, :string, ""
   setting :global_pings_disable, :boolean, false
   setting :send_outbound_pings, :boolean, true
-  setting :email_from, :string, 'publify@example.com'
+  setting :email_from, :string, "publify@example.com"
   setting :allow_signup, :integer, 0
-  setting :date_format, :string, '%d/%m/%Y'
-  setting :time_format, :string, '%Hh%M'
+  setting :date_format, :string, "%d/%m/%Y"
+  setting :time_format, :string, "%Hh%M"
   setting :image_avatar_size, :integer, 48
   setting :image_thumb_size, :integer, 125
   setting :image_medium_size, :integer, 600
 
   # SEO
-  setting :meta_description, :string, ''
-  setting :meta_keywords, :string, ''
-  setting :google_analytics, :string, ''
+  setting :meta_description, :string, ""
+  setting :meta_keywords, :string, ""
+  setting :google_analytics, :string, ""
   setting :rss_description, :boolean, false
-  setting :rss_description_text, :string, <<EOS.strip_heredoc
+  setting :rss_description_text, :string, <<-HTML.strip_heredoc
     <hr />
     <p><small>Original article written by %author% and published on <a href='%blog_url%'>%blog_name%</a>
     | <a href='%permalink_url%'>direct link to this article</a>
     | If you are reading this article anywhere other than on <a href='%blog_url%'>%blog_name%</a>,
       it has been illegally reproduced and without proper authorization.</small></p>
-EOS
-  setting :permalink_format, :string, '/%year%/%month%/%day%/%title%'
+  HTML
+  setting :permalink_format, :string, "/%year%/%month%/%day%/%title%"
   setting :robots, :string, 'User-agent: *\nAllow: /\nDisallow: /admin\n'
-  setting :humans, :string, <<EOS.strip_heredoc
+  setting :humans, :string, <<-TEXT.strip_heredoc
     /* TEAM */
     Your title: Your name.
     Site: email, link to a contact form, etc.
     Twitter: your Twitter username.
 
     /* SITE */
-    Software: Publify [http://publify.co] #{PublifyCore::VERSION}
-EOS
-  setting :index_categories, :boolean, true # deprecated but still needed for backward compatibility
+    Software: Publify [https://publify.github.io/] #{PublifyCore::VERSION}
+  TEXT
+  # deprecated but still needed for backward compatibility
+  setting :index_categories, :boolean, true
   setting :unindex_categories, :boolean, false
-  setting :index_tags, :boolean, true # deprecated but still needed for backward compatibility
+  # deprecated but still needed for backward compatibility
+  setting :index_tags, :boolean, true
   setting :unindex_tags, :boolean, false
   setting :admin_display_elements, :integer, 10
-  setting :google_verification, :string, ''
-  setting :nofollowify, :boolean, true # deprecated but still needed for backward compatibility
+  setting :google_verification, :string, ""
+  # deprecated but still needed for backward compatibility
+  setting :nofollowify, :boolean, true
   setting :dofollowify, :boolean, false
-  setting :use_canonical_url, :boolean, false
   setting :use_meta_keyword, :boolean, true
-  setting :home_title_template, :string, '%blog_name% | %blog_subtitle%' # spec OK
-  setting :home_desc_template, :string, '%blog_name% | %blog_subtitle% | %meta_keywords%' # OK
-  setting :article_title_template, :string, '%title% | %blog_name%'
-  setting :article_desc_template, :string, '%excerpt%'
-  setting :page_title_template, :string, '%title% | %blog_name%'
-  setting :page_desc_template, :string, '%excerpt%'
-  setting :paginated_title_template, :string, '%blog_name% | %blog_subtitle% %page%'
-  setting :paginated_desc_template, :string, '%blog_name% | %blog_subtitle% | %meta_keywords% %page%'
-  setting :tag_title_template, :string, 'Tag: %name% | %blog_name% %page%'
-  setting :tag_desc_template, :string, '%name% | %blog_name% | %blog_subtitle% %page%'
-  setting :author_title_template, :string, '%author% | %blog_name%'
-  setting :author_desc_template, :string, '%author% | %blog_name% | %blog_subtitle%'
-  setting :archives_title_template, :string, 'Archives for %blog_name% %date% %page%'
-  setting :archives_desc_template, :string, 'Archives for %blog_name% %date% %page% %blog_subtitle%'
-  setting :search_title_template, :string, 'Results for %search% | %blog_name% %page%'
-  setting :search_desc_template, :string, 'Results for %search% | %blog_name% | %blog_subtitle% %page%'
-  setting :statuses_title_template, :string, 'Notes | %blog_name% %page%'
-  setting :statuses_desc_template, :string, 'Notes | %blog_name% | %blog_subtitle% %page%'
-  setting :status_title_template, :string, '%body% | %blog_name%'
-  setting :status_desc_template, :string, '%excerpt%'
-
-  setting :custom_tracking_field, :string, ''
+  setting :home_title_template, :string, "%blog_name% | %blog_subtitle%"
+  setting :home_desc_template, :string, "%blog_name% | %blog_subtitle% | %meta_keywords%"
+  setting :article_title_template, :string, "%title% | %blog_name%"
+  setting :article_desc_template, :string, "%excerpt%"
+  setting :page_title_template, :string, "%title% | %blog_name%"
+  setting :page_desc_template, :string, "%excerpt%"
+  setting :paginated_title_template, :string, "%blog_name% | %blog_subtitle% %page%"
+  setting :paginated_desc_template, :string,
+          "%blog_name% | %blog_subtitle% | %meta_keywords% %page%"
+  setting :tag_title_template, :string, "Tag: %name% | %blog_name% %page%"
+  setting :tag_desc_template, :string, "%name% | %blog_name% | %blog_subtitle% %page%"
+  setting :author_title_template, :string, "%author% | %blog_name%"
+  setting :author_desc_template, :string, "%author% | %blog_name% | %blog_subtitle%"
+  setting :archives_title_template, :string, "Archives for %blog_name% %date% %page%"
+  setting :archives_desc_template, :string,
+          "Archives for %blog_name% %date% %page% %blog_subtitle%"
+  setting :search_title_template, :string, "Results for %search% | %blog_name% %page%"
+  setting :search_desc_template, :string,
+          "Results for %search% | %blog_name% | %blog_subtitle% %page%"
+  setting :statuses_title_template, :string, "Notes | %blog_name% %page%"
+  setting :statuses_desc_template, :string, "Notes | %blog_name% | %blog_subtitle% %page%"
+  setting :status_title_template, :string, "%body% | %blog_name%"
+  setting :status_desc_template, :string, "%excerpt%"
+
+  setting :custom_tracking_field, :string, ""
   # setting :meta_author_template,       :string, "%blog_name% | %nickname%"
 
-  setting :twitter_consumer_key, :string, ''
-  setting :twitter_consumer_secret, :string, ''
-  setting :custom_url_shortener, :string, ''
+  setting :twitter_consumer_key, :string, ""
+  setting :twitter_consumer_secret, :string, ""
+  setting :custom_url_shortener, :string, ""
   setting :statuses_in_timeline, :boolean, true
 
   validate :permalink_has_identifier
@@ -147,13 +153,13 @@ EOS
 
   # Check that all required blog settings have a value.
   def configured?
-    settings.key?('blog_name')
+    settings.key?("blog_name")
   end
 
   # The +Theme+ object for the current theme.
   def current_theme(reload = nil)
     @current_theme = nil if reload
-    @current_theme ||= Theme.find(theme) || Theme.new('', '')
+    @current_theme ||= Theme.find(theme) || Theme.new("", "")
   end
 
   module BasedUrlFor
@@ -166,20 +172,22 @@ EOS
     def url_for(options = {}, extra_params = {})
       case options
       when String
+        options = options.sub(%r{^/}, "")
         url_generated = if extra_params[:only_path]
                           root_path
                         else
                           base_url
                         end
-        url_generated += "/#{options}" # They asked for 'url_for "/some/path"', so return it unedited.
+        # They asked for 'url_for "/some/path"', so return it unedited.
+        url_generated += "/#{options}"
         url_generated += "##{extra_params[:anchor]}" if extra_params[:anchor]
         url_generated
       when Hash
-        merged_opts = options.reverse_merge!(only_path: false, controller: '',
-                                             action: 'permalink',
+        merged_opts = options.reverse_merge!(only_path: false, controller: "",
+                                             action: "permalink",
                                              host: host_with_port,
                                              script_name: root_path)
-        cache_key = merged_opts.values.prepend('blog-urlfor-withbaseurl').join('-')
+        cache_key = merged_opts.values.prepend("blog-urlfor-withbaseurl").join("-")
         unless Rails.cache.exist?(cache_key)
           Rails.cache.write(cache_key, super(merged_opts))
         end
@@ -194,7 +202,7 @@ EOS
 
   # The URL for a static file.
   def file_url(filename)
-    if CarrierWave.configure { |config| config.storage.name == 'CarrierWave::Storage::Fog' }
+    if CarrierWave.configure { |config| config.storage.name == "CarrierWave::Storage::Fog" }
       filename
     else
       url_for filename, only_path: false
@@ -206,7 +214,8 @@ EOS
   end
 
   def per_page(format)
-    return limit_article_display if format.nil? || format == 'html'
+    return limit_article_display if format.nil? || format == "html"
+
     limit_rss_display
   end
 
@@ -220,12 +229,12 @@ EOS
   end
 
   def permalink_has_identifier
-    unless permalink_format =~ /(%title%)/
-      errors.add(:base, I18n.t('errors.permalink_need_a_title'))
+    unless /(%title%)/.match?(permalink_format)
+      errors.add(:base, I18n.t("errors.permalink_need_a_title"))
     end
 
-    if permalink_format =~ /\.(atom|rss)$/
-      errors.add(:permalink_format, I18n.t('errors.cant_end_with_rss_or_atom'))
+    if /\.(atom|rss)$/.match?(permalink_format)
+      errors.add(:permalink_format, I18n.t("errors.cant_end_with_rss_or_atom"))
     end
   end
 
@@ -240,6 +249,7 @@ EOS
   def has_twitter_configured?
     return false if twitter_consumer_key.nil? || twitter_consumer_secret.nil?
     return false if twitter_consumer_key.empty? || twitter_consumer_secret.empty?
+
     true
   end
 
@@ -259,10 +269,11 @@ EOS
 
   def split_base_url
     unless @split_base_url
-      unless base_url =~ /(https?):\/\/([^\/]*)(.*)/
-        raise "Invalid base_url: #{base_url}"
-      end
-      @split_base_url = { protocol: Regexp.last_match[1], host_with_port: Regexp.last_match[2], root_path: Regexp.last_match[3].gsub(%r{/$}, '') }
+      raise "Invalid base_url: #{base_url}" unless base_url =~ %r{(https?)://([^/]*)(.*)}
+
+      @split_base_url = { protocol: Regexp.last_match[1],
+                          host_with_port: Regexp.last_match[2],
+                          root_path: Regexp.last_match[3].gsub(%r{/$}, "") }
     end
     @split_base_url
   end

data/app/models/comment.rb

@@ -1,4 +1,6 @@
-require 'timeout'
+# frozen_string_literal: true
+
+require "timeout"
 
 class Comment < Feedback
   belongs_to :user, optional: true
@@ -7,14 +9,14 @@ class Comment < Feedback
 
   attr_accessor :referrer, :permalink
 
-  scope :spam, -> { where(state: 'spam') }
+  scope :spam, -> { where(state: "spam") }
   scope :not_spam, -> { where("state != 'spam'") }
-  scope :presumed_spam, -> { where(state: 'presumed_spam') }
-  scope :presumed_ham, -> { where(state: 'presumed_ham') }
-  scope :ham, -> { where(state: 'ham') }
+  scope :presumed_spam, -> { where(state: "presumed_spam") }
+  scope :presumed_ham, -> { where(state: "presumed_ham") }
+  scope :ham, -> { where(state: "ham") }
   scope :unconfirmed, -> { where(state: %w(presumed_spam presumed_ham)) }
 
-  scope :last_published, -> { published.limit(5).order('created_at DESC') }
+  scope :last_published, -> { published.limit(5).order("created_at DESC") }
 
   def notify_user_via_email(user)
     EmailNotify.send_comment(self, user) if user.notify_via_email?
@@ -40,15 +42,12 @@ class Comment < Feedback
 
   def article_allows_feedback?
     return true if article.allow_comments?
-    errors.add(:article, 'Article is not open to comments')
+
+    errors.add(:article, "Article is not open to comments")
     false
   end
 
   def originator
     author
   end
-
-  def content_fields
-    [:body]
-  end
 end

data/app/models/config_manager.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ConfigManager
   def self.append_features(base)
     super
@@ -35,7 +37,7 @@ module ConfigManager
         raw_value.nil? ? item.default : raw_value
       end
       if item.ruby_type == :boolean
-        send(:define_method, item.name + '?') do
+        send(:define_method, "#{item.name}?") do
           raw_value = settings[item.name]
           raw_value.nil? ? item.default : raw_value
         end
@@ -62,7 +64,7 @@ module ConfigManager
       case ruby_type
       when :boolean
         case value
-        when '0', 0, '', false, 'false', 'f', nil
+        when "0", 0, "", false, "false", "f", nil
           false
         else
           true

data/app/models/content.rb

@@ -1,38 +1,40 @@
-require 'set'
-require 'uri'
+# frozen_string_literal: true
 
-class Content < ActiveRecord::Base
+require "set"
+require "uri"
+
+class Content < ApplicationRecord
   include ContentBase
 
-  belongs_to :text_filter, optional: true
   belongs_to :user, optional: true, touch: true
   belongs_to :blog
 
   validates :blog, presence: true
 
-  has_one :redirect, dependent: :destroy, inverse_of: :contents
+  has_one :redirect, dependent: :destroy, inverse_of: :content
 
   has_many :triggers, as: :pending_item, dependent: :delete_all
   has_many :resources, inverse_of: :content, dependent: :nullify
   has_and_belongs_to_many :tags
 
-  scope :user_id, ->(user_id) { where('user_id = ?', user_id) }
-  scope :published, -> { where(state: 'published'). order(default_order) }
-  scope :published_at, ->(time_params) {
-    published.
-    where(published_at: PublifyTime.delta(*time_params)).
-    order('published_at DESC')
-  }
-  scope :not_published, -> { where.not(state: 'published') }
-  scope :drafts, -> { where(state: 'draft').order('created_at DESC') }
-  scope :no_draft, -> { where.not(state: 'draft').order('published_at DESC') }
+  scope :user_id, ->(user_id) { where("user_id = ?", user_id) }
+  scope :published, -> { where(state: "published").order(default_order) }
+  scope :published_at, lambda { |time_params|
+                         published.where(published_at: PublifyTime.delta(*time_params))
+                       }
+  scope :not_published, -> { where.not(state: "published") }
+  scope :drafts, -> { where(state: "draft").order("created_at DESC") }
+  scope :no_draft, -> { where.not(state: "draft").order("published_at DESC") }
   scope :searchstring, lambda { |search_string|
-    tokens = search_string.split(' ').map { |c| "%#{c.downcase}%" }
-    where('state = ? AND ' + (['(LOWER(body) LIKE ? OR LOWER(extended) LIKE ? OR LOWER(title) LIKE ?)'] * tokens.size).join(' AND '),
-          'published', *tokens.map { |token| [token] * 3 }.flatten)
+    tokens = search_string.split(" ").map { |c| "%#{c.downcase}%" }
+    matcher = "(LOWER(body) LIKE ? OR LOWER(extended) LIKE ? OR LOWER(title) LIKE ?)"
+    template = "state = ? AND #{([matcher] * tokens.size).join(" AND ")}"
+    where(template, "published", *tokens.map { |token| [token] * 3 }.flatten)
   }
 
-  scope :published_at_like, ->(date_at) { where(published_at: PublifyTime.delta_like(date_at)) }
+  scope :published_at_like, lambda { |date_at|
+                              where(published_at: PublifyTime.delta_like(date_at))
+                            }
 
   serialize :whiteboard
 
@@ -45,21 +47,12 @@ class Content < ActiveRecord::Base
     end
   end
 
-  # Set the text filter for this object.
-  # NOTE: Due to how Rails injects association methods, this cannot be put in ContentBase
-  # TODO: Allowing assignment of a string here is not very clean.
-  def text_filter=(filter)
-    filter_object = case filter
-                    when TextFilter
-                      filter
-                    else
-                      TextFilter.find_or_default(filter)
-                    end
-    self.text_filter_id = if filter_object
-                            filter_object.id
-                          else
-                            filter.to_i
-                          end
+  def author_name
+    if user.present? && user.name.present?
+      user.name
+    else
+      author
+    end
   end
 
   def shorten_url
@@ -67,6 +60,7 @@ class Content < ActiveRecord::Base
 
     if redirect.present?
       return if redirect.to_path == permalink_url
+
       redirect.to_path = permalink_url
       redirect.save
     else
@@ -85,9 +79,7 @@ class Content < ActiveRecord::Base
   def self.search_with(params)
     params ||= {}
     scoped = unscoped
-    if params[:searchstring].present?
-      scoped = scoped.searchstring(params[:searchstring])
-    end
+    scoped = scoped.searchstring(params[:searchstring]) if params[:searchstring].present?
 
     if params[:published_at].present? && /(\d\d\d\d)-(\d\d)/ =~ params[:published_at]
       scoped = scoped.published_at_like(params[:published_at])
@@ -98,8 +90,8 @@ class Content < ActiveRecord::Base
     end
 
     if params[:published].present?
-      scoped = scoped.published if params[:published].to_s == '1'
-      scoped = scoped.not_published if params[:published].to_s == '0'
+      scoped = scoped.published if params[:published].to_s == "1"
+      scoped = scoped.not_published if params[:published].to_s == "0"
     end
 
     scoped
@@ -109,19 +101,14 @@ class Content < ActiveRecord::Base
     self[:whiteboard] ||= {}
   end
 
-  def link_to_author?
-    user.email.present? && blog.link_to_author
-  end
-
-  def get_rss_description
-    return '' unless blog.rss_description
-    return '' unless respond_to?(:user) && user && user.name
+  def rss_description
+    return "" unless blog.rss_description
 
     rss_desc = blog.rss_description_text
-    rss_desc.gsub!('%author%', user.name)
-    rss_desc.gsub!('%blog_url%', blog.base_url)
-    rss_desc.gsub!('%blog_name%', blog.blog_name)
-    rss_desc.gsub!('%permalink_url%', permalink_url)
+    rss_desc.gsub!("%author%", author_name)
+    rss_desc.gsub!("%blog_url%", blog.base_url)
+    rss_desc.gsub!("%blog_name%", blog.blog_name)
+    rss_desc.gsub!("%permalink_url%", permalink_url)
     rss_desc
   end