skip over BADNUM in scatter select routine

- so that BADNUM items are not included in
  the 'plotly_selected' event data.

Author: etpinard

src/traces/scatter/select.js

@@ -10,6 +10,7 @@
 'use strict';
 
 var subtypes = require('./subtypes');
+var BADNUM = require('../../constants/numerical').BADNUM;
 
 var DESELECTDIM = 0.2;
 
@@ -40,6 +41,11 @@ module.exports = function selectPoints(searchInfo, polygon) {
             di = cd[i];
             x = xa.c2p(di.x);
             y = ya.c2p(di.y);
+
+            if(x === BADNUM || y === BADNUM) {
+                continue;
+            }
+
             if(polygon.contains([x, y])) {
                 selection.push({
                     curveNumber: curveNumber,

test/jasmine/tests/select_test.js

@@ -322,4 +322,31 @@ describe('select box and lasso', function() {
             done();
         });
     });
+
+    it('should skip over BADNUM items', function(done) {
+        var data = [{
+            mode: 'markers',
+            x: [null, undefined, NaN, 0, 'NA'],
+            y: [NaN, null, undefined, 0, 'NA']
+        }];
+        var layout = {
+            dragmode: 'select',
+            width: 400,
+            heigth: 400,
+        };
+        var gd = createGraphDiv();
+        var pts;
+
+        Plotly.plot(gd, data, layout).then(function() {
+            gd.on('plotly_selected', function(data) {
+                pts = data.points;
+            });
+
+            drag([[100, 100], [300, 300]]);
+            expect(pts.length).toEqual(1);
+            expect(pts[0].x).toEqual(0);
+            expect(pts[0].y).toEqual(0);
+        })
+        .then(done);
+    });
 });