enfore isPlainObject in src files

etpinard · etpinard · commit e1f6818acb1d · 2016-07-28T18:21:34.000-04:00
diff --git a/src/components/colorbar/has_colorbar.js b/src/components/colorbar/has_colorbar.js
@@ -9,10 +9,9 @@
 
 'use strict';
 
+var Lib = require('../../lib');
+
 
 module.exports = function hasColorbar(container) {
-    return (
-        typeof container.colorbar === 'object' &&
-        container.colorbar !== null
-    );
+    return Lib.isPlainObject(container.colorbar);
 };
diff --git a/src/components/colorscale/has_colorscale.js b/src/components/colorscale/has_colorscale.js
@@ -33,12 +33,12 @@ module.exports = function hasColorscale(trace, containerStr) {
     }
 
     return (
-        (typeof container === 'object' && container !== null) && (
+        Lib.isPlainObject(container) && (
             isArrayWithOneNumber ||
             container.showscale === true ||
             (isNumeric(container.cmin) && isNumeric(container.cmax)) ||
             isValidScale(container.colorscale) ||
-            (typeof container.colorbar === 'object' && container.colorbar !== null)
+            Lib.isPlainObject(container.colorbar)
         )
     );
 };
diff --git a/src/components/rangeslider/defaults.js b/src/components/rangeslider/defaults.js
@@ -13,10 +13,9 @@ var attributes = require('./attributes');
 
 
 module.exports = function supplyLayoutDefaults(layoutIn, layoutOut, axName, counterAxes) {
-
     if(!layoutIn[axName].rangeslider) return;
 
-    var containerIn = typeof layoutIn[axName].rangeslider === 'object' ?
+    var containerIn = Lib.isPlainObject(layoutIn[axName].rangeslider) ?
             layoutIn[axName].rangeslider : {},
         containerOut = layoutOut[axName].rangeslider = {};
 
diff --git a/src/plots/mapbox/layers.js b/src/plots/mapbox/layers.js
@@ -131,11 +131,8 @@ proto.dispose = function dispose() {
 function isVisible(opts) {
     var source = opts.source;
 
-    // For some weird reason Lib.isPlainObject fails
-    // to detect `source` as a plain object in nw.js 0.12.
-
     return (
-        typeof source === 'object' ||
+        Lib.isPlainObject(source) ||
         (typeof source === 'string' && source.length > 0)
     );
 }
diff --git a/src/traces/scatter/subtypes.js b/src/traces/scatter/subtypes.js
@@ -9,6 +9,8 @@
 
 'use strict';
 
+var Lib = require('../../lib');
+
 module.exports = {
     hasLines: function(trace) {
         return trace.visible && trace.mode &&
@@ -26,7 +28,7 @@ module.exports = {
     },
 
     isBubble: function(trace) {
-        return (typeof trace.marker === 'object' &&
-                    Array.isArray(trace.marker.size));
+        return Lib.isPlainObject(trace.marker) &&
+            Array.isArray(trace.marker.size);
     }
 };

Original file line number	Diff line number	Diff line change
`@@ -33,12 +33,12 @@ module.exports = function hasColorscale(trace, containerStr) {`
`33`	`33`	`}`
`34`	`34`
`35`	`35`	`return (`
`36`		`- (typeof container === 'object' && container !== null) && (`
	`36`	`+ Lib.isPlainObject(container) && (`
`37`	`37`	`isArrayWithOneNumber \|\|`
`38`	`38`	`container.showscale === true \|\|`
`39`	`39`	`(isNumeric(container.cmin) && isNumeric(container.cmax)) \|\|`
`40`	`40`	`isValidScale(container.colorscale) \|\|`
`41`		`- (typeof container.colorbar === 'object' && container.colorbar !== null)`
	`41`	`+ Lib.isPlainObject(container.colorbar)`
`42`	`42`	`)`
`43`	`43`	`);`
`44`	`44`	`};`