remove deprec

Author: soyuka

src/Metadata/Resource/Factory/AttributesResourceMetadataCollectionFactory.php

@@ -185,12 +185,6 @@ private function getOperationWithDefaults(ApiResource $resource, Operation $oper
                 continue;
             }
 
-            // TODO: remove in 3.0
-            if ($operation instanceof HttpOperation && 'getUriVariables' === $methodName && !$operation->getUriTemplate() && $operation instanceof CollectionOperationInterface && !$operation->getUriVariables()) {
-                trigger_deprecation('api-platform', '2.7', sprintf('Identifiers on "%s" are declared on the default #[ApiResource] but you did not specify identifiers on the collection operation. In 3.0 the collection operations can have identifiers, you should specify identifiers on the operation not on the resource to avoid unwanted behavior.', $operation->getShortName()));
-                continue;
-            }
-
             $operation = $operation->{'with'.substr($methodName, 3)}($value);
         }
 

src/Metadata/Resource/Factory/UriTemplateResourceMetadataCollectionFactory.php

@@ -101,6 +101,10 @@ private function generateUriTemplate(HttpOperation $operation): string
         $uriTemplate = sprintf('/%s', $this->pathSegmentNameGenerator->getSegmentName($operation->getShortName()));
         $uriVariables = $operation->getUriVariables() ?? [];
 
+        if (!\is_array($uriVariables)) {
+            dd($operation);
+        }
+
         if ($parameters = array_keys($uriVariables)) {
             if (($operation->getExtraProperties()['is_legacy_resource_metadata'] ?? false) && 1 < \count($uriVariables[$parameters[0]]->getIdentifiers() ?? [])) {
                 $parameters[0] = 'id';
@@ -126,7 +130,11 @@ private function configureUriVariables($operation)
             [] === $operation->getUriVariables() ||
             ($operation instanceof CollectionOperationInterface && null === $operation->getUriTemplate())
         )) {
-            return $operation;
+            if (null === $operation->getUriVariables()) {
+                return $operation;
+            }
+
+            return $this->normalizeUriVariables($operation);
         }
 
         if (!$operation->getUriVariables()) {
@@ -178,6 +186,7 @@ private function configureUriVariables($operation)
     private function normalizeUriVariables($operation)
     {
         $uriVariables = (array) ($operation->getUriVariables() ?? []);
+
         $normalizedUriVariables = [];
         $resourceClass = $operation->getClass();