ENH Tidy-up code

bin/mdphpcs

@@ -2,7 +2,7 @@
 <?php
 
 use PHP_CodeSniffer\Exceptions\DeepExitException;
-use SilverStripe\MD_PHP_CodeSniffer\Sniffer;
+use SilverStripe\MarkdownPhpCodeSniffer\Sniffer;
 
 // php_codesniffer autoloader (which itself includes the composer autoloader)
 $autoloadCandidates = [

composer.json

@@ -16,8 +16,8 @@
     },
     "autoload": {
         "psr-4": {
-            "SilverStripe\\MD_PHP_CodeSniffer\\": "src/",
-            "SilverStripe\\MD_PHP_CodeSniffer\\Tests\\": "tests/"
+            "SilverStripe\\MarkdownPhpCodeSniffer\\": "src/",
+            "SilverStripe\\MarkdownPhpCodeSniffer\\Tests\\": "tests/"
         }
     },
     "bin": [

src/CodeBlock.php

@@ -1,8 +1,10 @@
 <?php
 
-namespace SilverStripe\MD_PHP_CodeSniffer;
+namespace SilverStripe\MarkdownPhpCodeSniffer;
 
+use PHP_CodeSniffer\Config;
 use PHP_CodeSniffer\Files\DummyFile;
+use PHP_CodeSniffer\Ruleset;
 
 class CodeBlock extends DummyFile
 {
@@ -12,6 +14,21 @@ class CodeBlock extends DummyFile
 
     private string $finalContent = '';
 
+    public function __construct(
+        Ruleset $ruleset,
+        Config $config,
+        string $content,
+        string $path,
+        string $realPath,
+        int $num
+    ) {
+        parent::__construct($content, $ruleset, $config);
+
+        $this->path = $path;
+        $this->realPath = $realPath;
+        $this->num = $num;
+    }
+
     public function cleanUp()
     {
         $this->finalContent = $this->content ?? '';

src/FixerReport.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace SilverStripe\MD_PHP_CodeSniffer;
+namespace SilverStripe\MarkdownPhpCodeSniffer;
 
 use PHP_CodeSniffer\Exceptions\DeepExitException;
 use PHP_CodeSniffer\Files\File;

src/Sniffer.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace SilverStripe\MD_PHP_CodeSniffer;
+namespace SilverStripe\MarkdownPhpCodeSniffer;
 
 use League\CommonMark\Environment\Environment;
 use League\CommonMark\Extension\CommonMark\CommonMarkCoreExtension;
@@ -65,10 +65,14 @@ public function run(string $lintLanguage, bool $fixing, bool $usingExplicitStand
         // Add code blocks to the file list for linting
         $todo = [];
         foreach ($codeBlocks as $block) {
-            $dummy = new CodeBlock($block['content'], $sniffer->ruleset, $sniffer->config);
-            $dummy->num = $block['num'];
-            $dummy->path = $block['path'];
-            $dummy->realPath = $block['realpath'];
+            $dummy = new CodeBlock(
+                $sniffer->ruleset,
+                $sniffer->config,
+                $block['content'],
+                $block['path'],
+                $block['realpath'],
+                $block['num']
+            );
             $todo[] = $dummy;
         }
 
@@ -107,6 +111,7 @@ public function run(string $lintLanguage, bool $fixing, bool $usingExplicitStand
 
         $sniffer->reporter->printReports();
 
+        // These return values are directly from Runner::runPHPCS()
         if ($numErrors === 0) {
             // No errors found.
             return 0;
@@ -148,6 +153,7 @@ private function prepareConfig(bool $usingExplicitStandard, string $lintLanguage
             $config->standards = [__DIR__ . '/../phpcs.default.xml'];
         }
 
+        // Most of these overrides are directly from Runner::runPHPCBF()
         if ($fixing) {
             // Override some of the command line settings that might break the fixes.
             $config->generator = null;
@@ -228,7 +234,7 @@ private function sniff(Runner $sniffer, array $todo): int
         // Turn all sniff errors into exceptions.
         set_error_handler([$sniffer, 'handleErrors']);
 
-        $lastDir  = '';
+        $lastDir = '';
         $numBlocks = count($todo);
 
         // Process each block sequentially - running sniff in parallel isn't supported

tests/CodeBlockTest.php

@@ -1,12 +1,12 @@
 <?php
 
-namespace SilverStripe\MD_PHP_CodeSniffer\Test;
+namespace SilverStripe\MarkdownPhpCodeSniffer\Test;
 
 use ReflectionProperty;
 use PHP_CodeSniffer\Config;
 use PHP_CodeSniffer\Ruleset;
 use PHPUnit\Framework\TestCase;
-use SilverStripe\MD_PHP_CodeSniffer\CodeBlock;
+use SilverStripe\MarkdownPhpCodeSniffer\CodeBlock;
 
 class CodeBlockTest extends TestCase
 {
@@ -22,7 +22,7 @@ public static function setUpBeforeClass(): void
     public function testGetContent()
     {
         $config = new Config();
-        $block = new CodeBlock('This is the content', new Ruleset($config), $config);
+        $block = new CodeBlock(new Ruleset($config), $config, 'This is the content', '', '', 0);
 
         $this->assertSame('This is the content', $block->getContent());
 
@@ -34,7 +34,7 @@ public function testGetContent()
     public function testCleanup()
     {
         $config = new Config();
-        $block = new CodeBlock('This is the content', new Ruleset($config), $config);
+        $block = new CodeBlock(new Ruleset($config), $config, 'This is the content', '', '', 0);
         $block->cleanUp();
 
         $this->assertSame('This is the content', $block->getContent());

tests/SnifferFixTest.php

@@ -1,12 +1,12 @@
 <?php
 
-namespace SilverStripe\MD_PHP_CodeSniffer\Test;
+namespace SilverStripe\MarkdownPhpCodeSniffer\Test;
 
 use PHP_CodeSniffer\Files\FileList;
 use PHP_CodeSniffer\Ruleset;
 use PHPUnit\Framework\TestCase;
 use ReflectionMethod;
-use SilverStripe\MD_PHP_CodeSniffer\Sniffer;
+use SilverStripe\MarkdownPhpCodeSniffer\Sniffer;
 
 /**
  * Because of PHP CodeSniffer's reliance on constants, this has to be done separately from the other tests.

tests/SnifferTest.php

@@ -1,12 +1,12 @@
 <?php
 
-namespace SilverStripe\MD_PHP_CodeSniffer\Test;
+namespace SilverStripe\MarkdownPhpCodeSniffer\Test;
 
 use PHP_CodeSniffer\Files\FileList;
 use PHP_CodeSniffer\Ruleset;
 use PHPUnit\Framework\TestCase;
 use ReflectionMethod;
-use SilverStripe\MD_PHP_CodeSniffer\Sniffer;
+use SilverStripe\MarkdownPhpCodeSniffer\Sniffer;
 
 class SnifferTest extends TestCase
 {