PSR12.Traits.UseDeclaration: check spacing after use keyword for multi-line statements + fix typo

[jrfnl]

Description

PSR12.Traits.UseDeclaration: fix typo in error code

The processUseStatement() method checking single-line trait use statements checks the spacing after the use keyword, but the error did not have the correct error code.

PSR12.Traits.UseDeclaration: check spacing after use keyword for multi-line statements

While the processUseStatement() method checking single-line trait use statements would check the spacing after the use keyword, the processUseGroup() method checking multi-line trait use statements did not execute that same check, while the rule applies to both single- as well as multi-line use statements.

By moving the check for the spacing after the use keyword to the process() method, it will now be executed for both situations.

Tested by adjusting a pre-existing test.

Suggested changelog entry

• PSR12.Traits.UseDeclaration will now report on issues with spacing after the use keyword using the SpaceAfterUse errorcode.
• PSR12.Traits.UseDeclaration will now report on issues with spacing after the use keyword for multi-line trait use statements.

Types of changes

• [x] Bug fix (non-breaking change which fixes an issue)
• [x] New feature (non-breaking change which adds functionality)
• [ ] Breaking change (fix or feature that would cause existing functionality to change)
  • [ ] This change is only breaking for integrators, not for external standards or end-users.
• [ ] Documentation improvement

[jrfnl]

Closing as replaced by https://github.com/PHPCSStandards/PHP_CodeSniffer/pull/75

[jrfnl]

FYI: this fix is included in today's PHP_CodeSniffer 3.8.0 release.

As per #3932, development on PHP_CodeSniffer will continue in the PHPCSStandards/PHP_CodeSniffer repository. If you want to stay informed, you may want to start "watching" that repo (or watching releases from that repo).