PHPCSStandards / PHP_CodeSniffer / 7171974696

78%

master: 79%

push

github

Report Full: fix two line wrapping bugs (#125)

* Fix two line wrapping bugs in default report formatter

The first is that the ANSI escape codes applied to bold the message when
`-s` is used were not being taken into account when wrapping the lines
for width, causing some lines to be wrapped unnecessarily.

The second is that when lines were wrapped in the middle of a long
message, the `|` characters making up the table were being bolded along
with the message.

* Report Full: iterate on line wrapping bug fix

This commit takes the fix one step further by adding the padding only after the message has been word-wrapped.
Includes correct handling of padding for multi-line error messages.

It then takes the last line of the resulting message and determines in isolation whether the source code suffix can fit on that line or needs to be placed on a new line.


Co-authored-by: Brad Jorsch <anomie@users.sourceforge.net>
Co-authored-by: jrfnl <jrfnl@users.noreply.github.com>

0 of 24 new or added lines in 1 file covered. (0.0%)

15105 of 23591 relevant lines covered (64.03%)

5.39 hits per line