Update dependency squizlabs/php_codesniffer to v3.13.0

[renovate]

This PR contains the following updates:

Package	Change	Age	Adoption	Passing	Confidence
squizlabs/php_codesniffer	3.7.1 -> 3.13.0

---

Release Notes

PHPCSStandards/PHP_CodeSniffer (squizlabs/php_codesniffer)

v3.13.0

Compare Source

Added

• Added support for PHP 8.4 asymmetric visibility modifiers to the tokenizer. #​871
  • Thanks to [Daniel Scherzer][@​DanielEScherzer] for the patch.
• Added support for PHP 8.4 final properties to the following sniffs:
  • PSR2.Classes.PropertyDeclaration #​950
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patches.

Changed

• Generic.WhiteSpace.LanguageConstructSpacing: will now also check the spacing after the goto language construct keyword. #​917
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• The PSR2.Classes.PropertyDeclaration will now check that the final modifier keyword is placed before a visibility keyword. #​950
  • Errors will be reported via a new FinalAfterVisibility error code.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Improved Help information about the --reports CLI flag. #​1078
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• The documentation for the following sniffs has been improved:
  • PSR1.Files.SideEffects
  • PSR2.ControlStructures.SwitchDeclaration
  • PSR2.Namespaces.NamespaceDeclaration
  • Thanks to [Rodrigo Primo][@​rodrigoprimo] for the patches.
• Various housekeeping, including improvements to the tests and documentation.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for their contributions.

Deprecated

• Nearly everything which was soft deprecated before is now hard deprecated and will show deprecation notices:
  • This applies to:
    • All sniffs which will be removed in 4.0. #​888
    • The deprecated Generator methods. #​889
    • The old array property setting format (via comma separated strings). #​890
    • Sniffs not implementing the PHP_CodeSniffer\Sniffs\Sniff interface. #​891
    • Sniffs not following the naming conventions. #​892
    • Standards called Internal. #​893
    • Sniffs which don't listen for PHP, like JS/CSS specific sniffs. #​894
  • The deprecation notices can be silenced by using the -q (=quiet) CLI flag.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patches.

Fixed

• Fixed bug #​1040 : Generic.Strings.UnnecessaryHeredoc - false positive for heredocs containing escape sequences.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​1040 : Generic.Strings.UnnecessaryHeredoc - fixer would not clean up escape sequences which aren't necessary in nowdocs.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​1048 : A file under scan would sometimes be updated with partial fixes, even though the file "failed to fix".
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.

Other

Calling all testers!

The first beta release for PHP_CodeSniffer 4.0 has been tagged. Please help by testing the beta release and reporting any issues you run into.
Upgrade guides for both ruleset maintainers/end-users, as well as for sniff developers and integrators, have been published to the Wiki to help smooth the transition.

v3.12.2

Compare Source

Added

• Added support for PHP 8.4 final properties to the following sniffs:
  • Generic.PHP.LowerCaseConstant #​948
  • Generic.PHP.UpperCaseConstant #​948
  • Squiz.Commenting.DocCommentAlignment #​951
  • Squiz.Commenting.VariableComment #​949
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patches.

Changed

• Tokenizer/PHP: a PHP open tag at the very end of a file will now always be tokenized as T_OPEN_TAG, independently of the PHP version. #​937
  • Previously, a PHP open tag at the end of a file was not tokenized as an open tag on PHP < 7.4 and the tokenization would depend on the short_open_tag setting.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• PEAR.Commenting.FunctionComment: improved message for "blank lines between docblock and declaration" check. #​830
• The documentation for the following sniffs has been improved:
  • Generic.Functions.OpeningFunctionBraceBsdAllman
  • Generic.Functions.OpeningFunctionBraceKernighanRitchie
  • Generic.WhiteSpace.LanguageConstructSpacing
  • Thanks to [Rodrigo Primo][@​rodrigoprimo] for the patches.
• Various housekeeping, including improvements to the tests and documentation.
  • Thanks to [Rodrigo Primo][@​rodrigoprimo] and [Juliette Reinders Folmer][@​jrfnl] for their contributions.

Fixed

• Fixed bug #​830 : PEAR.Commenting.FunctionComment will no longer remove blank lines within attributes.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​929 : Generic.PHP.ForbiddenFunctions: prevent false positives/negatives for code interlaced with comments.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​934 : Generic.PHP.LowerCaseConstant and Generic.PHP.UpperCaseConstant will now correctly ignore DNF types for properties.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​936 : Squiz.Commenting.FunctionCommentThrowTag: sniff would bow out when function has attributes attached, leading to false negatives.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​940 : Squiz.Commenting.VariableComment: false positive for missing docblock for properties using DNF types.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​944 : Squiz.Commenting.FunctionComment did not support DNF/intersection types in @param tags.
  • Thanks to [Jeffrey Angenent][@​devfrey] for the patch.
• Fixed bug #​945 : Squiz.WhiteSpace.FunctionSpacing would get confused when there are two docblocks above a function declaration.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​947 : Squiz.Commenting.FunctionCommentThrowTag: prevent false positives/negatives for code interlaced with comments.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​951 : Squiz.Commenting.DocCommentAlignment did not examine docblocks for final classes.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​955 : Potential race condition, leading to a fatal error, when both the Diff + the Code reports are requested and caching is on.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​956 : Generic.WhiteSpace.ScopeIndent: undefined array index notice when running in debug mode.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.

v3.12.1

Compare Source

Added

• Documentation for the following sniffs:
  • Squiz.Commenting.BlockComment
  • Thanks to [Colin Stewart][@​costdev] for the patch.

Changed

• Generic.WhiteSpace.HereNowdocIdentifierSpacing: improved error message text.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Various housekeeping, including improvements to the tests and documentation.
  • Thanks to [Rodrigo Primo][@​rodrigoprimo] and [Juliette Reinders Folmer][@​jrfnl] for their contributions.

Deprecated

• The Generic.Functions.CallTimePassByReference sniff. See #​921.
  • This sniff will be removed in version 4.0.0.

Fixed

• Fixed bug #​906 : Fixer: prevent InvalidArgumentExceptions when displaying verbose information.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​907 : Tokenizer/PHP: tokenization of tokens related to union, intersection and DNF types in combination with PHP 8.4 final properties.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​908 : Tokenizer/PHP: tokenization of ? in nullable types for readonly properties.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​916 : Tokenizer/PHP: goto was not recognized as a terminating statement for a case/default in a switch control structure.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.

Other

• PHP_CodeSniffer 4.0 is coming soon! Interested in a sneak peek ? Join the live stream at any time on April 14, 15, 17 or 18.
  Read the open invitation (#​924) for all the details.

v3.12.0

Compare Source

Added

• Added support for PHP 8.4 final properties to File::getMemberProperties() through a new is_final array index in the return value. #​834
  • Thanks to [Daniel Scherzer][@​DanielEScherzer] for the patch.
• Generators/HTML: each section title now has a unique anchor link, which can be copied when hovering over a title. #​859
  • This should make sharing a link to a specific section of the documentation more straight-forward.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Documentation for the following sniffs:
  • Squiz.Classes.ClassFileName
  • Squiz.Classes.ValidClassName
  • Thanks to [Brian Dunne][@​braindawg] for the patches.

Changed

• PHPCBF: the messaging when no fixable errors are found will now distinguish between "No violations" (at all) versus "No fixable errors". #​806
  • Thanks to [Peter Wilson][@​peterwilsoncc] for the patch.
• The -h (Help) option now contains a more extensive list of "config" options which can be set. #​809
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Improved error message when invalid sniff codes are supplied to --sniffs or --exclude command line arguments. #​344
  • Thanks to [Dan Wallis][@​fredden] for the patch.
• Improved error message when an invalid generator name is supplied to the --generator command line argument. #​709, #​771
  • The generator name will now also always be handled case-insensitively, independently of the OS used.
  • Thanks to [Rodrigo Primo][@​rodrigoprimo] for the patch.
• The user will be shown an informative error message for sniffs missing one of the required methods. #​873
  • Previously this would result in a fatal error.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Ruleset processing will now be allowed to run to its conclusion - barring critical errors - before displaying all ruleset errors in one go. #​857
  • Previously an error in a ruleset would cause PHPCS to exit immediately and show only one error at a time.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Generators: XML documentation files which don't contain any actual documentation will now silently be ignored. #​755
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Generators: when the title attribute is missing, the documentation generation will now fall back to the sniff name as the title. #​820
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Generators: cleaner output based on the elements of the documentation which are available. #​819, #​821
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Generators/HTML: improved display of code tables by using semantic HTML. #​854
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Squiz.Classes.ClassFileName: recommend changing the file name instead of changing the class name. #​845
  • This prevents unactionable recommendations due to the file name not translating to a valid PHP symbol name.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Squiz.Functions.FunctionDeclarationArgumentSpacing: incorrect spacing after a comma followed by a promoted property has an improved error message and will now be flagged with the SpacingBeforePropertyModifier or NoSpaceBeforePropertyModifier error codes. #​792
  • This was previously already flagged, but using either the SpacingBeforeHint or NoSpaceBeforeHint error code, which was misleading.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Squiz.Functions.FunctionDeclarationArgumentSpacing: the sniff will now also check the spacing after property modifiers for promoted properties in constructor methods. #​792
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Squiz.WhiteSpace.ScopeKeywordSpacing: the sniff will now also check the spacing after the final and abstract modifier keywords. #​604
  • Thanks to [Klaus Purer][@​klausi] for the patch.
• The following sniff(s) have received efficiency improvements:
  • Squiz.WhiteSpace.ScopeKeywordSpacing
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patches.
• Incorrectly set inline properties (in test case files) will be silently ignored again. #​884
  • This removes the Internal.PropertyDoesNotExist error code.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• The AbstractMethodUnitTest class will now flag duplicate test case markers in a test case file. #​773
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Various housekeeping, including improvements to the tests and documentation.
  • Thanks to [Asis Pattisahusiwa][@​asispts], [Dan Wallis][@​fredden], [Rodrigo Primo][@​rodrigoprimo] and [Juliette Reinders Folmer][@​jrfnl] for their contributions.

Deprecated

All deprecation are slated for removal in PHP_CodeSniffer 4.0.

• Support for sniffs not implementing the PHPCS Sniff interface. See #​694.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Support for including sniffs which don't comply with the PHPCS naming conventions (by referencing the sniff file directly). See #​689.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Support for external standards named "Internal". See #​799.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• The following Generator methods are now (soft) deprecated. See #​755:
  • PHP_CodeSniffer\Generators\Text::printTitle() in favour of PHP_CodeSniffer\Generators\Text::getFormattedTitle()
  • PHP_CodeSniffer\Generators\Text::printTextBlock() in favour of PHP_CodeSniffer\Generators\Text::getFormattedTextBlock()
  • PHP_CodeSniffer\Generators\Text::printCodeComparisonBlock() in favour of PHP_CodeSniffer\Generators\Text::getFormattedCodeComparisonBlock()
  • PHP_CodeSniffer\Generators\Markdown::printHeader() in favour of PHP_CodeSniffer\Generators\Markdown::getFormattedHeader()
  • PHP_CodeSniffer\Generators\Markdown::printFooter() in favour of PHP_CodeSniffer\Generators\Markdown::getFormattedFooter()
  • PHP_CodeSniffer\Generators\Markdown::printTextBlock() in favour of PHP_CodeSniffer\Generators\Markdown::getFormattedTextBlock()
  • PHP_CodeSniffer\Generators\Markdown::printCodeComparisonBlock() in favour of PHP_CodeSniffer\Generators\Markdown::getFormattedCodeComparisonBlock()
  • PHP_CodeSniffer\Generators\HTML::printHeader() in favour of PHP_CodeSniffer\Generators\HTML::getFormattedHeader()
  • PHP_CodeSniffer\Generators\HTML::printToc() in favour of PHP_CodeSniffer\Generators\HTML::getFormattedToc()
  • PHP_CodeSniffer\Generators\HTML::printFooter() in favour of PHP_CodeSniffer\Generators\HTML::getFormattedFooter()
  • PHP_CodeSniffer\Generators\HTML::printTextBlock() in favour of PHP_CodeSniffer\Generators\HTML::getFormattedTextBlock()
  • PHP_CodeSniffer\Generators\HTML::printCodeComparisonBlock() in favour of PHP_CodeSniffer\Generators\HTML::getFormattedCodeComparisonBlock()
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.

Fixed

• Fixed bug #​794 : Generators: prevent fatal error when the XML documentation does not comply with the expected format.
  • It is recommended to validate XML documentation files against the XSD file: https://phpcsstandards.github.io/PHPCSDevTools/phpcsdocs.xsd
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​814 : Generic.NamingConventions.ConstructorName: prevent potential fatal errors during live coding.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​816 : File::getDeclarationName(): prevent incorrect result for unfinished closures during live coding.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​817 : Squiz.Classes.ValidClassName: ignore comments when determining the name to be validated.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​825 : Squiz.Classes.ClassDeclaration: false positives when the next thing after a class was a function with an attribute attached.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​826 : Squiz.WhiteSpace.FunctionSpacing: prevent incorrect some results when attributes are attached to a function.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​827 : PEAR.Functions.FunctionDeclaration: fixer conflict over an unfinished closure during live coding.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​828 : Squiz.WhiteSpace.MemberVarSpacing: allow for readonly properties.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​832 : Squiz.WhiteSpace.MemberVarSpacing: prevent potential fixer conflict during live coding.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​833 : Squiz.PHP.EmbeddedPhp: fixer conflict when a PHP open tag for a multi-line snippet is found on the same line as a single-line embedded PHP snippet.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​833 : Squiz.PHP.EmbeddedPhp: incorrect indent calculation in certain specific situations.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​835 : Generic.PHP.DisallowShortOpenTag: don't act on parse errors.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​838 : Squiz.PHP.EmbeddedPhp: no new line before close tag was incorrectly enforced when a preceding OO construct or function had a trailing comment after the close curly.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​840 : Squiz.WhiteSpace.MemberVarSpacing: more accurate reporting on blank lines in the property "pre-amble" (i.e. docblock, attributes).
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​845 : Squiz.Classes.ClassFileName: don't throw an incorrect error for an unfinished OO declaration during live coding.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​865 : Setting an array property to an empty array from an XML ruleset now works correctly.
  • Previously, the property value would be set to [0 => ''].
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​866 : Squiz.WhiteSpace.FunctionOpeningBraceSpace: XML docs were not accessible due to an issue with the file name.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.

Other

• A new wiki page is available to clarify the difference between a project ruleset and an external standard.
  • This wiki page also contains detailed information about the naming conventions external standards must comply with.
• A new XMLLint validate action runner is available which can be used in CI to validate rulesets for PHP_CodeSniffer against the XSD.

v3.11.3

Compare Source

Changed

• Generic.ControlStructures.InlineControlStructure no longer unnecessarily listens for T_SWITCH tokens. #​595
  • Thanks to [Rodrigo Primo][@​rodrigoprimo] for the patch.
• Squiz.Functions.FunctionDeclarationArgumentSpacing: improvements to error message for SpaceBeforeComma error. #​783
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• The following sniff(s) have received efficiency improvements:
  • Squiz.Functions.FunctionDeclarationArgumentSpacing
  • Thanks to [Dan Wallis][@​fredden] and [Juliette Reinders Folmer][@​jrfnl] for the patches.
• Various housekeeping, including improvements to the tests and documentation.
  • Thanks to [Michał Bundyra][@​michalbundyra], [Rodrigo Primo][@​rodrigoprimo] and [Juliette Reinders Folmer][@​jrfnl] for their contributions.

Fixed

• Fixed bug #​620 : Squiz.Functions.FunctionDeclarationArgumentSpacing: newlines after type will now be handled by the fixer. This also prevents a potential fixer conflict.
  • Thanks to [Dan Wallis][@​fredden] for the patch.
• Fixed bug #​782 : Tokenizer/PHP: prevent an "Undefined array key" notice during live coding for unfinished arrow functions.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​783 : Squiz.Functions.FunctionDeclarationArgumentSpacing: new line after reference token was not flagged nor fixed.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​783 : Squiz.Functions.FunctionDeclarationArgumentSpacing: new line after variadic token was not flagged nor fixed.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​783 : Squiz.Functions.FunctionDeclarationArgumentSpacing: new line before/after the equal sign for default values was not flagged nor fixed when equalsSpacing was set to 0.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​783 : Squiz.Functions.FunctionDeclarationArgumentSpacing: fixer conflict when a new line is found before/after the equal sign for default values and equalsSpacing was set to 1.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​783 : Squiz.Functions.FunctionDeclarationArgumentSpacing: fixer for spacing before/after equal sign could inadvertently remove comment.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​783 : Squiz.Functions.FunctionDeclarationArgumentSpacing: fixer will now handle comments between the end of a parameter and a comma more cleanly.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​784 : Squiz.WhiteSpace.FunctionSpacing: prevent fixer conflict when a multi-line docblock would start on the same line as the function close curly being examined.
  • Thanks to [Klaus Purer][@​klausi] for the patch

v3.11.2

Compare Source

Changed

• Generators/HTML + Markdown: the output will now be empty (no page header/footer) when there are no docs to display. #​687
  • This is in line with the Text Generator which already didn't produce output if there are no docs.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Generators/HTML: only display a Table of Contents when there is more than one sniff with documentation. #​697
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Generators/HTML: improved handling of line breaks in <standard> blocks. #​723
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Generators/Markdown: improved compatibility with the variety of available markdown parsers. #​722
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Generators/Markdown: improved handling of line breaks in <standard> blocks. #​737
  • This prevents additional paragraphs from being displayed as code blocks.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Generic.NamingConventions.UpperCaseConstantName: the exact token containing the non-uppercase constant name will now be identified with more accuracy. #​665
  • Thanks to [Rodrigo Primo][@​rodrigoprimo] for the patch.
• Generic.Functions.OpeningFunctionBraceKernighanRitchie: minor improvement to the error message wording. #​736
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Various housekeeping, including improvements to the tests and documentation.
  • Thanks to [Rodrigo Primo][@​rodrigoprimo] and [Juliette Reinders Folmer][@​jrfnl] for their contributions.

Fixed

• Fixed bug #​527 : Squiz.Arrays.ArrayDeclaration: short lists within a foreach condition should be ignored.
  • Thanks to [Rodrigo Primo][@​rodrigoprimo] for the patch.
• Fixed bug #​665 : Generic.NamingConventions.UpperCaseConstantName: false positives and false negatives when code uses unconventional spacing and comments when calling define().
  • Thanks to [Rodrigo Primo][@​rodrigoprimo] for the patch.
• Fixed bug #​665 : Generic.NamingConventions.UpperCaseConstantName: false positive when a constant named DEFINE is encountered.
  • Thanks to [Rodrigo Primo][@​rodrigoprimo] for the patch.
• Fixed bug #​665 : Generic.NamingConventions.UpperCaseConstantName: false positive for attribute class called define.
  • Thanks to [Rodrigo Primo][@​rodrigoprimo] for the patch.
• Fixed bug #​665 : Generic.NamingConventions.UpperCaseConstantName: false positive when handling the instantiation of a class named define.
  • Thanks to [Rodrigo Primo][@​rodrigoprimo] for the patch.
• Fixed bug #​688 : Generators/Markdown could leave error_reporting in an incorrect state.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​698 : Generators/Markdown : link in the documentation footer would not parse as a link.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​738 : Generators/Text: stray blank lines after code sample titles.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​739 : Generators/HTML + Markdown: multi-space whitespace within a code sample title was folded into a single space.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.

v3.11.1

Compare Source

Changed

• Output from the --generator=... feature will respect the OS-expected EOL char in more places. #​671
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Various housekeeping, including improvements to the tests and documentation.
  • Thanks to [Bartosz Dziewoński][@​MatmaRex] and [Juliette Reinders Folmer][@​jrfnl] for their contributions.

Fixed

• Fixed bug #​674 : Generic.WhiteSpace.HereNowdocIdentifierSpacing broken XML documentation
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Fixed bug #​675 : InvalidArgumentException when a ruleset includes a sniff by file name and the included sniff does not comply with the PHPCS naming conventions.
  • Notwithstanding this fix, it is strongly recommended to ensure custom sniff classes comply with the PHPCS naming conventions.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.

v3.11.0

Compare Source

Added

• Runtime support for PHP 8.4. All known PHP 8.4 deprecation notices have been fixed.
  • Syntax support for new PHP 8.4 features will follow in a future release.
  • If you find any PHP 8.4 deprecation notices which were missed, please report them.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patches.
• Tokenizer support for PHP 8.3 "yield from" expressions with a comment between the keywords. #​529, #​647
  • Sniffs explicitly handling T_YIELD_FROM tokens may need updating. The PR description contains example code for use by sniff developers.
  • Additionally, the following sniff has been updated to support "yield from" expressions with comments:
    • Generic.WhiteSpace.LanguageConstructSpacing
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• New Generic.WhiteSpace.HereNowdocIdentifierSpacing sniff. #​586, #​637
  • Forbid whitespace between the <<< and the identifier string in heredoc/nowdoc start tokens.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• New Generic.Strings.UnnecessaryHeredoc sniff. #​633
  • Warns about heredocs without interpolation or expressions in the body text and can auto-fix these to nowdocs.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Documentation for the following sniffs:
  • Generic.Arrays.ArrayIndent
  • Squiz.PHP.Heredoc
  • Thanks to [Rodrigo Primo][@​rodrigoprimo] and [Juliette Reinders Folmer][@​jrfnl] for the patches.

Changed

• The Common::getSniffCode() method will now throw an InvalidArgumentException exception if an invalid $sniffClass is passed. #​524, #​625
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Documentation generated using the --generator=... feature will now always be presented in natural order based on the sniff name(s). #​668
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Minor improvements to the display of runtime information. #​658
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Squiz.Commenting.PostStatementComment: trailing annotations in PHP files will now be reported under a separate, non-auto-fixable error code AnnotationFound. #​560, #​627
  • This prevents (tooling related) annotations from taking on a different meaning when moved by the fixer.
  • The separate error code also allows for selectively excluding it to prevent the sniff from triggering on trailing annotations, while still forbidding other trailing comments.
  • Thanks to [Rodrigo Primo][@​rodrigoprimo] for the patch.
• Squiz.ControlStructures.ForEachLoopDeclaration: the SpacingAfterOpen error code has been replaced by the SpaceAfterOpen error code. The latter is a pre-existing code. The former appears to have been a typo. #​582
  • Thanks to [Dan Wallis][@​fredden] for the patch.
• The following sniff(s) have received efficiency improvements:
  • Generic.Classes.DuplicateClassName
  • Generic.NamingConventions.ConstructorName
  • Thanks to [Rodrigo Primo][@​rodrigoprimo] and [Juliette Reinders Folmer][@​jrfnl] for the patches.
• Various housekeeping, including improvements to the tests and documentation.
  • Thanks to [Rodrigo Primo][@​rodrigoprimo] and [Juliette Reinders Folmer][@​jrfnl] for their contributions.

Fixed

• Fixed bug #​3808 : Generic.WhiteSpace.ScopeIndent would throw false positive for tab indented multi-token yield from expression.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
• Fixed bug #​630 : The tokenizer could inadvertently transform "normal" parentheses to DNF parentheses, when a function call was preceded by a switch-case / alternative syntax control structure colon.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
• Fixed bug #​645 : On PHP 5.4, if yield was used as the declaration name for a function declared to return by reference, the function name would incorrectly be tokenized as T_YIELD instead of T_STRING.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
• Fixed bug #​647 : Tokenizer not applying tab replacement in single token "yield from" keywords.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
• Fixed bug #​647 : Generic.WhiteSpace.DisallowSpaceIndent did not flag space indentation in multi-line yield from.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
• Fixed bug #​647 : Generic.WhiteSpace.DisallowTabIndent did not flag tabs inside yield from.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
• Fixed bug #​652 : Generic.NamingConventions.ConstructorName: false positives for PHP-4 style calls to PHP-4 style parent constructor when a method with the same name as the parent class was called on another class.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
• Fixed bug #​652 : Generic.NamingConventions.ConstructorName: false negatives for PHP-4 style calls to parent constructor for function calls with whitespace and comments in unconventional places.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
• Fixed bug #​653 : Generic.Classes.DuplicateClassName : the sniff did not skip namespace keywords used as operators, which could lead to false positives.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
• Fixed bug #​653 : Generic.Classes.DuplicateClassName : sniff going into an infinite loop during live coding.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
• Fixed bug #​653 : Generic.Classes.DuplicateClassName : false positives/negatives when a namespace declaration contained whitespace or comments in unconventional places.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
• Fixed bug #​653 : Generic.Classes.DuplicateClassName : namespace for a file going in/out of PHP was not remembered/applied correctly.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch

v3.10.3

Compare Source

Changed

• Various housekeeping, including improvements to the tests and documentation.
  • Thanks to [Rodrigo Primo][@​rodrigoprimo] and [Juliette Reinders Folmer][@​jrfnl] for their contributions.

Fixed

• Fixed bug #​553 : Squiz.Classes.SelfMemberReference: false negative(s) when namespace operator was encountered between the namespace declaration and the OO declaration.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
• Fixed bug #​579 : AbstractPatternSniff: potential PHP notice during live coding.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
• Fixed bug #​580 : Squiz.Formatting.OperatorBracket: potential PHP notice during live coding.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
• Fixed bug #​581 : PSR12.ControlStructures.ControlStructureSpacing: prevent fixer conflict by correctly handling multiple empty newlines before the first condition in a multi-line control structure.
  • Thanks to [Dan Wallis][@​fredden] for the patch.
• Fixed bug #​585 : Tokenizer not applying tab replacement in heredoc/nowdoc openers.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
• Fixed bug #​588 : Squiz.PHP.EmbeddedPhp false positive when checking spaces after a PHP short open tag.
  • Thanks to [Rodrigo Primo][@​rodrigoprimo] for the patch.
• Fixed bug #​597 : Generic.PHP.LowerCaseKeyword did not flag nor fix non-lowercase anonymous class keywords.
  • Thanks to [Marek Štípek][@​maryo] for the patch.
• Fixed bug #​598 : Squiz.PHP.DisallowMultipleAssignments: false positive on assignments to variable property on object stored in array.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch
• Fixed bug #​608 : Squiz.Functions.MultiLineFunctionDeclaration did not take (parameter) attributes into account when checking for one parameter per line.
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch

Other

• The provenance of PHAR files associated with a release can now be verified via GitHub Artifact Attestations using the GitHub CLI tool with the following command: gh attestation verify [phpcs|phpcbf].phar -o PHPCSStandards. #​574
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.

v3.10.2

Compare Source

Changed

• The following sniff(s) have received efficiency improvements:
  • Generic.Functions.FunctionCallArgumentSpacing
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• The array format of the information passed to the Reports::generateFileReport() method is now documented in the Reports interface. [#​523]
  • Thanks to [Juliette Reinders Folmer][@​jrfnl] for the patch.
• Various housekeeping, including improvements to the tests and documentation.
  • Thanks to [Bill Ruddock][@​biinari], [Dan Wallis][@​fredden], [Klaus Purer][@​klausi], [Rodrigo Primo][@​rodrigoprimo] and [Juliette Reinders Folmer][@​jrfnl] for their contributions.

Fixed

• Fixed bug [#​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][@​jrfnl] for the patch
• Fixed bug [#​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 chec

---

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.

---

• If you want to rebase/retry this PR, check this box

---

This PR was generated by Mend Renovate. View the repository job log.