Better support for array field key/value when either key or value is …

…empty (fixes #1972)
getgrav · Nov 17, 2020 · ed8f24a · ed8f24a
1 parent d63dfc3
commit ed8f24a
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,6 +5,7 @@
     * Auto-link a plugin/theme license in details if it starts with `http`
     * Allow to fallback to `docs:` instead of `readme:`
     * Forward a `sid` to GPM when downloading a premium package
+    * Better support for array field key/value when either key or value is empty [#1972](https://github.com/getgrav/grav-plugin-admin/issues/1972)
 1. [](#bugfix)
     * Fixed Safari issue with new ACL picker field [#1955](https://github.com/getgrav/grav-plugin-admin/issues/1955)
     * Stop propagation of ACL add button in ACL picker [flex-objects#83](https://github.com/trilbymedia/grav-plugin-flex-objects/issues/83)

diff --git a/themes/grav/app/forms/fields/array.js b/themes/grav/app/forms/fields/array.js
@@ -111,7 +111,13 @@ export default class ArrayField {
         let escaped_name = !template.isValueOnly() ? keyElement.val() : this.getIndexFor(element);
         escaped_name = escaped_name.toString().replace(/\[/g, '%5B').replace(/]/g, '%5D');
         let name = `${template.getName()}[${escaped_name}]`;
-        valueElement.attr('name', !valueElement.val() ? template.getName() : name);
+
+        if (!template.isValueOnly() && (!keyElement.val() && !valueElement.val())) {
+            valueElement.attr('name', '');
+        } else {
+            // valueElement.attr('name', !valueElement.val() ? template.getName() : name);
+            valueElement.attr('name', name);
+        }
 
         this.refreshNames(template);
     }