-
-
Notifications
You must be signed in to change notification settings - Fork 41
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
Issue #44 returned #76
Comments
Just discovered this issue. After reformat code, my app was broken. |
Same here |
Same here with |
I think this issue is related |
fine a self resolving issue for me :) |
Same here. It happens with any kind of annotation. In this case, the <?php
namespace App;
use Acelaya\ZsmAnnotatedServices\Annotation\Inject; // <- Unused
class Foo
{
/**
* @Inject({Cache::class})
*/
public function __construct(Cache $cache)
{
$this->cache = $cache;
}
} But not in this case: <?php
namespace App;
use Acelaya\ZsmAnnotatedServices\Annotation as DI;
class Foo
{
/**
* @DI\Inject({Cache::class})
*/
public function __construct(Cache $cache)
{
$this->cache = $cache;
}
} |
Can confirm this is also happening to me, on 2016.3.1 and latest plugin. |
@Haehnchen you probably mean that it isn't due to the plugin, but to the PHPStorm API right? #76 (comment) |
Same here, tried with any non-aliased annotation, but everything is detected as unused. This goes for SensioLabs Route, Template, Security; Symfony Route; Custom Annotations. I think it's really broken :( |
I can confirm that this issue is solved by this build: |
I'm locally (EAP) running: |
Thank you @iltar 😄 |
For those (like me 10 minutes ago) who don't know where to find EAP builds of PhpStorm. |
I commented on #44 by mistake. I just upgraded IntelliJ to Bugged:
Works:
|
fyi: |
Opening this because I don't know how to reopen #44, all the relevant info is there.
The text was updated successfully, but these errors were encountered: