Ruby Critic

Updated 2015-09-13 10:38:27 +0100

db/migrate / 20150913074747_set_last_petition_created_at.rb

18 lines of codes

3 methods

1.9 complexity/method

1 churn

5.62 complexity

0 duplications

class SetLastPetitionCreatedAt < ActiveRecord::Migration                                        IrresponsibleModule              
      SetLastPetitionCreatedAt has no descriptive comment          
  
  class Petition < ActiveRecord::Base; end                                        IrresponsibleModule              
      SetLastPetitionCreatedAt::Petition has no descriptive comment          
  
  class Site < ActiveRecord::Base; end                                        IrresponsibleModule              
      SetLastPetitionCreatedAt::Site has no descriptive comment          
  

  def up
    Site.update_all(last_petition_created_at: last_petition_created_at)
  end

  def down                                        UtilityFunction              
      SetLastPetitionCreatedAt#down doesn't depend on instance state (maybe move it to another class?)          
  
    Site.update_all(last_petition_created_at: nil)
  end

  private

  def last_petition_created_at                                        UtilityFunction              
      SetLastPetitionCreatedAt#last_petition_created_at doesn't depend on instance state (maybe move it to another class?)          
  
    Petition.maximum(:created_at) || Time.current
  end
end