uglifycss - npm Package Compare versions

Comparing version 0.0.22 to 0.0.23

package.json

@@ -5,3 +5,3 @@ {
     "description": "Port of YUI CSS Compressor to NodeJS",
-    "version": "0.0.22",
+    "version": "0.0.23",
     "keywords": [
@@ -8,0 +8,0 @@ "css",

uglifycss-lib.js

@@ -34,2 +34,3 @@ /**
 var PATH_SEP = path.sep;
+var ___PRESERVED_TOKEN_ = "___PRESERVED_TOKEN_";
 
@@ -110,32 +111,41 @@ var pathResolve = PATH_SEP === "/" ? path.posix.resolve : path.win32.resolve;
             token = css.substring(startIndex, endIndex).replace(/(^\s*|\s*$)/g, "");
-            if (token.slice(0, 18) === "___PRESERVED_TOKEN_") {
-                continue;
-            }
-            if (terminator === "'" || terminator === '"') {
-                token = token.slice(1, -1);
-            } else if (terminator === ")") {
-                terminator = "";
-            }
+            if (token.slice(0, 19) !== ___PRESERVED_TOKEN_) {
 
-            // build path of detected urls:
-            target = options.target.slice();
-            token = token.split(SEP).join(PATH_SEP); // assuming urls in css use "/"
-            url = pathResolve(options.source.join(PATH_SEP), token).split(PATH_SEP);
+                if (terminator === "'" || terminator === '"') {
+                    token = token.slice(1, -1);
+                } else if (terminator === ")") {
+                    terminator = "";
+                }
 
-            file = url.pop();
+                if (options.convertUrls) {
 
-            // remove common part of both paths
-            while (target[0] === url[0]) {
-                target.shift();
-                url.shift();
-            }
+                    // build path of detected urls:
+                    target = options.target.slice();
+                    token = token.split(SEP).join(PATH_SEP); // assuming urls in css use "/"
+                    url = pathResolve(options.source.join(PATH_SEP), token).split(PATH_SEP);
 
-            target.fill("..");
-            url = terminator + target.concat(url, file).join(SEP) + terminator;
+                    file = url.pop();
 
-            preservedTokens.push(url);
+                    // remove common part of both paths
+                    while (target[0] === url[0]) {
+                        target.shift();
+                        url.shift();
+                    }
 
-            preserver = "url(___PRESERVED_TOKEN_" + (preservedTokens.length - 1) + "___)";
-            sb.push(preserver);
+                    target.fill("..");
+                    url = terminator + target.concat(url, file).join(SEP) + terminator;
 
+                } else {
+                    url = terminator + token + terminator;
+                }
+
+                preservedTokens.push(url);
+
+                preserver = "url(" + ___PRESERVED_TOKEN_ + (preservedTokens.length - 1) + "___)";
+                sb.push(preserver);
+
+            } else {
+                sb.push("url(" + token + ")");
+            }
+
             appendIndex = endIndex + 1;
@@ -228,3 +238,3 @@
 
-            preserver = "url(___PRESERVED_TOKEN_" + (preservedTokens.length - 1) + "___)";
+            preserver = "url(" + ___PRESERVED_TOKEN_ + (preservedTokens.length - 1) + "___)";
             sb.push(preserver);
@@ -318,3 +328,3 @@
             preservedTokens.push(part);
-            buffer[index] = "___PRESERVED_TOKEN_" + (preservedTokens.length - 1) + "___";
+            buffer[index] = ___PRESERVED_TOKEN_ + (preservedTokens.length - 1) + "___";
         }
@@ -440,5 +450,3 @@ };
     content = extractDataUrls(content, preservedTokens);
-    if (options.convertUrls) {
-        content = convertRelativeUrls(content, options, preservedTokens);
-    }
+    content = convertRelativeUrls(content, options, preservedTokens);
     content = collectComments(content, comments);
@@ -460,3 +468,3 @@
         preservedTokens.push(token);
-        return quote + "___PRESERVED_TOKEN_" + (preservedTokens.length - 1) + "___" + quote;
+        return quote + ___PRESERVED_TOKEN_ + (preservedTokens.length - 1) + "___" + quote;
     });
@@ -480,3 +488,3 @@
             }
-            content = content.replace(placeholder,  "___PRESERVED_TOKEN_" + (preservedTokens.length - 1) + "___");
+            content = content.replace(placeholder, ___PRESERVED_TOKEN_ + (preservedTokens.length - 1) + "___");
             continue;
@@ -489,3 +497,3 @@ }
             preservedTokens.push("\\");
-            content = content.replace(placeholder,  "___PRESERVED_TOKEN_" + (preservedTokens.length - 1) + "___");
+            content = content.replace(placeholder, ___PRESERVED_TOKEN_ + (preservedTokens.length - 1) + "___");
             i = i + 1; // attn: advancing the loop
@@ -495,3 +503,3 @@ preservedTokens.push("");
                 "___PRESERVE_CANDIDATE_COMMENT_" + i + "___",
-                "___PRESERVED_TOKEN_" + (preservedTokens.length - 1) + "___"
+                ___PRESERVED_TOKEN_ + (preservedTokens.length - 1) + "___"
             );
@@ -508,3 +516,3 @@ continue;
                     preservedTokens.push("");
-                    content = content.replace(placeholder,  "___PRESERVED_TOKEN_" + (preservedTokens.length - 1) + "___");
+                    content = content.replace(placeholder, ___PRESERVED_TOKEN_ + (preservedTokens.length - 1) + "___");
                 }
@@ -552,3 +560,3 @@ }
         );
-        return "___PRESERVED_TOKEN_" + (preservedTokens.length - 1) + "___";
+        return ___PRESERVED_TOKEN_ + (preservedTokens.length - 1) + "___";
     });
@@ -560,3 +568,3 @@
         preservedTokens.push(f1);
-        return "filter:progid:DXImageTransform.Microsoft.Matrix(___PRESERVED_TOKEN_" + (preservedTokens.length - 1) + "___);";
+        return "filter:progid:DXImageTransform.Microsoft.Matrix(" + ___PRESERVED_TOKEN_ + (preservedTokens.length - 1) + "___);";
     });
@@ -587,3 +595,3 @@
             preservedTokens.push('0' + g2);
-            return g1 + "___PRESERVED_TOKEN_" + (preservedTokens.length - 1) + "___";
+            return g1 + ___PRESERVED_TOKEN_ + (preservedTokens.length - 1) + "___";
         });
@@ -601,3 +609,3 @@
                 preservedTokens.push('0%');
-                f0.push("___PRESERVED_TOKEN_" + (preservedTokens.length - 1) + "___");
+                f0.push(___PRESERVED_TOKEN_ + (preservedTokens.length - 1) + "___");
             } else {
@@ -781,3 +789,3 @@ f0.push(part);
     for (i = preservedTokens.length - 1; i >= 0 ; i--) {
-        content = content.replace("___PRESERVED_TOKEN_" + i + "___", preservedTokens[i], "g");
+        content = content.replace(___PRESERVED_TOKEN_ + i + "___", preservedTokens[i], "g");
     }
@@ -784,0 +792,0 @@

Fixed alerts

Major refactor

Supply chain risk

Package has recently undergone a major refactor. It may be unstable or indicate significant internal changes. Use caution when updating to versions that include significant changes.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

36446

increased by0.86%

Lines of code

697

increased by0.58%

Number of medium supply chain risk alerts

0

decreased by-100%

No dependency changes