move validation to separated funcion

Author: Michael Laktionov

main.js

@@ -16,16 +16,23 @@ var device = UA.getDevice();
 var engine = UA.getEngine();
 var os = UA.getOS();
 var ua = UA.getUA();
+
 var setDefaults = function setDefaults(p) {
   var d = arguments.length > 1 && arguments[1] !== undefined ? arguments[1] : 'none';
   return p ? p : d;
 };
-var isIOS13Check = function isIOS13Check(type) {
+var getNavigatorInstance = function getNavigatorInstance() {
   if (typeof window !== 'undefined') {
     if (window.navigator || navigator) {
-      return navigator.platform.includes(type) || navigator.platform === 'MacIntel' && navigator.maxTouchPoints > 1 && !window.MSStream;
+      return window.navigator || navigator;
     }
   }
+
+  return false;
+};
+var isIOS13Check = function isIOS13Check(type) {
+  var nav = getNavigatorInstance();
+  return nav && (nav.platform.includes(type) || nav.platform === 'MacIntel' && nav.maxTouchPoints > 1 && !window.MSStream);
 };
 
 function _typeof(obj) {
@@ -419,21 +426,14 @@ var isIEType = function isIEType() {
 };
 
 var isElectronType = function isElectronType() {
-  if (typeof window !== 'undefined') {
-    if (window.navigator || navigator) {
-      var _ua = navigator.userAgent.toLowerCase();
-
-      return typeof _ua === 'string' ? _ua.includes('electron') : false;
-    }
-  }
+  var nav = getNavigatorInstance();
+  var ua = nav && nav.userAgent.toLowerCase();
+  return typeof ua === 'string' ? ua.includes('electron') : false;
 };
 
 var getIOS13 = function getIOS13() {
-  if (typeof window !== 'undefined') {
-    if (window.navigator || navigator) {
-      return (/iPad|iPhone|iPod/.test(navigator.platform) || navigator.platform === 'MacIntel' && navigator.maxTouchPoints > 1) && !window.MSStream;
-    }
-  }
+  var nav = getNavigatorInstance();
+  return nav && (/iPad|iPhone|iPod/.test(nav.platform) || nav.platform === 'MacIntel' && nav.maxTouchPoints > 1) && !window.MSStream;
 };
 
 var getIPad13 = function getIPad13() {

src/components/helpers/get-ua-data.js

@@ -14,15 +14,22 @@ export const mockUserAgent = userAgent => {
   window.navigator.__defineGetter__('userAgent', () => userAgent);
 };
 
+//TODO: Rework structure, move helper functions to utils
 export const setDefaults = (p, d = 'none') => (p ? p : d);
 
-export const isIOS13Check = type => {
+export const getNavigatorInstance = () => {
   if (typeof window !== 'undefined') {
     if (window.navigator || navigator) {
-      return (
-        navigator.platform.includes(type) ||
-        (navigator.platform === 'MacIntel' && navigator.maxTouchPoints > 1 && !window.MSStream)
-      );
+      return window.navigator || navigator;
     }
   }
+
+  return false;
+};
+
+export const isIOS13Check = type => {
+  const nav = getNavigatorInstance();
+  return (
+    nav && (nav.platform.includes(type) || (nav.platform === 'MacIntel' && nav.maxTouchPoints > 1 && !window.MSStream))
+  );
 };

src/components/helpers/selectors.js

@@ -1,4 +1,4 @@
-import { os, device, browser, ua, engine, setDefaults, isIOS13Check } from './get-ua-data';
+import { os, device, browser, ua, engine, setDefaults, isIOS13Check, getNavigatorInstance } from './get-ua-data';
 import { BROWSER_TYPES, DEVICE_TYPES, OS_TYPES } from './types';
 
 const isMobileType = () => device.type === DEVICE_TYPES.MOBILE;
@@ -31,25 +31,19 @@ const isMobileSafariType = () => browser.name === BROWSER_TYPES.MOBILE_SAFARI;
 const isOperaType = () => browser.name === BROWSER_TYPES.OPERA;
 const isIEType = () => browser.name === BROWSER_TYPES.INTERNET_EXPLORER || browser.name === BROWSER_TYPES.IE;
 const isElectronType = () => {
-  if (typeof window !== 'undefined') {
-    if (window.navigator || navigator) {
-      const ua = navigator.userAgent.toLowerCase();
+  const nav = getNavigatorInstance();
+  const ua = nav && nav.userAgent.toLowerCase();
 
-      return typeof ua === 'string' ? ua.includes('electron') : false;
-    }
-  }
+  return typeof ua === 'string' ? ua.includes('electron') : false;
 };
 
 const getIOS13 = () => {
-  if (typeof window !== 'undefined') {
-    if (window.navigator || navigator) {
-      return (
-        (/iPad|iPhone|iPod/.test(navigator.platform) ||
-          (navigator.platform === 'MacIntel' && navigator.maxTouchPoints > 1)) &&
-        !window.MSStream
-      );
-    }
-  }
+  const nav = getNavigatorInstance();
+  return (
+    nav &&
+    (/iPad|iPhone|iPod/.test(nav.platform) || (nav.platform === 'MacIntel' && nav.maxTouchPoints > 1)) &&
+      !window.MSStream
+  );
 };
 
 const getIPad13 = () => isIOS13Check('iPad');