How to remove multiline comment?

[oldy777]

Hi, I trying to make new rule and stuck with removing multiline comment.
It looks like:

 /**
     * @DI\InjectParams({
     *     "repository" = @DI\Inject("repository"),
     *     "option" = @DI\Inject("%option%"),
     * })
 */

I use

$inf = $constructMethod->getAttribute(AttributeKey::PHP_DOC_INFO);
$inf->removeByName('DI');

but this command removes only 1 line

    ---------- begin diff ----------
--- Original
+++ New
@@ -21,7 +21,6 @@
     /**
-     * @DI\InjectParams({
      *     "repository" = @DI\Inject("repository"),
      *     "option" = @DI\Inject("%option%"),
      * })
    ----------- end diff -----------

Also I tried to make custom TagValueNode, but didn't find a way to registered new PhpDocNodeFactory.

Need help :)
rector version dev-master(v0.7.0)

[TomasVotruba]

Hi, failing test case PR is needed.

Also keep the use imports

[TomasVotruba]

Any update?

[oldy777]

I will try to do it on weekends.

[TomasVotruba]

Demo should be enough for me: https://getrector.org/demo

[oldy777]

Cool! https://getrector.org/demo/4fb4118d-63cb-4c54-8bc2-082be6ddf328

[TomasVotruba]

The demo link you provided doesn't show any diff nor bug.

[TomasVotruba]

Have you had something like this in mind?

https://github.com/rectorphp/rector/blob/71f5a43b89d658cc9000af5020d342168af5d478/docs/AllRectorsOverview.md#jms

[oldy777]

Yes, that's exactly what I needed. Thank you very much

[oldy777]

@TomasVotruba Hi! I trying to run RemoveJmsInjectParamsAnnotationRector on my codebase and seems it doesn't work.

I created rector.yaml

services:
    Rector\JMS\Rector\ClassMethod\RemoveJmsInjectParamsAnnotationRector: null

Then run
php vendor/bin/rector process utils/Test.php --dry-run
Annotation was not deleted. But if I run your test, it works.
Test.php I created from your test example. On real files it doesn't works too.

And one more problem, I can't run rector unit tests:
PHP Fatal error: Cannot declare class Rector\Architecture\Tests\Rector\Class_\ConstructorInjectionToActionInjectionRector\Fixture\DuplicateController, because the name is already in use in /tmp/rector_temp_tests/original_066106da16a7_duplicate.php on line 34

I removed /tmp/rector_temp_tests but it not helped

[TomasVotruba]

Could you provide demo link?
https://getrector.org/demo

[oldy777]

@TomasVotruba There is no such rector
https://getrector.org/demo/3ea88594-ebc8-498f-9cf4-519247e9ba0c