From b7bb9e0f271f699ce7a5b08950580f4f2dc6b0c2 Mon Sep 17 00:00:00 2001
From: Ivo Valchev <Ivo.V.Valchev@gmail.com>
Date: Fri, 30 Jul 2021 14:27:17 +0200
Subject: [PATCH] `imagelist` and `filelist` fields can be emptied

---
 assets/js/app/editor/Components/Filelist.vue  | 6 +++---
 assets/js/app/editor/Components/Imagelist.vue | 7 +++----
 src/Entity/Field/FilelistField.php            | 4 +++-
 src/Entity/Field/ImagelistField.php           | 4 +++-
 4 files changed, 12 insertions(+), 9 deletions(-)

diff --git a/assets/js/app/editor/Components/Filelist.vue b/assets/js/app/editor/Components/Filelist.vue
index 647aa4157..84bb0940f 100644
--- a/assets/js/app/editor/Components/Filelist.vue
+++ b/assets/js/app/editor/Components/Filelist.vue
@@ -22,6 +22,9 @@
                 @moveFileDown="onMoveFileDown"
             ></editor-file>
         </div>
+        <div v-if="containerFiles.length === 0">
+            <input :name="name" value="" type="hidden" />
+        </div>
 
         <button class="btn btn-tertiary" type="button" :disabled="!allowMore" @click="addFile">
             <i class="fas fa-fw fa-plus"></i>
@@ -104,9 +107,6 @@ export default {
         onRemoveFile(elem) {
             let fieldNumber = this.getFieldNumberFromElement(elem);
             this.containerFiles.splice(fieldNumber, 1);
-            if (this.containerFiles.length === 0) {
-                this.addFile();
-            }
         },
         fieldName(index) {
             return this.name + '[' + index + ']';
diff --git a/assets/js/app/editor/Components/Imagelist.vue b/assets/js/app/editor/Components/Imagelist.vue
index a234952c4..7c41a1ad9 100644
--- a/assets/js/app/editor/Components/Imagelist.vue
+++ b/assets/js/app/editor/Components/Imagelist.vue
@@ -25,6 +25,9 @@
                 @moveImageDown="onMoveImageDown"
             ></editor-image>
         </div>
+        <div v-if="getActiveImageFields().length === 0">
+            <input :name="name" value="" type="hidden" />
+        </div>
 
         <button class="btn btn-tertiary" type="button" :disabled="!allowMore" @click="addImage">
             <i class="fas fa-fw fa-plus"></i>
@@ -114,10 +117,6 @@ export default {
             let updatedImage = this.containerImages[fieldNumber];
             updatedImage.hidden = true;
             this.$set(this.containerImages, fieldNumber, updatedImage);
-
-            if (this.getActiveImageFields().length === 0) {
-                this.addImage();
-            }
         },
         fieldName(index) {
             return this.name + '[' + index + ']';
diff --git a/src/Entity/Field/FilelistField.php b/src/Entity/Field/FilelistField.php
index f703204c7..62f5a0126 100644
--- a/src/Entity/Field/FilelistField.php
+++ b/src/Entity/Field/FilelistField.php
@@ -26,7 +26,9 @@ class FilelistField extends Field implements FieldInterface, ListFieldInterface,
      */
     public function getRawValue(): array
     {
-        return (array) parent::getValue() ?: [];
+        $value = parent::getValue();
+
+        return empty(current($value)) ? [] : (array) $value;
     }
 
     /**
diff --git a/src/Entity/Field/ImagelistField.php b/src/Entity/Field/ImagelistField.php
index 8f8f8f198..d130ab724 100644
--- a/src/Entity/Field/ImagelistField.php
+++ b/src/Entity/Field/ImagelistField.php
@@ -26,7 +26,9 @@ class ImagelistField extends Field implements FieldInterface, ListFieldInterface
      */
     public function getRawValue(): array
     {
-        return (array) parent::getValue() ?: [];
+        $value = parent::getValue();
+
+        return empty(current($value)) ? [] : (array) $value;
     }
 
     /**