Ruby Critic

Updated 2019-04-01 22:22:43 +0100
db/migrate / 20190401040652_create_trending_domains.rb

14 lines of codes
1 methods
9.4 complexity/method
1 churn
9.4 complexity
0 duplications
class CreateTrendingDomains < ActiveRecord::Migration                                        IrresponsibleModule              
      CreateTrendingDomains has no descriptive comment          
  
  def change                                        TooManyStatements              
      CreateTrendingDomains#change has approx 8 statements          
  
    create_table :trending_domains do |t|                                        UncommunicativeVariableName              
      CreateTrendingDomains#change has the variable name 't'          
  
      t.belongs_to :petition, index: true, foreign_key: true                                        FeatureEnvy              
      CreateTrendingDomains#change refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  
  
      t.string :domain, limit: 100, null: false                                        FeatureEnvy              
      CreateTrendingDomains#change refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  
  
      t.integer :count, null: false                                        FeatureEnvy              
      CreateTrendingDomains#change refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  
  
      t.datetime :starts_at, null: false                                        FeatureEnvy              
      CreateTrendingDomains#change refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  
  
      t.timestamps null: false                                        FeatureEnvy              
      CreateTrendingDomains#change refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  
  
    end

    add_index :trending_domains, [:created_at, :count], order: { starts_at: :desc, count: :desc }
    add_index :trending_domains, [:domain, :petition_id]
  end
end