fix: misnamed, so unused, variable in UriVariablesResolverTrait

[paullallier]

I suspect this has no impact since everywhere using $context['uri_variables_map'] falls back on $operation->getUriVariables() anyway.

Q	A
Branch?	3.1 (or 3.0 if still getting bugfixes)
Tickets	New issue
License	MIT
Doc PR	n/a

[dunglas]

Thanks! Your patch should improve performance.

[paullallier]

I suspect we could remove the call to $operation->getUriVariables() in these two functions too, which is probably more of a performance improvement. I wasn't sure enough to do it though. What do you think? I'm not certainly enough that $context['uri_variables_map'] will always exist when these functions are called.

core/src/Api/UriVariablesConverter.php

Line 47 in 6b79b6f

$uriVariablesDefinitions = $operation->getUriVariables() ?? [];

core/src/Metadata/UriVariablesConverter.php

Line 46 in 6b79b6f

$uriVariablesDefinitions = $operation->getUriVariables() ?? [];

[soyuka]

In the UriVariblesConverter we want to get $operation->getUriVariables() we can't be sure the $context exists.