Way to allow empty methods to have braces on the same line (useful for PHP 8 property promotion)

[simPod]

I have sniffs enabled to ensure that function braces are on the separate lines si this style is enforced:

public function __construct(A $a)
{
    $this->a = $a;
}

and this is not allowed

public function __construct(A $a) {
    $this->a = $a;
}

However, e.g. for property promotion it would be useful to have a way to allow this as it saves 2 lines:

public function __construct(private A $a) {}

Though I don't see a way to configure it while enforcing the style mentioned above for non empty functions:

Squiz.Functions.MultiLineFunctionDeclaration.BraceOnSameLine: Opening brace should be on a new line
Squiz.WhiteSpace.ScopeClosingBrace.ContentBefore: Closing brace must be on a line by itself

[gsherwood]

There is no option for this in PHPCS as none of the sniffs change their rules specifically for constructor property promotion. A new feature would need to be added for this.

See also #3278

[simPod]

I see, #3278 is pretty same.

[thiagobraga]

For now I'm using this approach, but it would be nice if it could be configured in phpcs.xml.dist specific rule or disabled just for that function, using @phpcs:disable.

/**
 * CheckUserLoginService constructor
 *
 * @phpcs:disable Squiz.WhiteSpace.ScopeClosingBrace
 * @param CompanyRepository $companyRepository
 * @param ColorService $colorService
 */
public function __construct(
    protected CompanyRepository $companyRepository,
    protected ColorService $colorService
) {}

/**
 * @phpcs:enable Squiz.WhiteSpace.ScopeClosingBrace
 * @param Type $var
 */
public function FunctionName(Type $var = null)
{} // <--- Error here. OK 👍

[simPod]

public function __construct(
    protected CompanyRepository $companyRepository,
    protected ColorService $colorService
    // phpcs:ignore Squiz.WhiteSpace.ScopeClosingBrace
) {}

[paveljanda]

Hi, is there any other development regarding this error?