custom pattern, concatenation, media query

[sKopheK]

Hello,

I wanted to use this plugin to update URLs after processed CSS file is placed into a subfolder so they still point to proper resources, but found out rules within media queries are processed twice, see example: https://runkit.com/skophek/5ab57de05bbffa0012a596d4

[peschee]

@dschenk @mzdr can one of you have a look?

[mzdr]

Hey @sKopheK,

thanks for taking the time to report this issue!

The problem is that we loop over all nodes and use .replaceValues() regardless if it’s just a declaration or a whole tree of rules. That’s why replacement is happening multiple times in such cases.

postcss-replace/index.js

Lines 40 to 50 in 2e74c9a

	nodeWalker((node) => {
	// Node
	if (node.text) {
	node.text = node.text.replace(...replacementArgs)
	}
	// Container
	else if (node.replaceValues) {
	node.replaceValues(...replacementArgs)
	}
	});

This is indeed a bug and I’m already fixing it right now. 👍