Fixing params that are not string

Author: iruzevic

Diff for: CHANGELOG.md

@@ -4,6 +4,16 @@ All notable changes to this project will be documented in this file.
 
 This projects adheres to [Semantic Versioning](https://semver.org/) and [Keep a CHANGELOG](https://keepachangelog.com/).
 
+## [3.1.0]
+
+### Removed
+
+- `FD_FIELD_NAMES_TAGS` and `FD_FIELD_NAMES_FULL` constants.
+
+### Changed
+
+- backend logic for processing params that are array.
+
 ## [3.0.18]
 
 ### Added
@@ -320,6 +330,7 @@ This projects adheres to [Semantic Versioning](https://semver.org/) and [Keep a
 
 - Initial production release.
 
+[3.1.0]: https://github.com/infinum/eightshift-forms-utils/compare/3.0.18...3.1.0
 [3.0.18]: https://github.com/infinum/eightshift-forms-utils/compare/3.0.17...3.0.18
 [3.0.17]: https://github.com/infinum/eightshift-forms-utils/compare/3.0.16...3.0.17
 [3.0.16]: https://github.com/infinum/eightshift-forms-utils/compare/3.0.15...3.0.16

Diff for: src/Config/UtilsConfig.php

@@ -501,8 +501,6 @@ class UtilsConfig
 	public const FD_ICON = 'icon';
 	public const FD_FIELDS = 'fields';
 	public const FD_FIELD_NAMES = 'fieldNames';
-	public const FD_FIELD_NAMES_TAGS = 'fieldNamesTags';
-	public const FD_FIELD_NAMES_FULL = 'fieldNamesFull';
 	public const FD_STEPS_SETUP = 'stepsSetup';
 	public const FD_RESPONSE_OUTPUT_DATA = 'responseOutputData';
 	public const FD_PARAMS_ORIGINAL = 'paramsOriginal';

Diff for: src/Helpers/UtilsGeneralHelper.php

@@ -307,8 +307,6 @@ public static function getFormDetails(string $formId): array
 			UtilsConfig::FD_FIELDS => [],
 			UtilsConfig::FD_FIELDS_ONLY => [],
 			UtilsConfig::FD_FIELD_NAMES => [],
-			UtilsConfig::FD_FIELD_NAMES_TAGS => [],
-			UtilsConfig::FD_FIELD_NAMES_FULL => [],
 			UtilsConfig::FD_STEPS_SETUP => [],
 		];
 
@@ -393,19 +391,11 @@ public static function getFormDetails(string $formId): array
 				continue;
 			}
 
-			$output[UtilsConfig::FD_FIELD_NAMES_FULL][] = $value;
-
 			if (isset($ignoreBlocks[$blockItemName])) {
 				continue;
 			}
 
 			$output[UtilsConfig::FD_FIELD_NAMES][] = $value;
-
-			if ($blockItemName === 'file') {
-				continue;
-			}
-
-			$output[UtilsConfig::FD_FIELD_NAMES_TAGS][] = $value;
 		}
 
 		// Check if this form uses steps.

Diff for: src/Rest/Routes/AbstractUtilsBaseRoute.php

@@ -181,7 +181,7 @@ static function ($item) {
 						);
 
 						// Append values to the first value.
-						$innerNotEmpty[0]['value'] = \implode(UtilsConfig::DELIMITER, $multiple);
+						$innerNotEmpty[0]['value'] = $multiple;
 
 						return $innerNotEmpty[0];
 					}
@@ -304,17 +304,22 @@ static function ($item) {
 
 					// File.
 					if ($fieldType === 'file') {
-						$output['files'][$key] = $fieldValue ? \array_merge(
-							$value,
-							[
-								'value' => \array_map(
-									function ($item) {
-										return UtilsUploadHelper::getFilePath($item);
-									},
-									\explode(UtilsConfig::DELIMITER, $fieldValue)
-								),
-							]
-						) : $value;
+						$output['files'][$key] = $value;
+
+						if (!$fieldValue) {
+							$output['files'][$key]['value'] = [];
+						} else {
+							if (!\is_array($fieldValue)) {
+								$fieldValue = [$fieldValue];
+							}
+
+							$output['files'][$key]['value'] = \array_map(
+								static function (string $file) {
+									return UtilsUploadHelper::getFilePath($file);
+								},
+								$fieldValue
+							);
+						}
 						break;
 					}
 
@@ -325,15 +330,13 @@ function ($item) {
 
 					// Checkbox.
 					if ($fieldType === 'checkbox') {
-						$fieldValue = \explode(UtilsConfig::DELIMITER, $fieldValue);
-					}
-
-					// Select multiple.
-					if ($fieldType === 'select' && \gettype($fieldValue) === 'array') {
-						$value['value'] = \implode(UtilsConfig::DELIMITER, $fieldValue);
+						if (!$fieldValue) {
+							$value['value'] = [];
+						} else {
+							$value['value'] = \is_string($fieldValue) ? [$fieldValue] : $fieldValue;
+						}
 					}
 
-					$output['paramsRaw'][$fieldName] = $fieldValue;
 					$output['params'][$key] = $value;
 
 					break;
@@ -471,8 +474,6 @@ protected function getFormDetailsApi($request): array
 			$output[UtilsConfig::FD_FIELDS] = $formDetails[UtilsConfig::FD_FIELDS] ?? [];
 			$output[UtilsConfig::FD_FIELDS_ONLY] = $formDetails[UtilsConfig::FD_FIELDS_ONLY] ?? [];
 			$output[UtilsConfig::FD_FIELD_NAMES] = $formDetails[UtilsConfig::FD_FIELD_NAMES] ?? [];
-			$output[UtilsConfig::FD_FIELD_NAMES_TAGS] = $formDetails[UtilsConfig::FD_FIELD_NAMES_TAGS] ?? [];
-			$output[UtilsConfig::FD_FIELD_NAMES_FULL] = $formDetails[UtilsConfig::FD_FIELD_NAMES_FULL] ?? [];
 			$output[UtilsConfig::FD_STEPS_SETUP] = $formDetails[UtilsConfig::FD_STEPS_SETUP] ?? [];
 		}