PHPCSStandards / PHP_CodeSniffer / 12736878529

79%

Build Type

push

github

Committed by web-flow

Commit Message

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.

Coverage Stats

2 of 0 new or added lines in 0 files covered. (NaN%)

4 existing lines in 1 file now uncovered.

24479 of 31253 relevant lines covered (78.33%)

66.01 hits per line