PHPCSStandards / PHP_CodeSniffer / 12736878529

79%

push

github

Squiz/FunctionSpacing: fixer is broken with doc comment on closing brace line (#784)

The fixer in Squiz.WhiteSpace.FunctionSpacing gets confused when there is a doc comment on the closing brace line.

Generally speaking this sniff doesn't care about other content on the same line as the function declaration.

But in the case of a "trailing" docblock, this meant that the sniff was adding the new lines _within_ the doc block, which, in turn, meant it would still not see the "blank lines after" in the next loop, as the added lines would be seen as doc comment lines, not blank lines. Hence, the fixer conflict.

Fixed now.

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

24479 of 31253 relevant lines covered (78.33%)

66.01 hits per line