Merge branch '5.2' into 5.3

* 5.2:
  [Cache] clarify RedisStore::evaluate()
  [Messenger] Fixed BC layer for RedeliveryStamp
  [DoctrineBridge] fix setting default mapping type to attribute/annotation on php 8/7 respectively
  do not render the same label id attribute twice
  make AbstractDataCollector extend DataCollector to reduce boilerplate

Author: nicolas-grekas

Store/RedisStore.php

@@ -281,19 +281,15 @@ private function evaluate(string $script, string $resource, array $args)
             }
 
             return $result;
-
-            return $this->redis->_instance($this->redis->_target($resource))->eval($script, array_merge([$resource], $args), 1);
         }
 
-        if ($this->redis instanceof \Predis\ClientInterface) {
-            try {
-                return $this->redis->eval(...array_merge([$script, 1, $resource], $args));
-            } catch (ServerException $e) {
-                throw new LockStorageException($e->getMessage(), $e->getCode(), $e);
-            }
-        }
+        \assert($this->redis instanceof \Predis\ClientInterface);
 
-        throw new InvalidArgumentException(sprintf('"%s()" expects being initialized with a Redis, RedisArray, RedisCluster or Predis\ClientInterface, "%s" given.', __METHOD__, get_debug_type($this->redis)));
+        try {
+            return $this->redis->eval(...array_merge([$script, 1, $resource], $args));
+        } catch (ServerException $e) {
+            throw new LockStorageException($e->getMessage(), $e->getCode(), $e);
+        }
     }
 
     private function getUniqueToken(Key $key): string