Fixed bug #2465 : Excluding a sniff by path is not working

gsherwood · gsherwood · commit a121fdbebcbe · 2019-04-01T15:49:21.000+11:00
diff --git a/package.xml b/package.xml
@@ -31,6 +31,7 @@ http://pear.php.net/dtd/package-2.0.xsd">
   - Fixed bug #2452 : LowercasePHPFunctions sniff failing on "new \File()"
   - Fixed bug #2453 : Squiz.CSS.SemicolonSpacingSniff false positive when style name proceeded by an asterisk
     -- Thanks to Juliette Reinders Folmer for the patch
+  - Fixed bug #2465 : Excluding a sniff by path is not working
  </notes>
  <contents>
   <dir name="/">
diff --git a/src/Ruleset.php b/src/Ruleset.php
@@ -467,7 +467,7 @@ public function processRuleset($rulesetPath, $depth=0)
                     } else {
                         $excludedSniffs = array_merge(
                             $excludedSniffs,
-                            $this->expandRulesetReference($exclude['name'], $rulesetDir, ($depth + 1))
+                            $this->expandRulesetReference((string) $exclude['name'], $rulesetDir, ($depth + 1))
                         );
                     }
                 }//end foreach

Original file line number	Diff line number	Diff line change
`@@ -467,7 +467,7 @@ public function processRuleset($rulesetPath, $depth=0)`
`467`	`467`	`} else {`
`468`	`468`	`$excludedSniffs = array_merge(`
`469`	`469`	`$excludedSniffs,`
`470`		`- $this->expandRulesetReference($exclude['name'], $rulesetDir, ($depth + 1))`
	`470`	`+ $this->expandRulesetReference((string) $exclude['name'], $rulesetDir, ($depth + 1))`
`471`	`471`	`);`
`472`	`472`	`}`
`473`	`473`	`}//end foreach`