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 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

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

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 = "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

        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))

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 ||

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

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

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 "tickler-empty-nd" if

Avoid too many return statements within this method.

Open

      page.search   { return true }

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

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

Avoid too many return statements within this method.

Open

    return false

Avoid too many return statements within this method.

Open

      page.stats    { return @todo.completed? }

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

        return "tickler-empty-nd" if

Avoid too many return statements within this method.

Open

      page.stats    { 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

        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

      page.todo     { return !update_needs_to_remove_todo_from_container }

Avoid too many return statements within this method.

Open

    return "completed_containeritems" if todo.completed?

Avoid too many return statements within this method.

Open

    return "c#{todo.context_id}items"

Avoid too many return statements within this method.

Open

      page.search   { return false }

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 "p#{todo.project_id}items" if source_view_is :project

Avoid too many return statements within this method.

Open

      return ""

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 'context'

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?

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

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

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

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