fix: register new subscriber in test setup factory

ezsystems · Apr 23, 2024 · 9d104aa · 9d104aa
1 parent eae6c1b
commit 9d104aa
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 16 deletions.
diff --git a/eZ/Publish/Core/settings/tests/integration_legacy.yml b/eZ/Publish/Core/settings/tests/integration_legacy.yml
@@ -33,6 +33,7 @@ services:
             - ['addSubscriber', ['@eZ\Publish\Core\Search\Common\EventSubscriber\SectionEventSubscriber']]
             - ['addSubscriber', ['@eZ\Publish\Core\Search\Common\EventSubscriber\TrashEventSubscriber']]
             - ['addSubscriber', ['@eZ\Publish\Core\Search\Common\EventSubscriber\UserEventSubscriber']]
+            - ['addSubscriber', ['@Ibexa\Core\Persistence\Legacy\Content\Mapper\ResolveVirtualFieldSubscriber']]
 
     Doctrine\Common\EventManager: ~
 

diff --git a/src/lib/Persistence/Legacy/Content/Mapper/ResolveVirtualFieldSubscriber.php b/src/lib/Persistence/Legacy/Content/Mapper/ResolveVirtualFieldSubscriber.php
@@ -103,26 +103,34 @@ public function persistExternalStorageField(ResolveMissingFieldEvent $event): vo
             $this->getDefaultStorageValue()
         );
 
-        $result = $storage->storeFieldData(
-            $content->versionInfo,
-            $field,
-            $event->getContext()
-        );
-
-        if ($result === true) {
-            $storageValue = new StorageFieldValue();
-            $converter = $this->converterRegistry->getConverter($fieldDefinition->fieldType);
-            $converter->toStorageValue(
-                $field->value,
-                $storageValue
-            );
-
-            $this->contentGateway->updateField(
+        if ($field->value->data !== null) {
+            $result = $storage->storeFieldData(
+                $content->versionInfo,
                 $field,
-                $storageValue
+                []
             );
+
+            if ($result === true) {
+                $storageValue = new StorageFieldValue();
+                $converter = $this->converterRegistry->getConverter($fieldDefinition->fieldType);
+                $converter->toStorageValue(
+                    $field->value,
+                    $storageValue
+                );
+
+                $this->contentGateway->updateField(
+                    $field,
+                    $storageValue
+                );
+            }
         }
 
+        $storage->getFieldData(
+            $content->versionInfo,
+            $field,
+            []
+        );
+
         $event->setField($field);
     }