Update JavaScript to 4.2.3

PerfectlyNormal · PerfectlyNormal · commit 94cca2eb3349 · 2016-04-09T12:45:18.000+02:00
diff --git a/app/assets/javascripts/highcharts.js b/app/assets/javascripts/highcharts.js
@@ -2,7 +2,7 @@
 // @compilation_level SIMPLE_OPTIMIZATIONS
 
 /**
- * @license Highcharts JS v4.2.2 (2016-02-04)
+ * @license Highcharts JS v4.2.3 (2016-02-08)
  *
  * (c) 2009-2016 Torstein Honsi
  *
@@ -59,7 +59,7 @@
         charts = [],
         chartCount = 0,
         PRODUCT = 'Highcharts',
-        VERSION = '4.2.2',
+        VERSION = '4.2.3',
 
         // some constants for frequently used strings
         DIV = 'div',
@@ -1533,7 +1533,7 @@
             useUTC: true,
             //timezoneOffset: 0,
             canvasToolsURL: 'http://code.highcharts.com/modules/canvas-tools.js',
-            VMLRadialGradientURL: 'http://code.highcharts.com/4.2.2/gfx/vml-radial-gradient.png'
+            VMLRadialGradientURL: 'http://code.highcharts.com/4.2.3/gfx/vml-radial-gradient.png'
         },
         chart: {
             //animation: true,
@@ -2482,7 +2482,7 @@
                 key,
                 attribs = {},
                 normalizer,
-                strokeWidth = rect.strokeWidth || wrapper.strokeWidth || 0;
+                strokeWidth = wrapper.strokeWidth || 0;
 
             normalizer = mathRound(strokeWidth) % 2 / 2; // mathRound because strokeWidth can sometimes have roundoff errors
 
@@ -3957,16 +3957,16 @@
                 };
 
             if (strokeWidth !== UNDEFINED) {
-                attribs.strokeWidth = strokeWidth;
+                wrapper.strokeWidth = strokeWidth;
                 attribs = wrapper.crisp(attribs);
             }
 
             if (r) {
                 attribs.r = r;
             }
 
-            wrapper.rSetter = function (value) {
-                attr(this.element, {
+            wrapper.rSetter = function (value, key, element) {
+                attr(element, {
                     rx: value,
                     ry: value
                 });
@@ -10545,7 +10545,7 @@
 
             var chart = this.chart;
 
-            if (!defined(hoverChartIndex) || !charts[hoverChartIndex].mouseIsDown) {
+            if (!defined(hoverChartIndex) || !charts[hoverChartIndex] || !charts[hoverChartIndex].mouseIsDown) {
                 hoverChartIndex = chart.index;
             }
 
@@ -12852,8 +12852,9 @@
                         })
                         .add();
                 } else {
+                    plotBorder.strokeWidth = -plotBorderWidth;
                     plotBorder.animate(
-                        plotBorder.crisp({ x: plotLeft, y: plotTop, width: plotWidth, height: plotHeight, strokeWidth: -plotBorderWidth }) //#3282 plotBorder should be negative
+                        plotBorder.crisp({ x: plotLeft, y: plotTop, width: plotWidth, height: plotHeight }) //#3282 plotBorder should be negative
                     );
                 }
             }
@@ -13364,8 +13365,8 @@
 
             // If no x is set by now, get auto incremented value. All points must have an
             // x value, however the y value can be null to create a gap in the series
-            if (point.x === UNDEFINED && series) {
-                point.x = x === UNDEFINED ? series.autoIncrement() : x;
+            if (typeof point.x !== 'number' && series) {
+                point.x = x === undefined ? series.autoIncrement() : x;
             }
 
             return point;
@@ -14351,8 +14352,8 @@
         /**
          * Return the series points with null points filtered out
          */
-        getValidPoints: function () {
-            return grep(this.points, function (point) {
+        getValidPoints: function (points) {
+            return grep(points || this.points, function (point) {
                 return !point.isNull;
             });
         },
@@ -14811,11 +14812,28 @@
             var series = this,
                 options = series.options,
                 step = options.step,
+                reversed,
                 graphPath = [],
                 gap;
 
             points = points || series.points;
 
+            // Bottom of a stack is reversed
+            reversed = points.reversed;
+            if (reversed) {
+                points.reverse();
+            }
+            // Reverse the steps (#5004)
+            step = { right: 1, center: 2 }[step] || (step && 3);
+            if (step && reversed) {
+                step = 4 - step;
+            }
+
+            // Remove invalid points, especially in spline (#5015)
+            if (options.connectNulls && !nullsAsZeroes && !connectCliffs) {
+                points = this.getValidPoints(points);
+            }
+
             // Build the line
             each(points, function (point, i) {
 
@@ -14847,14 +14865,14 @@
 
                     } else if (step) {
 
-                        if (step === 'right') {
+                        if (step === 1) { // right
                             pathToPoint = [
                                 L,
                                 lastPoint.plotX,
                                 plotY
                             ];
                         
-                        } else if (step === 'center') {
+                        } else if (step === 2) { // center
                             pathToPoint = [
                                 L,
                                 (lastPoint.plotX + plotX) / 2,
@@ -15327,11 +15345,7 @@
 
             // Start the recursive build process with a clone of the points array and null points filtered out (#3873)
             function startRecursive() {
-                var points = grep(series.points || [], function (point) { // #4390
-                    return point.y !== null;
-                });
-
-                series.kdTree = _kdtree(points, dimensions, dimensions);
+                series.kdTree = _kdtree(series.getValidPoints(), dimensions, dimensions);
             }
             delete series.kdTree;
 
@@ -16564,7 +16578,8 @@
 
             topPath = getGraphPath.call(this, graphPoints, true, true);
         
-            bottomPath = getGraphPath.call(this, bottomPoints.reverse(), true, true);
+            bottomPoints.reversed = true;
+            bottomPath = getGraphPath.call(this, bottomPoints, true, true);
             if (bottomPath.length) {
                 bottomPath[0] = L;
             }
@@ -16660,16 +16675,18 @@
                     lastY = lastPoint.plotY,
                     nextX = nextPoint.plotX,
                     nextY = nextPoint.plotY,
-                    correction;
+                    correction = 0;
 
                 leftContX = (smoothing * plotX + lastX) / denom;
                 leftContY = (smoothing * plotY + lastY) / denom;
                 rightContX = (smoothing * plotX + nextX) / denom;
                 rightContY = (smoothing * plotY + nextY) / denom;
 
-                // have the two control points make a straight line through main point
-                correction = ((rightContY - leftContY) * (rightContX - plotX)) /
-                    (rightContX - leftContX) + plotY - rightContY;
+                // Have the two control points make a straight line through main point
+                if (rightContX !== leftContX) { // #5016, division by zero
+                    correction = ((rightContY - leftContY) * (rightContX - plotX)) /
+                        (rightContX - leftContX) + plotY - rightContY;
+                }
 
                 leftContY += correction;
                 rightContY += correction;
diff --git a/app/assets/javascripts/highcharts/highcharts-3d.js b/app/assets/javascripts/highcharts/highcharts-3d.js
@@ -2,7 +2,7 @@
 // @compilation_level SIMPLE_OPTIMIZATIONS
 
 /**
- * @license Highcharts JS v4.2.2 (2016-02-04)
+ * @license Highcharts JS v4.2.3 (2016-02-08)
  *
  * 3D features for Highcharts JS
  *
diff --git a/app/assets/javascripts/highcharts/highcharts-more.js b/app/assets/javascripts/highcharts/highcharts-more.js
@@ -2,7 +2,7 @@
 // @compilation_level SIMPLE_OPTIMIZATIONS
 
 /**
- * @license Highcharts JS v4.2.2 (2016-02-04)
+ * @license Highcharts JS v4.2.3 (2016-02-08)
  *
  * (c) 2009-2016 Torstein Honsi
  *
@@ -816,11 +816,10 @@ var arrayMin = Highcharts.arrayMin,
             linePath = [].concat(lowerPath, higherPath);
 
             // For the area path, we need to change the 'move' statement into 'lineTo' or 'curveTo'
-            if (!this.chart.polar) {
+            if (!this.chart.polar && higherAreaPath[0] === 'M') {
                 higherAreaPath[0] = 'L'; // this probably doesn't work for spline        
             }
             this.areaPath = this.areaPath.concat(lowerPath, higherAreaPath);
-        
             return linePath;
         },
 
diff --git a/app/assets/javascripts/highcharts/modules/broken-axis.js b/app/assets/javascripts/highcharts/modules/broken-axis.js
@@ -1,5 +1,5 @@
 /**
- * Highcharts JS v4.2.2 (2016-02-04)
+ * Highcharts JS v4.2.3 (2016-02-08)
  * Highcharts Broken Axis module
  * 
  * License: www.highcharts.com/license
diff --git a/app/assets/javascripts/highcharts/modules/canvas-tools.js b/app/assets/javascripts/highcharts/modules/canvas-tools.js
@@ -2908,7 +2908,7 @@ if (CanvasRenderingContext2D) {
 		});
 	}
 }/**
- * @license Highcharts JS v4.2.2 (2016-02-04)
+ * @license Highcharts JS v4.2.3 (2016-02-08)
  * CanVGRenderer Extension module
  *
  * (c) 2011-2016 Torstein Honsi, Erik Olsson
diff --git a/app/assets/javascripts/highcharts/modules/data.js b/app/assets/javascripts/highcharts/modules/data.js
@@ -1,5 +1,5 @@
 /**
- * @license Highcharts JS v4.2.2 (2016-02-04)
+ * @license Highcharts JS v4.2.3 (2016-02-08)
  * Data module
  *
  * (c) 2012-2016 Torstein Honsi
diff --git a/app/assets/javascripts/highcharts/modules/exporting.js b/app/assets/javascripts/highcharts/modules/exporting.js
@@ -1,5 +1,5 @@
 /**
- * @license Highcharts JS v4.2.2 (2016-02-04)
+ * @license Highcharts JS v4.2.3 (2016-02-08)
  * Exporting module
  *
  * (c) 2010-2016 Torstein Honsi
diff --git a/app/assets/javascripts/highcharts/modules/heatmap.js b/app/assets/javascripts/highcharts/modules/heatmap.js
@@ -1,5 +1,5 @@
 /**
- * @license Highcharts JS v4.2.2 (2016-02-04)
+ * @license Highcharts JS v4.2.3 (2016-02-08)
  *
  * (c) 2011-2016 Torstein Honsi
  *
diff --git a/app/assets/javascripts/highcharts/modules/no-data-to-display.js b/app/assets/javascripts/highcharts/modules/no-data-to-display.js
@@ -1,5 +1,5 @@
 /**
- * @license Highcharts JS v4.2.2 (2016-02-04)
+ * @license Highcharts JS v4.2.3 (2016-02-08)
  * Plugin for displaying a message when there is no data visible in chart.
  *
  * (c) 2010-2016 Highsoft AS
diff --git a/app/assets/javascripts/highcharts/modules/offline-exporting.js b/app/assets/javascripts/highcharts/modules/offline-exporting.js
@@ -1,5 +1,5 @@
 /**
- * @license Highcharts JS v4.2.2 (2016-02-04)
+ * @license Highcharts JS v4.2.3 (2016-02-08)
  * Client side exporting module
  *
  * (c) 2015 Torstein Honsi / Oystein Moseng
diff --git a/app/assets/javascripts/highcharts/modules/solid-gauge.js b/app/assets/javascripts/highcharts/modules/solid-gauge.js
@@ -1,5 +1,5 @@
 /**
- * @license  Highcharts JS v4.2.2 (2016-02-04)
+ * @license  Highcharts JS v4.2.3 (2016-02-08)
  * Solid angular gauge module
  *
  * (c) 2010-2016 Torstein Honsi
diff --git a/app/assets/javascripts/highcharts/modules/treemap.js b/app/assets/javascripts/highcharts/modules/treemap.js
@@ -1,5 +1,5 @@
 /**
- * @license Highcharts JS v4.2.2 (2016-02-04)
+ * @license Highcharts JS v4.2.3 (2016-02-08)
  *
  * (c) 2014 Highsoft AS
  * Authors: Jon Arild Nygard / Oystein Moseng

Original file line number	Diff line number	Diff line change
`@@ -2,7 +2,7 @@`
`2`	`2`	`// @compilation_level SIMPLE_OPTIMIZATIONS`
`3`	`3`
`4`	`4`	`/**`
`5`		`- * @license Highcharts JS v4.2.2 (2016-02-04)`
	`5`	`+ * @license Highcharts JS v4.2.3 (2016-02-08)`
`6`	`6`	`*`
`7`	`7`	`* 3D features for Highcharts JS`
`8`	`8`	`*`
Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`/**`
`2`		`- * Highcharts JS v4.2.2 (2016-02-04)`
	`2`	`+ * Highcharts JS v4.2.3 (2016-02-08)`
`3`	`3`	`* Highcharts Broken Axis module`
`4`	`4`	`*`
`5`	`5`	`* License: www.highcharts.com/license`
Original file line number	Diff line number	Diff line change
`@@ -2908,7 +2908,7 @@ if (CanvasRenderingContext2D) {`
`2908`	`2908`	`});`
`2909`	`2909`	`}`
`2910`	`2910`	`}/**`
`2911`		`- * @license Highcharts JS v4.2.2 (2016-02-04)`
	`2911`	`+ * @license Highcharts JS v4.2.3 (2016-02-08)`
`2912`	`2912`	`* CanVGRenderer Extension module`
`2913`	`2913`	`*`
`2914`	`2914`	`* (c) 2011-2016 Torstein Honsi, Erik Olsson`