PHP_CodeSniffer

PHP_CodeSniffer copied to clipboard

Added some missing documentation ## Description This is just some house-keeping to keep the docs up to date. I had to spend some time reviewing the php code to figure...

richard-hp

A sniff similar to `Squiz.Classes.SelfMemberReference.NotUsed` for the new statement would be useful. `Squiz.Classes.SelfMemberReference` is only looking for `T_DOUBLE_COLON` and that does not include "new ClassName"

umherirrender

Feature Requst: Overwrite File locations

1

## Context At places like https://github.com/squizlabs/PHP_CodeSniffer/blob/7566b4d89de1d08d2a8ad85910507d6633a35d28/src/Reports/Code.php#L169 we report the full file location. `basepath` is a config property that can be used to remove it, but doesn't fully work for this....

MakerTim

`phpcs:enable` can sometimes override `phpcs:ignore`

6

## Describe the bug `phpcs:enable` can sometimes wind up overriding a later `phpcs:ignore` for the rule. This can particularly happen when multiple `phpcs:enable` comments are present in the file. ###...

anomiex

## Describe the bug If a closure/anonymous function is used as a callback in a function call, then indentations doesn't get fixed with `Generic.WhiteSpace.ScopeIndent` sniff property `exact` set to `true`....

yCodeTech

False positive Squiz.PHP.NonExecutableCode.Unreachable when using goto statement

1

**Code sample** ```php

Chi-teck

Generic.Formatting.MultipleStatementAlignment false positive for `match()`

2

## Describe the bug `Generic.Formatting.MultipleStatementAlignment` doesn't work correctly with `match()`. ### Code sample ```php ./packages ``` ### To reproduce Steps to reproduce the behavior: 1. Create a file called `test.php`...

LastDragon-ru

Generic.WhiteSpace.ScopeIndent false positive with multiline `yield from`

1

**Describe the bug** Generic.WhiteSpace.ScopeIndent reports multiline `yield from` statements as being incorrectly indented even if the indentation is correct. I've only tested this with tab indentation. **Code sample** ```php ```...

Daimona

Generic.WhiteSpace.ScopeIndent false positive with nested `match`-es

2

## Describe the bug The ScopeIndent sniff suggests a quite strange indentation when processing multi-nested `match`-statements ### Code sample ```php

worried-networking

Inject the Config in the Runner constructor

1

Inject the configuration object in the constructor instead of creating the instance without having the ability to change it. Currently it's not possible to change the configuration because it's using...

BafS