False positive for php:S125 - commented out code

The current version of the VSCode Sonarlint plugin (v1.17.0) produces a false positive error. It detects the second line in the following comment wrongly as commented out code:

			// Regarding $mode in recursive calls:
			// * self is a must to have the fallback item itself included;
			// * respect the original caller about whether to include variants or not;
			// * others should be excluded as they'll be handled here in loops.

Hello @micgro42 ,
thank you very much for your feedback.

I have verified your example. We will now look at how we can avoid such false positives.
I will come back to you when we have decided on further steps.

Best,
Nils

1 Like

Hi @micgro42,

we decide to revise the implementation of the rule to reduce the false positive noise to a minimum.
I have created the ticket SONARPHP-1028, so you can be up to date. Thanks again for your report. We will announce it also here when the problem is resolved.

Best,
Nils

1 Like

Thank you for looking into this, I really appreciate it! :slight_smile: