Ruby Critic Logo
Updated

db/migrate / 20160910054223_drop_email_sent_receipts.rb

A
17 lines of codes
2 methods
5.1 complexity/method
1 churn
10.26 complexity
0 duplications
  1. class DropEmailSentReceipts < ActiveRecord::Migration
    1. 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
    1. TooManyStatements
      DropEmailSentReceipts#down has approx 7 statements
    create_table :email_sent_receipts do |t|
    1. UncommunicativeVariableName
      DropEmailSentReceipts#down has the variable name 't'
    t.references :signature, index: true, foreign_key: true
    1. 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
    1. 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
    1. 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
    1. 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
    1. 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
    1. 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