MyEMS/myems

Function highchart has 323 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  function highchart(highchartsNGUtils, $timeout) {

    // acceptable shared state
    var seriesId = 0;
    var ensureIds = function (series) {

Function link has 179 lines of code (exceeds 25 allowed). Consider refactoring.

Open

      link: function (scope, element, attrs) {
        // We keep some chart-specific variables here as a closure
        // instead of storing them on 'scope'.

        // prevSeriesOptions is maintained by processSeries

File highcharts-ng.js has 439 lines of code (exceeds 250 allowed). Consider refactoring.

Open

if (typeof module !== 'undefined' && typeof exports !== 'undefined' && module.exports === exports){
  module.exports = 'highcharts-ng';
}

(function () {

Function getMergedOptions has 78 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    var getMergedOptions = function (scope, element, config) {
      var mergedOptions = {};

      var defaultOptions = {
        chart: {

Function highchartsNG has 77 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  function highchartsNG($window, $rootScope, lazyload, basePath, modules) {
    var readyQueue = [];
    var loading = false;
    return {
      lazyLoad:lazyload,

Function processSeries has 66 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        var processSeries = function(series, seriesOld) {
          var i;
          var ids = [];

          if(series) {

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

Open

      ready: function (callback, thisArg) {
        if (typeof $window.Highcharts !== 'undefined' || !lazyload) {
          callback();
        } else {
          readyQueue.push([callback, thisArg]);

Avoid deeply nested control flow statements.

Open

                    if (angular.equals(sCopy, s)) {
                      chartSeries.addPoint(ptNew, false);
                    }
                    
                    // Check if the data change was a push and shift operation

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

Open

  function highchartsNG($window, $rootScope, lazyload, basePath, modules) {

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                scope.$apply(function () {
                  scope.config[axisName].currentMin = e[axisName][0].min;
                  scope.config[axisName].currentMax = e[axisName][0].max;
                });

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                scope.$apply(function () {
                  scope.config[axisName].currentMin = thisChart[axisName][0].dataMin;
                  scope.config[axisName].currentMax = thisChart[axisName][0].dataMax;
                });

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

              scope.config[axisName].currentMin = this[axisName][0].min || scope.config[axisName].currentMin;

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

              scope.config[axisName].currentMax = this[axisName][0].max || scope.config[axisName].currentMax;

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

          if (enabled) {
            chart.credits.show();
          } else if (chart.credits) {
            chart.credits.hide();
          }

• Read up
  Read up