holderdeord/hdo-transcript-search

View on GitHub

Showing 56 of 95 total issues

Function matchMedia has 27 lines of code (exceeds 25 allowed). Consider refactoring.
Open

    window.matchMedia = function(media) {
        var mql         = localMatchMedia(media),
            listeners   = [],
            index       = 0;

Severity: Minor
Found in webapp/public/matchMedia.addListener.js - About 1 hr to fix

    Function renderQueries has 26 lines of code (exceeds 25 allowed). Consider refactoring.
    Open

        renderQueries() {
            // see chartist.scss for colors
            var queries = this.state.queries.map((q, i) => {
                let className = Colors.colorAt(i);
                let query;
    Severity: Minor
    Found in webapp/src/frontend/components/Timeline.js - About 1 hr to fix

      Method significant_terms has 26 lines of code (exceeds 25 allowed). Consider refactoring.
      Open

        def significant_terms(query = nil, algorithm = :mutual_information, size = 10)
          response = client.search(
            index: 'hdo-transcripts',
            type: 'speech',
            body: {
      Severity: Minor
      Found in scripts/significant_terms.rb - About 1 hr to fix

        Function render has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.
        Open

            render() {
                if (this.state.queries.length) {
                    this.chartOptions.axisY.onlyInteger = this.props.unit === 'count';
        
                    return (
        Severity: Minor
        Found in webapp/src/frontend/components/Timeline.js - 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

        Function render has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.
        Open

            render() {
                let speeches = this.props.speeches;
        
                if (!speeches.length) {
                    return null;
        Severity: Minor
        Found in webapp/src/frontend/components/SpeechInContext.js - 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 build_party_cache has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.
        Open

              def build_party_cache
                return unless @party_cache.empty?
        
                @logger.info "building name -> party cache, this could take a while"
        
        
        Severity: Minor
        Found in indexer/lib/hdo-transcript-indexer.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 calculate has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.
        Open

          def calculate(word_left, word_right, values_left, values_right)
            return if word_left == word_right
            return if ignored?(word_left) || ignored?(word_right)
            return if word_left.length < @options[:min_length] || word_right.length < @options[:min_length]
            return if (values_left.inject(&:+) <= @options[:threshold]) || (values_right.inject(&:+) <= @options[:threshold])
        Severity: Minor
        Found in scripts/correlation_finder.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 parse has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.
        Open

                def parse(file, options = {})
                  case file
                  when /s(\d{2})(\d{2})(\d{2}).*\.xml$/i
                    short_year = $1
                    month      = $2.to_i
        Severity: Minor
        Found in indexer/lib/hdo-transcript-indexer/converter.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

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

            getHitStream(opts) {
                let delimiter = opts.format === 'csv' ? ',' : '\t';
                let scrollId = null;
                let tsvHeaders = TSV_HEADERS;
                if (opts.extra_fields) {
        Severity: Minor
        Found in webapp/src/backend/search-api.js - 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

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

        window.matchMedia || (window.matchMedia = function() {
            "use strict";
        
            // For browsers that support matchMedium api such as IE 9 and webkit
            var styleMedia = (window.styleMedia || window.media);
        Severity: Minor
        Found in webapp/public/matchMedia.js - 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 method.
        Open

            return if (values_left.count { |v| v > 0 } < 2) || (values_right.count { |v| v > 0 } < 2)
        Severity: Major
        Found in scripts/correlation_finder.rb - About 30 mins to fix

          Avoid too many return statements within this function.
          Open

                      return 'Ukjent';
          Severity: Major
          Found in webapp/src/frontend/components/Lix.js - About 30 mins to fix

            Avoid too many return statements within this function.
            Open

                        return 'Meget vanskelig, byråkratnorsk';
            Severity: Major
            Found in webapp/src/frontend/components/Lix.js - About 30 mins to fix

              Function _handleMouseOver has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.
              Open

                  _handleMouseOver(event) {
                      let e = event.nativeEvent;
                      let val = e.srcElement.getAttribute('ct:value');
              
                      if (val) {
              Severity: Minor
              Found in webapp/src/frontend/components/BaseChart.js - 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

              Function getDescription has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.
              Open

                  getDescription(lixScore) {
                      lixScore = +lixScore.toFixed();
              
                      if (lixScore < 30) {
                          return 'Veldig lettlest / barnebøker';
              Severity: Minor
              Found in webapp/src/frontend/components/Lix.js - 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 fetch_session has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.
              Open

                    def fetch_session(session)
                      if session.split("-").first.to_i < 2008
                        # fetch the older transcripts that we were emailed
                        dest = @data_dir.join('old-data.zip')
              
              
              Severity: Minor
              Found in indexer/lib/hdo-transcript-indexer.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