diff --git a/eZ/Publish/Core/FieldType/Url/UrlStorage/Gateway/DoctrineStorage.php b/eZ/Publish/Core/FieldType/Url/UrlStorage/Gateway/DoctrineStorage.php index 61d7a6b97f..e627e69eb7 100644 --- a/eZ/Publish/Core/FieldType/Url/UrlStorage/Gateway/DoctrineStorage.php +++ b/eZ/Publish/Core/FieldType/Url/UrlStorage/Gateway/DoctrineStorage.php @@ -132,7 +132,7 @@ public function insertUrl($url) /** * Return a list of URLs used by the given field and version. * - * array An array of URLs, with urls as keys + * string[] An array of URLs */ public function getUrlsFromUrlLink(int $fieldId, int $versionNo): array { @@ -163,12 +163,8 @@ public function getUrlsFromUrlLink(int $fieldId, int $versionNo): array $statement = $selectQuery->execute(); $rows = $statement->fetchFirstColumn(); - $result = []; - foreach ($rows as $url) { - $result[$url] = true; - } - return $result; + return $rows; } /** diff --git a/tests/integration/Core/FieldType/Url/UrlStorage/Gateway/UrlDoctrineStorageGatewayTest.php b/tests/integration/Core/FieldType/Url/UrlStorage/Gateway/UrlDoctrineStorageGatewayTest.php index aa0f72581d..c290fa05a7 100644 --- a/tests/integration/Core/FieldType/Url/UrlStorage/Gateway/UrlDoctrineStorageGatewayTest.php +++ b/tests/integration/Core/FieldType/Url/UrlStorage/Gateway/UrlDoctrineStorageGatewayTest.php @@ -31,9 +31,9 @@ public function testGetUrlsFromUrlLink(): void $gateway->linkUrl($urlIds[1], 12, 2); $gateway->linkUrl($urlIds[2], 14, 1); - self::assertEquals(['https://ibexa.co/example1' => true, 'https://ibexa.co/example2' => true], $gateway->getUrlsFromUrlLink(10, 1), 'Did not get expected urlS for field 10'); - self::assertEquals(['https://ibexa.co/example2' => true], $gateway->getUrlsFromUrlLink(12, 2), 'Did not get expected url for field 12'); - self::assertEquals(['https://ibexa.co/example3' => true], $gateway->getUrlsFromUrlLink(14, 1), 'Did not get expected url for field 14'); + self::assertEquals(['https://ibexa.co/example1', 'https://ibexa.co/example2'], $gateway->getUrlsFromUrlLink(10, 1), 'Did not get expected urls for field 10'); + self::assertEquals(['https://ibexa.co/example2'], $gateway->getUrlsFromUrlLink(12, 2), 'Did not get expected url for field 12'); + self::assertEquals(['https://ibexa.co/example3'], $gateway->getUrlsFromUrlLink(14, 1), 'Did not get expected url for field 14'); self::assertEquals([], $gateway->getUrlsFromUrlLink(15, 1), 'Expected no urls for field 15'); }