PHPCSStandards / PHP_CodeSniffer / 12720586641

78%

master: 79%

push

github

Fix fixer conflict: PSR12/Squiz.Functions.FunctionDeclarationArgumentSpacing

This commit resolves the a fixer conflict between
`Squiz.Functions.FunctionDeclarationArgumentSpacing` and
`Squiz.WhiteSpace.SuperfluousWhitespace`.

This is done by changing the logic within the
`FunctionDeclarationArgumentSpacing` sniff to no longer look at (the length of)
only one `T_WHITESPACE` token, but instead consider all `T_WHITESPACE` tokens
between the type and parameter tokens, and validate on their content (not just
length).

As part of this, the error message has been changed slightly in some
circumstances. The error code has not been changed.
* When there are only space characters between the two tokens, the error
  message remains unchanged.
* When there are tabs or newlines included between the two tokens, these are
  rendered as part of the error message.

14 of 14 new or added lines in 1 file covered. (100.0%)

18967 of 24618 relevant lines covered (77.05%)

76.31 hits per line