Add null-data guard and regression tests for Block::rewriteIds

- Skip block elements with null data to avoid a potential fatal in
  Localizedfields::rewriteIds (setLocalizedValue() on null)
- Add regression tests covering relation rewriting inside a Block and
  inside a Block nested in Localizedfields (Service::rewriteIds seam,
  as used by "Paste recursive, updating references")

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Author: kingjia90

models/DataObject/ClassDefinition/Data/Block.php

@@ -488,14 +488,19 @@ public function rewriteIds(mixed $container, array $idMapping, array $params = [
                         continue;
                     }
 
+                    $blockElementData = $blockElement->getData();
+                    if ($blockElementData === null) {
+                        continue;
+                    }
+
                     $blockElement->setData(
                         $fieldDefinition->rewriteIds(
                             $container,
                             $idMapping,
                             [
                                 ...$params,
                                 // \Pimcore\Model\DataObject\ClassDefinition\Data::getDataFromObjectParam() reads 'injectedData'
-                                'injectedData' => $blockElement->getData(),
+                                'injectedData' => $blockElementData,
                             ],
                         ),
                     );

tests/Model/DataType/BlockTest.php

@@ -48,6 +48,7 @@ public function tearDown(): void
     protected function setUpTestClasses(): void
     {
         $this->tester->setupPimcoreClass_Block();
+        $this->tester->setupPimcoreClass_RelationTest();
     }
 
     /**
@@ -243,4 +244,81 @@ public function testBlockDataFromReferences(): void
             Cache::getHandler()->setHandleCli(false);
         }
     }
+
+    /**
+     * Verifies that relations inside a Block are rewritten by Service::rewriteIds
+     * (used by "Paste recursive, updating references")
+     *
+     * @throws Exception
+     */
+    public function testRewriteIdsInsideBlock(): void
+    {
+        $oldTarget = $this->createRelationTestObject('rewrite-old-target');
+        $newTarget = $this->createRelationTestObject('rewrite-new-target');
+
+        $object = $this->createBlockObject();
+        $data = [
+            'blockinput' => new BlockElement('blockinput', 'input', 'test-input'),
+            'blockadvancedRelations' => new BlockElement(
+                'blockadvancedRelations',
+                'advancedManyToManyRelation',
+                [new DataObject\Data\ElementMetadata('blockadvancedRelations', [], $oldTarget)]
+            ),
+        ];
+        $object->setTestblock([$data]);
+        $object->save();
+
+        $object = DataObject::getById($object->getId(), ['force' => true]);
+        Service::rewriteIds($object, ['object' => [$oldTarget->getId() => $newTarget->getId()]]);
+
+        $rewritten = $object->getTestblock()[0]['blockadvancedRelations']->getData();
+        $this->assertEquals($newTarget->getId(), $rewritten[0]->getElement()->getId());
+
+        //rewritten reference should survive save & reload
+        $object->save();
+        $object = DataObject::getById($object->getId(), ['force' => true]);
+
+        $reloaded = $object->getTestblock()[0]['blockadvancedRelations']->getData();
+        $this->assertEquals($newTarget->getId(), $reloaded[0]->getElement()->getId());
+    }
+
+    /**
+     * Verifies that relations inside a Block nested in Localizedfields are rewritten
+     * by Service::rewriteIds (used by "Paste recursive, updating references")
+     *
+     * @throws Exception
+     */
+    public function testRewriteIdsInsideLocalizedBlock(): void
+    {
+        $oldTarget = TestHelper::createEmptyObject();
+        $newTarget = TestHelper::createEmptyObject();
+
+        $object = $this->createBlockObject();
+        $data = [
+            'lblockadvancedRelations' => new BlockElement(
+                'lblockadvancedRelations',
+                'advancedManyToManyRelation',
+                [new DataObject\Data\ElementMetadata('lblockadvancedRelations', [], $oldTarget)]
+            ),
+        ];
+        $object->setLtestblock([$data], 'de');
+        $object->save();
+
+        $object = DataObject::getById($object->getId(), ['force' => true]);
+        Service::rewriteIds($object, ['object' => [$oldTarget->getId() => $newTarget->getId()]]);
+
+        $rewritten = $object->getLtestblock('de')[0]['lblockadvancedRelations']->getData();
+        $this->assertEquals($newTarget->getId(), $rewritten[0]->getElement()->getId());
+    }
+
+    protected function createRelationTestObject(string $key): DataObject\RelationTest
+    {
+        $object = new DataObject\RelationTest();
+        $object->setParent(Service::createFolderByPath('__test/relationobjects'));
+        $object->setKey($key);
+        $object->setPublished(true);
+        $object->save();
+
+        return $object;
+    }
 }