Ruby Critic

Updated 2016-07-14 20:13:05 +0100
db/migrate / 20160713124623_create_rate_limits.rb

13 lines of codes
1 methods
11.5 complexity/method
1 churn
11.54 complexity
0 duplications
class CreateRateLimits < ActiveRecord::Migration                                        IrresponsibleModule              
      CreateRateLimits has no descriptive comment          
  
  def change                                        TooManyStatements              
      CreateRateLimits#change has approx 8 statements          
  
    create_table :rate_limits do |t|                                        UncommunicativeVariableName              
      CreateRateLimits#change has the variable name 't'          
  
      t.integer :burst_rate, null: false, default: 1                                        FeatureEnvy              
      CreateRateLimits#change refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  5                  6                  
  
      t.integer :burst_period, null: false, default: 60                                        FeatureEnvy              
      CreateRateLimits#change refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  5                  6                  
  
      t.integer :sustained_rate, null: false, default: 5                                        FeatureEnvy              
      CreateRateLimits#change refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  5                  6                  
  
      t.integer :sustained_period, null: false, default: 300                                        FeatureEnvy              
      CreateRateLimits#change refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  5                  6                  
  
      t.string  :domain_whitelist, null: false, limit: 10000, default: ""                                        FeatureEnvy              
      CreateRateLimits#change refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  5                  6                  
  
      t.string  :ip_whitelist, null: false, limit: 10000, default: ""                                        FeatureEnvy              
      CreateRateLimits#change refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  5                  6                  
  
      t.timestamps null: false                                        FeatureEnvy              
      CreateRateLimits#change refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  5                  6                  
  
    end
  end
end