dereuromark/cakephp-ide-helper

Method with same name already defined in this class

LordSimal opened this issue · 4 comments

LordSimal commented

I am using the 1.15.0 version of the ide-helper plugin and enabled arrayAsGenerics support in my config.

  'IdeHelper' => [
    'arrayAsGenerics' => true
  ]

Now when I re-do the annoations PHPStorm seems to not like the new array annotations.

image

Is this a PHPStorm issue?

dereuromark commented

See https://twitter.com/dereuromark/status/1455496522049728517 and linked jetbrains ticket.

Nothing we can do at this point than upvoting and making sure with more pressure they fix their IDE
Until then you can ignore it, as it seems to still somehow partially work, or use the current config (default) of not using this syntax - which also means you need to make sure the CS ruleset ignores method annotations.

  • 👍1
LordSimal commented

thanks, I knew I have seen that somewhere but didn't look into twitter ^^
already upvoted that issue 👍

dereuromark commented

That said, I think I must have missed some, as the last ones are still old syntax:

* @method \App\Model\Entity\ReleaseGroup[]|\Cake\Datasource\ResultSetInterface saveManyOrFail(iterable $entities, $options = [])
* @method \App\Model\Entity\ReleaseGroup[]|\Cake\Datasource\ResultSetInterface|false deleteMany(iterable $entities, $options = [])
* @method \App\Model\Entity\ReleaseGroup[]|\Cake\Datasource\ResultSetInterface deleteManyOrFail(iterable $entities, $options = [])

If you have time and want to look into that, would be greatly appreciated.

dereuromark commented

Never mind, those are supposed to stay as they are not simple generics, but collection object.