validator is working

Author: SanderCokart

assets/js/components/tables/tag/TagTable.js

@@ -37,8 +37,6 @@ const TagTable = () => {
         if (!context.tags) context.read();
     }, [context]);
 
-    console.log(message);
-
     return (
         <TableContainer component={Paper}>
             <Table size="small">
@@ -49,7 +47,7 @@ const TagTable = () => {
                         </TableCell>
                     </TableRow>
                     <TableRow>
-                        <TableCell>{message.level === 'error' ? message.text.join('\n') : 'Tag Name'}</TableCell>
+                        <TableCell>Tag Name</TableCell>
                         <TableCell align="right">Actions</TableCell>
                     </TableRow>
                 </TableHead>

assets/js/contexts/TagContext.js

@@ -10,7 +10,7 @@ class TagContextProvider extends React.Component {
             tags:      null,
             isLoading: false,
             message:   {
-                text:  '',
+                text:  null,
                 level: null,
             },
 
@@ -29,11 +29,11 @@ class TagContextProvider extends React.Component {
         this.setState({isLoading: true});
         try {
             const r = await axios.post('/api/tag/create', data);
-            const {createdTag, message} = r.data;
+            const createdTag = r.data.tag, message = r.data.message;
 
-            if (message.level === 'error') {
+            if (message !== undefined && message.level === 'error') {
                 this.setState({
-                    message:   message.text,
+                    message:   message,
                     isLoading: false,
                 });
             } else {
@@ -42,8 +42,8 @@ class TagContextProvider extends React.Component {
                     isLoading: false,
                 });
             }
-
         } catch (e) {
+            console.error(e);
             this.setState({
                 errors:    e,
                 isLoading: false,
@@ -82,16 +82,21 @@ class TagContextProvider extends React.Component {
      */
     async delete(data) {
         try {
+            if (this.state.isLoading) return;
+
+            this.setState({isLoading: true});
             const r = await axios.delete('/api/tag/delete/' + data.id);
             const newTags = [...this.state.tags].filter(tag => tag.id !== data.id);
 
             this.setState({
-                tags: newTags,
+                tags:      newTags,
+                isLoading: false,
             });
 
         } catch (e) {
             this.setState({
-                error: e,
+                error:     e,
+                isLoading: false,
             });
         }
     }

config/packages/prod/deprecations.yaml

@@ -0,0 +1,8 @@
+# As of Symfony 5.1, deprecations are logged in the dedicated "deprecation" channel when it exists
+#monolog:
+#    channels: [deprecation]
+#    handlers:
+#        deprecation:
+#            type: stream
+#            channels: [deprecation]
+#            path: "%kernel.logs_dir%/%kernel.environment%.deprecations.log"

src/Controller/TagController.php

@@ -2,11 +2,11 @@
 
 namespace App\Controller;
 
-use App\Controller\Services\Message;
 use App\Entity\Tag;
 use App\Repository\TagRepository;
 use Doctrine\ORM\EntityManagerInterface;
 use Exception;
+use Symfony\Bridge\Doctrine\Validator\Constraints\UniqueEntityValidator;
 use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
 use Symfony\Component\HttpFoundation\JsonResponse;
 use Symfony\Component\HttpFoundation\Request;
@@ -66,11 +66,17 @@ public function create(Request $request)
         $tag = new Tag();
         $tag->setName($content->name);
 
-        $tagNameError = $this->validator->validateProperty($tag, 'name');
+        $tagNameErrors = $this->validator->validate($tag);
 
         $errors = [];
-        if (sizeof($tagNameError) > 0) {
-            $errors[] = $tagNameError[0]->getMessage();
+        if (sizeof($tagNameErrors) > 0) {
+
+//            enable this to generate an array of all errors instead of just one
+//            foreach ($tagNameErrors as $tagNameError) {
+//                $errors[] = $tagNameError->getMessage();
+//            }
+
+            $errors[] = $tagNameErrors[0]->getMessage();
 
             return $this->json(
                 ['message' => ['text' => $errors, 'level' => 'error']]

src/Entity/Tag.php

@@ -4,13 +4,15 @@
 
 use App\Repository\TagRepository;
 use Doctrine\ORM\Mapping as ORM;
+use Symfony\Bridge\Doctrine\Validator\Constraints\UniqueEntity;
 use Symfony\Component\Serializer\Encoder\JsonEncoder;
 use Symfony\Component\Serializer\Normalizer\ObjectNormalizer;
 use Symfony\Component\Serializer\Serializer;
 use Symfony\Component\Validator\Constraints as Assert;
 
 /**
  * @ORM\Entity(repositoryClass=TagRepository::class)
+ * @UniqueEntity(message="The tag name must be unique.", fields={"name"})
  */
 class Tag
 {
@@ -22,7 +24,7 @@ class Tag
     private $id;
 
     /**
-     * @ORM\Column(type="string", length=255, nullable=false)
+     * @ORM\Column(type="string", name="name", length=255, nullable=false, unique=true)
      * @Assert\NotBlank(message="The tag name cannot be blank.")
      * @Assert\Length(
      *      min = 2,
@@ -60,4 +62,5 @@ public function normalize()
 
         return $serializer->normalize($this, 'json');
     }
+
 }

src/Migrations/Version20200722212009.php src/Migrations/Version20200723224647.phpsrc/Migrations/Version20200722212009.php renamed to src/Migrations/Version20200723224647.php

@@ -10,7 +10,7 @@
 /**
  * Auto-generated Migration: Please modify to your needs!
  */
-final class Version20200722212009 extends AbstractMigration
+final class Version20200723224647 extends AbstractMigration
 {
     public function getDescription() : string
     {

src/Migrations/Version20200723225030.php

@@ -0,0 +1,31 @@
+<?php
+
+declare(strict_types=1);
+
+namespace DoctrineMigrations;
+
+use Doctrine\DBAL\Schema\Schema;
+use Doctrine\Migrations\AbstractMigration;
+
+/**
+ * Auto-generated Migration: Please modify to your needs!
+ */
+final class Version20200723225030 extends AbstractMigration
+{
+    public function getDescription() : string
+    {
+        return '';
+    }
+
+    public function up(Schema $schema) : void
+    {
+        // this up() migration is auto-generated, please modify it to your needs
+        $this->addSql('CREATE UNIQUE INDEX UNIQ_389B7835E237E06 ON tag (name)');
+    }
+
+    public function down(Schema $schema) : void
+    {
+        // this down() migration is auto-generated, please modify it to your needs
+        $this->addSql('DROP INDEX UNIQ_389B7835E237E06 ON tag');
+    }
+}

tests/bootstrap.php

@@ -0,0 +1,11 @@
+<?php
+
+use Symfony\Component\Dotenv\Dotenv;
+
+require dirname(__DIR__).'/vendor/autoload.php';
+
+if (file_exists(dirname(__DIR__).'/config/bootstrap.php')) {
+    require dirname(__DIR__).'/config/bootstrap.php';
+} elseif (method_exists(Dotenv::class, 'bootEnv')) {
+    (new Dotenv())->bootEnv(dirname(__DIR__).'/.env');
+}