[TASK] Replace GeneralUtility::_GP

sypets · Oct 27, 2024 · 1d8fea7 · 1d8fea7
1 parent abb4556
commit 1d8fea7
Showing 1 changed file with 11 additions and 7 deletions.
diff --git a/Classes/FormEngine/CustomEvaluation/ExcludeLinkTargetsLinkTargetEvaluation.php b/Classes/FormEngine/CustomEvaluation/ExcludeLinkTargetsLinkTargetEvaluation.php
@@ -4,8 +4,6 @@
 
 namespace Sypets\Brofix\FormEngine\CustomEvaluation;
 
-use TYPO3\CMS\Core\Utility\GeneralUtility;
-
 /**
  * Validation of input field
  * tx_brofix_exclude_link_target.linktarget
@@ -56,12 +54,15 @@ public function deevaluateFieldValue(array $parameters): string
      * Normalize linktype for domains, this should contain only
      * the domain, not the scheme.
      *
-     * @param string $value
+     * @param string $linkTarget
      * @return string
      */
-    protected function normalizeLinkTarget(string $value): string
+    protected function normalizeLinkTarget(string $linkTarget): string
     {
-        $formData = GeneralUtility::_GP('data');
+        $formData = $_POST['data'] ?? [];
+        if (!$formData) {
+            return $linkTarget;
+        }
         $id = key($formData['tx_brofix_exclude_link_target']);
         $data = $formData['tx_brofix_exclude_link_target'][$id];
 
@@ -70,8 +71,11 @@ protected function normalizeLinkTarget(string $value): string
 
         if ($match === 'domain' && $linkType === 'external') {
             // extract domain
-            $value = preg_replace('#(?:https?://)?([^/]*).*#', '$1', $value);
+            $linkTarget = preg_replace('#(?:https?://)?([^/]*).*#', '$1', $linkTarget);
         }
-        return $value;
+        if ($match === 'exact' && $linkType === 'external' && preg_match('#^https?://#', $linkTarget) !== 1) {
+            $linkTarget = 'https://' . $linkTarget;
+        }
+        return $linkTarget;
     }
 }