Ruby Critic

Updated 2015-07-01 12:04:51 +0100
db/migrate / 20150701145201_create_government_responses.rb

13 lines of codes
1 methods
8.2 complexity/method
1 churn
8.21 complexity
38 duplications
class CreateGovernmentResponses < ActiveRecord::Migration                                        IrresponsibleModule              
      CreateGovernmentResponses has no descriptive comment          
  
  def change                                        DuplicateCode              
      Similar code found in 2 nodes              Locations:                  0                  1                  
  
                                        TooManyStatements              
      CreateGovernmentResponses#change has approx 7 statements          
  
    create_table :government_responses do |t|                                        UncommunicativeVariableName              
      CreateGovernmentResponses#change has the variable name 't'          
  
      t.references :petition                                        FeatureEnvy              
      CreateGovernmentResponses#change refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  
  
      t.string :summary, limit: 500, null: false                                        FeatureEnvy              
      CreateGovernmentResponses#change refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  
  
      t.text :details                                        FeatureEnvy              
      CreateGovernmentResponses#change refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  
  
      t.timestamps null: false                                        FeatureEnvy              
      CreateGovernmentResponses#change refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  
  
    end

    add_index :government_responses, :petition_id, unique: true
    add_foreign_key :government_responses, :petitions, on_delete: :cascade
  end
end