[Symfony73] Skip with @translate docblock on ConstraintOptionsToNamedArgumentsRector (#920)

* [Symfony73] Skip with @translate docblock on ConstraintOptionsToNamedArgumentsRector

* fix

* fix

* final touch: also skip comment

Author: samsonasik

rules-tests/Symfony73/Rector/Class_/ConstraintOptionsToNamedArgumentsRector/Fixture/skip_with_translate_comment.php.inc

@@ -0,0 +1,24 @@
+<?php
+
+namespace Rector\Symfony\Tests\Symfony73\Rector\Class_\ConstraintOptionsToNamedArgumentsRector\Fixture;
+
+use Symfony\Component\Form\AbstractType;
+use Symfony\Component\Form\Extension\Core\Type\TextType;
+use Symfony\Component\Form\FormBuilderInterface;
+use Symfony\Component\Validator\Constraints\NotBlank;
+
+class SkipWithTranslateComment extends AbstractType
+{
+    public function buildForm(FormBuilderInterface $builder, array $options): void
+    {
+        $builder
+            ->add('name', TextType::class, [
+                'constraints' => [
+                    new NotBlank(['message' =>
+                        // @Translate
+                        'Name is required.'
+                    ]),
+                ],
+            ]);
+    }
+}

rules-tests/Symfony73/Rector/Class_/ConstraintOptionsToNamedArgumentsRector/Fixture/skip_with_translate_docblock.php.inc

@@ -0,0 +1,21 @@
+<?php
+
+namespace Rector\Symfony\Tests\Symfony73\Rector\Class_\ConstraintOptionsToNamedArgumentsRector\Fixture;
+
+use Symfony\Component\Form\AbstractType;
+use Symfony\Component\Form\Extension\Core\Type\TextType;
+use Symfony\Component\Form\FormBuilderInterface;
+use Symfony\Component\Validator\Constraints\NotBlank;
+
+class SkipWithTranslateDocblock extends AbstractType
+{
+    public function buildForm(FormBuilderInterface $builder, array $options): void
+    {
+        $builder
+            ->add('name', TextType::class, [
+                'constraints' => [
+                    new NotBlank(['message' => /** @Translate */ 'Name is required.']),
+                ],
+            ]);
+    }
+}

rules/Symfony73/Rector/Class_/ConstraintOptionsToNamedArgumentsRector.php

@@ -110,6 +110,7 @@ public function refactor(Node $node): ?Node
 
         $array = $node->args[0]->value;
         $namedArgs = [];
+        $oldTokens = $this->file->getOldTokens();
 
         foreach ($array->items as $item) {
             if (! $item instanceof ArrayItem) {
@@ -130,6 +131,22 @@ public function refactor(Node $node): ?Node
                 continue;
             }
 
+            $lastTokenKey = $item->key->getEndTokenPos();
+            $startTokenValue = $item->value->getStartTokenPos();
+
+            while ($lastTokenKey < $startTokenValue) {
+                ++$lastTokenKey;
+
+                if (! isset($oldTokens[$lastTokenKey])) {
+                    break;
+                }
+
+                $token = $oldTokens[$lastTokenKey];
+                if ($token->is([T_DOC_COMMENT, T_COMMENT])) {
+                    return null;
+                }
+            }
+
             $arg = new Arg($item->value);
             $arg->name = new Identifier($keyValue);