Method show_empty_message_in_source_container has a Cognitive Complexity of 45 (exceeds 5 allowed). Consider refactoring.

Open

  def show_empty_message_in_source_container
    container_id = ""
    source_view do |page|
      page.project  {
        container_id = "p#{@original_item_project_id}empty-nd" if @remaining_in_context == 0

• Read up
  Read up

File todos_helper.rb has 410 lines of code (exceeds 250 allowed). Consider refactoring.

Open

module TodosHelper

  def remote_star_icon(todo=@todo)
    link_to( image_tag_for_star(todo),
      toggle_star_todo_path(todo),

Method project_and_context_links has a Cognitive Complexity of 20 (exceeds 5 allowed). Consider refactoring.

Open

  def project_and_context_links(todo, parent_container_type, opts = {})
    str = ''
    if todo.completed?
      str += todo.context.name unless opts[:suppress_context]
      should_suppress_project = opts[:suppress_project] || todo.project.nil?

• Read up
  Read up

Consider simplifying this complex logical expression.

Open

    return (@remaining_in_context == 0 && (@todo_hidden_state_changed && @todo.hidden?)) ||
      (@remaining_in_context == 0 && @todo_was_deferred_from_active_state) ||
      (@remaining_in_context == 0 && @tag_was_removed) ||
      (@remaining_in_context == 0 && @todo.completed? && !(@original_item_was_deferred || @original_item_was_hidden)) if source_view_is(:tag)

Method empty_container_msg_div_id has a Cognitive Complexity of 17 (exceeds 5 allowed). Consider refactoring.

Open

  def empty_container_msg_div_id(todo = @todo || @successor)
    raise Exception.new, "no @todo or @successor set" if !todo

    source_view do |page|
      page.project  {

• Read up
  Read up

Method empty_container_msg_div_id has 45 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  def empty_container_msg_div_id(todo = @todo || @successor)
    raise Exception.new, "no @todo or @successor set" if !todo

    source_view do |page|
      page.project  {

Method should_show_new_item has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring.

Open

  def should_show_new_item
    source_view do |page|
      page.todo { return !@todo.hidden? }
      page.deferred { return @todo.deferred? || @todo.pending? }
      page.context {

• Read up
  Read up

Method update_needs_to_hide_context has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring.

Open

  def update_needs_to_hide_context
    return (@remaining_in_context == 0 && (@todo_hidden_state_changed && @todo.hidden?)) ||
      (@remaining_in_context == 0 && @todo_was_deferred_from_active_state) ||
      (@remaining_in_context == 0 && @tag_was_removed) ||
      (@remaining_in_context == 0 && @todo.completed? && !(@original_item_was_deferred || @original_item_was_hidden)) if source_view_is(:tag)

• Read up
  Read up

Method item_container_id has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

  def item_container_id (todo)
    return "hiddenitems"              if source_view_is(:tag) && todo.hidden?
    return "c#{todo.context_id}items" if source_view_is :deferred
    return @new_due_id                if source_view_is :calendar
    return "tickleritems"             if !source_view_is(:todo) && (todo.deferred? || todo.pending?)

• Read up
  Read up

Method feed_content_for_todo has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

  def feed_content_for_todo(todo)
    item_notes = todo.notes ? todo.rendered_notes : ''
    due = todo.due ? content_tag(:div, t('todos.feeds.due', :date => format_date(todo.due))) : ''
    done = todo.completed? ? content_tag(:div, t('todos.feeds.completed', :date => format_date(todo.completed_at))) : ''
    context_link = link_to(context_url(todo.context), todo.context.name)

• Read up
  Read up

Consider simplifying this complex logical expression.

Open

        return "tickler-empty-nd" if
          @todo_was_deferred_from_active_state ||
          @todo_was_blocked_from_active_state ||
          @todo_was_destroyed_from_deferred_state ||
          @todo_was_created_deferred ||

Consider simplifying this complex logical expression.

Open

        container_id = "hidden-empty-nd" if (@remaining_hidden_count == 0 && !@todo.hidden? && @todo_hidden_state_changed) ||
          (@remaining_hidden_count == 0 && @todo.completed? && @original_item_was_hidden)

Consider simplifying this complex logical expression.

Open

        return "tickler-empty-nd" if
          @todo_was_deferred_from_active_state ||
          @todo_was_blocked_from_active_state ||
          @todo_was_destroyed_from_deferred_state ||
          @todo_was_created_deferred ||

Consider simplifying this complex logical expression.

Open

        return "tickler-empty-nd" if
          @todo_was_deferred_from_active_state ||
          @todo_was_blocked_from_active_state ||
          @todo_was_destroyed_from_deferred_state ||
          @todo_was_created_deferred ||

Consider simplifying this complex logical expression.

Open

        return "tickler-empty-nd" if
          @todo_was_deferred_from_active_state ||
          @todo_was_blocked_from_active_state ||
          @todo_was_destroyed_from_deferred_state ||
          @todo_was_created_deferred ||

Consider simplifying this complex logical expression.

Open

        container_id = "tickler-empty-nd" if (todo_was_removed_from_deferred_or_blocked_container && @remaining_deferred_or_pending_count == 0) ||
          (@original_item_was_deferred && @remaining_deferred_or_pending_count == 0 && (@todo.completed? || @tag_was_removed))

Avoid too many return statements within this method.

Open

      page.todo     { return @context_changed || @todo.hidden? || @todo.deferred? || @todo.pending?}

Avoid too many return statements within this method.

Open

        return "c#{todo.context_id}empty-nd"

Avoid too many return statements within this method.

Open

        return (@todo.active? && @todo.project && @todo.project.id == @default_project.id) ||
          (@todo.project.hidden? && @todo.project_hidden?) ||
          ( @todo.deferred? && (@todo.project.id == @default_project.id) ) ||
          @todo.pending?

Avoid too many return statements within this method.

Open

        return "c#{todo.context_id}empty-nd"

Avoid too many return statements within this method.

Open

    return false

Avoid too many return statements within this method.

Open

        return "empty_#{@new_due_id}"

Avoid too many return statements within this method.

Open

      page.stats    { return @todo.completed? }

Avoid too many return statements within this method.

Open

        return "tickler-empty-nd" if

Avoid too many return statements within this method.

Open

    return "c#{todo.context_id}empty-nd"

Avoid too many return statements within this method.

Open

    return 'context'

Avoid too many return statements within this method.

Open

      page.tag      { return !update_needs_to_remove_todo_from_container && !@tag_was_removed }

Avoid too many return statements within this method.

Open

      page.tag      { return update_needs_to_remove_todo_from_container && !@tag_was_removed}

Avoid too many return statements within this method.

Open

        return "tickler-empty-nd" if

Avoid too many return statements within this method.

Open

      page.tag      { return (@context_changed && !@todo.hidden?) || @tag_was_removed || @todo_hidden_state_changed || @todo_deferred_state_changed }

Avoid too many return statements within this method.

Open

    return "c#{todo.context_id}items"

Avoid too many return statements within this method.

Open

    return false

Avoid too many return statements within this method.

Open

      page.search   { return false }

Avoid too many return statements within this method.

Open

      page.todo     { return !update_needs_to_remove_todo_from_container }

Avoid too many return statements within this method.

Open

    return false

Avoid too many return statements within this method.

Open

      page.stats    { return false }

Avoid too many return statements within this method.

Open

      page.todo     { return @context_changed && !(@todo.deferred? || @todo.pending? || @todo.hidden?) }

Avoid too many return statements within this method.

Open

      page.search   { return true }

Avoid too many return statements within this method.

Open

    return "completed_containeritems" if todo.completed?

Avoid too many return statements within this method.

Open

      page.stats    { return !@todo.completed? }

Avoid too many return statements within this method.

Open

    return false

Avoid too many return statements within this method.

Open

    return "p#{todo.project_id}items" if source_view_is :project

Avoid too many return statements within this method.

Open

      return ""

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

Open

  def remote_defer_menu_item(days, todo)
    url = {:controller => 'todos', :action => 'defer', :id => todo.id, :days => days,
      :_source_view => (@source_view.underscore.gsub(/\s+/,'_') rescue "")}
    url[:_tag_name] = @tag_name if @source_view == 'tag'

• Read up
  Read up

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

Open

  def staleness_class(item)
    if item.due || item.completed?
      return ""
    elsif item.created_at < current_user.time - (prefs.staleness_starts * 3).days
      return " stale_l3"

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

      page.context  {
        container_id = "c#{@original_item_context_id}empty-nd" if @remaining_in_context == 0
        container_id = "tickler-empty-nd" if todo_was_removed_from_deferred_or_blocked_container && @remaining_deferred_or_pending_count == 0
        container_id = "empty-d" if @completed_count && @completed_count == 0 && !@todo.completed?
      }

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

      page.project  {
        container_id = "p#{@original_item_project_id}empty-nd" if @remaining_in_context == 0
        container_id = "tickler-empty-nd" if todo_was_removed_from_deferred_or_blocked_container && @remaining_deferred_or_pending_count == 0
        container_id = "empty-d" if @completed_count && @completed_count == 0 && !@todo.completed?
      }

• Read up
  Read up