[PhpDocInfo] Fixes Partial value update got duplicated on DoctrineAnnotationTagValueNode change

[samsonasik]

Fixes rectorphp/rector#7020

[samsonasik]

It seems happen because before @Route, there is SpacelessPhpDocTagNode

     * @Since("6.4.0.1")

which a annotation class use Shopware\Core\Framework\Routing\Annotation\Since;, so the duplication happen when there is multiple annotation class, changed after 1 annotation class, so it possibly cause parent -> child duplication.

[TomasVotruba]

Just checking, is this still valid? What is a blocker here to move it forward?

[samsonasik]

@TomasVotruba it is, it somehow duplicated when 2 SpacelessPhpDocTagNode exists, when next is changed

[samsonasik]

Finally 🎉 All checks have passed 🎉 @TomasVotruba it is ready for review.

[samsonasik]

it seems duplicating again when changing below DoctrineAnnotationTagValueNode:

     * @required
     * @Route("/api/_admin/reset-excluded-search-term", name="api.admin.reset-excluded-search-term", methods={"POST"})
     *

it got :

      * @Route("/api/_admin/reset-excluded-search-term", name="api.admin.reset-excluded-search-term", methods={"POST"}, defaults={})
+     * @Route("/api/_admin/reset-excluded-search-term", name="api.admin.reset-excluded-search-term", methods={"POST"})

[samsonasik]

mark as draft again, it seems need to check against non SpacelessPhpDocTagNode too.

[samsonasik]

Finally Fixed 🎉🎉🎉 All checks have passed 🎉 @TomasVotruba it is ready for review.

[samsonasik]

I am merging it ;)

[TomasVotruba]

👍

[samsonasik]

New downgrade error seems unrelated, symfony cache trait and test http client can be ignored from parallellint check.