Ruby Critic

Updated 2016-09-10 07:02:58 +0100
db/migrate / 20160910054223_drop_email_sent_receipts.rb

17 lines of codes
2 methods
5.1 complexity/method
1 churn
10.26 complexity
0 duplications
class DropEmailSentReceipts < ActiveRecord::Migration                                        IrresponsibleModule              
      DropEmailSentReceipts has no descriptive comment          
  
  def up
    remove_foreign_key :email_sent_receipts, column: :signature_id
    drop_table :email_sent_receipts
  end

  def down                                        TooManyStatements              
      DropEmailSentReceipts#down has approx 7 statements          
  
    create_table :email_sent_receipts do |t|                                        UncommunicativeVariableName              
      DropEmailSentReceipts#down has the variable name 't'          
  
      t.references :signature, index: true, foreign_key: true                                        FeatureEnvy              
      DropEmailSentReceipts#down refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  5                  
  
      t.datetime :government_response                                        FeatureEnvy              
      DropEmailSentReceipts#down refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  5                  
  
      t.datetime :debate_outcome                                        FeatureEnvy              
      DropEmailSentReceipts#down refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  5                  
  
      t.timestamps null: false                                        FeatureEnvy              
      DropEmailSentReceipts#down refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  5                  
  
      t.datetime :debate_scheduled                                        FeatureEnvy              
      DropEmailSentReceipts#down refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  5                  
  
      t.datetime :petition_email                                        FeatureEnvy              
      DropEmailSentReceipts#down refers to 't' more than self (maybe move it to another class?)              Locations:                  0                  1                  2                  3                  4                  5                  
  
    end
  end
end