use named arguments

Author: jrushlow

src/Security/SecurityControllerBuilder.php

@@ -35,7 +35,7 @@ public function addLoginMethod(ClassSourceManipulator $manipulator): void
 
         // @legacy Refactor when annotations are no longer supported
         if ($this->phpCompatUtil->canUseAttributes()) {
-            $loginMethodBuilder->addAttribute($manipulator->buildAttributeNode(Route::class, ['/login', 'name' => 'app_login']));
+            $loginMethodBuilder->addAttribute($manipulator->buildAttributeNode(Route::class, ['path' => '/login', 'name' => 'app_login']));
         } else {
             $loginMethodBuilder->setDocComment(<<< 'EOT'
 /**
@@ -90,7 +90,7 @@ public function addLogoutMethod(ClassSourceManipulator $manipulator): void
 
         // @legacy Refactor when annotations are no longer supported
         if ($this->phpCompatUtil->canUseAttributes()) {
-            $logoutMethodBuilder->addAttribute($manipulator->buildAttributeNode(Route::class, ['/logout', 'name' => 'app_logout']));
+            $logoutMethodBuilder->addAttribute($manipulator->buildAttributeNode(Route::class, ['path' => '/logout', 'name' => 'app_logout']));
         } else {
             $logoutMethodBuilder->setDocComment(<<< 'EOT'
 /**

src/Util/ClassSourceManipulator.php

@@ -933,9 +933,7 @@ public function buildAttributeNode(string $attributeClass, array $options, ?stri
 
         $context = $this;
         $nodeArguments = array_map(static function ($option, $value) use ($context) {
-            $ident = \is_int($option) ? null : new Node\Identifier($option);
-
-            return new Node\Arg($context->buildNodeExprByValue($value), false, false, [], $ident);
+            return new Node\Arg($context->buildNodeExprByValue($value), false, false, [], new Node\Identifier($option));
         }, array_keys($options), array_values($options));
 
         $class = $attributePrefix ? sprintf('%s\\%s', $attributePrefix, Str::getShortClassName($attributeClass)) : Str::getShortClassName($attributeClass);