savonrb/savon

View on GitHub

Showing 14 of 14 total issues

Class GlobalOptions has 52 methods (exceeds 20 allowed). Consider refactoring.
Open

  class GlobalOptions < Options
    include SharedOptions

    def initialize(options = {})
      @option_type = :global
Severity: Major
Found in lib/savon/options.rb - About 7 hrs to fix

    Class Builder has 29 methods (exceeds 20 allowed). Consider refactoring.
    Open

      class Builder
        attr_reader :multipart
    
        SCHEMA_TYPES = {
          "xmlns:xsd" => "http://www.w3.org/2001/XMLSchema",
    Severity: Minor
    Found in lib/savon/builder.rb - About 3 hrs to fix

      File options.rb has 286 lines of code (exceeds 250 allowed). Consider refactoring.
      Open

      require "logger"
      require "httpi"
      
      module Savon
        class Options
      Severity: Minor
      Found in lib/savon/options.rb - About 2 hrs to fix

        Method to_hash has a Cognitive Complexity of 19 (exceeds 5 allowed). Consider refactoring.
        Open

            def to_hash(hash, path)
              return hash unless hash
              return hash.map { |value| to_hash(value, path) } if hash.is_a?(Array)
              return hash.to_s unless hash.is_a?(Hash)
        
        
        Severity: Minor
        Found in lib/savon/qualified_message.rb - About 2 hrs 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

        Class Response has 21 methods (exceeds 20 allowed). Consider refactoring.
        Open

          class Response
            CRLF = /\r\n/
            WSP  = /[#{%Q|\x9\x20|}]/
        
            def initialize(http, globals, locals)
        Severity: Minor
        Found in lib/savon/response.rb - About 2 hrs to fix

          Method configure_ssl has a Cognitive Complexity of 14 (exceeds 5 allowed). Consider refactoring.
          Open

              def configure_ssl
                @http_request.auth.ssl.ssl_version   = @globals[:ssl_version]       if @globals.include? :ssl_version
                @http_request.auth.ssl.min_version   = @globals[:ssl_min_version]   if @globals.include? :ssl_min_version
                @http_request.auth.ssl.max_version   = @globals[:ssl_max_version]   if @globals.include? :ssl_max_version
          
          
          Severity: Minor
          Found in lib/savon/request.rb - About 1 hr 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 initialize has 27 lines of code (exceeds 25 allowed). Consider refactoring.
          Open

              def initialize(options = {})
                @option_type = :global
          
                defaults = {
                  :encoding                    => "UTF-8",
          Severity: Minor
          Found in lib/savon/options.rb - About 1 hr to fix

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

                def initialize(message_tag, namespace_identifier, types, used_namespaces, message, element_form_default, key_converter, unwrap)
            Severity: Major
            Found in lib/savon/message.rb - About 1 hr to fix

              Method verify_message! has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.
              Open

                  def verify_message!
                    return if @expected[:message].eql? :any
                    unless equals_except_any(@expected[:message], @actual[:message])
                      expected_message = "  with this message: #{@expected[:message].inspect}" if @expected[:message]
                      expected_message ||= "  with no message."
              Severity: Minor
              Found in lib/savon/mock/expectation.rb - About 55 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 equals_except_any has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.
              Open

                  def equals_except_any(msg_expected, msg_real)
                    return true if msg_expected === msg_real
                    return false if (msg_expected.nil? || msg_real.nil?) # If both are nil has returned true
                    msg_expected.each do |key, expected_value|
                      next if (expected_value == :any &&  msg_real.include?(key))
              Severity: Minor
              Found in lib/savon/mock/expectation.rb - About 55 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 build_xml has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.
              Open

                  def build_xml
                    tag(builder, :Envelope, namespaces_with_globals) do |xml|
                      tag(xml, :Header, header_attributes) { xml << header.to_s } unless header.empty?
                      tag(xml, :Body, body_attributes) do
                        if @globals[:no_message_tag]
              Severity: Minor
              Found in lib/savon/builder.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 to_s has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.
              Open

                  def to_s
                    message_is_xml = @message =~ /^</
                    has_filters    = @filters.any?
                    pretty_print   = @pretty_print
              
              
              Severity: Minor
              Found in lib/savon/log_message.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 initialize has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.
              Open

                  def initialize(globals, locals)
                    @gyoku_options  = { :key_converter => globals[:convert_request_keys_to] }
              
                    @wsse_auth      = locals[:wsse_auth].nil? ? globals[:wsse_auth] : locals[:wsse_auth]
                    @wsse_timestamp = locals[:wsse_timestamp].nil? ? globals[:wsse_timestamp] : locals[:wsse_timestamp]
              Severity: Minor
              Found in lib/savon/header.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 add_attachments_to_multipart_message has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.
              Open

                  def add_attachments_to_multipart_message(multipart_message)
                    if @locals[:attachments].is_a? Hash
                      # hash example: { 'att1' => '/path/to/att1', 'att2' => '/path/to/att2' }
                      @locals[:attachments].each do |identifier, attachment|
                        add_attachment_to_multipart_message(multipart_message, attachment, identifier)
              Severity: Minor
              Found in lib/savon/builder.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