PHPCSStandards / PHP_CodeSniffer / 14313595379

78%

master: 79%

Build Type

push

github

Committed by jrfnl

Commit Message

Squiz/OperatorBracket: stability tweak

The `$allowed` tokens array is joined after declaration with the `Tokens::$operators` array. This is only reliable if the keys in both arrays are unique (or intended to be overwritten).
However, with the `$allowed` array being numerically indexed and PHP native tokens also being integers, this was not the case and could lead to one of the tokens in the original array (unintentionally) being disregarded/overwritten by one of the tokens in the `Tokens::$operators` array.

Fixed now, by making setting the tokens constants both as the key as well as the value, making the entries in the array suitable for an array join operation.

Run Details

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

4 existing lines in 1 file now uncovered.

19311 of 24903 relevant lines covered (77.54%)

77.57 hits per line