Show messages, when an element can not be deleted.

Fixes issue #43.

src/Controller/AdminPages/BaseAdminController.php

@@ -43,10 +43,14 @@ declare(strict_types=1);
 namespace App\Controller\AdminPages;
 
 use App\DataTables\LogDataTable;
+use App\Entity\Attachments\AttachmentType;
 use App\Entity\Base\AbstractNamedDBElement;
+use App\Entity\Base\AbstractPartsContainingDBElement;
 use App\Entity\Base\AbstractStructuralDBElement;
 use App\Entity\Base\PartsContainingRepositoryInterface;
 use App\Entity\LabelSystem\LabelProfile;
+use App\Entity\PriceInformations\Currency;
+use App\Entity\UserSystem\Group;
 use App\Entity\UserSystem\User;
 use App\Events\SecurityEvent;
 use App\Events\SecurityEvents;
@@ -385,6 +389,35 @@ abstract class BaseAdminController extends AbstractController
         if ($this->isCsrfTokenValid('delete'.$entity->getId(), $request->request->get('_token'))) {
             $entityManager = $this->getDoctrine()->getManager();
 
+            //Check if we can delete the part (it must not contain Parts)
+            if ($entity instanceof AbstractPartsContainingDBElement) {
+                /** @var AbstractPartsContainingRepository $repo */
+                $repo = $this->entityManager->getRepository($this->entity_class);
+                if ($repo->getPartsCount($entity) > 0) {
+                    $this->addFlash('error', 'entity.delete.must_not_contain_parts');
+                    return $this->redirectToRoute($this->route_base.'_new');
+                }
+            } elseif ($entity instanceof AttachmentType) {
+                if ($entity->getAttachmentsForType()->count() > 0) {
+                    $this->addFlash('error', 'entity.delete.must_not_contain_attachments');
+                    return $this->redirectToRoute($this->route_base.'_new');
+                }
+            } elseif ($entity instanceof Currency) {
+                if ($entity->getPricedetails()->count() > 0) {
+                    $this->addFlash('error', 'entity.delete.must_not_contain_prices');
+                    return $this->redirectToRoute($this->route_base.'_new');
+                }
+            } elseif ($entity instanceof Group) {
+                if ($entity->getUsers()->count() > 0) {
+                    $this->addFlash('error', 'entity.delete.must_not_contain_users');
+                    return $this->redirectToRoute($this->route_base.'_new');
+                }
+            } elseif ($entity instanceof User) {
+                //TODO: Find a better solution
+                $this->addFlash('error', 'Currently it is not possible to delete a user, as this would break the log... This will be implemented later...');
+                return $this->redirectToRoute($this->route_base.'_new');
+            }
+
             //Check if we need to remove recursively
             if ($entity instanceof AbstractStructuralDBElement && $request->get('delete_recursive', false)) {
                 $recursionHelper->delete($entity, false);

src/Entity/Attachments/AttachmentType.php

@@ -91,7 +91,7 @@ class AttachmentType extends AbstractStructuralDBElement
      */
     public function getAttachmentsForType(): Collection
     {
-        return $this->attachments;
+        return $this->attachments_with_type;
     }
 
     /**

src/Entity/PriceInformations/Currency.php

@@ -45,6 +45,7 @@ namespace App\Entity\PriceInformations;
 use App\Entity\Attachments\CurrencyAttachment;
 use App\Entity\Base\AbstractStructuralDBElement;
 use App\Entity\Parameters\CurrencyParameter;
+use Doctrine\Common\Collections\ArrayCollection;
 use Doctrine\Common\Collections\Collection;
 use Doctrine\ORM\Mapping as ORM;
 use Symfony\Bridge\Doctrine\Validator\Constraints\UniqueEntity;
@@ -103,6 +104,25 @@ class Currency extends AbstractStructuralDBElement
      */
     protected $parameters;
 
+    /** @var Collection<int, Pricedetail>
+     * @ORM\OneToMany(targetEntity="App\Entity\PriceInformations\Pricedetail", mappedBy="currency")
+     */
+    protected $pricedetails;
+
+    public function __construct()
+    {
+        $this->pricedetails = new ArrayCollection();
+        parent::__construct();
+    }
+
+    /**
+     * @return Collection
+     */
+    public function getPricedetails(): Collection
+    {
+        return $this->pricedetails;
+    }
+
     /**
      * Returns the 3 letter ISO code of this currency.
      *

src/Entity/PriceInformations/Pricedetail.php

@@ -82,7 +82,7 @@ class Pricedetail extends AbstractDBElement implements TimeStampableInterface
     /**
      * @var ?Currency The currency used for the current price information.
      *                If this is null, the global base unit is assumed.
-     * @ORM\ManyToOne(targetEntity="Currency")
+     * @ORM\ManyToOne(targetEntity="Currency", inversedBy="pricedetails")
      * @ORM\JoinColumn(name="id_currency", referencedColumnName="id", nullable=true)
      * @Selectable()
      */

src/Entity/UserSystem/Group.php

@@ -47,6 +47,7 @@ use App\Entity\Base\AbstractStructuralDBElement;
 use App\Entity\Parameters\GroupParameter;
 use App\Security\Interfaces\HasPermissionsInterface;
 use App\Validator\Constraints\ValidPermission;
+use Doctrine\Common\Collections\ArrayCollection;
 use Doctrine\Common\Collections\Collection;
 use Doctrine\ORM\Mapping as ORM;
 use Symfony\Component\Validator\Constraints as Assert;
@@ -102,10 +103,12 @@ class Group extends AbstractStructuralDBElement implements HasPermissionsInterfa
      */
     protected $parameters;
 
+
     public function __construct()
     {
         parent::__construct();
         $this->permissions = new PermissionsEmbed();
+        $this->users = new ArrayCollection();
     }
 
     /**
@@ -136,4 +139,9 @@ class Group extends AbstractStructuralDBElement implements HasPermissionsInterfa
     {
         return $this->permissions;
     }
+
+    public function getUsers(): Collection
+    {
+        return $this->users;
+    }
 }