Added snif for print_r

Author: mmaas

MoxioSniffs/Sniffs/PHP/DisallowPrintrWithoutWrapperSniff.php

@@ -0,0 +1,26 @@
+<?php
+declare(strict_types=1);
+
+namespace Moxio\CodeSniffer\MoxioSniffs\Sniffs\PHP;
+
+
+use Moxio\CodeSniffer\MoxioSniffs\Sniffs\AbstractFunctionCallSniff;
+use PHP_CodeSniffer\Files\File;
+
+final class DisallowPrintrWithoutWrapperSniff extends AbstractFunctionCallSniff
+{
+
+	protected function registerFunctions()
+	{
+		return ['print_r'];
+	}
+
+	protected function processFunctionCall(File $phpcsFile, $functionName, $functionNamePtr, $argumentPtrs)
+	{
+		if (count($argumentPtrs) < 2) {
+			$error = "Do not commit code that prints directly to the console. Add 'true' or use '\MXO_Debug_Helper::print_r()'";
+			$phpcsFile->addError($error, $functionNamePtr, 'Disallowed');
+		}
+	}
+
+}

MoxioSniffs/Sniffs/Tests/PHP/DisallowPrintrWithoutWrapperSniffTest.inc

@@ -0,0 +1,6 @@
+<?php
+print_r('foo');
+print_r(['a']);
+print_r(['a' => 1, 'b' => 2]);
+print_r(['a'], true);
+print_r(['a' => 1, 'b' => 2], true);

MoxioSniffs/Sniffs/Tests/PHP/DisallowPrintrWithoutWrapperSniffTest.php

@@ -0,0 +1,23 @@
+<?php
+namespace Moxio\CodeSniffer\MoxioSniffs\Sniffs\Tests\PHP;
+
+use Moxio\CodeSniffer\MoxioSniffs\Sniffs\PHP\DisallowPrintrWithoutWrapperSniff;
+use Moxio\CodeSniffer\MoxioSniffs\Sniffs\Tests\AbstractSniffTest;
+
+class DisallowPrintrWithoutWrapperSniffTest extends AbstractSniffTest
+{
+    protected function getSniffClass()
+    {
+        return DisallowPrintrWithoutWrapperSniff::class;
+    }
+
+    public function testSniff()
+    {
+        $file = __DIR__ . '/DisallowPrintrWithoutWrapperSniffTest.inc';
+        $this->assertFileHasErrorsOnLines($file, [
+            2,
+            3,
+            4,
+        ]);
+    }
+}