3.10.2 - 2024-07-22

[jrfnl]

Changed

• The following sniff(s) have received efficiency improvements:
  • Generic.Functions.FunctionCallArgumentSpacing
  • Thanks to Juliette Reinders Folmer for the patch.
• The array format of the information passed to the Reports::generateFileReport() method is now documented in the Reports interface. Documentation: improve information about Report data format + remove unused foreach keys #523
  • Thanks to Juliette Reinders Folmer for the patch.
• Various housekeeping, including improvements to the tests and documentation.
  • Thanks to Bill Ruddock, Dan Wallis, Klaus Purer, Rodrigo Primo and Juliette Reinders Folmer for their contributions.

Fixed

• Fixed bug Generic/FunctionCallArgumentSpacing: bug fix - ignore commas in nested match structures + minor efficiency tweak #513 : Generic.Functions.FunctionCallArgumentSpacing did not ignore the body of a match expressions passed as a function argument, which could lead to false positives.
  • Thanks to Juliette Reinders Folmer for the patch
• Fixed bug Generic/DisallowTabIndent: do not auto-fix heredoc/nowdoc closer indent #533 : Generic.WhiteSpace.DisallowTabIndent: tab indentation for heredoc/nowdoc closers will no longer be auto-fixed to prevent parse errors. The issue will still be reported.
  • The error code for heredoc/nowdoc indentation using tabs has been made more specific - TabsUsedHeredocCloser - to allow for selectively excluding the indentation check for heredoc/nowdoc closers.
  • Thanks to Juliette Reinders Folmer for the patch
• Fixed bug Generic/DisallowSpaceIndent: flag heredoc/nowdoc closer using space indent #534 : Generic.WhiteSpace.DisallowSpaceIndent did not report on space indentation for PHP 7.3 flexible heredoc/nowdoc closers.
  • Closers using space indentation will be reported with a dedicated error code: SpacesUsedHeredocCloser.
  • Thanks to Juliette Reinders Folmer for the patch
• Fixed bug Squiz.PHP.DisallowMultipleAssignments false positive for list assignments after embedded PHP without semicolon #537 : Squiz.PHP.DisallowMultipleAssignments false positive for list assignments at the start of a new PHP block after an embedded PHP statement.
  • Thanks to Juliette Reinders Folmer for the patch
• Fixed bug Squiz/DisallowMultipleAssignments: false positive when checking for default values and function is missing the closing parenthesis #551 : Squiz.PHP.DisallowMultipleAssignments prevent false positive for function parameters during live coding.
  • Thanks to Rodrigo Primo for the patch.
• Fixed bug Generic/UselessOverridingMethod: improve handling of PHP open/close tags between statements #554 : Generic.CodeAnalysis.UselessOverridingMethod edge case false negative when the call to the parent method would end on a PHP close tag.
  • Thanks to Juliette Reinders Folmer for the patch
• Fixed bug Squiz/SelfMemberReference: handle namespace declaration ending on PHP close tag #555 : Squiz.Classes.SelfMemberReference edge case false negative when the namespace declaration would end on a PHP close tag.
  • Thanks to Juliette Reinders Folmer for the patch

---

New Contributors

The PHP_CodeSniffer project is happy to welcome the following new contributor:
@biinari (with a whopping 9 merged PRs!)

We also welcome @klausi back to the project.

Furthermore, the project is happy to welcome @JetBrains as a corporate funder for the project, joining @Automattic, @humanmade and @godaddy.

Statistics

Closed: 2 issues
Merged: 30 pull requests

If you like to stay informed about releases and more, follow @phpcs on Mastodon or @PHP_CodeSniffer on X.

---

This discussion was created from the release 3.10.2 - 2024-07-22.