Merge pull request #23 from disstudio/fix/export-serialization-groups

fix: export serialization groups

Author: TheMilek

README.md

@@ -111,12 +111,15 @@ Symfony Flex, it's much quicker!
 
     ```yaml
     # config/packages/sylius_import_export.yaml
+    imports:
+        - { resource: "@SyliusImportExportBundle/config/config.yaml" }
+
     sylius_import_export:
         export:
             resources:
                 sylius.order: ~
                 app.brand:
-                    serialization_group: 'app:brand:export'
+                    serialization_groups: ['app:brand:export']
                     sections:
                         - 'Sylius\Bundle\AdminBundle\SectionResolver\AdminSection'
                     provider: 'sylius_import_export.provider.resource_data.orm'
@@ -128,6 +131,15 @@ Symfony Flex, it's much quicker!
     ```
    For a more detailed overview check the [configuration reference](docs/configuration_reference.md).
 
+4. Configure routes:
+
+    ```yaml
+    # config/routes/sylius_import_export.yaml
+    imports:
+        resource: "@SyliusImportExportBundle/config/routes.yaml"
+    ```
+
+
 ## File Storage
 
 ### Export Files

UPGRADE.md

@@ -1,5 +1,31 @@
 # UPGRADE
 
+## From v0.2.0 to v0.2.1
+
+### Configuration key serialization_group renamed to serialization_groups
+
+The configuration key for export resources has been renamed to reflect that it can handle multiple groups.
+
+Before:
+
+```yaml
+sylius_import_export:
+    export:
+        resources:
+            app.brand:
+                serialization_group: 'app:brand:export'
+```
+
+After:
+
+```yaml
+sylius_import_export:
+    export:
+        resources:
+            app.brand:
+                serialization_groups: ['app:brand:export']
+```
+
 ## From v0.1.x to v0.2.0
 
 ### `BatchedExportDataManager` constructor signature changed

docs/configuration_reference.md

@@ -18,7 +18,7 @@ sylius_import_export:
     export:
         resources:
             app.brand:
-                serialization_group: 'app:brand:export'
+                serialization_groups: ['app:brand:export']
                 provider: 'sylius_import_export.provider.resource_data.dbal'
                 sections:
                     - 'admin'
@@ -42,8 +42,8 @@ sylius_import_export:
             # The alias of a resource as configured within the ResourceBundle
             # It can be retried by using the `console/bin sylius:debug:resource` console command.
             <resource_alias>:
-                # The group used for data serialization, when not specified, defaults to "sylius_import_export:export"
-                serialization_group: <string>
+                # The groups used for data serialization, when not specified, defaults to ["sylius_import_export:export"]
+                serialization_groups: [<string>, ...]
                 # Resource specific overwrite of the default_provider
                 provider: <service_id>
                 # Resource specific overwrite of the default section

src/DependencyInjection/Configuration.php

@@ -13,7 +13,6 @@
 
 namespace Sylius\ImportExport\DependencyInjection;
 
-use Sylius\ImportExport\Serializer\DefaultSerializationGroups;
 use Symfony\Component\Config\Definition\Builder\ArrayNodeDefinition;
 use Symfony\Component\Config\Definition\Builder\TreeBuilder;
 use Symfony\Component\Config\Definition\ConfigurationInterface;
@@ -59,9 +58,9 @@ private function addExportConfiguration(ArrayNodeDefinition $node): void
                                     })
                                 ->end()
                                 ->children()
-                                    ->scalarNode('serialization_group')
-                                        ->cannotBeEmpty()
-                                        ->defaultValue(DefaultSerializationGroups::EXPORT_GROUP)
+                                    ->arrayNode('serialization_groups')
+                                        ->scalarPrototype()->end()
+                                        ->defaultValue(['DefaultSerializationGroups::EXPORT_GROUP'])
                                     ->end()
                                     ->scalarNode('provider')
                                         ->defaultNull()

src/Provider/Parameters/GridExportParametersProvider.php

@@ -21,7 +21,7 @@
 
 final class GridExportParametersProvider implements GridExportParametersProviderInterface
 {
-    /** @param array<string, array{serialization_group: string}> $resourceExportConfiguration */
+    /** @param array<string, array{serialization_groups: string[]}> $resourceExportConfiguration */
     public function __construct(
         private GridProviderInterface $gridProvider,
         private ParametersParserInterface $parametersParser,
@@ -34,7 +34,7 @@ public function getParameters(MetadataInterface $metadata, string $gridName, Req
         $gridConfiguration = $this->gridProvider->get($gridName);
 
         $resourceExportConfiguration = $this->resourceExportConfiguration[$metadata->getAlias()] ?? [];
-        $serializationGroup = $resourceExportConfiguration['serialization_group'] ?? DefaultSerializationGroups::EXPORT_GROUP;
+        $serializationGroups = $resourceExportConfiguration['serialization_groups'] ?? [DefaultSerializationGroups::EXPORT_GROUP];
 
         $parameters = $this->parametersParser->parseRequestValues(
             $gridConfiguration->getDriverConfiguration(),
@@ -43,7 +43,7 @@ public function getParameters(MetadataInterface $metadata, string $gridName, Req
 
         return array_merge($parameters, [
             'grid' => $gridName,
-            'serialization_group' => $serializationGroup,
+            'serialization_groups' => $serializationGroups,
         ]);
     }
 }