updated the code to be more clear

5 years ago · 4d8c781369
parent 42b0f185e2
commit 4d8c781369
1 changed files with 5 additions and 1 deletions
--- a/src/parser-browsers.js
+++ b/src/parser-browsers.js
@ -590,7 +590,11 @@ const browsersList = [
  {
    test: [/.*/i],
    describe(ua) {
-      const regexp = ua.search('\\(') === -1 ? /^(.*)\/(.*) / : /^(.*)\/(.*)[ \t]\((.*)/;
+	  // Define the regexp depending if the ua has device specifications
+	  const regexpWithoutParenthesis = /^(.*)\/(.*) /;
+	  const regexpWithParenthesis = /^(.*)\/(.*)[ \t]\((.*)/;
+	  const hasDeviceSpec = ua.search('\\(') !== -1;
+      const regexp = hasDeviceSpec ? regexpWithoutParenthesis : regexpWithParenthesis;
      return {
        name: Utils.getFirstMatch(regexp, ua),
        version: Utils.getSecondMatch(regexp, ua),