trusty-cms 4.0.1 → 4.1.3

data/lib/tasks/upgrade_to_devise.rake

@@ -17,7 +17,7 @@ namespace :import do
         admin: legacy.admin,
         designer: legacy.designer,
         content_editor: legacy.content_editor,
-        notes: legacy.notes
+        notes: legacy.notes,
       )
       u.save!
     end

data/lib/translation_support.rb

@@ -1,54 +1,54 @@
 class TranslationSupport
   class << self
-
-    #Retrieve US word set
-    def get_translation_keys(language_root, suffix=nil)
+    # Retrieve US word set
+    def get_translation_keys(language_root, suffix = nil)
       (dummy_comments, words) = read_file("#{language_root}/en#{suffix}.yml", 'en')
       words
     end
 
-    #Retrieve comments, translation data in hash form
+    # Retrieve comments, translation data in hash form
     def read_file(filename, basename)
-      (comments, data) = IO.read(filename).split(/\n#{basename}:\s*\n/)   #Add error checking for failed file read?
-      return comments, create_hash(data, basename)
+      (comments, data) = IO.read(filename).split(/\n#{basename}:\s*\n/) # Add error checking for failed file read?
+      [comments, create_hash(data, basename)]
     end
 
-    #Creates hash of translation data
-    def create_hash(data, basename)
+    # Creates hash of translation data
+    def create_hash(data, _basename)
       words = Hash.new
       return words if !data
+
       parent = Array.new
       previous_key = 'base'
       data.split("\n").each do |w|
         next if w.strip.blank?
+
         (key, value) = w.split(':', 2)
         value ||= ''
-        shift = (key =~ /\w/)/2 - parent.size                             #Determine level of current key in comparison to parent array
-        key = key.sub(/^\s+/,'')
-        parent << previous_key if shift > 0                               #If key is child of previous key, add previous key as parent
-        (shift * -1).times { parent.pop } if shift < 0                      #If key is not related to previous key, remove parent keys
-        previous_key = key                                                #Track key in case next key is child of this key
+        shift = (key =~ /\w/) / 2 - parent.size # Determine level of current key in comparison to parent array
+        key = key.sub(/^\s+/, '')
+        parent << previous_key if shift > 0 # If key is child of previous key, add previous key as parent
+        (shift * -1).times { parent.pop } if shift < 0 # If key is not related to previous key, remove parent keys
+        previous_key = key # Track key in case next key is child of this key
         words[parent.join(':') + ':' + key] = value unless key.blank?
       end
       words
     end
 
     def open_available_tags(filename)
-      data = YAML::load(File.open("#{filename}"))
+      data = YAML::load(File.open(filename.to_s))
       data.to_s
     end
 
-    #Writes to file from translation data hash structure
-    def write_file(filename,basename,comments,words)
-      File.open(filename, "w") do |log|
-        log.puts(comments+"\n"+basename+": \n")
-        words.sort.each do |k,v|
+    # Writes to file from translation data hash structure
+    def write_file(filename, basename, comments, words)
+      File.open(filename, 'w') do |log|
+        log.puts(comments + "\n" + basename + ": \n")
+        words.sort.each do |k, v|
           keys = k.split(':')
-          (keys.size-1).times { keys[keys.size-1] = '  ' + keys[keys.size-1] }   #Add indentation for children keys
-          log.puts(keys[keys.size-1]+':'+v+"\n")
+          (keys.size - 1).times { keys[keys.size - 1] = '  ' + keys[keys.size - 1] } # Add indentation for children keys
+          log.puts(keys[keys.size - 1] + ':' + v + "\n")
         end
       end
     end
-
   end
 end

data/lib/trusty_cms.rb

@@ -1,7 +1,7 @@
-TRUSTY_CMS_ROOT = File.expand_path(File.join(File.dirname(__FILE__), "..")) unless defined? TRUSTY_CMS_ROOT
+TRUSTY_CMS_ROOT = File.expand_path(File.join(File.dirname(__FILE__), '..')) unless defined? TRUSTY_CMS_ROOT
 
 unless defined? TrustyCms::VERSION
   module TrustyCms
-    VERSION = '4.0.1'
+    VERSION = '4.1.3'.freeze
   end
 end

data/lib/trusty_cms/admin_ui.rb

@@ -18,10 +18,10 @@ module TrustyCms
       end
 
       def [](id)
-        unless id.kind_of? Integer
-          self.find {|subnav_item| subnav_item.name.to_s.titleize == id.to_s.titleize }
-        else
+        if id.is_a? Integer
           super
+        else
+          find { |subnav_item| subnav_item.name.to_s.titleize == id.to_s.titleize }
         end
       end
 
@@ -29,6 +29,7 @@ module TrustyCms
         options = args.extract_options!
         item = args.size > 1 ? deprecated_add(*(args << caller)) : args.first
         raise DuplicateTabNameError.new("duplicate tab name `#{item.name}'") if self[item.name]
+
         item.tab = self if item.respond_to?(:tab=)
         if options.empty?
           super(item)
@@ -79,8 +80,9 @@ module TrustyCms
       attr_reader :name, :url
       attr_accessor :tab
 
-      def initialize(name, url = "#")
-        @name, @url = name, url
+      def initialize(name, url = '#')
+        @name = name
+        @url = url
       end
 
       def visible?(user)
@@ -92,8 +94,9 @@ module TrustyCms
       end
 
       private
+
       def visible_by_controller?(user)
-        params = TrustyCms::Application.routes.recognize_path(url, :method => :get)
+        params = TrustyCms::Application.routes.recognize_path(url, method: :get)
         if params && params[:controller]
           klass = "#{params[:controller].camelize}Controller".constantize
           klass.user_has_access_to_action?(user, params[:action])
@@ -131,24 +134,24 @@ module TrustyCms
     end
 
     def initialize_nav
-      @nav = NavTab.new("Tab Container")
+      @nav = NavTab.new('Tab Container')
       load_default_nav
     end
 
     def load_default_nav
-      content = nav_tab("Content")
-      content << nav_item("Pages", "/admin/pages")
+      content = nav_tab('Content')
+      content << nav_item('Pages', '/admin/pages')
       nav << content
 
-      design = nav_tab("Design")
-      design << nav_item("Layouts", "/admin/layouts")
+      design = nav_tab('Design')
+      design << nav_item('Layouts', '/admin/layouts')
       nav << design
 
-      settings = nav_tab("Settings")
-      settings << nav_item("General", "/admin/configuration")
-      settings << nav_item("Personal", "/admin/preferences")
-      settings << nav_item("Users", "/admin/users")
-      settings << nav_item("Extensions", "/admin/extensions")
+      settings = nav_tab('Settings')
+      settings << nav_item('General', '/admin/configuration')
+      settings << nav_item('Personal', '/admin/preferences')
+      settings << nav_item('Users', '/admin/users')
+      settings << nav_item('Extensions', '/admin/extensions')
       nav << settings
     end
 

data/lib/trusty_cms/admin_ui/region_partials.rb

@@ -1,6 +1,6 @@
 class TrustyCms::AdminUI::RegionPartials
   def initialize(template)
-    @partials = Hash.new {|h,k| h[k] = "<strong>`#{k}' default partial not found!</strong>" }
+    @partials = Hash.new { |h, k| h[k] = "<strong>`#{k}' default partial not found!</strong>" }
     @template = template
   end
 
@@ -8,13 +8,14 @@ class TrustyCms::AdminUI::RegionPartials
     @partials[key.to_s]
   end
 
-  def method_missing(method, *args, &block)
+  def method_missing(method, *_args, &block)
     if block_given?
       # Ruby 1.9.2 yields self in instance_eval... see https://gist.github.com/479572
       # lambdas are as strict as methods in 1.9.x, making sure that the args match, Procs are not.
-      if RUBY_VERSION =~ /^1\.9/ and block.lambda? and block.arity != 1
+      if RUBY_VERSION =~ /^1\.9/ && block.lambda? && (block.arity != 1)
         raise "You can only pass a proc ('Proc.new') or a lambda that takes exactly one arg (for self) to TrustyCms::AdminUI::RegionPartials' method_missing."
       end
+
       @partials[method.to_s] = @template.capture(&block)
     else
       @partials[method.to_s]

data/lib/trusty_cms/admin_ui/region_set.rb

@@ -1,7 +1,6 @@
 class TrustyCms::AdminUI::RegionSet
-
   def initialize
-    @regions = Hash.new do |h,k|
+    @regions = Hash.new do |h, k|
       h[k] = []
     end
     yield self if block_given?
@@ -11,8 +10,9 @@ class TrustyCms::AdminUI::RegionSet
     @regions[region.to_sym]
   end
 
-  def add(region=nil, partial=nil, options={})
-    raise ArgumentError, "You must specify a region and a partial" unless region and partial
+  def add(region = nil, partial = nil, options = {})
+    raise ArgumentError, 'You must specify a region and a partial' unless region && partial
+
     if options[:before]
       index = @regions[region].empty? ? 0 : (@regions[region].index(options[:before]) || @regions[region].size)
       self[region].insert(index, partial)
@@ -31,5 +31,4 @@ class TrustyCms::AdminUI::RegionSet
       super
     end
   end
-
 end

data/lib/trusty_cms/available_locales.rb

@@ -1,5 +1,4 @@
 module TrustyCms::AvailableLocales
-
   # Returns the list of available locale files in options_for_select format.
   #
   def self.locales
@@ -8,12 +7,11 @@ module TrustyCms::AvailableLocales
       if File.exists?(path) && path !~ /_available_tags/
         locale_yaml = YAML.load_file(path)
         stem = File.basename(path, '.yml')
-        if locale_yaml[stem] && lang = locale_yaml[stem]["this_file_language"]
+        if locale_yaml[stem] && lang = locale_yaml[stem]['this_file_language']
           available_locales[lang] = stem
         end
       end
     end
-    available_locales.collect {|k,v| [k, v]}.sort_by { |s| s[0] }
+    available_locales.collect { |k, v| [k, v] }.sort_by { |s| s[0] }
   end
-
 end

data/lib/trusty_cms/config/definition.rb

@@ -1,7 +1,6 @@
 module TrustyCms
   class Config
     class Definition
-
       attr_reader :empty, :default, :type, :notes, :validate_with, :select_from, :allow_blank, :allow_display, :allow_change, :units, :definer
 
       # Configuration 'definitions' are metadata held in memory that add restriction and description to individual config entries.
@@ -22,8 +21,8 @@ module TrustyCms
       #
       # See the method documentation in TrustyCms::Config for options and conventions.
       #
-      def initialize(options={})
-        [:empty, :default, :type, :notes, :validate_with, :select_from, :allow_blank, :allow_change, :allow_display, :units, :definer].each do |attribute|
+      def initialize(options = {})
+        %i[empty default type notes validate_with select_from allow_blank allow_change allow_display units definer].each do |attribute|
           instance_variable_set "@#{attribute}".to_sym, options[attribute]
         end
       end
@@ -61,7 +60,7 @@ module TrustyCms
           choices = select_from
           choices = choices.call if choices.respond_to? :call
           choices = normalize_selection(choices)
-          choices.unshift ["",""] if allow_blank?
+          choices.unshift ['', ''] if allow_blank?
           choices
         end
       end
@@ -71,14 +70,14 @@ module TrustyCms
       #
       def normalize_selection(choices)
         choices = choices.to_a if Hash === choices
-        choices = choices.collect{|c| (c.is_a? Array) ? c : [c,c]}
+        choices = choices.collect { |c| (c.is_a? Array) ? c : [c, c] }
       end
 
       # If the config item is a selector and :select_from specifies [name, value] pairs (as hash or array),
       # this will return the name corresponding to the currently selected value.
       #
       def selected(value)
-        if value && selector? && pair = selection.find{|s| s.last == value}
+        if value && selector? && pair = selection.find { |s| s.last == value }
           pair.first
         end
       end
@@ -103,13 +102,18 @@ module TrustyCms
           setting.errors.add :value, :not_permitted unless selectable?(setting.value)
         end
         if integer?
-          Integer(setting.value) rescue setting.errors.add :value, :not_a_number
+          begin
+            Integer(setting.value)
+          rescue StandardError
+            setting.errors.add :value, :not_a_number
+          end
         end
       end
 
       # Returns true if the value is one of the permitted selections. Not case-sensitive.
       def selectable?(value)
         return true unless selector?
+
         selection.map(&:last).map(&:downcase).include?(value.downcase)
       end
 
@@ -135,7 +139,6 @@ module TrustyCms
       def hidden?
         true if allow_display == false
       end
-
     end
   end
 end

data/lib/trusty_cms/engine.rb

@@ -1,21 +1,21 @@
 module TrustyCms
   class Engine < Rails::Engine
-    paths["app/helpers"] = []
-    initializer "trusty_cms.assets.precompile" do |app|
+    paths['app/helpers'] = []
+    initializer 'trusty_cms.assets.precompile' do |app|
       app.config.assets.paths << Rails.root.join('../trusty-cms/node_modules')
       app.config.assets.precompile += %w(
-      admin/main.css admin.js
-      ckeditor/config.js
-      ckeditor/contents.css
-      ckeditor/config.js
-      ckeditor/styles.js
-      ckeditor/skins/moono/editor.css
-      ckeditor/lang/en.js
-      rad_social/rad_screen.css
-      rad_social/captcha.js
-      rad_social/rad_email.js
-      rad_social/rad_widget.js
-    )
+        admin/main.css admin.js
+        ckeditor/config.js
+        ckeditor/contents.css
+        ckeditor/config.js
+        ckeditor/styles.js
+        ckeditor/skins/moono/editor.css
+        ckeditor/lang/en.js
+        rad_social/rad_screen.css
+        rad_social/captcha.js
+        rad_social/rad_email.js
+        rad_social/rad_widget.js
+      )
     end
   end
 end

data/lib/trusty_cms/extension.rb

@@ -33,7 +33,7 @@ module TrustyCms
     end
 
     def migrations_path
-      File.join(self.root, 'db', 'migrate')
+      File.join(root, 'db', 'migrate')
     end
 
     def migrates_from
@@ -41,11 +41,11 @@ module TrustyCms
     end
 
     def routing_file
-      File.join(self.root, 'config', 'routes.rb')
+      File.join(root, 'config', 'routes.rb')
     end
 
     def load_initializers
-      Dir["#{self.root}/config/initializers/**/*.rb"].sort.each do |initializer|
+      Dir["#{root}/config/initializers/**/*.rb"].sort.each do |initializer|
         require initializer
       end
     end
@@ -53,7 +53,7 @@ module TrustyCms
     def migrator
       unless @migrator
         extension = self
-        @migrator = Class.new(ExtensionMigrator){ self.extension = extension }
+        @migrator = Class.new(ExtensionMigrator) { self.extension = extension }
       end
       @migrator
     end
@@ -62,7 +62,7 @@ module TrustyCms
       AdminUI.instance
     end
 
-    def tab(name, options={}, &block)
+    def tab(name, options = {}, &block)
       @the_tab = admin.nav[name]
       unless @the_tab
         @the_tab = TrustyCms::AdminUI::NavTab.new(name)
@@ -81,7 +81,7 @@ module TrustyCms
       if block_given?
         block.call(@the_tab)
       end
-      return @the_tab
+      @the_tab
     end
     alias :add_tab :tab
 
@@ -95,18 +95,16 @@ module TrustyCms
     #   ThirdPartyExtension.extend(MyExtension::IntegrationPoints)
     # end
     def extension_enabled?(extension)
-      begin
-        extension = (extension.to_s.camelcase + 'Extension').constantize
-        extension.enabled?
-      rescue NameError
-        false
-      end
+      extension = (extension.to_s.camelcase + 'Extension').constantize
+      extension.enabled?
+    rescue NameError
+      false
     end
 
     class << self
-
       def activate_extension
         return if instance.active?
+
         instance.activate_extension if instance.respond_to? :activate
         Dir["#{Rails.root}/config/routes/**/*.rb"].each do |route_file|
         end
@@ -115,6 +113,7 @@ module TrustyCms
 
       def deactivate_extension
         return unless instance.active?
+
         instance.active = false
         instance.deactivate if instance.respond_to? :deactivate
       end
@@ -124,14 +123,13 @@ module TrustyCms
         subclass.extension_name = subclass.name.to_name('Extension')
       end
 
-      def migrate_from(extension_name, until_migration=nil)
+      def migrate_from(extension_name, until_migration = nil)
         instance.migrates_from[extension_name] = until_migration
       end
 
-      def extension_config(&block)
+      def extension_config
         yield Rails.configuration
       end
-
     end
   end
 end