fix(Middleware): log deprecation when annotation was actually used

Signed-off-by: Arthur Schiwon <[email protected]>
nextcloud · Nov 13, 2024 · b45dc98 · b45dc98
1 parent 0da87f1
commit b45dc98
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/lib/private/AppFramework/Middleware/Security/CORSMiddleware.php b/lib/private/AppFramework/Middleware/Security/CORSMiddleware.php
@@ -101,12 +101,12 @@ public function beforeController($controller, $methodName) {
 	 */
 	protected function hasAnnotationOrAttribute(ReflectionMethod $reflectionMethod, string $annotationName, string $attributeClass): bool {
 		if ($this->reflector->hasAnnotation($annotationName)) {
+			$this->logger->debug($reflectionMethod->getDeclaringClass()->getName() . '::' . $reflectionMethod->getName() . ' uses the @' . $annotationName . ' annotation and should use the #[' . $attributeClass . '] attribute instead');
 			return true;
 		}
 
 
 		if (!empty($reflectionMethod->getAttributes($attributeClass))) {
-			$this->logger->debug($reflectionMethod->getDeclaringClass()->getName() . '::' . $reflectionMethod->getName() . ' uses the @' . $annotationName . ' annotation and should use the #[' . $attributeClass . '] attribute instead');
 			return true;
 		}