Ruby Critic

Updated 2019-05-10 20:21:50 +0100
db/migrate / 20190501120348_create_domains.rb

14 lines of codes
1 methods
10.1 complexity/method
1 churn
10.1 complexity
0 duplications
class CreateDomains < ActiveRecord::Migration                                        IrresponsibleModule              
      CreateDomains has no descriptive comment          
  
  def change                                        TooManyStatements              
      CreateDomains#change has approx 8 statements          
  
    create_table :domains do |t|                                        UncommunicativeVariableName              
      CreateDomains#change has the variable name 't'          
  
      t.belongs_to :canonical_domain, index: true                                        FeatureEnvy              
      CreateDomains#change refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  
  
      t.string     :name, limit: 100, null: false                                        FeatureEnvy              
      CreateDomains#change refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  
  
      t.string     :strip_characters, limit: 10                                        FeatureEnvy              
      CreateDomains#change refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  
  
      t.string     :strip_extension, limit: 10, default: "+"                                        FeatureEnvy              
      CreateDomains#change refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  
  
      t.timestamps null: false                                        FeatureEnvy              
      CreateDomains#change refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  
  
    end

    add_index :domains, :name, unique: true
    add_foreign_key :domains, :domains, column: :canonical_domain_id
  end
end