trusty-cms 4.1.0 → 4.1.5

data/app/models/status.rb

@@ -3,7 +3,8 @@ class Status
 
   def initialize(options = {})
     options = options.symbolize_keys
-    @id, @name = options[:id], options[:name]
+    @id = options[:id]
+    @name = options[:name]
   end
 
   def symbol
@@ -27,15 +28,14 @@ class Status
   end
 
   def self.selectable_values
-    self.selectable.map(&:name)
+    selectable.map(&:name)
   end
 
   @@statuses = [
-    Status.new(:id => 1,   :name => 'Draft'    ),
-    Status.new(:id => 50,  :name => 'Reviewed' ),
-    Status.new(:id => 90,  :name => 'Scheduled'),
-    Status.new(:id => 100, :name => 'Published'),
-    Status.new(:id => 101, :name => 'Hidden'   )
+    Status.new(id: 1,   name: 'Draft'),
+    Status.new(id: 50,  name: 'Reviewed'),
+    Status.new(id: 90,  name: 'Scheduled'),
+    Status.new(id: 100, name: 'Published'),
+    Status.new(id: 101, name: 'Hidden'),
   ]
-
 end

data/app/models/trusty_cms/config.rb

@@ -1,5 +1,4 @@
 module TrustyCms
-
   class << self
     def config_definitions
       @config_definitions ||= {}
@@ -67,7 +66,7 @@ module TrustyCms
     #   edit_setting('admin.name)
     #
 
-    self.table_name = "config"
+    self.table_name = 'config'
     after_save :update_cache
     attr_reader :definition
 
@@ -105,12 +104,12 @@ module TrustyCms
       end
 
       def to_hash
-        Hash[ *all.map { |pair| [pair.key, pair.value] }.flatten ]
+        Hash[*all.map { |pair| [pair.key, pair.value] }.flatten]
       end
 
       def initialize_cache
         TrustyCms::Config.ensure_cache_file
-        Rails.cache.write('TrustyCms::Config',TrustyCms::Config.to_hash)
+        Rails.cache.write('TrustyCms::Config', TrustyCms::Config.to_hash)
         Rails.cache.write('TrustyCms.cache_mtime', File.mtime(cache_file))
         Rails.cache.silence!
       end
@@ -121,6 +120,7 @@ module TrustyCms
 
       def stale_cache?
         return true unless TrustyCms::Config.cache_file_exists?
+
         Rails.cache.read('TrustyCms.cache_mtime') != File.mtime(cache_file)
       end
 
@@ -134,15 +134,15 @@ module TrustyCms
       end
 
       def cache_file
-        File.join(cache_path,'trusty_config_cache.txt')
+        File.join(cache_path, 'trusty_config_cache.txt')
       end
 
       def site_settings
-        @site_settings ||= %w{ site.title site.host dev.host local.timezone }
+        @site_settings ||= %w{site.title site.host dev.host local.timezone}
       end
 
       def default_settings
-        @default_settings ||= %w{ defaults.locale defaults.page.filter defaults.page.parts defaults.page.fields defaults.page.status }
+        @default_settings ||= %w{defaults.locale defaults.page.filter defaults.page.parts defaults.page.fields defaults.page.status}
       end
 
       def user_settings
@@ -161,7 +161,7 @@ module TrustyCms
       #
       def namespace(prefix, options = {}, &block)
         prefix = [options[:prefix], prefix].join('.') if options[:prefix]
-        with_options(options.merge(:prefix => prefix), &block)
+        with_options(options.merge(prefix: prefix), &block)
       end
 
       # Declares a setting definition that will constrain and support the use of a particular config entry.
@@ -176,7 +176,7 @@ module TrustyCms
       # * :allow_blank should be false if the config item must not be blank or nil
       # * :allow_change should be false if the config item can only be set, not changed. Add a default to specify an unchanging config entry.
       # * :allow_display should be false if the config item should not be showable in radius tags
-      #    
+      #
       #   TrustyCms.config do |config|
       #     config.define 'defaults.locale', :select_from => lambda { TrustyCms::AvailableLocales.locales }, :allow_blank => true
       #     config.define 'defaults.page.parts', :default => "Body,Extended"
@@ -191,7 +191,7 @@ module TrustyCms
       #
       # but at the moment that's only done in testing.
       #
-      def define(key, options={})
+      def define(key, options = {})
         called_from = caller.grep(/\/initializers\//).first
         if options.is_a? TrustyCms::Config::Definition
           definition = options
@@ -199,7 +199,7 @@ module TrustyCms
           key = [options[:prefix], key].join('.') if options[:prefix]
         end
 
-        definition ||= TrustyCms::Config::Definition.new(options.merge(:definer => called_from))
+        definition ||= TrustyCms::Config::Definition.new(options.merge(definer: called_from))
         definitions[key] = definition
 
         if self[key].nil? && !definition.default.nil?
@@ -216,9 +216,8 @@ module TrustyCms
       end
 
       def definition_for(key)
-        definitions[key] ||= TrustyCms::Config::Definition.new(:empty => true)
+        definitions[key] ||= TrustyCms::Config::Definition.new(empty: true)
       end
-
     end
 
     # The usual way to use a config item:
@@ -236,13 +235,14 @@ module TrustyCms
     def value=(param)
       newvalue = param.to_s
       if newvalue != self[:value]
-        raise ConfigError, "#{self.key} cannot be changed" unless settable? || self[:value].blank?
-        if boolean?
-          self[:value] = (newvalue == "1" || newvalue == "true") ? "true" : "false"
-        else
-          self[:value] = newvalue
-        end
-        self.save!
+        raise ConfigError, "#{key} cannot be changed" unless settable? || self[:value].blank?
+
+        self[:value] = if boolean?
+                         newvalue == '1' || newvalue == 'true' ? 'true' : 'false'
+                       else
+                         newvalue
+                       end
+        save!
       end
       self[:value]
     end
@@ -270,20 +270,21 @@ module TrustyCms
     # that does not restrict use.
     #
     def definition
-      @definition ||= self.class.definition_for(self.key)
+      @definition ||= self.class.definition_for(key)
     end
 
     # Returns true if the item key ends with '?' or the definition specifies :type => :boolean.
     #
     def boolean?
-      definition.boolean? || self.key.ends_with?("?")
+      definition.boolean? || key.ends_with?('?')
     end
 
     # Returns true if the item is boolean and true.
     #
     def checked?
       return nil if self[:value].nil?
-      boolean? && self[:value] == "true"
+
+      boolean? && self[:value] == 'true'
     end
 
     # Returns true if the item defintion includes a :select_from parameter that limits the range of permissible options.
@@ -302,11 +303,10 @@ module TrustyCms
       TrustyCms::Config.initialize_cache
     end
 
-    delegate :default, :type, :allow_blank?, :hidden?, :visible?, :settable?, :selection, :notes, :units, :to => :definition
+    delegate :default, :type, :allow_blank?, :hidden?, :visible?, :settable?, :selection, :notes, :units, to: :definition
 
     def validate
       definition.validate(self)
     end
-
   end
 end

data/app/models/trusty_cms/page_response_cache_director.rb

@@ -22,7 +22,7 @@ module TrustyCms
     private
 
     def non_cacheable_response
-      @listener.set_expiry nil, :private => true, "no-cache" => true
+      @listener.set_expiry nil, :private => true, 'no-cache' => true
       @listener.set_etag('')
     end
 
@@ -32,12 +32,11 @@ module TrustyCms
         timeout = @page.cache_timeout
       end
 
-      @listener.set_expiry timeout, :public => true, :private => false
+      @listener.set_expiry timeout, public: true, private: false
     end
 
     def cacheable?
       @listener.cacheable_request? && @page.cache?
     end
-
   end
 end

data/app/models/user.rb

@@ -1,10 +1,10 @@
 class User < ActiveRecord::Base
-  has_many :pages, :foreign_key => :created_by_id
+  has_many :pages, foreign_key: :created_by_id
   self.table_name = 'admins'
 
   # :confirmable, :lockable, :timeoutable and :omniauthable
   devise :database_authenticatable, :registerable,
-    :recoverable, :rememberable, :trackable, :validatable
+         :recoverable, :rememberable, :trackable, :validatable
 
   alias_attribute :created_by_id, :id
 
@@ -13,53 +13,54 @@ class User < ActiveRecord::Base
   validate :password_complexity
 
   # Default Order
-  default_scope {order('last_name')}
+  default_scope { order('last_name') }
 
   # Associations
-  belongs_to :created_by, :class_name => 'User'
-  belongs_to :updated_by, :class_name => 'User'
+  belongs_to :created_by, class_name: 'User'
+  belongs_to :updated_by, class_name: 'User'
 
   def has_role?(role)
     case role
     when :admin
-      self.admin?
+      admin?
     when :designer
-      self.designer?
+      designer?
     when :content_editor
-      self.content_editor?  
+      content_editor?
     else
       false
     end
   end
 
   def admin?
-    self.admin
+    admin
   end
 
   def designer?
-    self.designer
+    designer
   end
 
   def content_editor?
-    self.content_editor
+    content_editor
   end
 
-  def locale 
+  def locale
     'en'
   end
 
   def name
-    "#{self.first_name} #{self.last_name}"
+    "#{first_name} #{last_name}"
   end
 
   def password_required?
     return false if skip_password_validation
+
     super
   end
 
   def password_complexity
     return false if password.blank? || password =~ /^(?=.*?[A-Z])(?=.*?[a-z])(?=.*?[0-9])(?=.*?[#?!@$%^&*-]).{8,70}$/
+
     errors.add :password, 'Complexity requirement not met. Length should be 12 characters and include: 1 uppercase, 1 lowercase, 1 digit and 1 special character.'
   end
-
 end

data/app/models/user_action_observer.rb

@@ -4,7 +4,7 @@ class UserActionObserver < ActiveRecord::Observer
   def current_user=(user)
     self.class.current_user = user
   end
-  
+
   def self.current_user=(user)
     Thread.current[:current_user] = user
   end
@@ -18,10 +18,10 @@ class UserActionObserver < ActiveRecord::Observer
   end
 
   def before_create(model)
-    #model.created_by = self.current_user
+    # model.created_by = self.current_user
   end
 
   def before_update(model)
-    #model.updated_by = self.current_user
+    # model.updated_by = self.current_user
   end
 end

data/app/views/admin/assets/edit.html.haml

@@ -30,12 +30,3 @@
           = save_model_and_continue_editing_button(@asset)
           = t("or")
           = link_to t("cancel"), admin_assets_path
-  
-  - main.edit_regenerate do
-    - if current_user.admin? or current_user.designer?
-      %fieldset
-        = form_tag refresh_admin_asset_path(@asset), :method => :put do
-          %label= t("clipped_extension.regenerate_thumbnails")
-          %br
-          .asset_buttons
-            %input.button{ :type => "submit", :value => t("clipped_extension.regenerate") }

data/app/views/admin/assets/new.html.haml

@@ -18,6 +18,3 @@
           = save_model_and_continue_editing_button(@asset)
           = t("or")
           = link_to t("cancel"), admin_assets_path
-
-    - main.edit_regenerate do
-       

data/app/views/admin/assets/remove.html.haml

@@ -2,7 +2,7 @@
 = stylesheet_link_tag 'admin/assets'
 
 %h1= t("clipped_extension.remove_asset")
-%p.warning= t("clipped_extension.confirm_remove_asset")
+%h4.warning= t("clipped_extension.confirm_remove_asset").html_safe
 
 %table.index#assets{:cellpadding => "0", :cellspacing => "0", :border => "0"}
   %tbody

data/app/views/admin/layouts/remove.html.haml

@@ -1,12 +1,13 @@
+- body_classes << 'reversed'
+
 %h1= t('remove_layout')
 
-%p= t('text.layouts.remove_warning').html_safe
+%h4= t('text.layouts.remove_warning').html_safe
 
-%table.index#layouts
+%table.tablesaw.index#layouts
   %tbody
     %tr.node.level_1
       %td.layout
-        = image('layout', :alt => '')
         = h @layout.name
 
 = form_for [:admin, @layout], :html => { :method => 'delete', 'data-onsubmit_status'=>'Removing layout&#8230;' } do

data/app/views/admin/page_parts/_page_part.html.haml

@@ -3,7 +3,7 @@
     = hidden_field_tag "page[parts_attributes][#{page_part_counter}][name]", page_part.name, :id => "part_#{page_part.name.to_slug}_name"
     = hidden_field_tag "page[parts_attributes][#{page_part_counter}][id]", page_part.id, :id => "part_#{page_part_counter}_id", :class => 'id_input' unless page_part.new_record?
     = hidden_field_tag "page[parts_attributes][#{page_part_counter}][_destroy]", page_part._destroy, :class => 'delete_input'
-    %p
+    .content
       %label To use HTML or Radius tags, switch to plain text view by clicking the 'Source' button.
       %span.reference_links
         = t('reference')

data/app/views/admin/pages/remove.html.haml

@@ -1,9 +1,10 @@
 - pages = @page.children.count > 1 ? t('pages') : t('page')
+- body_classes << 'reversed'
 
 %h1
   = t('remove_pages', :pages => pages)
 
-%p
+%h4
   = t('text.pages.remove_warning', :pages => pages).html_safe
 
 .inset

data/app/views/admin/sites/index.haml

@@ -31,7 +31,9 @@
               = h(site.base_domain)
           - tbody.modify_cell do
             %td.remove
-              = link_to image('remove.png', :alt => 'Remove'), admin_site_path(site), :method => :delete, :onclick => "return confirm('Are you sure you want to permanently remove #{site.name}?')"
+              = link_to admin_site_path(site), :method => :delete, :onclick => "return confirm('Are you sure you want to permanently remove #{site.name}?')" do
+                %i.fas.fa-minus-circle
+                = t('remove')
           - tbody.order_cell do
             %td.order
               = order_links(site).html_safe

data/app/views/admin/snippets/remove.html.haml

@@ -1,13 +1,14 @@
+- body_classes << 'reversed'
+
 %h1= t('remove_snippet')
 
-%p 
-  = t('text.snippets.remove_warning')
+%h4
+  = t('text.snippets.remove_warning').html_safe
 
-%table.index#snippets
+%table.tablesaw.index#snippets
   %tbody
     %tr.node.level_1
       %td.snippet
-        = image('snippet', :alt => '')
         %span.title= @snippet.name
 
 = form_for [:admin, @snippet], :html => {:method => :delete, 'data-onsubmit_status'=>'Removing snippet&#8230;'} do

data/app/views/admin/users/remove.html.haml

@@ -1,7 +1,7 @@
 %h1= t('remove_user')
 
-%p
-  = t('text.users.remove_warning')
+%h4
+  = t('text.users.remove_warning').html_safe
 
 %table.index#users
   %tbody

data/app/views/rad_social_mailer/social_mail_form.html.haml

@@ -1,6 +1,6 @@
 = stylesheet_link_tag('rad_social/rad_screen')
-= javascript_include_tag('https://www.google.com/recaptcha/api/js/recaptcha_ajax.js')
 = javascript_include_tag('rad_social/rad_email')
+= csrf_meta_tags
 
 .rad-dialog-titlebar
   %span.rad-dialog-title
@@ -31,8 +31,6 @@
         = email_message
   %input{:type=>'hidden', :name=>'subject', :value=> email_subject}
   %input{:type=>'hidden', :id=>'rs-base-message', :value=> email_message}
-  #recaptcha-container.captcha-container
-    = recaptcha_v3 action: 'emailSeats'
   %input{:type=>'hidden', :name=>'submit_url', :value=> email_action_url}
   %input.button.primary-button{:id => 'rad_email_submit', :type=>:button ,:value=>'Submit'}
   .loader

data/app/views/widget/_email_form.html.haml

@@ -1,5 +1,3 @@
-- javascript('rad_social/captcha')
-
 = form_tag
 %form{:id=>'rad_social_email_form', :class=>'rad-email-form'}
   %label{:for => 'from'}
@@ -20,6 +18,3 @@
   %input{:type=>'hidden', :name=>'submit_url', :value=>'contact/email'}
   %input.button.primary-button{:id => 'email_submit', :type=>:button ,:value=>'Submit'}
   .loader
-
-  #recaptcha-container.captcha-container
-    = recaptcha_v3 action: 'emailSeats'