Merge "Fix unexpected duplicating of uncalled mixin rules"

lib/Less/Tree/Selector.php

@@ -101,12 +101,16 @@ public function CacheElements() {
 				$css .= $v->value;
 				continue;
 			}
+
+			if ( isset( $v->value->value ) && !is_object( $v->value->value ) ) {
+				$css .= $v->value->value;
+				continue;
+			}
+
 			if ( ( $v->value instanceof Less_Tree_Selector || $v->value instanceof Less_Tree_Variable )
 				|| !is_string( $v->value->value ) ) {
 				$this->cacheable = false;
-			}
-			if ( isset( $v->value->value ) && !is_object( $v->value->value ) ) {
-				$css .= $v->value->value;
+				return;
 			}
 		}
 

test/Fixtures/less.php/css/T363076-mixins-copies-rules.css

@@ -0,0 +1,21 @@
+.btn {
+  color: blue;
+  font-weight: bold;
+}
+.btn[title] {
+  background: lightgreen;
+}
+.btn[disabled] {
+  cursor: not-allowed;
+  color: red;
+}
+.btn[test] {
+  background-color: pink;
+}
+.btn-specific {
+  color: blue;
+  font-weight: bold;
+}
+.btn-specific[title] {
+  background: lightgreen;
+}

test/Fixtures/less.php/less/T363076-mixins-copies-rules.less

@@ -0,0 +1,19 @@
+.btn {
+	color: blue;
+	font-weight: bold;
+
+	&[title] {
+	   background: lightgreen;
+	}
+ }
+ .btn[disabled] {
+	cursor: not-allowed;
+	color: red;
+ }
+ .btn[test] {
+	background-color: pink;
+ }
+
+ .btn-specific {
+	.btn();
+ }