search

Sorien / silex-idea-plugin

Idea plugin for Silex Framework - plugin is not compatible with PHPStorm 2016.2
MIT License
26 stars 8 forks source link

Autocomplete app global in Twig #27

Closed CarsonF closed 8 years ago

CarsonF commented 9 years ago

I think this one small thing would go a long ways. Should it be pretty easy to implement, right?

CarsonF commented 9 years ago

Was hoping I could persuade you to sneak this in before the rest of the twig stuffs. Given that you plan to do twig work for routes.

Sorien commented 9 years ago

there is same problem like routes ... how to detect global vars (indexer/dump/custom annotations) for non global vars its even worth and there are some planed changes in JetBrains twig implementation

CarsonF commented 9 years ago

Is it not safe to assume the app variable, specifically, is Silex?

It seemed like an edge case in my mind, which didn't have the detection problems.

Sorien commented 9 years ago

i would like to reuse s2 plugin and its twig implementation i'll talk to Haehnchen if we can make some extension points to reuse his code ... when it comes to autocompleting app it should be possible but i'd like to wait for https://youtrack.jetbrains.com/issue/WI-27708 don't want to make same thing two times

Edit: this issue https://youtrack.jetbrains.com/issue/WI-7093

Sorien commented 8 years ago

plugin will support only pimple DI, not enough time to finish whole silex support