Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Minor PS API changes for 10.0.x will break plugin #611

Closed
neuro159 opened this issue Nov 3, 2015 · 6 comments
Closed

Minor PS API changes for 10.0.x will break plugin #611

neuro159 opened this issue Nov 3, 2015 · 6 comments

Comments

@neuro159
Copy link

neuro159 commented Nov 3, 2015

com.jetbrains.php.lang.psi.resolve.types.PhpType#add(java.util.Collection<java.lang.String>) is removed.

please use series of com.jetbrains.php.lang.psi.resolve.types.PhpType#add(java.lang.String) calls.

@Haehnchen
Copy link
Owner

@neuro159 I think com.jetbrains.php.lang.psi.resolve.types.PhpType#add(com.intellij.util.io.StringRef) will also be removed?

Replacement especially for PhpTypeProvider2 would be nice. It's definitely a hack by now to simulate array return types. Is there a way do not break is feature?

fr.adrienbrault.idea.symfony2plugin.doctrine.ObjectRepositoryResultTypeProvider#getBySignature
if(name.equals("findAll") || name.equals("findBy")) {
  method.getType().add(phpClass.getFQN() + "[]");
  return phpNamedElementCollections;
}

nnnnnnnnnn

@emil-nasso
Copy link

Sorry for hijacking this issue. Does anyone know the status of this plugin in phpstorm 10? Will updating completely break it?

@Haehnchen
Copy link
Owner

PhpStorm 10 itself will work. some tests fail https://travis-ci.org/Haehnchen/idea-php-symfony2-plugin but not critical. but with next release 10.0.x we will get into trouble.

@apfelbox
Copy link

@Haehnchen could you please release a hotfix fixing this?

In the new EAP, every (!) single click in a PHP file triggers

16:30:15 NoSuchMethodError: com.jetbrains.php.lang.psi.resolve.types.PhpType.add(Ljava/util/Collection;)Lcom/jetbrains/php/lang/psi/resolve/types/PhpType;

@ByScripts
Copy link

@apfelbox I confirm.

@Haehnchen
Copy link
Owner

... possible tomorrow

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants