Undefined index: parenthesis_opener in .../Symfony/Sniffs/Errors/ExceptionMessageSniff.php on line 62

[pvlrmnnk]

Code example:

/**
 * {@inheritdoc}
 */
public function process(ContextInterface $context, Delegate $delegate): void
{
    try {
        $delegate->process($context);
    } catch (\Throwable $t) {
        // TODO: Implement later

        throw $t;
    }
}

Error:

FILE: /.../Middleware/ExceptionInterceptorMiddleware.php
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
 1 | ERROR | An error occurred during processing; checking has been aborted. The error message was: Undefined index: parenthesis_opener in
   |       | /.../vendor/escapestudios/symfony2-coding-standard/Symfony/Sniffs/Errors/ExceptionMessageSniff.php on line 62
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------

[notrix]

👍 same issue here

[ft-1408]

Same issue for me

[MatthieuSarter]

Same for me.

There are indeed two issues :

• when there is no open parenthesis between the throw and the end of the file, the above error is raised and check is aborted,
• when there is an open parentheis betewwen the throw and the end of the file, the content of the first parenthesis is checked, even if it is not related to the throw.

I wrote a fix, I will submit a pull request.

[COil]

Any status about this issue?

[djoos]

Sorry, I'm a bit swamped with work at the moment. When things calm down, I'll get into the issues and PRs here...

If someone can give Matthieu's fix a go and feed back here, that will be helpful in getting the PR out quicker!

[COil]

@djoos Just tested @MatthieuSarter the fix and I don't have the error anymore.

[COil]

Hello @djoos , any news concerning this? :thinking:

I have tested and it works.

[djoos]

Thanks for the nudge @COil - merged & tagged as 3.0.1