Sign upLogin

drhenner/ror_ecommerce

View on GitHub
Star
app/assets/javascripts/jquery/jquery-migrate-1.1.1.js

Summary

Maintainability
C
1 day
Test Coverage

File jquery-migrate-1.1.1.js has 371 lines of code (exceeds 250 allowed). Consider refactoring.
Open

/*!
 * jQuery Migrate - v1.1.1 - 2013-02-16
 * https://github.com/jquery/jquery-migrate
 * Copyright 2005, 2013 jQuery Foundation, Inc. and other contributors; Licensed MIT
 */
Severity: Minor
Found in app/assets/javascripts/jquery/jquery-migrate-1.1.1.js - About 4 hrs to fix

    Function attr has 43 lines of code (exceeds 25 allowed). Consider refactoring.
    Open

    jQuery.attr = function( elem, name, value, pass ) {
  var lowerName = name.toLowerCase(),
    nType = elem && elem.nodeType;

  if ( pass ) {
    Severity: Minor
    Found in app/assets/javascripts/jquery/jquery-migrate-1.1.1.js - About 1 hr to fix

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

        jQuery.clean = function( elems, context, fragment, scripts ) {
    // Set context per 1.8 logic
    context = context || document;
    context = !context.nodeType && context[0] || context;
    context = context.ownerDocument || context;
      Severity: Minor
      Found in app/assets/javascripts/jquery/jquery-migrate-1.1.1.js - About 1 hr to fix

        Avoid deeply nested control flow statements.
        Open

                  if ( typeof elem.getElementsByTagName !== "undefined" ) {
            // handleScript alters the DOM, so use jQuery.merge to ensure snapshot iteration
            jsTags = jQuery.grep( jQuery.merge( [], elem.getElementsByTagName("script") ), handleScript );

            // Splice the scripts into ret after their former ancestor and advance our index beyond them
        Severity: Major
        Found in app/assets/javascripts/jquery/jquery-migrate-1.1.1.js - About 45 mins to fix

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

          jQuery.event.remove = function( elem, types, handler, selector, mappedTypes ){
          Severity: Minor
          Found in app/assets/javascripts/jquery/jquery-migrate-1.1.1.js - About 35 mins to fix

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

            jQuery.event.add = function( elem, types, handler, data, selector ){
            Severity: Minor
            Found in app/assets/javascripts/jquery/jquery-migrate-1.1.1.js - About 35 mins to fix

              There are no issues that match your filters.

              Category
              Status