Ruby Critic

Updated 2015-08-07 15:47:01 +0100
db/migrate / 20150805142254_create_petition_emails.rb

14 lines of codes
1 methods
9.1 complexity/method
1 churn
9.06 complexity
0 duplications
class CreatePetitionEmails < ActiveRecord::Migration                                        IrresponsibleModule              
      CreatePetitionEmails has no descriptive comment          
  
  def change                                        TooManyStatements              
      CreatePetitionEmails#change has approx 8 statements          
  
    create_table :petition_emails do |t|                                        UncommunicativeVariableName              
      CreatePetitionEmails#change has the variable name 't'          
  
      t.references :petition                                        FeatureEnvy              
      CreatePetitionEmails#change refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  
  
      t.string :subject, null: false                                        FeatureEnvy              
      CreatePetitionEmails#change refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  
  
      t.text :body                                        FeatureEnvy              
      CreatePetitionEmails#change refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  
  
      t.string :sent_by                                        FeatureEnvy              
      CreatePetitionEmails#change refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  
  
      t.timestamps null: false                                        FeatureEnvy              
      CreatePetitionEmails#change refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  
  
    end

    add_index :petition_emails, :petition_id
    add_foreign_key :petition_emails, :petitions, on_delete: :cascade
  end
end