refactor: lib/util.js (#452)

* feat: refactor

* fix: revert

* fix: lint

Author: jamesgeorge007

lib/util.js

@@ -25,12 +25,11 @@ function getPaths(publicPath, compiler, url) {
         compilers[i].options.output.publicPath;
 
       if (compilerPublicPath) {
-        if (compilerPublicPath.indexOf('/') === 0) {
-          compilerPublicPathBase = compilerPublicPath;
-        } else {
-          // handle the case where compilerPublicPath is a URL with hostname
-          compilerPublicPathBase = parse(compilerPublicPath).pathname;
-        }
+        compilerPublicPathBase =
+          compilerPublicPath.indexOf('/') === 0
+            ? compilerPublicPath // eslint-disable-next-line
+            : // handle the case where compilerPublicPath is a URL with hostname
+              parse(compilerPublicPath).pathname;
 
         // check the url vs the path part of the compilerPublicPath
         if (url.indexOf(compilerPublicPathBase) === 0) {
@@ -66,21 +65,19 @@ module.exports = {
     const urlObject = parse(url);
     let filename;
 
+    const hostNameIsTheSame = localPrefix.hostname === urlObject.hostname;
+
     // publicPath has the hostname that is not the same as request url's, should fail
     if (
       localPrefix.hostname !== null &&
       urlObject.hostname !== null &&
-      localPrefix.hostname !== urlObject.hostname
+      !hostNameIsTheSame
     ) {
       return false;
     }
 
     // publicPath is not in url, so it should fail
-    if (
-      publicPath &&
-      localPrefix.hostname === urlObject.hostname &&
-      url.indexOf(publicPath) !== 0
-    ) {
+    if (publicPath && hostNameIsTheSame && url.indexOf(publicPath) !== 0) {
       return false;
     }