Remove opts global leak. Fix MadLittleMods#13 - v0.3.9

Author: MadLittleMods

CHANGELOG.md

@@ -1,3 +1,9 @@
+
+# v0.3.9 - 2015-6-29
+
+ - Remove `opts` global leak. Fix #13
+
+
 # v0.3.8 - 2015-5-28
 
  - Add support for pseudo selectors `:hover` `:before`

README.md

@@ -8,7 +8,7 @@ CSS variables or CSS Custom Properties limited subset polyfill/shim.
 
 We strive for the most complete transformation but we/no plugin can achieve true complete parity according to the [specification](http://dev.w3.org/csswg/css-variables/) because of the DOM cascade unknowns.
 
-## Latest Version: v0.3.8
+## Latest Version: v0.3.9
 ### [Changelog](https://github.com/MadLittleMods/postcss-css-variables/blob/master/CHANGELOG.md)
 
 ### Install
@@ -217,7 +217,7 @@ will be processed to:
 
 ## Interoperability
 
-`post-css-variables` plays really nice with [`postcss-nested`](https://github.com/postcss/postcss-nested) in order to get a larger subset of CSS variables features. *See [Nested rules, Usage section](#nested-rules)*
+`postcss-css-variables` plays really nice with [`postcss-nested`](https://github.com/postcss/postcss-nested) in order to get a larger subset of CSS variables features. *See [Nested rules, Usage section](#nested-rules)*
 
 If you are using [`postcss-custom-properties`](https://github.com/postcss/postcss-custom-properties) previously, we have a compatible feature set and more so you can switch over without having to refactor any of your code. You can just start writing the new awesome stuff.
 
@@ -230,7 +230,7 @@ There is another similar plugin available, [`postcss-custom-properties`](https:/
 
 ### Differences from `postcss-custom-properties`
 
-The main features that we`post-css-variables` add/provide are:
+The main features that we`postcss-css-variables` add/provide are:
 
  - No limitation on what scope CSS variables can be declared or used (`:root` or wherever)
  	 - Proper value substitution based on explicit DOM/structure traversal

index.js

@@ -57,8 +57,8 @@ var defaults = {
 };
 
 module.exports = postcss.plugin('postcss-css-variables', function(options) {
-	
-	opts = extend({}, defaults, options);
+
+	var opts = extend({}, defaults, options);
 
 	// Work with opts here
 
@@ -78,7 +78,7 @@ module.exports = postcss.plugin('postcss-css-variables', function(options) {
 
 		// Add the js defined variables `opts.variables` to the map
 		map = extend(
-			map, 
+			map,
 			Object.keys(opts.variables).reduce(function(prevVariableMap, variableName) {
 				var variableEntry = opts.variables[variableName];
 				// Automatically prefix any variable with `--` (CSS custom property syntax) if it doesn't have it already
@@ -187,7 +187,7 @@ module.exports = postcss.plugin('postcss-css-variables', function(options) {
 		var rulesThatHaveDeclarationsWithVariablesList = [];
 		css.eachRule(function(rule) {
 			var doesRuleUseVariables = rule.nodes.some(function(node) {
-				if(node.type == 'decl') {
+				if(node.type === 'decl') {
 					var decl = node;
 					// If it uses variables
 					// and is not a variable declarations that we may be preserving from earlier
@@ -223,16 +223,16 @@ module.exports = postcss.plugin('postcss-css-variables', function(options) {
 			// Resolve the declarations
 			rulesToWorkOn.forEach(function(ruleToWorkOn) {
 				ruleToWorkOn.nodes.slice(0).forEach(function(node) {
-					if(node.type == 'decl') {
+					if(node.type === 'decl') {
 						var decl = node;
-						resolveDecl(decl, map);
+						resolveDecl(decl, map, opts.preserve);
 					}
 				});
 			});
 
 		});
 
-		
+
 
 
 

lib/clone-splice-parent-onto-node-when.js

@@ -35,19 +35,19 @@ var cloneSpliceParentOntoNodeWhen = function(node, parent, /*optional*/whenCb) {
 	// Assign parents to our parent clones
 	cloneParentList.forEach(function(parentClone, index, cloneParentList) {
 		// Keep assigning parents detached until just very end
-		if(index+1 < cloneParentList.length) {
+		if(index + 1 < cloneParentList.length) {
 			//parentClone.moveTo(cloneParentList[index+1]);
-			parentClone.parent = cloneParentList[index+1];
+			parentClone.parent = cloneParentList[index + 1];
 		}
 	});
 
 
 	// Assign parents to our node clones
 	cloneList.forEach(function(clone, index, cloneList) {
 		// Keep assigning parents detached until just very end
-		if(index+1 < cloneList.length) {
+		if(index + 1 < cloneList.length) {
 			//clone.moveTo(cloneList[index+1]);
-			clone.parent = cloneList[index+1];
+			clone.parent = cloneList[index + 1];
 		// Then splice on the new parent scope
 		} else {
 			// Set the highest parent ancestor to back to where we should splice in
@@ -61,4 +61,4 @@ var cloneSpliceParentOntoNodeWhen = function(node, parent, /*optional*/whenCb) {
 	return cloneList[0];
 };
 
-module.exports = cloneSpliceParentOntoNodeWhen;
+module.exports = cloneSpliceParentOntoNodeWhen;

lib/find-node-ancestor-with-selector.js

@@ -5,10 +5,9 @@ var generateScopeList = require('./generate-scope-list');
 var findNodeAncestorWithSelector = function(selector, node) {
 	var matchingNode;
 
-	var currentNode = node;
-	var stillFindingNode = true;
 	// Keep going until we run out of parents to search
 	// or we found the node
+	var currentNode = node;
 	while(currentNode.parent && !matchingNode) {
 		// A trick to get the selector split up. Generate a scope list on a clone(clean parent)
 		var currentNodeScopeList = generateScopeList(currentNode.clone(), true);
@@ -30,4 +29,4 @@ var findNodeAncestorWithSelector = function(selector, node) {
 	return matchingNode;
 };
 
-module.exports = findNodeAncestorWithSelector;
+module.exports = findNodeAncestorWithSelector;

lib/gather-variable-dependencies.js

@@ -47,4 +47,4 @@ var gatherVariableDependencies = function(variablesUsed, map, _dependencyVariabl
 	};
 };
 
-module.exports = gatherVariableDependencies;
+module.exports = gatherVariableDependencies;

lib/generate-descendant-pieces-from-selector.js

@@ -20,4 +20,4 @@ var generateDescendantPiecesFromSelector = function(selector) {
 		});
 };
 
-module.exports = generateDescendantPiecesFromSelector;
+module.exports = generateDescendantPiecesFromSelector;

lib/generate-direct-descendant-pieces-from-selector.js

@@ -18,4 +18,4 @@ var generateDirectDescendantPiecesFromSelector = function(selector) {
 		});
 };
 
-module.exports = generateDirectDescendantPiecesFromSelector;
+module.exports = generateDirectDescendantPiecesFromSelector;

lib/generate-scope-list.js

@@ -50,7 +50,7 @@ var generateScopeList = function(node, /*optional*/includeSelf) {
 
 				// Add to the front of the array
 				scopeStringPieces.unshift.apply(scopeStringPieces, descendantPieces);
-				
+
 				return scopeStringPieces;
 			});
 		});
@@ -66,6 +66,6 @@ var generateScopeList = function(node, /*optional*/includeSelf) {
 	}
 
 	return selectorScopeList;
-}
+};
 
-module.exports = generateScopeList;
+module.exports = generateScopeList;

lib/is-node-under-scope.js

@@ -8,4 +8,4 @@ var isNodeUnderScope = function(node, scopeNode, /*optional*/ignorePseudo) {
 	return isUnderScope(nodeScopeList, scopeNodeScopeList, ignorePseudo);
 };
 
-module.exports = isNodeUnderScope;
+module.exports = isNodeUnderScope;