Method update has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

  def update
    @sub_request = SubRequest.find(params[:id])
    return unless user_is_owner_or_admin_of(@sub_request.shift, current_department)
    begin
      SubRequest.transaction do

• Read up
  Read up

Method detailed_stats has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

  def detailed_stats(start_date, end_date)
    shifts_set = shifts.on_days(start_date, end_date).active
    detailed_stats = {}

    shifts_set.each do |s|

• Read up
  Read up

Method search has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

  def search
    users = current_department.active_users
    start_date = interpret_start
    end_date = interpret_end
    #filter results if we are searching

• Read up
  Read up

Method check_for_conflicts has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

  def self.check_for_conflicts(time_slots, wipe, time_slot_scope)
    return "" if time_slots.empty?
    table = TimeSlot.arel_table
    time_slots_with_conflict = Array.new
    time_slots.each_slice(450) do |tss|

• Read up
  Read up

Method narrow_down has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

  def narrow_down(payforms)
    if ( !params[:unsubmitted] and !params[:submitted] and !params[:approved] and !params[:skipped] and !params[:printed]  )
      params[:unsubmitted] = params[:submitted] = params[:approved] = true
    end
    scope = []

• Read up
  Read up

Avoid too many return statements within this method.

Open

      return ["user_profile_entries[#{id}]", :content, options.map{|opt| [opt, opt]}, {selected: content}]

Avoid too many return statements within this method.

Open

        return false

Avoid too many return statements within this method.

Open

      return false

Avoid too many return statements within this method.

Open

      return options.map{|v| ["data_fields[#{id}]", 1, v]}

Avoid too many return statements within this method.

Open

      return "Future"

Avoid too many return statements within this method.

Open

      return false

Avoid too many return statements within this method.

Open

      return "Completed"

Avoid too many return statements within this method.

Open

        return options.map{|v| ["user_profile_entries[#{id}]", v]}

Avoid too many return statements within this method.

Open

        return options.map{|v| ["user_profile_entries[#{id}]", v, chosen.include?(v) ? {checked: true} : {}]}

Avoid too many return statements within this method.

Open

      return options.map{|v| ["user_profile_entries[#{id}]", :content, v, content==v ? {checked: true} : {}]}

Avoid too many return statements within this method.

Open

      return false

Avoid too many return statements within this method.

Open

        return false

Method no_concurrent_timeslots has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

  def no_concurrent_timeslots
    dont_conflict_with_self = (self.new_record? ? "" : "AND id != #{self.id}")

    if self.calendar.active
      c = TimeSlot.where("start < ? AND end > ? AND location_id = ? AND active = ? #{dont_conflict_with_self}", self.end, self.start, self.location, true).count

• Read up
  Read up

Method create has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

  def create
    parse_date_and_time_output(params[:sub_request])
    join_date_and_time(params[:sub_request])
    @sub_request = SubRequest.new(params[:sub_request])
    @sub_request.shift = Shift.find(params[:shift_id])

• Read up
  Read up

Method submit has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

  def submit
    @payform = Payform.find(params[:id])
    return unless user_is_owner_or_admin_of(@payform, @payform.department)
    @payform.submitted = Time.now

• Read up
  Read up