Update number checks to include isFinite

Author: tripp

src/gallery-charts-stockindicators/js/CanvasAxis.js

@@ -81,10 +81,10 @@ Y.CanvasAxis = Y.Base.create("canvasAxis", Y.AxisBase, [Y.Renderer], Y.merge(Y.A
                 explicitLabels = this._labelValuesExplicitlySet ? this.get("labelValues") : null,
                 direction = (position === "left" || position === "right") ? "vertical" : "horizontal";
             if(margin) {
-                marginLeft = typeof margin.left === "number" ? margin.left : 0;
-                marginRight = typeof margin.right === "number" ? margin.right : 0;
-                marginTop = typeof margin.top === "number" ? margin.top : 0;
-                marginBottom = typeof margin.bottom === "number" ? margin.bottom : 0;
+                marginLeft = typeof margin.left === "number" && isFinite(margin.left) ? margin.left : 0;
+                marginRight = typeof margin.right === "number" && isFinite(margin.right) ? margin.right : 0;
+                marginTop = typeof margin.top === "number" && isFinite(margin.top) ? margin.top : 0;
+                marginBottom = typeof margin.bottom === "number" && isFinite(margin.bottom) ? margin.bottom : 0;
             }
             //need to defaultMargins method to the layout classes.
             for(i in defaultMargins)

src/gallery-charts-stockindicators/js/Crosshair.js

@@ -107,7 +107,7 @@ Y.Crosshair.prototype = {
             for(i = 0; i < len; i = i + 1) {
                 graph = series[i];
                 y = graph.coords[index];
-                isNumber = typeof y === "number";
+                isNumber = typeof y === "number" && isFinite(y);
 
                 if(graph.marker) {
                     if(isNumber) {

src/gallery-charts-stockindicators/js/MultipleLineSeries.js

@@ -69,7 +69,7 @@ Y.MultipleLineSeries = Y.Base.create("multipleLineSeries", Y.CartesianSeries, [Y
         {
             nextX = Math.round(xcoords[i] * 1000)/1000;
             nextY = Math.round(ycoords[i] * 1000)/1000;
-            pointValid = typeof nextX === "number" && typeof nextY === "number";
+            pointValid = typeof nextX === "number" && isFinite(nextX)  && typeof nextY === "number" && isFinite(nextY);
             if(pointValid) {
                 thresholdIndex = 0;
                 if(thresholds) {
@@ -91,7 +91,7 @@ Y.MultipleLineSeries = Y.Base.create("multipleLineSeries", Y.CartesianSeries, [Y
                         m = Math.round(((nextY - lastValidY) / (nextX - lastValidX)) * 1000)/1000;
                         intersectX = ((thresholdCoords[thresholdIndex] - nextY)/m) + nextX;
                         intersectY = thresholdCoords[thresholdIndex];
-                        if(typeof lastPathIndex === "number") {
+                        if(typeof lastPathIndex === "number" && isFinite(lastPathIndex)) {
                             this._lineTo(graphPaths[lastPathIndex], intersectX, intersectY);
                         }
                         this._moveTo(graphPaths[pathIndex], intersectX, intersectY);

src/gallery-charts-stockindicators/js/StockIndicatorsChart.js

@@ -1065,7 +1065,7 @@ Y.StockIndicatorsChart = Y.Base.create("stockIndicatorsChart",  Y.Widget, [Y.Ren
             config.legend.width = graphConfig.width;
         }
         legend = this._addLegend(config, cb);
-        if(thresholdStyles && typeof previousClose === "number") {
+        if(thresholdStyles && typeof previousClose === "number" && isFinite(previousClose)) {
             thresholdConfig = {
                 type: "thresholdline",
                 thresholds: [previousClose],