Phpinspectionsea: Inspection suggest usage of static:: when the method name begins with static

[thexpand]

When a method is defined in a class via annotations like this: @method StaticBlockManager staticBlock() and later used withing the $this context $this->staticBlock()->get("CONTACTS") the inspections suggest that I use static:: instead of $this->.

Here's an attachment:

[kalessil]

Thank you for reporting @thexpand.
What happens if you apply the suggested change?

[thexpand]

@kalessil It becomes static::staticBlock()->getTitle("CONTACTS")

[kalessil]

And then, does application get broken or keep running?

[thexpand]

No, it's broken, because it is not a static method. I'm using ZendFramework and in the template, I'm using a fake class, where I write my @method doc blocks (for auto-suggest in the template). The reason why I'm doing this is because the view helpers in ZendFramework are called via the magic method __call().

You can take a look how it's written here:
https://github.com/zendframework/zend-view/blob/master/src/Renderer/PhpRenderer.php#L35

I do the same, but my method's name is staticBlock()

[kalessil]

Oh, I see now. This is a bug in IDE itself:

• https://bitbucket.org/kalessil/phpinspectionsea/issues/300/false-positive-in-static-methods
• https://youtrack.jetbrains.com/issue/WI-33569

Please vote for it and I'll write a workaround.

[thexpand]

@kalessil I voted, but it seems that they have abandoned the issue. It's more than an year old.

[kalessil]

Ths. Once I have done with the workaround, I'll ask guys to close the issue.

[kalessil]

Fixed.