[TASK] Accept only alphanumeric values for forms marker names

Classes/Finisher/Forms/FormsFinisher.php

@@ -83,9 +83,13 @@ class FormsFinisher extends AbstractFinisher {
 
 			$formElementProperties = $formElement->getProperties();
 			if (isset($formElementProperties['markerName']) && \trim($formElementProperties['markerName']) !== '') {
-				$markers[\trim($formElementProperties['markerName'])] = $value;
+				$markerName = \trim($formElementProperties['markerName']);
 			} else {
-				$markers[$identifier] = $value;
+				$markerName = $identifier;
+			}
+			$markerName = preg_replace('/[^a-zA-Z0-9]/', '', $markerName);
+			if (!isset($markers[$markerName])) {
+				$markers[$markerName] = $value;
 			}
 		}
 

Classes/XClass/Form/FormEditorController.php

@@ -168,13 +168,22 @@ class FormEditorController extends \TYPO3\CMS\Form\Controller\FormEditorControll
 			if (isset($element['properties']['markerName']) && $element['properties']['markerName'] !== '') {
 				$markerName = $element['properties']['markerName'];
 			}
-
-			$markers[] = [
-				'identifier' => $markerName,
-				'type' => MailTemplateService::MARKER_TYPE_STRING,
-				'value' => $element['label'],
-				'description' => $element['label']
-			];
+			$markerName = preg_replace('/[^a-zA-Z0-9]/', '', $markerName);
+			$duplicateMarker = FALSE;
+			foreach ($markers as $marker) {
+				if ($marker['identifier'] === $markerName) {
+					$duplicateMarker = TRUE;
+					break;
+				}
+			}
+			if (!$duplicateMarker) {
+				$markers[] = [
+					'identifier' => $markerName,
+					'type' => MailTemplateService::MARKER_TYPE_STRING,
+					'value' => $element['label'],
+					'description' => $element['label']
+				];
+			}
 		}
 
 		$registrationService = $this->objectManager->get(RegisterService::class);