angelakuo/citydogshare

View on GitHub

Showing 2,136 of 3,976 total issues

Function _dialogDatepicker has 5 arguments (exceeds 4 allowed). Consider refactoring.
Open

    _dialogDatepicker: function(input, date, onSelect, settings, pos) {

    Function switchClass has 5 arguments (exceeds 4 allowed). Consider refactoring.
    Open

        switchClass: function( remove, add, speed, easing, callback) {

      Function ArrayContaining has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.
      Open

      getJasmineRequireObj().ArrayContaining = function(j$) {
        function ArrayContaining(sample) {
          this.sample = sample;
        }
      
      

      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 all_info has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.
      Open

        def all_info
          filtered_dogs_hash = Dog.all.select { |dog|
            valid_dog = true
            if params[:available] == "true" && !dog.available then valid_dog = false end
            if params[:available] == "false" && dog.available then valid_dog = false end
      Severity: Minor
      Found in app/controllers/dogs_controller.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 valid_start_end_dates? has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.
      Open

        def valid_start_end_dates?
          errors.add(:start_date, "Start date has passed") if start_date.present? and Date.today > start_date
          errors.add(:end_date, "End date has passed") if end_date.present? and Date.today > end_date
          if start_date.present? and end_date.present?
            errors.add(:start_end_date, "Start date must be before end date") if start_date > end_date
      Severity: Minor
      Found in app/models/event.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

      Avoid too many return statements within this function.
      Open

                  return sortInput ?
                      ( indexOf( sortInput, a ) - indexOf( sortInput, b ) ) :
                      0;

        Avoid too many return statements within this function.
        Open

                return ret == null ? undefined : ret;

          Avoid too many return statements within this function.
          Open

                      return root.ready !== undefined ?
                          root.ready( selector ) :
          
                          // Execute immediately if ready is not present
                          selector( jQuery );

            Avoid too many return statements within this function.
            Open

                                return results;

              Avoid too many return statements within this function.
              Open

                  return select( selector.replace( rtrim, "$1" ), context, results, seed );

                Avoid too many return statements within this function.
                Open

                                return this.constructor( context ).find( selector );

                  Avoid too many return statements within this function.
                  Open

                                          return results;

                    Avoid too many return statements within this function.
                    Open

                                return ret;

                      Avoid too many return statements within this function.
                      Open

                                  return this;

                        Avoid too many return statements within this function.
                        Open

                                return elem[ name ];

                          Avoid too many return statements within this function.
                          Open

                                  return compare & 4 ? -1 : 1;

                            Avoid too many return statements within this function.
                            Open

                                        return value;

                              Avoid too many return statements within this function.
                              Open

                                                  return results;

                                Avoid too many return statements within this function.
                                Open

                                        return jQuery.makeArray( selector, this );

                                  Avoid too many return statements within this function.
                                  Open

                                      return '';
                                    Severity
                                    Category
                                    Status
                                    Source
                                    Language