codeplant/simple-navigation

View on GitHub

Showing 9 of 9 total issues

Class Item has 24 methods (exceeds 20 allowed). Consider refactoring.
Open

  class Item
    attr_reader :key,
                :name,
                :sub_navigation,
                :url,
Severity: Minor
Found in lib/simple_navigation/item.rb - About 2 hrs to fix

    Method initialize has 6 arguments (exceeds 4 allowed). Consider refactoring.
    Open

        def initialize(container, key, name, url = nil, opts = {}, &sub_nav_block)
    Severity: Minor
    Found in lib/simple_navigation/item.rb - About 45 mins to fix

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

          def active_navigation_item(options = {}, value_for_nil = nil)
            if options[:level].nil? || options[:level] == :all
              options[:level] = :leaves
            end
            container = active_navigation_item_container(options)
      Severity: Minor
      Found in lib/simple_navigation/helpers.rb - About 45 mins to fix

      Cognitive Complexity

      Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

      A method's cognitive complexity is based on a few simple rules:

      • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
      • Code is considered more complex for each "break in the linear flow of the code"
      • Code is considered more complex when "flow breaking structures are nested"

      Further reading

      Method item has 5 arguments (exceeds 4 allowed). Consider refactoring.
      Open

          def item(key, name, url = nil, options = {}, &block)
      Severity: Minor
      Found in lib/simple_navigation/item_container.rb - About 35 mins to fix

        Method selected_by_autohighlight? has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.
        Open

            def selected_by_autohighlight?
              return false unless auto_highlight?
              return false unless self.url
        
              root_path_match? ||
        Severity: Minor
        Found in lib/simple_navigation/item.rb - About 35 mins to fix

        Cognitive Complexity

        Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

        A method's cognitive complexity is based on a few simple rules:

        • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
        • Code is considered more complex for each "break in the linear flow of the code"
        • Code is considered more complex when "flow breaking structures are nested"

        Further reading

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

              def a_tags(item_container)
                item_container.items.each_with_object([]) do |item, list|
                  next unless item.selected?
                  list << tag_for(item)
        
        
        Severity: Minor
        Found in lib/simple_navigation/renderer/breadcrumbs.rb - About 35 mins to fix

        Cognitive Complexity

        Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

        A method's cognitive complexity is based on a few simple rules:

        • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
        • Code is considered more complex for each "break in the linear flow of the code"
        • Code is considered more complex when "flow breaking structures are nested"

        Further reading

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

            def level_for_item(navi_key)
              return level if self[navi_key]
        
              items.each do |item|
                next unless item.sub_navigation
        Severity: Minor
        Found in lib/simple_navigation/item_container.rb - About 35 mins to fix

        Cognitive Complexity

        Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

        A method's cognitive complexity is based on a few simple rules:

        • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
        • Code is considered more complex for each "break in the linear flow of the code"
        • Code is considered more complex when "flow breaking structures are nested"

        Further reading

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

            def items(items_provider = nil, &block)
              if (items_provider && block) || (items_provider.nil? && block.nil?)
                fail('please specify either items_provider or block, but not both')
              end
        
        
        Severity: Minor
        Found in lib/simple_navigation/configuration.rb - About 25 mins to fix

        Cognitive Complexity

        Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

        A method's cognitive complexity is based on a few simple rules:

        • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
        • Code is considered more complex for each "break in the linear flow of the code"
        • Code is considered more complex when "flow breaking structures are nested"

        Further reading

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

              def render(item_container)
                if skip_if_empty? && item_container.empty?
                  ''
                else
                  tag = options[:ordered] ? :ol : :ul
        Severity: Minor
        Found in lib/simple_navigation/renderer/list.rb - About 25 mins to fix

        Cognitive Complexity

        Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

        A method's cognitive complexity is based on a few simple rules:

        • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
        • Code is considered more complex for each "break in the linear flow of the code"
        • Code is considered more complex when "flow breaking structures are nested"

        Further reading

        Severity
        Category
        Status
        Source
        Language