Merge pull request #210 from PHPCSStandards/develop

Release PHPCSExtra 1.0.2

Author: jrfnl

CHANGELOG.md

@@ -14,6 +14,17 @@ This projects adheres to [Keep a CHANGELOG](http://keepachangelog.com/) and uses
 
 _Nothing yet._
 
+## [1.0.2] - 2023-01-10
+
+### Changed
+
+#### Universal
+
+* `Universal.CodeAnalysis.ConstructorDestructorReturn`: the sniff will now respect a potentially set [`php_version` configuration option][php_version-config] and only report on PHP4-style constructors when the `php_version` is below `'80000'`. Thanks [@anomiex] for reporting! [#207], [#208]
+
+[#207]: https://github.com/PHPCSStandards/PHPCSExtra/issues/207
+[#208]: https://github.com/PHPCSStandards/PHPCSExtra/pull/208
+
 
 ## [1.0.1] - 2023-01-05
 
@@ -221,7 +232,6 @@ The upgrade to PHPCSUtils 1.0.0-alpha4 took care of a number of bugs, which pote
 
 [php-manual-dirname]:           https://www.php.net/function.dirname
 [php-rfc-negative_array_index]: https://wiki.php.net/rfc/negative_array_index
-[php_version-config]:           https://github.com/squizlabs/PHP_CodeSniffer/wiki/Configuration-Options#setting-the-php-version
 [ESLint "no lonely if"]:        https://eslint.org/docs/rules/no-lonely-if
 [PHPCSUtils 1.0.0-alpha4]:      https://github.com/PHPCSStandards/PHPCSUtils/releases/tag/1.0.0-alpha4
 
@@ -385,8 +395,10 @@ This initial alpha release contains the following sniffs:
     Individual sub-types can be allowed by excluding specific error codes.
 
 [Composer PHPCS plugin]: https://github.com/PHPCSStandards/composer-installer
+[php_version-config]:    https://github.com/squizlabs/PHP_CodeSniffer/wiki/Configuration-Options#setting-the-php-version
 
 [Unreleased]: https://github.com/PHPCSStandards/PHPCSExtra/compare/stable...HEAD
+[1.0.2]: https://github.com/PHPCSStandards/PHPCSExtra/compare/1.0.1...1.0.2
 [1.0.1]: https://github.com/PHPCSStandards/PHPCSExtra/compare/1.0.0...1.0.1
 [1.0.0]: https://github.com/PHPCSStandards/PHPCSExtra/compare/1.0.0-rc1...1.0.0
 [1.0.0-RC1]: https://github.com/PHPCSStandards/PHPCSExtra/compare/1.0.0-alpha3...1.0.0-rc1

README.md

@@ -177,7 +177,6 @@ The sniff will make a distinction between keys which will be duplicate in all PH
 If a [`php_version` configuration option][php_version-config] has been passed to PHPCS using either `--config-set` or `--runtime-set`, it will be respected by the sniff and only report duplicate keys for the configured PHP version.
 
 [php-rfc-negative_array_index]: https://wiki.php.net/rfc/negative_array_index
-[php_version-config]:           https://github.com/squizlabs/PHP_CodeSniffer/wiki/Configuration-Options#setting-the-php-version
 
 #### `Universal.Arrays.MixedArrayKeyTypes` :books:
 
@@ -218,6 +217,9 @@ Require a consistent modifier keyword order for class declarations.
 * Disallows return type declarations on constructor/destructor methods - error code: `ReturnTypeFound`, auto-fixable.
 * Discourages constructor/destructor methods returning a value - error code: `ReturnValueFound`.
 
+If a [`php_version` configuration option][php_version-config] has been passed to PHPCS using either `--config-set` or `--runtime-set`, it will be respected by the sniff.
+In effect, this means that the sniff will only report on PHP4-style constructors if the configured PHP version is less than 8.0.
+
 #### `Universal.CodeAnalysis.ForeachUniqueAssignment` :wrench: :books:
 
 Detects `foreach` control structures which use the same variable for both the key as well as the value assignment as this will lead to unexpected - and most likely unintended - behaviour.
@@ -478,3 +480,5 @@ This code is released under the GNU Lesser General Public License (LGPLv3). For
 [phpcs-gh]:              https://github.com/squizlabs/PHP_CodeSniffer
 [phpcsutils-gh]:         https://github.com/PHPCSStandards/PHPCSUtils
 [composer-installer-gh]: https://github.com/PHPCSStandards/composer-installer
+
+[php_version-config]:    https://github.com/squizlabs/PHP_CodeSniffer/wiki/Configuration-Options#setting-the-php-version

Universal/Sniffs/CodeAnalysis/ConstructorDestructorReturnSniff.php

@@ -14,6 +14,7 @@
 use PHP_CodeSniffer\Sniffs\Sniff;
 use PHP_CodeSniffer\Util\Tokens;
 use PHPCSUtils\BackCompat\BCFile;
+use PHPCSUtils\BackCompat\Helper;
 use PHPCSUtils\Tokens\Collections;
 use PHPCSUtils\Utils\FunctionDeclarations;
 use PHPCSUtils\Utils\GetTokensAsString;
@@ -67,7 +68,12 @@ public function process(File $phpcsFile, $stackPtr)
         if ($functionNameLC === '__construct' || $functionNameLC === '__destruct') {
             $functionType = \sprintf('A "%s()" magic method', $functionNameLC);
         } else {
-            // This may be a PHP 4-style constructor.
+            // If the PHP version is explicitly set to PHP 8.0 or higher, ignore PHP 4-style constructors.
+            if ((int) Helper::getConfigData('php_version') >= 80000) {
+                return;
+            }
+
+            // This may be a PHP 4-style constructor which should be handled.
             $OOName = ObjectDeclarations::getName($phpcsFile, $scopePtr);
 
             if (empty($OOName) === true) {

Universal/Tests/CodeAnalysis/ConstructorDestructorReturnUnitTest.inc renamed to Universal/Tests/CodeAnalysis/ConstructorDestructorReturnUnitTest.1.inc

@@ -0,0 +1,22 @@
+<?php
+
+/*
+ * This test file is run with php_version set to a value of 80000 or higher.
+ *
+ * The PHP4-style constructor should no longer be recognized as a constructor.
+ * No errors should be thrown for it, nor any auto-fixes made.
+ */
+class ReturnsAValue {
+    public function __construct(): self {
+        return $this;
+    }
+
+    public function __destruct():string {
+        return 'destructed';
+    }
+
+    function returnsavalue(): string
+    {
+        return 'php4style';
+    }
+}

Universal/Tests/CodeAnalysis/ConstructorDestructorReturnUnitTest.inc.fixed renamed to Universal/Tests/CodeAnalysis/ConstructorDestructorReturnUnitTest.1.inc.fixed

@@ -0,0 +1,22 @@
+<?php
+
+/*
+ * This test file is run with php_version set to a value of 80000 or higher.
+ *
+ * The PHP4-style constructor should no longer be recognized as a constructor.
+ * No errors should be thrown for it, nor any auto-fixes made.
+ */
+class ReturnsAValue {
+    public function __construct() {
+        return $this;
+    }
+
+    public function __destruct() {
+        return 'destructed';
+    }
+
+    function returnsavalue(): string
+    {
+        return 'php4style';
+    }
+}

Universal/Tests/CodeAnalysis/ConstructorDestructorReturnUnitTest.2.inc

@@ -0,0 +1,22 @@
+<?php
+
+/*
+ * This test file is run with php_version set to a value of 70999 or lower.
+ *
+ * The PHP4-style constructor should be recognized as a constructor and handled as usual.
+ */
+
+class ReturnsAValue {
+    public function __construct(): self {
+        return $this;
+    }
+
+    public function __destruct():string {
+        return 'destructed';
+    }
+
+    function returnsavalue(): string
+    {
+        return 'php4style';
+    }
+}

Universal/Tests/CodeAnalysis/ConstructorDestructorReturnUnitTest.2.inc.fixed

@@ -0,0 +1,22 @@
+<?php
+
+/*
+ * This test file is run with php_version set to a value of 70999 or lower.
+ *
+ * The PHP4-style constructor should be recognized as a constructor and handled as usual.
+ */
+
+class ReturnsAValue {
+    public function __construct() {
+        return $this;
+    }
+
+    public function __destruct() {
+        return 'destructed';
+    }
+
+    function returnsavalue()
+    {
+        return 'php4style';
+    }
+}

Universal/Tests/CodeAnalysis/ConstructorDestructorReturnUnitTest.3.inc

@@ -0,0 +1,14 @@
+<?php
+
+/*
+ * This test file is only here to reset the value of the php_version configuration option.
+ *
+ * The PHP4-style constructor should be recognized as a constructor and handled as usual.
+ */
+
+class ReturnsAValue {
+    function returnsavalue(): string
+    {
+        return 'php4style';
+    }
+}