Merge pull request #1396 from bolt/bugfix/wrap-list-fields-with-1-ele…

…ment-in-array

Wrap list fields with 1 element in array

src/Entity/Field.php

@@ -221,7 +221,7 @@ public function getParsedValue()
             $count = count($value);
             if ($count === 0) {
                 return null;
-            } elseif ($count === 1 && array_keys($value)[0] === 0) {
+            } elseif ($count === 1 && array_keys($value)[0] === 0 && ! $this instanceof ListFieldInterface) {
                 return reset($value);
             }
         }

src/Entity/Field/FilelistField.php

@@ -7,12 +7,13 @@
 use Bolt\Configuration\Content\ContentType;
 use Bolt\Entity\Field;
 use Bolt\Entity\FieldInterface;
+use Bolt\Entity\ListFieldInterface;
 use Doctrine\ORM\Mapping as ORM;
 
 /**
  * @ORM\Entity
  */
-class FilelistField extends Field implements FieldInterface
+class FilelistField extends Field implements FieldInterface, ListFieldInterface
 {
     public const TYPE = 'filelist';
 

src/Entity/Field/ImagelistField.php

@@ -7,12 +7,13 @@
 use Bolt\Configuration\Content\ContentType;
 use Bolt\Entity\Field;
 use Bolt\Entity\FieldInterface;
+use Bolt\Entity\ListFieldInterface;
 use Doctrine\ORM\Mapping as ORM;
 
 /**
  * @ORM\Entity
  */
-class ImagelistField extends Field implements FieldInterface
+class ImagelistField extends Field implements FieldInterface, ListFieldInterface
 {
     public const TYPE = 'imagelist';
 
@@ -37,7 +38,7 @@ public function getValue(): array
             $imageField = new ImageField();
             $imageField->setName((string) $key);
             $imageField->setValue($image);
-            array_push($result, $imageField);
+            $result[] = $imageField;
         }
 
         return $result;

src/Entity/ListFieldInterface.php

@@ -0,0 +1,12 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Bolt\Entity;
+
+/**
+ * Any Field that has an array of fields as its value must implement this interface.
+ */
+interface ListFieldInterface
+{
+}