No functional changes.

Universal/DuplicateArrayKey: add support for detecting duplicate key PHP cross-version

How array keys for array items without keys are being determined, has changed in PHP 8.0.

Previ...

Universal/DisallowStandalonePostIncrementDecrement: allow for statement ending with close tag

A stand-alone statement doesn't necessarily have to end with a semi-colon. It can also end on a ...

Universal/ForeachUniqueAssignment: improve analysis for foreach list assignments

Universal/DisallowUse[Class|Const|Function]: minor code coverage tweak

Universal/NoReservedKeywordParameterNames: various improvements

This commit:
* Prevents "accidentally correct" handling of code like `foreach ($data as [$id => ...

These tests weren't testing the sniff well enough.

Ignore a statement which should never be reachable (defense in depth code).

... to confirm that properties set via the constructor are also checked.

Properties set via the...

While parameters (variables) are case-sensitive in PHP, keywords are not.

This commit improves ...

... to safeguard bowing out for function declarations without parameters and that the sniff stay...

... which can be made now support for PHPCS < 3.7.1 has been dropped.

The sniff now only listen...

✨ New Modernize.FunctionCalls.Dirname sniff

This new sniff will detect and, when possible, auto-fix two typical code modernizations which ca...

Includes updating CI and other dev related files to take the new standard into account.

Universal/SeparateFunctionsFromOO: update tests for arrow functions

As the sniff uses predefined token collections from PHPCSUtils, it automatically takes PHP 7.4 a...

Universal/SeparateFunctionsFromOO: update tests for enums

Universal/DisallowLonelyIf: bow out early for a specific situation (PHP close tag)

As the sniff uses predefined token collections from PHPCSUtils, it automatically takes PHP 8.1 `...

If an inner "if/else" chain using alternative syntax has a PHP close tag after the `endif` inste...

Universal/DisallowLonelyIf: add extra test

Universal/NoLeadingBackslash: examine imports within a group use statement

.. to cover an edge case which is already handled correctly.

Previously the sniff would only examine the start of a complete import statement, now it will al...

Universal/OneStatementInShortEchoTag: prevent false positive

Universal/RequireFinalClass: various tweaks

Universal/DisallowFinalClass: various tweaks

Universal/RequireAnonClassParentheses: remove redundant condition

Universal/DisallowAnonClassParentheses: remove redundant condition

... and annotate why the condition has been removed, as well as annotating a second likely redun...

As things were, the sniff would bow out if the opening brace for the class could not be found (y...

As things were, the sniff would bow out if the opening brace for the class could not be found (y...

As things were, the sniff would bow out if the opening brace for the class could not be found (y...

As things were, the sniff would bow out if the opening brace for the class could not be found (y...

In convoluted code, which I hope to never encounter in real life, it _could_ be possible to enco...

When a class is declared as `abstract`, it would attempt to record both for the `abstract`, as w...

GH Actions: bust the cache semi-regularly

Universal/DisallowAlternativeSyntax: bug fix - handle if/elseif statements in one go

As things were, the sniff would examine - and potentially fix - each control structure keyword w...

Universal/DisallowAlternativeSyntax: bug fix - ignore inline HTML in nested closed scopes

... as the HTML in that case is not necessarily echo-ed out when the control structure is execut...

Universal/DisallowAlternativeSyntax: improve error message

Group the setting of the `$code` and `$data` variables for the error message with the conditions...

Improve the readability of the error message by moving the variable part to the end of the message.

When the `$allowWithInlineHTML` property is set to `true`, the error message could be regarded a...

Universal/DisallowAlternativeSyntax: various tweaks

* Minor tweak to the metric name.
* Also record metric for single line/no body control structures.

When a control structure condition is directly followed by a PHP close tag, a semicolon is silen...

While empty "leafs" of control structures is allowed, it is less common, so make the code used f...

ControlStructures::hasBody(): handle no body due to PHP close tag

When a control structure condition is directly followed by a PHP close tag, a semicolon is silen...

GH Actions: bust the cache semi-regularly

GH Actions: bust the cache semi-regularly

GH Actions: bust the cache semi-regularly

Caches used in GH Actions do not get updated, they can only be replaced by a different cache wit...

Caches used in GH Actions do not get updated, they can only be replaced by a different cache wit...

Caches used in GH Actions do not get updated, they can only be replaced by a different cache wit...

Caches used in GH Actions do not get updated, they can only be replaced by a different cache wit...

Universal/Disallow[Long|Short]ListSyntax: improve metrics

Universal/DisallowShortArraySyntax: record metrics

Record both the `yes` as well as the `no`, which makes the metric feature complete.

... as recording just and only the `no` is not a useful metric and adding the logic to also reco...

It's quite straight forward to record informative metrics for this sniff, so let's.

Universal/DisallowShortArraySyntax: don't skip over short lists

Universal/DisallowShortListSyntax: bug fix - don't skip over nested brackets

... as short arrays can contain short lists and those still need to be examined.

Includes test ...

... as for some unfanthomable reason (short) arrays can be used as keys for short lists and if s...

Universal/DisallowShortListSyntax: allow for tokenizer issue in PHPCS 3.7.1

... which was fixed in 3.7.2.

The utility methods used from PHPCSUtils 1.0.0-alpha4 already tak...

Universal/AlphabeticExtendsImplements: docs/test update for enums

Universal/DisallowUseClass: "docs" update for enums

Universal/[Require/Disallow]FinalClass: docs/test update for readonly classes

This commit adds tests confirming that `enum ... implements ...` statements are handled correctl...

This updates the phrasing referencing the different OO structures to include enums in:
* The err...

... to ensure the fixer continues to work correctly when the class is also `readonly`.

What with PHP 8.2 introducing `readonly` classes, the metric name `Class declaration type` would...

Universal/OneStatementInShortEchoTag: improve error code

As the name of the sniff is `OneStatementInShortEchoTag`, "Found" would imply that one statement...

Universal/ConstructorDestructorReturn: improve return type fixer

... to prevent it from removing comments.

Includes adjusting a pre-existing test to cover the c...

Rename namespace prefix from `Dealerdirect` to `PHPCSStandards`

QA: always declare metric names as class constants

.. to reduce the risk of metrics not recording correctly due to typos in one of the instances re...

NormalizedArrays/CommaAfterLast: improve fixer for flexible heredoc/nowdoc

ObjectDeclarations::getClassProperties(): add token pointer indexes to return array

... as quite some sniffs will need those anyway and it prevents having to duplicate the token wa...

Collections: add new `constantModifierKeywords()` method

When comma's are enforced after the last entry of a (multi-line) array, the fixer would previous...

:sparkles: New `Universal.Constants.ModifierKeywordOrder` sniff