Improve maintainability because developers will not wonder what the variable is used for. If a private field is declared but not used in the program, it can be considered dead code and should therefore be removed.

Looking at IntelliJ's code, it looks like it failed to create a key store. The 'more' link shows the complete error: SonarLint will try to set in its HTTP library the Intellij's truststore and keystore even if the connection is not secure. I've added a picture of the popup and the complete error.ĭoes anybody know if this is on purpose as described above, or a bug? This runs fine, but it does produce the 'php s1068' warning below. I've written this example: class TestClass It can of course be a bug (false positive) in sonarlint I'm not really sure why this is: It might be that it is considered a bug in the code to access the current class by name rather then by self or even static, but I would expect that to be the error as this implies you can safely remove the property, which is not the case. Sonarlint seems to think that a private static property that is accessed within the class (by classname, not by self) is unused.

Sonarlint phpstorm