Code style fixes

src/Core/Domain/Form/DataGridType.php

@@ -4,24 +4,17 @@
 
 use ForkCMS\Modules\Backend\Domain\Action\ModuleAction;
 use ForkCMS\Modules\Internationalisation\Domain\Locale\Locale;
-use ForkCMS\Modules\Internationalisation\Domain\Translation\TranslationKey;
 use Pageon\DoctrineDataGridBundle\DataGrid\DataGrid;
-use ReflectionObject;
 use Symfony\Component\Form\AbstractType;
 use Symfony\Component\Form\FormInterface;
 use Symfony\Component\Form\FormView;
 use Symfony\Component\OptionsResolver\OptionsResolver;
-use Symfony\Contracts\Translation\TranslatorInterface;
 
 /**
  * This form type is a workaround for showing a datagrid in a form.
  */
 final class DataGridType extends AbstractType
 {
-    public function __construct(private readonly TranslatorInterface $translator)
-    {
-    }
-
     public function getBlockPrefix(): string
     {
         return 'data_grid';

src/Core/Domain/Util/ArrayUtil.php

@@ -13,13 +13,14 @@ final class ArrayUtil
     public static function flatten(array $array, string $prefix = ''): array
     {
         $result = [];
-        foreach($array as $key=>$value) {
-            if(is_array($value)) {
+        foreach ($array as $key => $value) {
+            if (is_array($value)) {
                 $result += self::flatten($value, $prefix . $key . '.');
             } else {
                 $result[$prefix . $key] = $value;
             }
         }
+
         return $result;
     }
 }

src/Modules/Backend/tests/Backend/Actions/UserEditTest.php

@@ -34,7 +34,6 @@ public function testEditWithoutChanges(): void
     {
         $user = self::loadPage();
         self::assertEmptyFormSubmission('user', 0, 'Save');
-        self::getClient()->followRedirect();
         self::assertCurrentUrlEndsWith('/private/en/backend/user-index');
         self::assertDataGridHasLink($user->getEmail());
         self::assertResponseContains('The settings for "' . $user->getDisplayName() . '" were saved.');

src/Modules/ContentBlocks/Domain/ContentBlock/ContentBlock.php

@@ -49,7 +49,8 @@ class ContentBlock
     #[ORM\Column(type: 'integer')]
     private int $id;
 
-    #[ORM\ManyToOne(targetEntity: Block::class, cascade: ['persist'], fetch: 'EAGER')]
+    #[ORM\ManyToOne(targetEntity: Block::class, cascade: ['persist'], fetch: 'EAGER', nullable: false)]
+    #[ORM\JoinColumn(nullable: false)]
     private Block $widget;
 
     #[ORM\Column(type: 'string', options: ['default' => self::DEFAULT_TEMPLATE])]

src/Modules/ContentBlocks/Domain/ContentBlock/ContentBlockDataTransferObject.php

@@ -15,7 +15,7 @@ abstract class ContentBlockDataTransferObject
 
     public Block $widget;
 
-    public readonly int $revisionId;
+    public int $revisionId;
 
     #[Assert\NotBlank(message: 'err.FieldIsRequired')]
     public string $title;

src/Modules/ContentBlocks/module.xml

@@ -18,8 +18,9 @@
   </authors>
   <description>
     <![CDATA[
-      The content blocks module provides the possibility to re-use content on multiple pages, wherefore the content can be managed in 1 single point, but the changes are immediately visible on all pages it occurs.
-      Content blocks are exceptionally well-suited for recurring content like contact details or call-to-actions.
+The content blocks module provides the possibility to re-use content on multiple pages.
+The content can be managed in one place and changes will affect all pages where the content block is used.
+Some examples for these content blocks are: footer doormats, addresses, social media blocks, call to actions, ...
     ]]>
   </description>
   <events>

src/Modules/ContentBlocks/templates/Frontend/Widgets/Default.html.twig

@@ -1,5 +1,5 @@
 {% if content_block is defined and content_block.text %}
-  <div class="module-content-blocks widget-content-blocks-default content-block-{{ content_block.id }}" id="{{ content_block.id }}">
+  <div class="module-content-blocks widget-content-blocks-default content-block-{{ content_block.id }}">
     {{ content_block.text|parseEditor }}
   </div>
 {% endif %}

src/Modules/Frontend/Domain/Block/BlockRepository.php

@@ -18,7 +18,8 @@
  */
 final class BlockRepository extends SortableRepository implements ServiceEntityRepositoryInterface
 {
-    public function __construct(EntityManagerInterface $entityManager,
+    public function __construct(
+        EntityManagerInterface $entityManager,
         private readonly EventDispatcherInterface $eventDispatcher
     ) {
         parent::__construct($entityManager, $entityManager->getClassMetadata(Block::class));

src/Modules/Internationalisation/Backend/Actions/TranslationAdd.php

@@ -27,7 +27,7 @@ protected function getFormResponse(Request $request): ?Response
             formType: TranslationType::class,
             formData: new CreateTranslation($translation),
             redirectResponse: new RedirectResponse(TranslationIndex::getActionSlug()->generateRoute($this->router)),
-            successFlashMessageCallback: static fn (FormInterface $form) => FlashMessage::success(
+            successFlashMessageCallback: fn (FormInterface $form) => FlashMessage::success(
                 'EntityAdded',
                 ['entity' => $this->translator->trans($form->getData()->getEntity()->getTranslatable())]
             ),

src/Modules/Internationalisation/Backend/Actions/TranslationEdit.php

@@ -31,7 +31,7 @@ protected function getFormResponse(Request $request): ?Response
             formType: TranslationType::class,
             formData: new ChangeTranslation($translation),
             redirectResponse: new RedirectResponse(TranslationIndex::getActionSlug()->generateRoute($this->router)),
-            successFlashMessageCallback: static fn (FormInterface $form) => FlashMessage::success(
+            successFlashMessageCallback: fn (FormInterface $form) => FlashMessage::success(
                 'EntityEdited',
                 ['entity' => $this->translator->trans($form->getData()->getEntity()->getTranslatable())]
             ),