[Template ] unique name template (#2822)

claroline · Jul 5, 2024 · af940ac · af940ac
1 parent fd3a092
commit af940ac
Show file tree

Hide file tree

Showing 3 changed files with 34 additions and 12 deletions.
diff --git a/src/main/core/Entity/Template/Template.php b/src/main/core/Entity/Template/Template.php
@@ -19,12 +19,8 @@
 
 /**
  * @ORM\Entity
- * @ORM\Table(
- *     name="claro_template",
- *     uniqueConstraints={
- *         @ORM\UniqueConstraint(name="template_unique_name", columns={"claro_template_type", "entity_name"})
- *     }
- * )
+ *
+ * @ORM\Table(name="claro_template")
  */
 class Template
 {
@@ -34,6 +30,7 @@ class Template
 
     /**
      * @ORM\ManyToOne(targetEntity="Claroline\CoreBundle\Entity\Template\TemplateType")
+     *
      * @ORM\JoinColumn(name="claro_template_type", nullable=false, onDelete="CASCADE")
      *
      * @var TemplateType

diff --git a/src/main/core/Installation/Migrations/Version20240705075828.php b/src/main/core/Installation/Migrations/Version20240705075828.php
@@ -0,0 +1,30 @@
+<?php
+
+namespace Claroline\CoreBundle\Installation\Migrations;
+
+use Doctrine\DBAL\Schema\Schema;
+use Doctrine\Migrations\AbstractMigration;
+
+/**
+ * Auto-generated migration based on mapping information: modify it with caution.
+ *
+ * Generation date: 2024/07/05 07:58:29
+ */
+final class Version20240705075828 extends AbstractMigration
+{
+    public function up(Schema $schema): void
+    {
+        $this->addSql('
+            DROP INDEX template_unique_name ON claro_template
+        ');
+    }
+
+    public function down(Schema $schema): void
+    {
+        $this->addSql('
+            CREATE UNIQUE INDEX claro_template_type_entity_name ON claro_template (
+                claro_template_type, entity_name
+            )
+        ');
+    }
+}
diff --git a/src/main/core/Resources/modules/administration/template/components/form.jsx b/src/main/core/Resources/modules/administration/template/components/form.jsx
@@ -35,12 +35,7 @@ const TemplateForm = (props) =>
             type: 'string',
             label: trans('name'),
             required: true,
-            disabled: (template) => template.system,
-            options: {
-              unique: {
-                check: ['apiv2_template_get', {field: 'name'}]
-              }
-            }
+            disabled: (template) => template.system
           }, {
             name: 'defineAsDefault',
             type: 'boolean',