CED-1124 NGSTACK-831 edit infocollection

bundle/Controller/Admin/AdminController.php

@@ -1,31 +1,45 @@
 <?php
 
+declare(strict_types=1);
+
 namespace Netgen\Bundle\InformationCollectionBundle\Controller\Admin;
 
-use Ibexa\Contracts\Core\Repository\Values\Content\Content;
-use Ibexa\Core\MVC\Symfony\Security\Authorization\Attribute;
+use Doctrine\ORM\NonUniqueResultException;
+use eZ\Publish\API\Repository\Values\ContentType\ContentType;
 use Ibexa\Bundle\Core\Controller;
 use Ibexa\Contracts\Core\Repository\ContentService;
+use Ibexa\Contracts\Core\Repository\Values\Content\Content;
 use Ibexa\Contracts\Core\SiteAccess\ConfigResolverInterface;
+use Ibexa\Core\MVC\Symfony\Security\Authorization\Attribute;
+use Netgen\Bundle\InformationCollectionBundle\Form\Payload\InformationCollectionStruct;
+use Netgen\Bundle\InformationCollectionBundle\Form\Builder\FormBuilder;
 use Netgen\InformationCollection\API\Persistence\Anonymizer\Anonymizer;
 use Netgen\InformationCollection\API\Service\InformationCollection;
 use Netgen\InformationCollection\API\Value\Collection;
 use Netgen\InformationCollection\API\Value\Filter\CollectionFields;
+use Netgen\InformationCollection\API\Value\Filter\CollectionId;
 use Netgen\InformationCollection\API\Value\Filter\Collections;
 use Netgen\InformationCollection\API\Value\Filter\ContentId;
 use Netgen\InformationCollection\API\Value\Filter\Contents;
 use Netgen\InformationCollection\API\Value\Filter\Query;
 use Netgen\InformationCollection\API\Value\Filter\SearchQuery;
+use Netgen\InformationCollection\Core\Factory\FieldDataFactory;
 use Netgen\InformationCollection\Core\Pagination\InformationCollectionCollectionListAdapter;
 use Netgen\InformationCollection\Core\Pagination\InformationCollectionCollectionListSearchAdapter;
 use Netgen\InformationCollection\Core\Pagination\InformationCollectionContentsAdapter;
+use Netgen\InformationCollection\Doctrine\Repository\EzInfoCollectionAttributeRepository;
+use Netgen\InformationCollection\Doctrine\Repository\EzInfoCollectionRepository;
 use Pagerfanta\Adapter\AdapterInterface;
 use Pagerfanta\Pagerfanta;
 use Symfony\Component\HttpFoundation\RedirectResponse;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
 use Symfony\Contracts\Translation\TranslatorInterface;
 
+use function array_merge;
+use function count;
+use function time;
+
 class AdminController extends Controller
 {
     protected InformationCollection $service;
@@ -36,25 +50,39 @@ class AdminController extends Controller
 
     protected Anonymizer $anonymizer;
 
+    protected FieldDataFactory $factory;
+    protected EzInfoCollectionRepository $infoCollectionRepository;
+
+    protected EzInfoCollectionAttributeRepository $infoCollectionAttributeRepository;
+
+    protected FormBuilder $builder;
+
     private TranslatorInterface $translator;
 
     public function __construct(
         InformationCollection $service,
         Anonymizer $anonymizer,
         ContentService $contentService,
         ConfigResolverInterface $configResolver,
-        TranslatorInterface $translator
-    )
-    {
+        TranslatorInterface $translator,
+        FieldDataFactory $factory,
+        EzInfoCollectionRepository $infoCollectionRepository,
+        EzInfoCollectionAttributeRepository $infoCollectionAttributeRepository,
+        FormBuilder $builder
+    ) {
         $this->service = $service;
         $this->contentService = $contentService;
         $this->configResolver = $configResolver;
         $this->anonymizer = $anonymizer;
         $this->translator = $translator;
+        $this->factory = $factory;
+        $this->infoCollectionRepository = $infoCollectionRepository;
+        $this->infoCollectionAttributeRepository = $infoCollectionAttributeRepository;
+        $this->builder = $builder;
     }
 
     /**
-     * Displays overview page
+     * Displays overview page.
      */
     public function overviewAction(Request $request): Response
     {
@@ -63,27 +91,27 @@ public function overviewAction(Request $request): Response
         $adapter = new InformationCollectionContentsAdapter($this->service, Query::countQuery());
         $pager = $this->getPager($adapter, (int) $request->query->get('page'));
 
-        return $this->render("@NetgenInformationCollection/admin/overview.html.twig", ['objects' => $pager]);
+        return $this->render('@NetgenInformationCollection/admin/overview.html.twig', ['objects' => $pager]);
     }
 
     /**
-     * Displays list of collection for selected Content
+     * Displays list of collection for selected Content.
      */
     public function collectionListAction(Request $request, Content $content): Response
     {
         $this->checkReadPermissions();
 
         $adapter = new InformationCollectionCollectionListAdapter($this->service, ContentId::withContentId($content->id));
-        $pager = $this->getPager($adapter, (int)$request->query->get('page'));
+        $pager = $this->getPager($adapter, (int) $request->query->get('page'));
 
-        return $this->render("@NetgenInformationCollection/admin/collection_list.html.twig", [
+        return $this->render('@NetgenInformationCollection/admin/collection_list.html.twig', [
             'objects' => $pager,
             'content' => $content,
         ]);
     }
 
     /**
-     * Handles collection search
+     * Handles collection search.
      */
     public function searchAction(Request $request, Content $content): Response
     {
@@ -92,9 +120,10 @@ public function searchAction(Request $request, Content $content): Response
         $query = SearchQuery::withContentAndSearchText($content->id, $request->query->get('searchText'));
 
         $adapter = new InformationCollectionCollectionListSearchAdapter($this->service, $query);
-        $pager = $this->getPager($adapter, (int)$request->query->get('page'));
+        $pager = $this->getPager($adapter, (int) $request->query->get('page'));
 
-        return $this->render("@NetgenInformationCollection/admin/collection_list.html.twig",
+        return $this->render(
+            '@NetgenInformationCollection/admin/collection_list.html.twig',
             [
                 'objects' => $pager,
                 'content' => $content,
@@ -103,20 +132,20 @@ public function searchAction(Request $request, Content $content): Response
     }
 
     /**
-     * Displays individual collection details
+     * Displays individual collection details.
      */
     public function viewAction(Collection $collection): Response
     {
         $this->checkReadPermissions();
 
-        return $this->render("@NetgenInformationCollection/admin/view.html.twig", [
+        return $this->render('@NetgenInformationCollection/admin/view.html.twig', [
             'collection' => $collection,
             'content' => $collection->getContent(),
         ]);
     }
 
     /**
-     * Handles actions performed on overview page
+     * Handles actions performed on overview page.
      */
     public function handleContentsAction(Request $request): RedirectResponse
     {
@@ -132,7 +161,6 @@ public function handleContentsAction(Request $request): RedirectResponse
         }
 
         if ($request->request->has('DeleteCollectionByContentAction')) {
-
             $this->checkDeletePermissions();
 
             $query = new Contents($contents);
@@ -150,7 +178,7 @@ public function handleContentsAction(Request $request): RedirectResponse
     }
 
     /**
-     * Handles actions performed on collection list page
+     * Handles actions performed on collection list page.
      */
     public function handleCollectionListAction(Request $request): RedirectResponse
     {
@@ -167,10 +195,9 @@ public function handleCollectionListAction(Request $request): RedirectResponse
         }
 
         if ($request->request->has('DeleteCollectionAction')) {
-
             $this->checkDeletePermissions();
 
-            $query = new Collections($contentId, $collections);
+            $query = new Collections((int)$contentId, $collections);
 
             $this->service->deleteCollections($query);
 
@@ -180,7 +207,6 @@ public function handleCollectionListAction(Request $request): RedirectResponse
         }
 
         if ($request->request->has('AnonymizeCollectionAction')) {
-
             $this->checkAnonymizePermissions();
 
             foreach ($collections as $collection) {
@@ -198,7 +224,7 @@ public function handleCollectionListAction(Request $request): RedirectResponse
     }
 
     /**
-     * Handles action on collection details page
+     * Handles action on collection details page.
      */
     public function handleCollectionAction(Request $request): RedirectResponse
     {
@@ -219,7 +245,6 @@ public function handleCollectionAction(Request $request): RedirectResponse
         }
 
         if ($request->request->has('DeleteFieldAction')) {
-
             $this->checkDeletePermissions();
 
             $query = new CollectionFields($contentId, $collectionId, $fields);
@@ -232,7 +257,6 @@ public function handleCollectionAction(Request $request): RedirectResponse
         }
 
         if ($request->request->has('AnonymizeFieldAction')) {
-
             $this->checkAnonymizePermissions();
 
             $this->anonymizer->anonymizeCollection($collectionId, $fields);
@@ -243,25 +267,22 @@ public function handleCollectionAction(Request $request): RedirectResponse
         }
 
         if ($request->request->has('DeleteCollectionAction')) {
-
             $this->checkDeletePermissions();
 
-            $query = new Collections($contentId, [$collectionId]);
+            $query = new Collections((int)$contentId, [$collectionId]);
             $this->service->deleteCollections($query);
 
-            $this->addFlashMessage("success", "collection_removed");
+            $this->addFlashMessage('success', 'collection_removed');
 
             return $this->redirectToRoute('netgen_information_collection.route.admin.collection_list', ['contentId' => $contentId]);
-
         }
 
         if ($request->request->has('AnonymizeCollectionAction')) {
-
             $this->checkAnonymizePermissions();
 
             $this->anonymizer->anonymizeCollection($collectionId);
 
-            $this->addFlashMessage("success", "collection_anonymized");
+            $this->addFlashMessage('success', 'collection_anonymized');
 
             return $this->redirectToRoute('netgen_information_collection.route.admin.view', ['collectionId' => $collectionId]);
         }
@@ -271,10 +292,95 @@ public function handleCollectionAction(Request $request): RedirectResponse
         return $this->redirectToRoute('netgen_information_collection.route.admin.view', ['collectionId' => $collectionId]);
     }
 
+    /**
+     * @throws NonUniqueResultException
+     */
+    public function editAction(Request $request, int $collectionId): RedirectResponse|Response
+    {
+        $this->checkEditPermissions();
+
+        $collection = $this->service->getCollection(new CollectionId($collectionId));
+
+        $location = $collection->getContent()->contentInfo->getMainLocation();
+
+        if ($location === null) {
+            throw $this->createNotFoundException();
+        }
+
+        $form = $this->builder->createUpdateFormForLocation($location, $collection)->getForm();
+
+        $form->handleRequest($request);
+
+        if ($form->isSubmitted() && $form->isValid()) {
+            /** @var InformationCollectionStruct $struct */
+            $struct = $form->getData()->payload;
+
+            /** @var ContentType $contentType */
+            $contentType = $form->getData()->definition;
+
+            $ezInfoCollection = $this->infoCollectionRepository->find($collectionId);
+
+            if ($ezInfoCollection === null) {
+                throw $this->createNotFoundException();
+            }
+
+            $ezInfoCollection->setModified(time());
+
+            $this->infoCollectionRepository->save($ezInfoCollection);
+
+            foreach ($struct->getCollectedFields() as $fieldDefIdentifier => $value) {
+                if ($value === null) {
+                    continue;
+                }
+
+                $fieldDefinition = $contentType->getFieldDefinition($fieldDefIdentifier);
+
+                if ($fieldDefinition === null) {
+                    continue;
+                }
+
+                $legacyValue = $this->factory->getLegacyValue($value, $fieldDefinition);
+
+                $ezInfoAttribute = $this->infoCollectionAttributeRepository->findByCollectionIdAndFieldDefinitionId(
+                    $collectionId,
+                    $fieldDefinition->id
+                );
+
+                if ($ezInfoAttribute === null) {
+                    $ezInfoAttribute = $this->infoCollectionAttributeRepository->getInstance();
+                    $ezInfoAttribute->setContentObjectId($collection->getContent()->id);
+                    $ezInfoAttribute->setContentObjectAttributeId($collection->getContent()->getField($fieldDefinition->identifier)->id);
+                    $ezInfoAttribute->setContentClassAttributeId($fieldDefinition->id);
+                    $ezInfoAttribute->setInformationCollectionId($collection->getId());
+                }
+
+                $ezInfoAttribute->setDataInt($legacyValue->getDataInt());
+                $ezInfoAttribute->setDataFloat($legacyValue->getDataFloat());
+                $ezInfoAttribute->setDataText($legacyValue->getDataText());
+
+                $this->infoCollectionAttributeRepository->save($ezInfoAttribute);
+            }
+
+            return $this->redirectToRoute(
+                'netgen_information_collection.route.admin.view',
+                [
+                    'contentId' => $location->contentInfo->id,
+                    'collectionId' => $collection->getId(),
+                ]
+            );
+        }
+
+        return $this->render('@NetgenInformationCollection/admin/edit.html.twig', [
+            'collection' => $collection,
+            'content' => $location->getContent(),
+            'form' => $form->createView(),
+        ]);
+    }
+
     /**
      * Adds a flash message with specified parameters.
      */
-    protected function addFlashMessage(string $messageType, string $message, int $count = 1, array $parameters = array()): void
+    protected function addFlashMessage(string $messageType, string $message, int $count = 1, array $parameters = []): void
     {
         $parameters = array_merge($parameters, ['count' => $count]);
 
@@ -289,7 +395,7 @@ protected function addFlashMessage(string $messageType, string $message, int $co
     }
 
     /**
-     * Returns configured instance of Pagerfanta
+     * Returns configured instance of Pagerfanta.
      */
     protected function getPager(AdapterInterface $adapter, int $currentPage): Pagerfanta
     {
@@ -320,4 +426,10 @@ protected function checkAnonymizePermissions(): void
         $attribute = new Attribute('infocollector', 'anonymize');
         $this->denyAccessUnlessGranted($attribute);
     }
+
+    protected function checkEditPermissions(): void
+    {
+        $attribute = new Attribute('infocollector', 'edit');
+        $this->denyAccessUnlessGranted($attribute);
+    }
 }

bundle/Controller/Admin/Export/Export.php

@@ -6,7 +6,7 @@
 
 use Ibexa\Contracts\Core\Repository\ContentService;
 use Ibexa\Core\MVC\Symfony\Security\Authorization\Attribute;
-use Netgen\Bundle\InformationCollectionBundle\Form\ExportType;
+use Netgen\Bundle\InformationCollectionBundle\Form\Type\ExportType;
 use Netgen\InformationCollection\API\Service\Exporter;
 use Netgen\InformationCollection\API\Value\Export\ExportCriteria;
 use Netgen\InformationCollection\Core\Export\ExportResponseFormatterRegistry;