refactor: code

Author: alexander-akait

src/plugins/postcss-url-parser.js

@@ -48,7 +48,7 @@ function getWebpackIgnoreCommentValue(index, nodes, inBetween) {
   return matched && matched[2] === "true";
 }
 
-function shouldHandleURL(url, declaration, result, options) {
+function shouldHandleURL(url, declaration, result, isSupportDataURLInNewURL) {
   if (url.length === 0) {
     result.warn(`Unable to find uri in '${declaration.toString()}'`, {
       node: declaration,
@@ -57,7 +57,7 @@ function shouldHandleURL(url, declaration, result, options) {
     return false;
   }
 
-  if (isDataUrl(url) && options.isSupportDataURLInNewURL) {
+  if (isDataUrl(url) && isSupportDataURLInNewURL) {
     try {
       decodeURIComponent(url);
     } catch (ignoreError) {
@@ -74,7 +74,7 @@ function shouldHandleURL(url, declaration, result, options) {
   return true;
 }
 
-function parseDeclaration(declaration, key, result, options) {
+function parseDeclaration(declaration, key, result, isSupportDataURLInNewURL) {
   if (!needParseDeclaration.test(declaration[key])) {
     return;
   }
@@ -141,7 +141,9 @@ function parseDeclaration(declaration, key, result, options) {
       url = normalizeUrl(url, isStringValue);
 
       // Do not traverse inside `url`
-      if (!shouldHandleURL(url, declaration, result, options)) {
+      if (
+        !shouldHandleURL(url, declaration, result, isSupportDataURLInNewURL)
+      ) {
         // eslint-disable-next-line consistent-return
         return false;
       }
@@ -197,7 +199,9 @@ function parseDeclaration(declaration, key, result, options) {
           url = normalizeUrl(url, isStringValue);
 
           // Do not traverse inside `url`
-          if (!shouldHandleURL(url, declaration, result, options)) {
+          if (
+            !shouldHandleURL(url, declaration, result, isSupportDataURLInNewURL)
+          ) {
             // eslint-disable-next-line consistent-return
             return false;
           }
@@ -240,7 +244,9 @@ function parseDeclaration(declaration, key, result, options) {
           let url = normalizeUrl(value, true);
 
           // Do not traverse inside `url`
-          if (!shouldHandleURL(url, declaration, result, options)) {
+          if (
+            !shouldHandleURL(url, declaration, result, isSupportDataURLInNewURL)
+          ) {
             // eslint-disable-next-line consistent-return
             return false;
           }
@@ -282,11 +288,12 @@ const plugin = (options = {}) => {
 
       return {
         Declaration(declaration) {
+          const { isSupportDataURLInNewURL } = options;
           const parsedURL = parseDeclaration(
             declaration,
             "value",
             result,
-            options
+            isSupportDataURLInNewURL
           );
 
           if (!parsedURL) {