ibexa · webhdx · Feb 3, 2023 · Jan 17, 2023 · Jan 17, 2023 · Jan 17, 2023
diff --git a/src/bundle/DependencyInjection/Configuration/Parser/AdminUiForms.php b/src/bundle/DependencyInjection/Configuration/Parser/AdminUiForms.php
@@ -29,6 +29,7 @@ class AdminUiForms extends AbstractParser
 {
     public const FORM_TEMPLATES_PARAM = 'admin_ui_forms.content_edit_form_templates';
     public const FIELD_TYPES_PARAM = 'admin_ui_forms.content_edit.fieldtypes';
+    public const META_FIELD_GROUPS_LIST_PARAM = 'admin_ui_forms.content_edit.meta_field_groups_list';
     public const CONTENT_TYPE_FIELD_TYPES_PARAM = 'admin_ui_forms.content_type_edit.field_types';
     public const CONTENT_TYPE_DEFAULT_META_FIELD_TYPE_GROUP_PARAM =
         'admin_ui_forms.content_type_edit.default_meta_field_type_group';
@@ -87,6 +88,12 @@ public function addSemanticConfig(NodeBuilder $nodeBuilder)
                                     ->end()
                                 ->end()
                             ->end()
+                            ->arrayNode('meta_field_groups_list')
+                                ->performNoDeepMerging()
+                                ->scalarPrototype()->end()
+                                ->info('List of field groups that would be placed under a meta section in a content form')
+                                ->defaultValue(['metadata'])
+                            ->end()
                         ->end()
                     ->end()
                     ->arrayNode('content_type_edit')
@@ -156,6 +163,12 @@ public function mapConfig(
             unset($scopeSettings['admin_ui_forms']['content_edit']['fieldtypes']);
         }
 
+        if (!empty($scopeSettings['admin_ui_forms']['content_edit']['meta_field_groups_list'])) {
+            $scopeSettings['admin_ui_forms.content_edit.meta_field_groups_list'] =
+                $scopeSettings['admin_ui_forms']['content_edit']['meta_field_groups_list'];
+            unset($scopeSettings['admin_ui_forms']['content_edit']['meta_field_groups_list']);
+        }
+
         if (!empty($scopeSettings['admin_ui_forms']['content_type_edit']['field_types'])) {
             $scopeSettings['admin_ui_forms.content_type_edit.field_types'] =
                 $scopeSettings['admin_ui_forms']['content_type_edit']['field_types'];
@@ -178,6 +191,11 @@ public function mapConfig(
             $currentScope,
             $scopeSettings['admin_ui_forms.content_edit.fieldtypes'] ?? []
         );
+        $contextualizer->setContextualParameter(
+            self::META_FIELD_GROUPS_LIST_PARAM,
+            $currentScope,
+            $scopeSettings['admin_ui_forms.content_edit.meta_field_groups_list'] ?? []
+        );
         $contextualizer->setContextualParameter(
             self::CONTENT_TYPE_FIELD_TYPES_PARAM,
             $currentScope,
@@ -197,6 +215,7 @@ public function postMap(array $config, ContextualizerInterface $contextualizer)
     {
         $contextualizer->mapConfigArray('admin_ui_forms.content_edit_form_templates', $config);
         $contextualizer->mapConfigArray('admin_ui_forms.content_edit.fieldtypes', $config);
+        $contextualizer->mapConfigArray('admin_ui_forms.content_edit.meta_field_groups_list', $config);
         $contextualizer->mapConfigArray('admin_ui_forms.content_type_edit.field_types', $config);
         $contextualizer->mapSetting('admin_ui_forms.content_type_edit.default_meta_field_type_group', $config);
     }

diff --git a/src/bundle/Resources/config/admin_ui_forms.yaml b/src/bundle/Resources/config/admin_ui_forms.yaml
@@ -2,6 +2,8 @@ system:
     admin_group:
         admin_ui_forms:
             content_edit:
+                meta_field_groups_list:
+                    - metadata
                 form_templates:
                     - { template: '@ibexadesign/content/form_fields.html.twig', priority: 0 }
 

diff --git a/src/bundle/Resources/views/themes/admin/content/components/meta_fields.html.twig b/src/bundle/Resources/views/themes/admin/content/components/meta_fields.html.twig
@@ -5,7 +5,7 @@
 {% block sections %}
     {% for identifier in meta_fields %}
         {% embed '@ibexadesign/ui/component/anchor_navigation/section.html.twig' with {'form': form, 'identifier': identifier} %}
-            {% set data_id = 'ibexa-edit-content-sections-meta-' ~ identifier %}
+            {% set data_id = '#ibexa-edit-content-sections-meta-' ~ identifier %}
             {% block content %}
                 {% from '@ibexadesign/content/edit_macros.html.twig' import render_form_field %}
 

diff --git a/src/lib/Component/Content/ContentEditMetaFieldsComponent.php b/src/lib/Component/Content/ContentEditMetaFieldsComponent.php
@@ -9,7 +9,10 @@
 namespace Ibexa\AdminUi\Component\Content;
 
 use Ibexa\Contracts\AdminUi\Component\Renderable;
+use Ibexa\Contracts\Core\Repository\Values\ContentType\ContentType;
+use Ibexa\Contracts\Core\Repository\Values\ContentType\FieldDefinitionCollection;
 use Ibexa\Contracts\Core\SiteAccess\ConfigResolverInterface;
+use Ibexa\Core\Repository\Values\ContentType\FieldDefinition;
 use Twig\Environment;
 
 class ContentEditMetaFieldsComponent implements Renderable
@@ -39,17 +42,21 @@ public function render(array $parameters = []): string
         $contentType = $parameters['content_type'];
 
         $metaFieldTypeIdentifiers = $this->getMetaFieldTypeIdentifiers();
+        $metaFieldDefinitionCollection = $this->getMetaFieldDefinitionCollection($contentType);
 
-        if (empty($metaFieldTypeIdentifiers)) {
+        if (empty($metaFieldTypeIdentifiers) && $metaFieldDefinitionCollection->isEmpty()) {
             return self::NO_CONTENT;
         }
 
-        $parameters['meta_fields'] = [];
+        $metaFields = $this->mapMetaFieldDefinitionCollectionToIdentifiers($metaFieldDefinitionCollection);
+
         foreach ($metaFieldTypeIdentifiers as $identifier) {
             $fields = $contentType->getFieldDefinitionsOfType($identifier);
-            $parameters['meta_fields'] = array_merge($parameters['meta_fields'], array_column($fields->toArray(), 'identifier'));
+            $metaFields = array_merge($metaFields, array_column($fields->toArray(), 'identifier'));
         }
 
+        $parameters['meta_fields'] = array_unique($metaFields);
+
         return $this->twig->render(
             '@ibexadesign/content/components/meta_fields.html.twig',
             $parameters
@@ -70,4 +77,21 @@ private function getMetaFieldTypeIdentifiers(): array
             )
         );
     }
+
+    private function getMetaFieldDefinitionCollection(ContentType $contentType): FieldDefinitionCollection
+    {
+        $metaFieldGroups = $this->configResolver->getParameter(
+            'admin_ui_forms.content_edit.meta_field_groups_list'
+        );
+
+        return $contentType->fieldDefinitions->filter(
+            static fn (FieldDefinition $field): bool => true === in_array($field->fieldGroup, $metaFieldGroups),
+        );
+    }
+
+    private function mapMetaFieldDefinitionCollectionToIdentifiers(
+        FieldDefinitionCollection $metaFieldDefinitionCollection
+    ): array {
+        return array_column($metaFieldDefinitionCollection->toArray(), 'identifier');
+    }
 }
diff --git a/src/lib/EventListener/SetViewParametersListener.php b/src/lib/EventListener/SetViewParametersListener.php
@@ -27,6 +27,7 @@
 use Ibexa\Core\MVC\Symfony\View\View;
 use Ibexa\Core\MVC\Symfony\View\ViewEvents;
 use Symfony\Component\EventDispatcher\EventSubscriberInterface;
+use Symfony\Component\Form\FormInterface;
 
 /**
  * @todo It should use ViewEvents::FILTER_VIEW_PARAMETERS event instead.
@@ -276,7 +277,18 @@ private function getIgnoredContentFields(array $fieldsData): array
             $ignoredFieldIdentifiers[] = $fieldIdentifier;
         }
 
-        return $ignoredFieldIdentifiers;
+        $metaFieldGroups = $this->configResolver->getParameter(
+            'admin_ui_forms.content_edit.meta_field_groups_list'
+        );
+        $metaFieldIdentifiers = array_keys(
+            array_filter(
+                $fieldsData,
+                static fn (FormInterface $field): bool => true
+                    === in_array($field->getData()->fieldDefinition->fieldGroup, $metaFieldGroups)
+            )
+        );
+
+        return array_merge($ignoredFieldIdentifiers, $metaFieldIdentifiers);
     }
 }
 

diff --git a/src/lib/Form/Provider/GroupedNonMetaFormFieldsProvider.php b/src/lib/Form/Provider/GroupedNonMetaFormFieldsProvider.php
@@ -10,6 +10,7 @@
 
 use Ibexa\Contracts\ContentForms\Content\Form\Provider\GroupedContentFormFieldsProviderInterface;
 use Ibexa\Contracts\Core\SiteAccess\ConfigResolverInterface;
+use Symfony\Component\Form\FormInterface;
 
 final class GroupedNonMetaFormFieldsProvider implements GroupedContentFormFieldsProviderInterface
 {
@@ -28,16 +29,31 @@ public function __construct(
     public function getGroupedFields(array $fieldsDataForm): array
     {
         $identifiers = $this->getMetaFields();
+        $metaFieldGroups = $this->getMetaFieldGroups();
+
+        $metaFieldIdentifiers = array_keys(
+            array_filter(
+                $fieldsDataForm,
+                static fn (FormInterface $field): bool => true
+                    === in_array($field->getData()->fieldDefinition->fieldGroup, $metaFieldGroups)
+            )
+        );
 
         $groupedFields = $this->innerGroupedContentFormFieldsProvider->getGroupedFields($fieldsDataForm);
         foreach ($groupedFields as $group => $fields) {
             $groupedFields[$group] = array_filter(
                 $fields,
-                static function (string $fieldIdentifier) use ($fieldsDataForm, $identifiers): bool {
+                static function (string $fieldIdentifier) use (
+                    $fieldsDataForm,
+                    $identifiers,
+                    $metaFieldIdentifiers
+                ): bool {
                     $fieldData = $fieldsDataForm[$fieldIdentifier]->getNormData();
                     $fieldTypeIdentifier = $fieldData->fieldDefinition->fieldTypeIdentifier;
+                    $fieldIdentifier = $fieldData->fieldDefinition->identifier;
 
-                    return !in_array($fieldTypeIdentifier, $identifiers, true);
+                    return !in_array($fieldTypeIdentifier, $identifiers, true)
+                        && !in_array($fieldIdentifier, $metaFieldIdentifiers);
                 }
             );
         }
@@ -59,4 +75,12 @@ private function getMetaFields(): array
             )
         );
     }
+
+    /**
+     * @return array<string>
+     */
+    private function getMetaFieldGroups(): array
+    {
+        return $this->configResolver->getParameter('admin_ui_forms.content_edit.meta_field_groups_list');
+    }
 }
diff --git a/src/lib/Menu/ContentEditAnchorMenuBuilder.php b/src/lib/Menu/ContentEditAnchorMenuBuilder.php
@@ -126,8 +126,24 @@ private function getMetaFieldItems(ContentType $contentType): array
             static fn (array $config): bool => true === $config['meta']
         ));
 
+        $metaFieldGroups = $this->configResolver->getParameter(
+            'admin_ui_forms.content_edit.meta_field_groups_list'
+        );
+        $metaFieldDefinitionCollection = $contentType->fieldDefinitions->filter(
+            static fn (FieldDefinition $field): bool => true === in_array($field->fieldGroup, $metaFieldGroups),
+        );
+
         $items = [];
         $order = 0;
+        foreach ($metaFieldDefinitionCollection as $fieldDefinition) {
+            $order += self::ITEM_ORDER_SPAN;
+            $items[$fieldDefinition->identifier] = $this->createSecondLevelItem(
+                $fieldDefinition->identifier,
+                $fieldDefinition,
+                $order
+            );
+        }
+
         foreach ($metaFieldTypeIdentifiers as $metaFieldTypeIdentifier) {
             if (false === $contentType->hasFieldDefinitionOfType($metaFieldTypeIdentifier)) {
                 continue;
@@ -137,7 +153,9 @@ private function getMetaFieldItems(ContentType $contentType): array
             foreach ($fieldDefinitions as $fieldDefinition) {
                 $fieldDefIdentifier = $fieldDefinition->identifier;
                 $order += self::ITEM_ORDER_SPAN;
-                $items[$fieldDefIdentifier] = $this->createSecondLevelItem($fieldDefIdentifier, $fieldDefinition, $order);
+                if (!isset($items[$fieldDefIdentifier])) {
+                    $items[$fieldDefIdentifier] = $this->createSecondLevelItem($fieldDefIdentifier, $fieldDefinition, $order);
+                }
             }
         }
 

diff --git a/tests/bundle/DependencyInjection/Configuration/Parser/AdminUiFormsTest.php b/tests/bundle/DependencyInjection/Configuration/Parser/AdminUiFormsTest.php
@@ -116,6 +116,47 @@ public function testContentEditFieldTypesAreMapped(): void
 
         $this->parser->mapConfig($scopeSettings, $currentScope, $this->contextualizer);
     }
+
+    /**
+     * Test 'meta_field_groups_list' fieldtype settings are mapped.
+     */
+    public function testContentEditMetaFieldgroupListIsMapped(): void
+    {
+        $scopeSettings = [
+            'admin_ui_forms' => [
+                'content_edit' => [
+                    'meta_field_groups_list' => [
+                        'metadata',
+                        'seo',
+                    ],
+                ],
+            ],
+        ];
+        $currentScope = 'admin_group';
+
+        $this->contextualizer
+            ->expects($this->atLeast(2))
+            ->method('setContextualParameter')
+            ->withConsecutive(
+                [
+                    AdminUiForms::FORM_TEMPLATES_PARAM,
+                    $currentScope,
+                    [],
+                ],
+                [
+                    AdminUiForms::FIELD_TYPES_PARAM,
+                    $currentScope,
+                    [],
+                ],
+                [
+                    AdminUiForms::META_FIELD_GROUPS_LIST_PARAM,
+                    $currentScope,
+                    ['metadata', 'seo'],
+                ],
+            );
+
+        $this->parser->mapConfig($scopeSettings, $currentScope, $this->contextualizer);
+    }
 }
 
 class_alias(AdminUiFormsTest::class, 'EzSystems\EzPlatformAdminUiBundle\Tests\DependencyInjection\Configuration\Parser\AdminUiFormsTest');
diff --git a/tests/lib/EventListener/SetViewParametersListenerTest.php b/tests/lib/EventListener/SetViewParametersListenerTest.php
@@ -75,13 +75,17 @@ public function setUp(): void
         $this->configResolver = $this->createMock(ConfigResolverInterface::class);
         $this->configResolver
             ->method('getParameter')
-            ->with('admin_ui_forms.content_edit.fieldtypes')
-            ->willReturn(
+            ->withConsecutive(
+                ['admin_ui_forms.content_edit.fieldtypes'],
+                ['admin_ui_forms.content_edit.meta_field_groups_list']
+            )
+            ->willReturnOnConsecutiveCalls(
                 [
                     'ibexa_taxonomy_entry_assignment' => [
                         'meta' => true,
                     ],
-                ]
+                ],
+                ['metadata']
             );
 
         $this->groupedContentFormFieldsProvider = $this->createMock(GroupedContentFormFieldsProviderInterface::class);
@@ -227,15 +231,35 @@ public function testSetUserUpdateViewTemplateParameters(): void
     public function testSetContentFieldsParameters(): void
     {
         $fields = [
-            'name' => 'ezstring',
-            'short_name' => 'ezstring',
-            'description' => 'ezrichtext',
-            'tags' => 'ibexa_taxonomy_entry_assignment',
+            'name' => [
+                'type' => 'ezstring',
+                'group' => 'content',
+            ],
+            'short_name' => [
+                'type' => 'ezstring',
+                'group' => 'content',
+            ],
+            'description' => [
+                'type' => 'ezrichtext',
+                'group' => 'content',
+            ],
+            'tags' => [
+                'type' => 'ibexa_taxonomy_entry_assignment',
+                'group' => 'content',
+            ],
+            'metadata_field' => [
+                'type' => 'ezstring',
+                'group' => 'metadata',
+            ],
         ];
 
         $fieldsDataChildren = [];
-        foreach ($fields as $identifier => $type) {
-            $fieldsDataChildren[$identifier] = $this->createFieldMock($identifier, $type);
+        foreach ($fields as $identifier => $data) {
+            $fieldsDataChildren[$identifier] = $this->createFieldMock(
+                $identifier,
+                $data['type'],
+                $data['group']
+            );
         }
 
         $fieldsDataForm = $this->createMock(FormInterface::class);
@@ -255,7 +279,7 @@ public function testSetContentFieldsParameters(): void
         $groupedFields = [
             'Content' => ['name', 'short_name', 'description'],
         ];
-        $ignoredContentFields = ['tags'];
+        $ignoredContentFields = ['tags', 'metadata_field'];
 
         $this->groupedContentFormFieldsProvider
             ->method('getGroupedFields')
@@ -347,7 +371,7 @@ private function generateUser(int $ownerId): APIUser
     /**
      * @return \Ibexa\Contracts\Core\Repository\Values\Content\Field|\PHPUnit\Framework\MockObject\MockObject
      */
-    private function createFieldMock(string $identifier, string $type): MockObject
+    private function createFieldMock(string $identifier, string $type, string $fieldGroup = 'content'): MockObject
     {
         $data = new FieldData([
             'field' => new Field([
@@ -356,6 +380,7 @@ private function createFieldMock(string $identifier, string $type): MockObject
             ]),
             'fieldDefinition' => new FieldDefinition([
                 'fieldTypeIdentifier' => $type,
+                'fieldGroup' => $fieldGroup,
             ]),
         ]);