Add database indices to improve performance

src/Entity/Attachments/Attachment.php

@@ -34,7 +34,12 @@ use LogicException;
  * Class Attachment.
  *
  * @ORM\Entity(repositoryClass="App\Repository\AttachmentRepository")
- * @ORM\Table(name="`attachments`")
+ * @ORM\Table(name="`attachments`", indexes={
+ *    @ORM\Index(name="attachments_idx_id_element_id_class_name", columns={"id", "element_id", "class_name"}),
+ *    @ORM\Index(name="attachments_idx_class_name_id", columns={"class_name", "id"}),
+ *    @ORM\Index(name="attachment_name_idx", columns={"name"}),
+ *    @ORM\Index(name="attachment_element_idx", columns={"class_name", "element_id"})
+ * })
  * @ORM\InheritanceType("SINGLE_TABLE")
  * @ORM\DiscriminatorColumn(name="class_name", type="string")
  * @ORM\DiscriminatorMap({
@@ -104,7 +109,7 @@ abstract class Attachment extends AbstractNamedDBElement
     /**
      * @var AttachmentType
      * @ORM\ManyToOne(targetEntity="AttachmentType", inversedBy="attachments_with_type")
-     * @ORM\JoinColumn(name="type_id", referencedColumnName="id")
+     * @ORM\JoinColumn(name="type_id", referencedColumnName="id", nullable=false)
      * @Selectable()
      * @Assert\NotNull(message="validator.attachment.must_not_be_null")
      */

src/Entity/LogSystem/AbstractLogEntry.php

@@ -71,7 +71,11 @@ use Psr\Log\LogLevel;
  * This entity describes a entry in the event log.
  *
  * @ORM\Entity(repositoryClass="App\Repository\LogEntryRepository")
- * @ORM\Table("log")
+ * @ORM\Table("log", indexes={
+ *    @ORM\Index(name="log_idx_type", columns={"type"}),
+ *    @ORM\Index(name="log_idx_type_target", columns={"type", "target_type", "target_id"}),
+ *    @ORM\Index(name="log_idx_datetime", columns={"datetime"}),
+ * })
  * @ORM\InheritanceType("SINGLE_TABLE")
  * @ORM\DiscriminatorColumn(name="type", type="smallint")
  * @ORM\DiscriminatorMap({

src/Entity/Parameters/AbstractParameter.php

@@ -34,7 +34,11 @@ use function sprintf;
 
 /**
  * @ORM\Entity(repositoryClass="App\Repository\ParameterRepository")
- * @ORM\Table("parameters")
+ * @ORM\Table("parameters", indexes={
+ *    @ORM\Index(name="parameter_name_idx", columns={"name"}),
+ *    @ORM\Index(name="parameter_group_idx", columns={"param_group"}),
+ *    @ORM\Index(name="parameter_type_element_idx", columns={"type", "element_id"})
+ * })
  * @ORM\InheritanceType("SINGLE_TABLE")
  * @ORM\DiscriminatorColumn(name="type", type="smallint")
  * @ORM\DiscriminatorMap({
@@ -91,7 +95,6 @@ abstract class AbstractParameter extends AbstractNamedDBElement
 
     /**
      * @var string The unit in which the value values are given (e.g. V)
-     * @Assert\Length(max=5)
      * @ORM\Column(type="string", nullable=false)
      */
     protected string $unit = '';

src/Entity/Parts/Category.php

@@ -33,7 +33,10 @@ use Symfony\Component\Validator\Constraints as Assert;
  * Class AttachmentType.
  *
  * @ORM\Entity(repositoryClass="App\Repository\Parts\CategoryRepository")
- * @ORM\Table(name="`categories`")
+ * @ORM\Table(name="`categories`", indexes={
+ *     @ORM\Index(name="name_idx", columns={"name"}),
+ *     @ORM\Index(name="parent_name_idx", columns={"parent_id", "name"}),
+ * })
  */
 class Category extends AbstractPartsContainingDBElement
 {

src/Entity/Parts/Footprint.php

@@ -61,7 +61,10 @@ use Symfony\Component\Validator\Constraints as Assert;
  * Class Footprint.
  *
  * @ORM\Entity(repositoryClass="App\Repository\Parts\FootprintRepository")
- * @ORM\Table("`footprints`")
+ * @ORM\Table("`footprints`", indexes={
+ *     @ORM\Index(name="name_idx", columns={"name"}),
+ *     @ORM\Index(name="parent_name_idx", columns={"parent_id", "name"}),
+ * })
  */
 class Footprint extends AbstractPartsContainingDBElement
 {

src/Entity/Parts/Manufacturer.php

@@ -61,7 +61,10 @@ use Symfony\Component\Validator\Constraints as Assert;
  * Class Manufacturer.
  *
  * @ORM\Entity(repositoryClass="App\Repository\Parts\ManufacturerRepository")
- * @ORM\Table("`manufacturers`")
+ * @ORM\Table("`manufacturers`", indexes={
+ *     @ORM\Index(name="name_idx", columns={"name"}),
+ *     @ORM\Index(name="parent_name_idx", columns={"parent_id", "name"}),
+ * })
  */
 class Manufacturer extends AbstractCompany
 {

src/Entity/Parts/MeasurementUnit.php

@@ -55,7 +55,10 @@ use Symfony\Component\Validator\Constraints as Assert;
  * This could be something like N, grams, meters, etc...
  *
  * @ORM\Entity(repositoryClass="App\Repository\Parts\MeasurementUnitRepository")
- * @ORM\Table(name="`measurement_units`")
+ * @ORM\Table(name="`measurement_units`", indexes={
+ *     @ORM\Index(name="name_idx", columns={"name"}),
+ *     @ORM\Index(name="parent_name_idx", columns={"parent_id", "name"}),
+ * })
  * @UniqueEntity("unit")
  */
 class MeasurementUnit extends AbstractPartsContainingDBElement

src/Entity/Parts/Part.php

@@ -75,7 +75,10 @@ use Symfony\Component\Validator\Constraints as Assert;
  * Otherwise this class would be too big, to be maintained.
  *
  * @ORM\Entity(repositoryClass="App\Repository\PartRepository")
- * @ORM\Table("`parts`")
+ * @ORM\Table("`parts`", indexes={
+ *    @ORM\Index(name="parts_idx_datet_name_last_id_needs", columns={"datetime_added", "name", "last_modified", "id", "needs_review"}),
+ *    @ORM\Index(name="parts_idx_name", columns={"name"}),
+ * })
  */
 class Part extends AttachmentContainingDBElement
 {

src/Entity/Parts/PartLot.php

@@ -58,7 +58,10 @@ use Symfony\Component\Validator\Constraints as Assert;
  * It is the connection between a part and its store locations.
  *
  * @ORM\Entity()
- * @ORM\Table(name="part_lots")
+ * @ORM\Table(name="part_lots", indexes={
+ *    @ORM\Index(name="part_lots_idx_instock_un_expiration_id_part", columns={"instock_unknown", "expiration_date", "id_part"}),
+ *    @ORM\Index(name="part_lots_idx_needs_refill", columns={"needs_refill"}),
+ * })
  * @ORM\HasLifecycleCallbacks()
  * @ValidPartLot()
  */

src/Entity/Parts/Storelocation.php

@@ -61,7 +61,10 @@ use Symfony\Component\Validator\Constraints as Assert;
  * Class Store location.
  *
  * @ORM\Entity(repositoryClass="App\Repository\Parts\StorelocationRepository")
- * @ORM\Table("`storelocations`")
+ * @ORM\Table("`storelocations`", indexes={
+ *     @ORM\Index(name="name_idx", columns={"name"}),
+ *     @ORM\Index(name="parent_name_idx", columns={"parent_id", "name"}),
+ * })
  */
 class Storelocation extends AbstractPartsContainingDBElement
 {

src/Entity/Parts/Supplier.php

@@ -65,7 +65,10 @@ use Symfony\Component\Validator\Constraints as Assert;
  * Class Supplier.
  *
  * @ORM\Entity(repositoryClass="App\Repository\Parts\SupplierRepository")
- * @ORM\Table("`suppliers`")
+ * @ORM\Table("`suppliers`", indexes={
+ *     @ORM\Index(name="name_idx", columns={"name"}),
+ *     @ORM\Index(name="parent_name_idx", columns={"parent_id", "name"}),
+ * })
  */
 class Supplier extends AbstractCompany
 {

src/Entity/PriceInformations/Currency.php

@@ -59,7 +59,10 @@ use Symfony\Component\Validator\Constraints as Assert;
  *
  * @UniqueEntity("iso_code")
  * @ORM\Entity()
- * @ORM\Table(name="currencies")
+ * @ORM\Table(name="currencies", indexes={
+ *     @ORM\Index(name="name_idx", columns={"name"}),
+ *     @ORM\Index(name="parent_name_idx", columns={"parent_id", "name"}),
+ * })
  */
 class Currency extends AbstractStructuralDBElement
 {

src/Entity/PriceInformations/Orderdetail.php

@@ -66,7 +66,9 @@ use Symfony\Component\Validator\Constraints as Assert;
 /**
  * Class Orderdetail.
  *
- * @ORM\Table("`orderdetails`")
+ * @ORM\Table("`orderdetails`", indexes={
+ *    @ORM\Index(name="orderdetails_supplier_part_nr", columns={"supplierpartnr"}),
+ * })
  * @ORM\Entity()
  * @ORM\HasLifecycleCallbacks()
  * @UniqueEntity({"supplierpartnr", "supplier", "part"})

src/Entity/PriceInformations/Pricedetail.php

@@ -66,7 +66,10 @@ use Symfony\Component\Validator\Constraints as Assert;
  * Class Pricedetail.
  *
  * @ORM\Entity()
- * @ORM\Table("`pricedetails`")
+ * @ORM\Table("`pricedetails`", indexes={
+ *     @ORM\Index(name="pricedetails_idx_min_discount", columns={"min_discount_quantity"}),
+ *     @ORM\Index(name="pricedetails_idx_min_discount_price_qty", columns={"min_discount_quantity", "price_related_quantity"}),
+ * })
  * @ORM\HasLifecycleCallbacks()
  * @UniqueEntity(fields={"min_discount_quantity", "orderdetail"})
  */

src/Entity/UserSystem/Group.php

@@ -56,7 +56,10 @@ use Symfony\Component\Validator\Constraints as Assert;
  * This entity represents an user group.
  *
  * @ORM\Entity()
- * @ORM\Table("`groups`")
+ * @ORM\Table("`groups`", indexes={
+ *     @ORM\Index(name="name_idx", columns={"name"}),
+ *     @ORM\Index(name="parent_name_idx", columns={"parent_id", "name"}),
+ * })
  */
 class Group extends AbstractStructuralDBElement implements HasPermissionsInterface
 {

src/Entity/UserSystem/User.php

@@ -80,7 +80,9 @@ use Symfony\Component\Validator\Constraints as Assert;
  * Also this entity is able to save some informations about the user, like the names, email-address and other info.
  *
  * @ORM\Entity(repositoryClass="App\Repository\UserRepository")
- * @ORM\Table("`users`")
+ * @ORM\Table("`users`", indexes={
+ *    @ORM\Index(name="user_idx_username", columns={"name"})
+ * })
  * @ORM\EntityListeners({"App\EntityListeners\TreeCacheInvalidationListener"})
  * @UniqueEntity("name", message="validator.user.username_already_used")
  */