feat: refactoring

Author: erdnaxelaweb

components/ImportExportBundle/src/bundle/Resources/config/item_value_transformer/transformers.yaml

@@ -64,3 +64,7 @@ services:
     AlmaviaCX\Bundle\IbexaImportExport\Item\ValueTransformer\Ibexa\HtmlToTextBlockTransformer:
         tags:
             - { name: almaviacx.import_export.item.value_transformer, alias: almaviacx.import_export.item.value_transformer.html_to_textblock}
+
+    AlmaviaCX\Bundle\IbexaImportExport\Item\ValueTransformer\Utils\DefaultTransformer:
+        tags:
+            - { name: almaviacx.import_export.item.value_transformer, alias: almaviacx.import_export.item.value_transformer.default}

components/ImportExportBundle/src/bundle/Resources/views/themes/admin/import_export/job/list.html.twig

@@ -80,15 +80,17 @@
             }]) %}
 
             {% set progress_bar %}
-                {% include '@ibexadesign/import_export/components/progress_bar.html.twig' with {processedCount: job.lastExecution.workflowState.offset, totalCount: job.lastExecution.workflowState.totalItemsCount} only %}
+                {% if job.lastExecution %}
+                    {% include '@ibexadesign/import_export/components/progress_bar.html.twig' with {processedCount: job.lastExecution.workflowState.offset, totalCount: job.lastExecution.workflowState.totalItemsCount} only %}
+                {% endif %}
             {% endset %}
             {% set body_row_cols = body_row_cols|merge([{
                 content: progress_bar,
                 raw: true,
             }]) %}
 
             {% set body_row_cols = body_row_cols|merge([
-                { content: status[job.lastExecution.status]  },
+                { content: job.lastExecution ? status[job.lastExecution.status] : null  },
                 { content: job.requestedDate|ibexa_full_datetime  },
             ]) %}
 

components/ImportExportBundle/src/lib/Item/ValueTransformer/Utils/DefaultTransformer.php

@@ -0,0 +1,23 @@
+<?php
+
+declare(strict_types=1);
+
+namespace AlmaviaCX\Bundle\IbexaImportExport\Item\ValueTransformer\Utils;
+
+use AlmaviaCX\Bundle\IbexaImportExport\Item\ValueTransformer\AbstractItemValueTransformer;
+use Symfony\Component\OptionsResolver\OptionsResolver;
+
+class DefaultTransformer extends AbstractItemValueTransformer
+{
+    protected function transform(mixed $value, array $options = [])
+    {
+        return $value ?? $options['default_value'];
+    }
+
+    protected function configureOptions(OptionsResolver $optionsResolver): void
+    {
+        parent::configureOptions($optionsResolver);
+        $optionsResolver->define('default_value')
+                        ->default(null);
+    }
+}

components/ImportExportBundle/src/lib/Writer/Ibexa/Content/IbexaContentUpdater.php

@@ -117,6 +117,14 @@ protected function handleLocation(
 
         foreach ($existingLocations as $existingLocation) {
             if ($existingLocation->parentLocationId === $parentLocationId) {
+                if ($existingLocation->hidden !== $hidden) {
+                    if ($hidden) {
+                        $this->repository->getLocationService()->hideLocation($existingLocation);
+                    } else {
+                        $this->repository->getLocationService()->unhideLocation($existingLocation);
+                    }
+                }
+
                 return $existingLocation;
             }
         }