JSLint: bug fix

jrfnl · jrfnl · commit e9a62e5d4037 · 2021-02-13T09:20:15.000+01:00
The `$name` passed to `Config::getExecutablePath()` appears to be incorrect which would result in a duplicate `jslint jslint` command.
diff --git a/src/Standards/Squiz/Sniffs/Debug/JSLintSniff.php b/src/Standards/Squiz/Sniffs/Debug/JSLintSniff.php
@@ -48,7 +48,7 @@ public function register()
      */
     public function process(File $phpcsFile, $stackPtr)
     {
-        $rhinoPath  = Config::getExecutablePath('jslint');
+        $rhinoPath  = Config::getExecutablePath('rhino');
         $jslintPath = Config::getExecutablePath('jslint');
         if ($rhinoPath === null || $jslintPath === null) {
             return;

Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ public function register()`
`48`	`48`	`*/`
`49`	`49`	`public function process(File $phpcsFile, $stackPtr)`
`50`	`50`	`{`
`51`		`- $rhinoPath = Config::getExecutablePath('jslint');`
	`51`	`+ $rhinoPath = Config::getExecutablePath('rhino');`
`52`	`52`	`$jslintPath = Config::getExecutablePath('jslint');`
`53`	`53`	`if ($rhinoPath === null \|\| $jslintPath === null) {`
`54`	`54`	`return;`