EZP-31136: Fixed removal of outdated links when publishing

eZ/Publish/Core/FieldType/Tests/Url/UrlStorageTest.php

@@ -36,6 +36,15 @@ public function testStoreFieldDataWithExistingUrl()
             ->method('linkUrl')
             ->with(12, 42, 24);
 
+        $gateway
+            ->expects($this->once())
+            ->method('unlinkUrl')
+            ->with(
+                42,
+                24,
+                [12]
+            );
+
         $storage = $this->getPartlyMockedStorage($gateway);
         $result = $storage->storeFieldData($versionInfo, $field, $this->getContext());
 
@@ -67,6 +76,15 @@ public function testStoreFieldDataWithNewUrl()
             ->method('linkUrl')
             ->with(12, 42, 24);
 
+        $gateway
+            ->expects($this->once())
+            ->method('unlinkUrl')
+            ->with(
+                42,
+                24,
+                [12]
+            );
+
         $storage = $this->getPartlyMockedStorage($gateway);
         $result = $storage->storeFieldData($versionInfo, $field, $this->getContext());
 
@@ -93,6 +111,10 @@ public function testStoreFieldDataWithEmptyUrl()
             ->expects($this->never())
             ->method('linkUrl');
 
+        $gateway
+            ->expects($this->never())
+            ->method('unlinkUrl');
+
         $storage = $this->getPartlyMockedStorage($gateway);
         $result = $storage->storeFieldData($versionInfo, $field, $this->getContext());
 

eZ/Publish/Core/FieldType/Url/UrlStorage.php

@@ -62,6 +62,12 @@ public function storeFieldData(VersionInfo $versionInfo, Field $field, array $co
 
         $this->gateway->linkUrl($urlId, $field->id, $versionInfo->versionNo);
 
+        $this->gateway->unlinkUrl(
+            $field->id,
+            $versionInfo->versionNo,
+            [$urlId]
+        );
+
         $field->value->data['urlId'] = $urlId;
 
         // Signals that the Value has been modified and that an update is to be performed