Replace instanceof Array with Array.isArray

Author: wojtekmaj

src/Calendar.jsx

@@ -63,7 +63,7 @@ function getValue(value, index) {
     return null;
   }
 
-  const rawValue = value instanceof Array && value.length === 2 ? value[index] : value;
+  const rawValue = Array.isArray(value) && value.length === 2 ? value[index] : value;
 
   if (!rawValue) {
     return null;
@@ -100,7 +100,7 @@ const getDetailValueTo = (args) => getDetailValue(args, 1);
 const getDetailValueArray = (args) => {
   const { value } = args;
 
-  if (value instanceof Array) {
+  if (Array.isArray(value)) {
     return value;
   }
 

src/shared/utils.js

@@ -75,12 +75,12 @@ export function getTileClasses({
     return classes;
   }
 
-  if (!(date instanceof Array) && !dateType) {
+  if (!(Array.isArray(date)) && !dateType) {
     throw new Error('getTileClasses(): Unable to get tile activity classes because one or more required arguments were not passed.');
   }
 
   const now = new Date();
-  const dateRange = date instanceof Array ? date : getRange(dateType, date);
+  const dateRange = Array.isArray(date) ? date : getRange(dateType, date);
 
   if (isValueWithinRange(now, dateRange)) {
     classes.push(`${className}--now`);
@@ -90,11 +90,11 @@ export function getTileClasses({
     return classes;
   }
 
-  if (!(value instanceof Array) && !valueType) {
+  if (!(Array.isArray(value)) && !valueType) {
     throw new Error('getTileClasses(): Unable to get tile activity classes because one or more required arguments were not passed.');
   }
 
-  const valueRange = value instanceof Array ? value : getRange(valueType, value);
+  const valueRange = Array.isArray(value) ? value : getRange(valueType, value);
 
   if (isRangeWithinRange(valueRange, dateRange)) {
     classes.push(`${className}--active`);

test/Test.jsx

@@ -118,7 +118,7 @@ export default class Test extends PureComponent {
       return dateToRender;
     };
 
-    if (value instanceof Array) {
+    if (Array.isArray(value)) {
       return (
         <p>
           {`Chosen date range: ${renderDate(value[0])} - ${renderDate(value[1])}`}