publify_core 9.1.0 → 9.2.0

data/app/models/page_sidebar.rb

@@ -1,6 +1,8 @@
+# frozen_string_literal: true
+
 class PageSidebar < Sidebar
-  display_name 'Page'
-  description 'Show pages for this blog'
+  display_name "Page"
+  description "Show pages for this blog"
 
   setting :maximum_pages, 10
 

data/app/models/ping.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class Ping < ApplicationRecord
   belongs_to :article
 end

data/app/models/post_type.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class PostType < ApplicationRecord
   validates :name, uniqueness: true
   validates :name, presence: true
@@ -5,7 +7,7 @@ class PostType < ApplicationRecord
   before_save :sanitize_title
 
   def name_is_not_read
-    errors.add(:name, I18n.t('errors.article_type_already_exist')) if name == 'read'
+    errors.add(:name, I18n.t("errors.article_type_already_exist")) if name == "read"
   end
 
   def sanitize_title

data/app/models/redirect.rb

@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 class Redirect < ApplicationRecord
-  belongs_to :contents, optional: true, touch: true
+  belongs_to :content, optional: true, touch: true
   belongs_to :blog
 
   validates :from_path, uniqueness: true
@@ -9,9 +11,12 @@ class Redirect < ApplicationRecord
   def full_to_path
     path = to_path
     # FIXME: Unify HTTP URI matchers
-    return path if path =~ %r{^(https?)://([^/]*)(.*)}
+    return path if %r{^(https?)://([^/]*)(.*)}.match?(path)
+
     url_root = blog.root_path
-    path = File.join(url_root, path) unless url_root.nil? || path[0, url_root.length] == url_root
+    unless url_root.nil? || path[0, url_root.length] == url_root
+      path = File.join(url_root, path)
+    end
     path
   end
 
@@ -24,7 +29,7 @@ class Redirect < ApplicationRecord
   end
 
   def to_url
-    raise 'Use #from_url'
+    raise "Use #from_url"
   end
 
   def from_url
@@ -34,8 +39,8 @@ class Redirect < ApplicationRecord
   private
 
   def random_token
-    characters = 'ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz1234567890'
-    temp_token = ''
+    characters = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz1234567890"
+    temp_token = ""
     srand
     6.times do
       pos = rand(characters.length)

data/app/models/redirection.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class Redirection < ApplicationRecord
   belongs_to :content
   belongs_to :redirect

data/app/models/resource.rb

@@ -1,5 +1,7 @@
-require 'carrierwave'
-require 'carrierwave/orm/activerecord'
+# frozen_string_literal: true
+
+require "carrierwave"
+require "carrierwave/orm/activerecord"
 
 class Resource < ApplicationRecord
   belongs_to :blog

data/app/models/search_sidebar.rb

@@ -1,7 +1,9 @@
+# frozen_string_literal: true
+
 class SearchSidebar < Sidebar
-  description 'Adds basic search sidebar in your Publify blog'
+  description "Adds basic search sidebar in your Publify blog"
 
-  setting :title, 'Search'
+  setting :title, "Search"
 end
 
 SidebarRegistry.register_sidebar SearchSidebar

data/app/models/sidebar.rb

@@ -1,4 +1,6 @@
-require 'sidebar_field'
+# frozen_string_literal: true
+
+require "sidebar_field"
 
 # This class cannot be autoloaded since other sidebar classes depend on it.
 class Sidebar < ApplicationRecord
@@ -24,7 +26,7 @@ class Sidebar < ApplicationRecord
   end
 
   def self.purge
-    delete_all('active_position is null and staged_position is null')
+    delete_all("active_position is null and staged_position is null")
   end
 
   def self.setting(key, default = nil, options = {})
@@ -55,7 +57,7 @@ class Sidebar < ApplicationRecord
     if desc
       @description = desc
     else
-      @description || ''
+      @description || ""
     end
   end
 
@@ -90,7 +92,7 @@ class Sidebar < ApplicationRecord
   end
 
   def html_id
-    short_name + '-' + id.to_s
+    "#{short_name}-#{id}"
   end
 
   def parse_request(_contents, _params); end
@@ -130,8 +132,11 @@ class Sidebar < ApplicationRecord
   end
 
   def admin_state
-    return :active if active_position && (staged_position == active_position || staged_position.nil?)
+    if active_position && (staged_position == active_position || staged_position.nil?)
+      return :active
+    end
     return :will_change_position if active_position != staged_position
+
     raise "Unknown admin_state: active: #{active_position}, staged: #{staged_position}"
   end
 end

data/app/models/static_sidebar.rb

@@ -1,18 +1,21 @@
+# frozen_string_literal: true
+
 class StaticSidebar < Sidebar
-  DEFAULT_TEXT = '
-<ul>
-  <li><a href="http://www.publify.co/" title="Publify">Publify</a></li>
-  <li><a href="http://t37.net/" title="Le Rayon UX">Frédéric</a></li>
-  <li><a href="http://www.matijs.net/" title="Matijs">Matijs</a></li>
-  <li><a href="http://elsif.fr/" title="Yannick">Yannick</a></li>
-  <li><a href="http://blog.ookook.fr/" title="Thomas">Thomas</a></li>
-  <li><a href="/admin">Admin</a></li>
-</ul>
+  DEFAULT_TEXT = <<~TEXT
+    <ul>
+      <li><a href="https://publify.github.io/" title="Publify">Publify</a></li>
+      <li><a href="http://t37.net/" title="Le Rayon UX">Frédéric</a></li>
+      <li><a href="http://www.matijs.net/" title="Matijs">Matijs</a></li>
+      <li><a href="http://elsif.fr/" title="Yannick">Yannick</a></li>
+      <li><a href="http://blog.ookook.fr/" title="Thomas">Thomas</a></li>
+      <li><a href="/admin">Admin</a></li>
+    </ul>
+  TEXT
 
-'.freeze
-  description 'Static content, like links to other sites, advertisements, or blog meta-information'
+  description "Static content, like links to other sites, advertisements," \
+    " or blog meta-information"
 
-  setting :title, 'Links'
+  setting :title, "Links"
   setting :body, DEFAULT_TEXT, input_type: :text_area
 end
 

data/app/models/tag.rb

@@ -1,6 +1,8 @@
+# frozen_string_literal: true
+
 class Tag < ApplicationRecord
   belongs_to :blog
-  has_and_belongs_to_many :contents, order: 'created_at DESC'
+  has_and_belongs_to_many :contents, order: "created_at DESC"
 
   validates :name, uniqueness: { scope: :blog_id }
   validates :blog, presence: true
@@ -12,10 +14,11 @@ class Tag < ApplicationRecord
 
   def self.create_from_article!(article)
     return if article.keywords.nil?
+
     tags = []
     Tag.transaction do
-      tagwords = article.keywords.to_s.scan(/((['"]).*?\2|[\.:[[:alnum:]]]+)/).map do |x|
-        x.first.tr("\"'", '')
+      tagwords = article.keywords.to_s.scan(/((['"]).*?\2|[.:[[:alnum:]]]+)/).map do |x|
+        x.first.tr("\"'", "")
       end
       tagwords.uniq.each do |tagword|
         tagname = tagword.to_url
@@ -35,18 +38,19 @@ class Tag < ApplicationRecord
 
   def self.find_all_with_content_counters
     Tag.joins(:contents).
-      where(contents: { state: 'published' }).
-      select(*Tag.column_names, 'COUNT(contents_tags.content_id) as content_counter').
+      where(contents: { state: "published" }).
+      select(*Tag.column_names, "COUNT(contents_tags.content_id) as content_counter").
       group(*Tag.column_names).
-      order('content_counter DESC').limit(1000)
+      order("content_counter DESC").limit(1000)
   end
 
   def self.find_with_char(char)
-    where('name LIKE ? ', "%#{char}%").order('name ASC')
+    where("name LIKE ? ", "%#{char}%").order("name ASC")
   end
 
   def self.collection_to_string(tags)
-    tags.map(&:display_name).sort.map { |name| name =~ / / ? "\"#{name}\"" : name }.join ', '
+    tags.map(&:display_name).sort.
+      map { |name| / /.match?(name) ? "\"#{name}\"" : name }.join ", "
   end
 
   def published_contents
@@ -58,10 +62,10 @@ class Tag < ApplicationRecord
   end
 
   def feed_url(format)
-    "#{permalink_url}.#{format.gsub(/\d/, '')}"
+    "#{permalink_url}.#{format.gsub(/\d/, "")}"
   end
 
   def permalink_url(_anchor = nil, only_path = false)
-    blog.url_for(controller: 'tags', action: 'show', id: permalink, only_path: only_path)
+    blog.url_for(controller: "tags", action: "show", id: permalink, only_path: only_path)
   end
 end

data/app/models/tag_sidebar.rb

@@ -1,6 +1,8 @@
+# frozen_string_literal: true
+
 class TagSidebar < Sidebar
-  display_name 'Tags'
-  description 'Show most popular tags for this blog'
+  display_name "Tags"
+  description "Show most popular tags for this blog"
 
   setting :maximum_tags, 20
 
@@ -11,8 +13,9 @@ class TagSidebar < Sidebar
 
   def sizes
     return @sizes if @sizes
+
     total = tags.reduce(0) { |sum, tag| sum + tag.content_counter }
-    average = total.to_f / @tags.size.to_f
+    average = total.to_f / @tags.size
     @sizes = tags.reduce({}) do |h, tag|
       size = tag.content_counter.to_f / average
       h.merge tag => [[2.0 / 3.0, size].max, 2].min * 100

data/app/models/text_filter.rb

@@ -1,15 +1,28 @@
-require 'net/http'
+# frozen_string_literal: true
 
-class TextFilter < ApplicationRecord
-  serialize :filters, Array
-  serialize :params, Hash
+require "net/http"
+
+class TextFilter
+  attr_accessor :description, :filters, :markup, :name, :params
+
+  def initialize(name: nil,
+                 description: nil,
+                 markup: nil,
+                 filters: [],
+                 params: nil)
+    @name = name
+    @description = description
+    @markup = markup
+    @filters = filters
+    @params = params
+  end
 
   def sanitize(*args, &blk)
     self.class.sanitize(*args, &blk)
   end
 
   def self.find_or_default(name)
-    find_by(name: name) || find_by(name: 'none')
+    make_filter(name) || none
   end
 
   def self.filter_text(text, filters)
@@ -17,8 +30,10 @@ class TextFilter < ApplicationRecord
 
     filters.each do |filter|
       next if filter.nil?
+
       filter_class = map[filter.to_s]
       next unless filter_class
+
       text = filter_class.filtertext(text)
     end
 
@@ -35,12 +50,16 @@ class TextFilter < ApplicationRecord
 
     help = []
     help.push(filter_map[markup])
-    filter_types['macropre'].sort_by(&:short_name).each { |f| help.push f }
-    filter_types['macropost'].sort_by(&:short_name).each { |f| help.push f }
+    filter_types["macropre"].sort_by(&:short_name).each { |f| help.push f }
+    filter_types["macropost"].sort_by(&:short_name).each { |f| help.push f }
     filters.each { |f| help.push(filter_map[f.to_s]) }
 
     help_text = help.map do |f|
-      f.help_text.blank? ? '' : "<h3>#{f.display_name}</h3>\n#{BlueCloth.new(f.help_text).to_html}\n"
+      if f.help_text.blank?
+        ""
+      else
+        "<h3>#{f.display_name}</h3>\n#{BlueCloth.new(f.help_text).to_html}\n"
+      end
     end
 
     help_text.join("\n")
@@ -52,10 +71,65 @@ class TextFilter < ApplicationRecord
     help = [filter_map[markup]]
     filters.each { |f| help.push(filter_map[f.to_s]) }
 
-    help_text = help.map do |f|
-      f.help_text.blank? ? '' : "#{BlueCloth.new(f.help_text).to_html}\n"
+    help.map do |f|
+      f.help_text.blank? ? "" : "#{BlueCloth.new(f.help_text).to_html}\n"
     end.join("\n")
+  end
+
+  def self.all
+    [
+      markdown,
+      smartypants,
+      markdown_smartypants,
+      textile,
+      none,
+    ]
+  end
+
+  def self.make_filter(name)
+    case name
+    when "markdown"
+      markdown
+    when "smartypants"
+      smartypants
+    when "markdown smartypants"
+      markdown_smartypants
+    when "textile"
+      textile
+    when "none"
+      none
+    end
+  end
+
+  def self.markdown
+    new(name: "markdown",
+        description: "Markdown",
+        markup: "markdown")
+  end
+
+  def self.smartypants
+    new(name: "smartypants",
+        description: "SmartyPants",
+        markup: "none",
+        filters: [:smartypants])
+  end
+
+  def self.markdown_smartypants
+    new(name: "markdown smartypants",
+        description: "Markdown with SmartyPants",
+        markup: "markdown",
+        filters: [:smartypants])
+  end
+
+  def self.textile
+    new(name: "textile",
+        description: "Textile",
+        markup: "textile")
+  end
 
-    help_text
+  def self.none
+    new(name: "none",
+        description: "None",
+        markup: "none")
   end
 end

data/app/models/trackback.rb

@@ -1,34 +1,30 @@
+# frozen_string_literal: true
+
 # Legacy class to support old feedback sent as trackbacks.
 class Trackback < Feedback
   content_fields :excerpt
   validates :title, :excerpt, :url, presence: true
 
-  # attr_accessible :url, :blog_name, :title, :excerpt, :ip, :published, :article_id
-
-  def initialize(*args, &block)
-    super(*args, &block)
-    self.title ||= url
-    self.blog_name ||= ''
-  end
-
   before_create :process_trackback
 
   def process_trackback
     if excerpt.length >= 251
       # this limits excerpt to 250 chars, including the trailing "..."
-      self.excerpt = excerpt[0..246] << '...'
+      self.excerpt = excerpt[0..246] << "..."
     end
   end
 
   def article_allows_feedback?
     return true if article.allow_pings?
-    errors.add(:article, 'Article is not pingable')
+
+    errors.add(:article, "Article is not pingable")
     false
   end
 
   def blog_allows_feedback?
     return true unless blog.global_pings_disable
-    errors.add(:article, 'Pings are disabled')
+
+    errors.add(:article, "Pings are disabled")
     false
   end
 

data/app/models/trigger.rb

@@ -1,20 +1,23 @@
+# frozen_string_literal: true
+
 class Trigger < ApplicationRecord
   belongs_to :pending_item, polymorphic: true
 
   class << self
-    def post_action(due_at, item, method = 'came_due')
+    def post_action(due_at, item, method = "came_due")
       create!(due_at: due_at, pending_item: item,
               trigger_method: method)
       fire
     end
 
     def fire
-      where('due_at <= ?', Time.zone.now).destroy_all
+      where("due_at <= ?", Time.zone.now).destroy_all
       true
     end
 
     def remove(pending_item, conditions = {})
       return if pending_item.new_record?
+
       conditions = conditions.merge(pending_item_id: pending_item.id,
                                     pending_item_type: pending_item.class.to_s)
       where(conditions).delete_all
@@ -25,7 +28,7 @@ class Trigger < ApplicationRecord
   before_destroy :trigger_pending_item
 
   def trigger_pending_item
-    pending_item.send(trigger_method) if pending_item
+    pending_item&.send(trigger_method)
     true
   end
 end