X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=src%2FWallabag%2FCoreBundle%2FForm%2FDataTransformer%2FStringToListTransformer.php;h=b712ad152c13e1fc1d8a0830eb7b7978d41244ad;hb=78833672469f7beb0c4a195aa0a76f7ca4133057;hp=332a91b87ac717797d03f65cf458a2ee1b63706f;hpb=f19f9f62d13c62f18884e8bd0fa67403e8cad8d7;p=github%2Fwallabag%2Fwallabag.git

diff --git a/src/Wallabag/CoreBundle/Form/DataTransformer/StringToListTransformer.php b/src/Wallabag/CoreBundle/Form/DataTransformer/StringToListTransformer.php
index 332a91b8..b712ad15 100644
--- a/src/Wallabag/CoreBundle/Form/DataTransformer/StringToListTransformer.php
+++ b/src/Wallabag/CoreBundle/Form/DataTransformer/StringToListTransformer.php
@@ -2,14 +2,22 @@
 
 namespace Wallabag\CoreBundle\Form\DataTransformer;
 
-use Doctrine\Common\Persistence\ObjectManager;
 use Symfony\Component\Form\DataTransformerInterface;
-use Symfony\Component\Form\Exception\TransformationFailedException;
 
+/**
+ * Transforms a comma-separated list to a proper PHP array.
+ * Example: the string "foo, bar" will become the array ["foo", "bar"].
+ */
 class StringToListTransformer implements DataTransformerInterface
 {
+    /**
+     * @var string
+     */
     private $separator;
 
+    /**
+     * @param string $separator The separator used in the list.
+     */
     public function __construct($separator = ',')
     {
         $this->separator = $separator;
@@ -34,16 +42,16 @@ class StringToListTransformer implements DataTransformerInterface
     /**
      * Transforms a string to a list.
      *
-     * @param  string $string
+     * @param string $string
      *
      * @return array|null
      */
     public function reverseTransform($string)
     {
-        if (!$string) {
-            return null;
+        if ($string === null) {
+            return;
         }
 
-        return array_filter(array_map('trim', explode($this->separator, $string)));
+        return array_values(array_filter(array_map('trim', explode($this->separator, $string))));
     }
 }