[WIP]Feature/doctrine support

[Haehnchen]

now we have also some doctrine support. :)

[Lumbendil]

It'd be great if you parsed the Entity annotation to know if the class has a custom repository, and in such case, changed the class return of getRepository to that class.

Other than that, great job! 👍

[Haehnchen]

yeah there is much todo. iam currently working on this :)

[Lumbendil]

Great! Then it'd be nice that you edited your issue and you marked it as [WIP] so it's possible to see that the PR isn't at a finished state.

[Haehnchen]

looks ready to me at least of:

• iam no parsing the position of the repository class, only use default one on entityname. too heavy in the moment
• if repository autocomplete is used on the repository class instead of the entity class, the later event for the entity autocomplete is not fired on phpstorm

[adrienbrault]

In the container.xml file, inside the <service id="doctrine.orm.default_entity_manager"> definition, there is somewhere:

<call method="setEntityNamespaces"> <argument type="collection"> <argument key="SupFarmBundle">SupFarmBundle\Entity</argument> <argument key="FOSUserBundle">FOS\UserBundle\Entity</argument> <argument key="SonataUserBundle">Sonata\UserBundle\Entity</argument> </argument> </call> 

Maybe it'd be better to rely on this to resolve AcmeFooBundle:User strings.

[adrienbrault]

I think that you could also support findXXX methods on the repository if you returned a "EntityFQN[]" type. This is probably possible since you can do @return User[] in docblocks.

[Lumbendil]

Also, in this PR or after this is merged it'd be nice to have support for the magic methods findByXXXX and findOneByXXXX

[Haehnchen]

thx for the new Symfony2InterfacesUtil. now Repository classes resolved corrent on interfaces

[Lumbendil]

Does this PR support resolution of target entities?

http://symfony.com/doc/master/cookbook/doctrine/resolve_target_entity.html

[adrienbrault]

isGetRepositoryCall would be a better name imho

[Haehnchen]

do anyone see problems on merging it now?. i have two TODOs mentioned in commits, but i want to look later for a better solution. currently this feature works good on a big symfony2 at worktime, and save me lot of time. furthermore i have many more in pipe and want to close this.

[adrienbrault]

Why not using a xpath query ?

[adrienbrault]

Please at least update your type provider with the depth trick like in https://github.com/adrienbrault/idea-php-symfony2-plugin/blob/master/src/fr/adrienbrault/idea/symfony2plugin/dic/SymfonyContainerTypeProvider.java

[stof]

you should also support \\Doctrine\\Common\\Persistence\\ObjectManager::getRepository for people getting the repository from the EntityManager

[lunetics]

+1 thumbs up for the good work @Haehnchen

[stof]

What if the user has an entity named Repository because he is managing a Repository ?

[adrienbrault]

If you really want to exclude the repositories, maybe using Symfony2InterfacesUtil.isInstanceOf would work

[adrienbrault]

Replacing / with \ should work, right ?

[adrienbrault]

Really awesome, thanks!