Ruby Critic

Updated 2017-09-09 06:41:29 +0100

app/jobs / notify_creators_that_parliament_is_dissolving_job.rb

15 lines of codes

2 methods

2.9 complexity/method

2 churn

5.71 complexity

0 duplications

code
2 smells

class NotifyCreatorsThatParliamentIsDissolvingJob < ApplicationJob                                        IrresponsibleModule              
      NotifyCreatorsThatParliamentIsDissolvingJob has no descriptive comment          
  
  queue_as :high_priority

  def perform
    petitions.find_each do |petition|
      NotifyCreatorThatParliamentIsDissolvingJob.perform_later(petition.creator)
    end
  end

  private

  def petitions                                        UtilityFunction              
      NotifyCreatorsThatParliamentIsDissolvingJob#petitions doesn't depend on instance state (maybe move it to another class?)          
  
    Petition.open_at_dissolution
  end
end