From 90dfe0dff5c6be52add2e95abb3d3c06b86e7701 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?K=C3=A9vin=20Letord?= Date: Wed, 4 Mar 2026 09:41:31 +0100 Subject: [PATCH] Fix deprecation warning for usort --- .../Services/Proxy/UseCases/UseCaseProxy.php | 27 ++++++++++++++----- 1 file changed, 21 insertions(+), 6 deletions(-) diff --git a/src/OpenClassrooms/UseCase/Application/Services/Proxy/UseCases/UseCaseProxy.php b/src/OpenClassrooms/UseCase/Application/Services/Proxy/UseCases/UseCaseProxy.php index 061a674..4bf38f0 100644 --- a/src/OpenClassrooms/UseCase/Application/Services/Proxy/UseCases/UseCaseProxy.php +++ b/src/OpenClassrooms/UseCase/Application/Services/Proxy/UseCases/UseCaseProxy.php @@ -179,8 +179,13 @@ private function sortPreStrategies() usort( $this->strategies, function (ProxyStrategyBag $s1, ProxyStrategyBag $s2) { - return array_search($s1->getType(), UseCaseProxy::$strategyPreOrder) > - array_search($s2->getType(), UseCaseProxy::$strategyPreOrder); + $pos1 = array_search($s1->getType(), UseCaseProxy::$strategyPreOrder); + $pos2 = array_search($s2->getType(), UseCaseProxy::$strategyPreOrder); + + $pos1 = (false === $pos1) ? PHP_INT_MAX : $pos1; + $pos2 = (false === $pos2) ? PHP_INT_MAX : $pos2; + + return $pos1 <=> $pos2; } ); } @@ -215,8 +220,13 @@ private function sortPostStrategies() usort( $this->strategies, function (ProxyStrategyBag $s1, ProxyStrategyBag $s2) { - return array_search($s1->getType(), UseCaseProxy::$strategyPostOrder) > - array_search($s2->getType(), UseCaseProxy::$strategyPostOrder); + $pos1 = array_search($s1->getType(), UseCaseProxy::$strategyPostOrder); + $pos2 = array_search($s2->getType(), UseCaseProxy::$strategyPostOrder); + + $pos1 = (false === $pos1) ? PHP_INT_MAX : $pos1; + $pos2 = (false === $pos2) ? PHP_INT_MAX : $pos2; + + return $pos1 <=> $pos2; } ); } @@ -242,8 +252,13 @@ private function sortOnExceptionStrategies() usort( $this->strategies, function (ProxyStrategyBag $s1, ProxyStrategyBag $s2) { - return array_search($s1->getType(), UseCaseProxy::$strategyOnExceptionOrder) > - array_search($s2->getType(), UseCaseProxy::$strategyOnExceptionOrder); + $pos1 = array_search($s1->getType(), UseCaseProxy::$strategyOnExceptionOrder); + $pos2 = array_search($s2->getType(), UseCaseProxy::$strategyOnExceptionOrder); + + $pos1 = (false === $pos1) ? PHP_INT_MAX : $pos1; + $pos2 = (false === $pos2) ? PHP_INT_MAX : $pos2; + + return $pos1 <=> $pos2; } ); }