Created a new case

Author: Grafikart

src/AppBundle/Form/DataTransformer/TagArrayToStringTransformer.php

@@ -54,8 +54,9 @@ public function reverseTransform($string)
             return [];
         }
 
-        $names = array_unique(array_map('trim', explode(',', $string)));
-
+        $names = array_filter(array_unique(array_map('trim', explode(',', $string))), function ($name) {
+            return !empty($name);
+        });
 
         // Get the current tags and find the new ones that should be created.
         $tags = $this->manager->getRepository(Tag::class)->findBy([

tests/AppBundle/Form/DataTransformer/TagArrayToStringTransformerTest.php

@@ -58,6 +58,16 @@ public function testCreateTheRightAmountOfTags()
         $this->assertSame('Hello', $tags[0]->getName());
     }
 
+    /**
+     * Too many commas.
+     */
+    public function testCreateTheRightAmountOfTagsWithTooManyCommas()
+    {
+        $transformer = $this->getMockedTransformer();
+        $this->assertCount(3, $transformer->reverseTransform('Hello, Demo,, How'));
+        $this->assertCount(3, $transformer->reverseTransform('Hello, Demo, How,'));
+    }
+
     /**
      * Spaces at the end (and begining) of a world shouldn't matter.
      */