rails_admin 0.6.0 → 0.6.1

data/spec/dummy_app/app/mongoid/image.rb

@@ -2,6 +2,6 @@ class Image
   include Mongoid::Document
   include Mongoid::Paperclip
 
-  has_mongoid_attached_file :file, :styles => { :medium => "300x300>", :thumb => "100x100>" }
+  has_mongoid_attached_file :file, styles: {medium: '300x300>', thumb: '100x100>'}
   validates_attachment_presence :file
 end

data/spec/dummy_app/app/mongoid/league.rb

@@ -1,14 +1,14 @@
 class League
   include Mongoid::Document
 
-  field :name, :type => String
+  field :name, type: String
   include Mongoid::Timestamps
 
-  has_many :divisions, :foreign_key => 'custom_league_id'
+  has_many :divisions, foreign_key: 'custom_league_id'
 
   validates_presence_of(:name)
 
   def custom_name
-    "League '#{self.name}'"
+    "League '#{name}'"
   end
 end

data/spec/dummy_app/app/mongoid/nested_field_test.rb

@@ -1,11 +1,11 @@
 class NestedFieldTest
   include Mongoid::Document
 
-  field :title, :type => String
-  belongs_to :field_test, :inverse_of => :nested_field_tests
-  belongs_to :another_field_test, :inverse_of => :nested_field_tests
+  field :title, type: String
+  belongs_to :field_test, inverse_of: :nested_field_tests
+  belongs_to :another_field_test, inverse_of: :nested_field_tests
   include Mongoid::Timestamps
 
-  has_one :comment, :as => :commentable, :autosave => true
-  accepts_nested_attributes_for :comment, :allow_destroy => true, :reject_if => proc { |attributes| attributes["content"].blank? }
+  has_one :comment, as: :commentable, autosave: true
+  accepts_nested_attributes_for :comment, allow_destroy: true, reject_if: proc { |attributes| attributes['content'].blank? }
 end

data/spec/dummy_app/app/mongoid/player.rb

@@ -3,37 +3,37 @@ class Player
   include Mongoid::Timestamps
   include ActiveModel::ForbiddenAttributesProtection
 
-  field :deleted_at, :type => DateTime
-  belongs_to :team, :inverse_of => :players
-  field :name, :type => String
-  field :position, :type => String
-  field :number, :type => Integer
-  field :retired, :type => Boolean, :default => false
-  field :injured, :type => Boolean, :default => false
-  field :born_on, :type => Date
-  field :notes, :type => String
-  field :suspended, :type => Boolean, :default => false
+  field :deleted_at, type: DateTime
+  belongs_to :team, inverse_of: :players
+  field :name, type: String
+  field :position, type: String
+  field :number, type: Integer
+  field :retired, type: Boolean, default: false
+  field :injured, type: Boolean, default: false
+  field :born_on, type: Date
+  field :notes, type: String
+  field :suspended, type: Boolean, default: false
 
   validates_presence_of(:name)
-  validates_numericality_of(:number, :only_integer => true)
-  validates_uniqueness_of(:number, :scope => :team_id, :message => "There is already a player with that number on this team")
+  validates_numericality_of(:number, only_integer: true)
+  validates_uniqueness_of(:number, scope: :team_id, message: 'There is already a player with that number on this team')
 
   validates_each :name do |record, attr, value|
-    record.errors.add(:base, "Player is cheating") if value.to_s =~ /on steroids/
+    record.errors.add(:base, 'Player is cheating') if value.to_s =~ /on steroids/
   end
 
-  has_one :draft, :dependent => :destroy
-  has_many :comments, :as => :commentable
+  has_one :draft, dependent: :destroy
+  has_many :comments, as: :commentable
 
   before_destroy :destroy_hook
 
   def destroy_hook; end
 
   def draft_id
-    self.draft.try :id
+    draft.try :id
   end
 
   def draft_id=(id)
-    self.draft = Draft.where(:_id => id).first
+    self.draft = Draft.where(_id: id).first
   end
 end

data/spec/dummy_app/app/mongoid/team.rb

@@ -5,41 +5,45 @@ class Team
   include Mongoid::Timestamps
 
   belongs_to :division
-  field :name, :type => String
-  field :logo_url, :type => String
-  field :manager, :type => String
-  field :ballpark, :type => String
-  field :mascot, :type => String
-  field :founded, :type => Integer
-  field :wins, :type => Integer
-  field :losses, :type => Integer
-  field :win_percentage, :type => Float
-  field :revenue, :type => BigDecimal
-  field :color, :type => String
-  field :custom_field, :type => String
+  field :name, type: String
+  field :logo_url, type: String
+  field :manager, type: String
+  field :ballpark, type: String
+  field :mascot, type: String
+  field :founded, type: Integer
+  field :wins, type: Integer
+  field :losses, type: Integer
+  field :win_percentage, type: Float
+  field :revenue, type: BigDecimal
+  field :color, type: String
+  field :custom_field, type: String
 
-  has_many :players, :inverse_of => :team, :order => :_id.asc
+  has_many :players, inverse_of: :team, order: :_id.asc
   has_and_belongs_to_many :fans
-  has_many :comments, :as => :commentable
+  has_many :comments, as: :commentable
 
-  validates_presence_of :division_id, :only_integer => true
+  validates_presence_of :division_id, only_integer: true
   validates_presence_of :manager
-  validates_numericality_of :founded, :only_integer => true
-  validates_numericality_of :wins, :only_integer => true
-  validates_numericality_of :losses, :only_integer => true
+  validates_numericality_of :founded, only_integer: true
+  validates_numericality_of :wins, only_integer: true
+  validates_numericality_of :losses, only_integer: true
   validates_numericality_of :win_percentage
-  validates_numericality_of :revenue, :allow_nil => true
+  validates_numericality_of :revenue, allow_nil: true
   # needed to force these attributes to :string type
-  validates_length_of :logo_url, :maximum => 255
-  validates_length_of :manager, :maximum => 100
-  validates_length_of :ballpark, :maximum => 100
-  validates_length_of :mascot, :maximum => 100
+  validates_length_of :logo_url, maximum: 255
+  validates_length_of :manager, maximum: 100
+  validates_length_of :ballpark, maximum: 100
+  validates_length_of :mascot, maximum: 100
 
   def player_names_truncated
-    players.map{|p| p.name}.join(", ")[0..32]
+    players.collect { |p| p.name }.join(', ')[0..32]
   end
 
   def color_enum
     ['white', 'black', 'red', 'green', 'blu<e>é']
   end
+
+  scope :green, -> { where(color: 'red') }
+  scope :red, -> { where(color: 'red') }
+  scope :white, -> { where(color: 'white') }
 end

data/spec/dummy_app/app/mongoid/user.rb

@@ -7,22 +7,22 @@ class User
   devise :database_authenticatable, :registerable, :recoverable, :rememberable, :trackable, :validatable
 
   ## Database authenticatable
-  field :email,              :type => String
-  field :encrypted_password, :type => String
+  field :email,              type: String
+  field :encrypted_password, type: String
 
   ## Recoverable
-  field :reset_password_token,   :type => String
-  field :reset_password_sent_at, :type => Time
+  field :reset_password_token,   type: String
+  field :reset_password_sent_at, type: Time
 
   ## Rememberable
-  field :remember_created_at, :type => Time
+  field :remember_created_at, type: Time
 
   ## Trackable
-  field :sign_in_count,      :type => Integer
-  field :current_sign_in_at, :type => Time
-  field :last_sign_in_at,    :type => Time
-  field :current_sign_in_ip, :type => String
-  field :last_sign_in_ip,    :type => String
+  field :sign_in_count,      type: Integer
+  field :current_sign_in_at, type: Time
+  field :last_sign_in_at,    type: Time
+  field :current_sign_in_ip, type: String
+  field :last_sign_in_ip,    type: String
 
   ## Encryptable
   # field :password_salt, :type => String
@@ -47,14 +47,14 @@ class User
   include Mongoid::Timestamps
 
   # Add Paperclip support for avatars
-  has_mongoid_attached_file :avatar, :styles => { :medium => "300x300>", :thumb => "100x100>" }
+  has_mongoid_attached_file :avatar, styles: {medium: '300x300>', thumb: '100x100>'}
 
-  field :roles, :type => Array
+  field :roles, type: Array
 
   def attr_accessible_role
     :custom_role
   end
 
   attr_accessor :delete_avatar
-  before_validation { self.avatar = nil if self.delete_avatar == '1' }
+  before_validation { self.avatar = nil if delete_avatar == '1' }
 end

data/spec/dummy_app/config/application.rb

@@ -1,12 +1,12 @@
 require File.expand_path('../boot', __FILE__)
 
-require "action_controller/railtie"
-require "action_mailer/railtie"
-require "sprockets/railtie"
+require 'action_controller/railtie'
+require 'action_mailer/railtie'
+require 'sprockets/railtie'
 
 begin
   require "#{CI_ORM}/railtie"
-rescue LoadError
+rescue LoadError # rubocop:disable HandleExceptions
 end
 
 # Require the gems listed in Gemfile, including any gems
@@ -18,7 +18,7 @@ module DummyApp
     # Settings in config/environments/* take precedence over those specified here.
     # Application configuration should go into files in config/initializers
     # -- all .rb files in that directory are automatically loaded.
-    config.eager_load_paths.reject!{ |p| p =~ /\/app\/(\w+)$/ && !%w(controllers helpers views).push(CI_ORM).include?($1) }
+    config.eager_load_paths.reject! { |p| p =~ %r{/app/(\w+)$} && !%w(controllers helpers views).push(CI_ORM).include?(Regexp.last_match[1]) }
     config.autoload_paths += %W(#{config.root}/app/#{CI_ORM} #{config.root}/app/#{CI_ORM}/concerns)
 
     # Set Time.zone default to the specified zone and make Active Record auto-convert to this zone.

data/spec/dummy_app/config/boot.rb

@@ -1,6 +1,4 @@
-unless defined?(CI_ORM)
-  CI_ORM = (ENV["CI_ORM"] || :active_record).to_sym
-end
+CI_ORM = (ENV['CI_ORM'] || :active_record).to_sym unless defined?(CI_ORM)
 require 'rubygems'
 
 # Set up gems listed in the Gemfile.

data/spec/dummy_app/config/environments/test.rb

@@ -14,7 +14,7 @@ DummyApp::Application.configure do
 
   # Configure static asset server for tests with Cache-Control for performance.
   config.serve_static_assets  = true
-  config.static_cache_control = "public, max-age=3600"
+  config.static_cache_control = 'public, max-age=3600'
 
   # Show full error reports and disable caching.
   config.consider_all_requests_local       = true

data/spec/dummy_app/config/initializers/devise.rb

@@ -41,12 +41,12 @@ Devise.setup do |config|
   # Configure which authentication keys should be case-insensitive.
   # These keys will be downcased upon creating or modifying a user and when used
   # to authenticate or find a user. Default is :email.
-  config.case_insensitive_keys = [ :email ]
+  config.case_insensitive_keys = [:email]
 
   # Configure which authentication keys should have whitespace stripped.
   # These keys will have whitespace before and after removed upon creating or
   # modifying a user and when used to authenticate or find a user. Default is :email.
-  config.strip_whitespace_keys = [ :email ]
+  config.strip_whitespace_keys = [:email]
 
   # Tell if authentication through request.params is enabled. True by default.
   # It can be set to an array that will enable params authentication only for the

data/spec/dummy_app/config/initializers/session_patch.rb

@@ -9,8 +9,7 @@ require 'action_dispatch/middleware/session/abstract_store'
 ActionDispatch::Session::StaleSessionCheck.module_eval do
   def stale_session_check!
     yield
-  rescue ArgumentError => argument_error
+  rescue ArgumentError
     {}
   end
 end
-

data/spec/dummy_app/config/initializers/session_store.rb

@@ -1,3 +1,3 @@
 # Be sure to restart your server when you modify this file.
 
-DummyApp::Application.config.session_store :cookie_store, :key => '_dummy_app_session'
+DummyApp::Application.config.session_store :cookie_store, key: '_dummy_app_session'

data/spec/dummy_app/config/initializers/wrap_parameters.rb

@@ -5,7 +5,7 @@
 
 # Enable parameter wrapping for JSON. You can disable this by setting :format to an empty array.
 ActiveSupport.on_load(:action_controller) do
-  wrap_parameters :format => [:json] if respond_to?(:wrap_parameters)
+  wrap_parameters format: [:json] if respond_to?(:wrap_parameters)
 end
 
 # To enable root element in JSON for ActiveRecord objects.

data/spec/dummy_app/config/routes.rb

@@ -1,10 +1,10 @@
 DummyApp::Application.routes.draw do
   # Needed for :show_in_app tests
-  resources :players, :only => [:show]
+  resources :players, only: [:show]
 
   devise_for :users
   mount RailsAdmin::Engine => '/admin', :as => 'rails_admin'
-  root :to => "rails_admin/main#dashboard"
+  root to: 'rails_admin/main#dashboard'
   # https://github.com/sferik/rails_admin/issues/362
   get ':controller(/:action(/:id(.:format)))'
 end

data/spec/dummy_app/db/migrate/00000000000001_create_divisions_migration.rb

@@ -3,7 +3,7 @@ class CreateDivisionsMigration < ActiveRecord::Migration
     create_table :divisions do |t|
       t.timestamps
       t.integer :league_id
-      t.string :name, :limit => 50, :null => false
+      t.string :name, limit: 50, null: false
     end
   end
 

data/spec/dummy_app/db/migrate/00000000000002_create_drafts_migration.rb

@@ -8,7 +8,7 @@ class CreateDraftsMigration < ActiveRecord::Migration
       t.integer :round
       t.integer :pick
       t.integer :overall
-      t.string :college, :limit => 100
+      t.string :college, limit: 100
       t.text :notes
     end
   end

data/spec/dummy_app/db/migrate/00000000000003_create_leagues_migration.rb

@@ -2,7 +2,7 @@ class CreateLeaguesMigration < ActiveRecord::Migration
   def self.up
     create_table :leagues do |t|
       t.timestamps
-      t.string :name, :limit => 50, :null => false
+      t.string :name, limit: 50, null: false
     end
   end
 

data/spec/dummy_app/db/migrate/00000000000004_create_players_migration.rb

@@ -4,11 +4,11 @@ class CreatePlayersMigration < ActiveRecord::Migration
       t.timestamps
       t.datetime :deleted_at
       t.integer :team_id
-      t.string :name, :limit => 100, :null => false
-      t.string :position, :limit => 50
-      t.integer :number, :null => false
-      t.boolean :retired, :default => false
-      t.boolean :injured, :default => false
+      t.string :name, limit: 100, null: false
+      t.string :position, limit: 50
+      t.integer :number, null: false
+      t.boolean :retired, default: false
+      t.boolean :injured, default: false
       t.date :born_on
       t.text :notes
     end

data/spec/dummy_app/db/migrate/00000000000005_create_teams_migration.rb

@@ -4,11 +4,11 @@ class CreateTeamsMigration < ActiveRecord::Migration
       t.timestamps
       t.integer :league_id
       t.integer :division_id
-      t.string :name, :limit => 50
-      t.string :logo_url, :limit => 255
-      t.string :manager, :limit => 100, :null => false
-      t.string :ballpark, :limit => 100
-      t.string :mascot, :limit => 100
+      t.string :name, limit: 50
+      t.string :logo_url, limit: 255
+      t.string :manager, limit: 100, null: false
+      t.string :ballpark, limit: 100
+      t.string :mascot, limit: 100
       t.integer :founded
       t.integer :wins
       t.integer :losses

data/spec/dummy_app/db/migrate/00000000000006_devise_create_users.rb

@@ -2,8 +2,8 @@ class DeviseCreateUsers < ActiveRecord::Migration
   def self.up
     create_table :users do |t|
       ## Database authenticatable
-      t.string :email,              :null => false, :default => ""
-      t.string :encrypted_password, :null => false, :default => ""
+      t.string :email,              null: false, default: ''
+      t.string :encrypted_password, null: false, default: ''
 
       ## Recoverable
       t.string   :reset_password_token
@@ -13,7 +13,7 @@ class DeviseCreateUsers < ActiveRecord::Migration
       t.datetime :remember_created_at
 
       ## Trackable
-      t.integer  :sign_in_count, :default => 0
+      t.integer  :sign_in_count, default: 0
       t.datetime :current_sign_in_at
       t.datetime :last_sign_in_at
       t.string   :current_sign_in_ip
@@ -39,8 +39,8 @@ class DeviseCreateUsers < ActiveRecord::Migration
       t.timestamps
     end
 
-    add_index :users, :email, :unique => true
-    add_index :users, :reset_password_token, :unique => true
+    add_index :users, :email, unique: true
+    add_index :users, :reset_password_token, unique: true
   end
 
   def self.down

data/spec/dummy_app/db/migrate/00000000000007_create_histories_table.rb

@@ -1,11 +1,11 @@
 class CreateHistoriesTable < ActiveRecord::Migration
-   def self.up
-     create_table :histories do |t|
-       t.string :message # title, name, or object_id
-       t.string :username
-       t.integer :item
-       t.string :table
-       t.timestamps
+  def self.up
+    create_table :histories do |t|
+      t.string :message # title, name, or object_id
+      t.string :username
+      t.integer :item
+      t.string :table
+      t.timestamps
     end
     add_index(:histories, [:item, :table])
   end