[TASK] EXT:redirects - Wrong variable name in phpdoc of class SourceHost 40/55440/6
authorJoerg Boesche <typo3@joergboesche.de>
Wed, 24 Jan 2018 07:54:06 +0000 (08:54 +0100)
committerAndreas Fernandez <a.fernandez@scripting-base.de>
Wed, 24 Jan 2018 10:48:13 +0000 (11:48 +0100)
- Resolves a wrong variable definition in phpdoc
- Adds type safe method parameter
- Adds return type string to method

Resolves: #83665
Releases: master
Change-Id: I0ed52e0b2086d6d006885964f480b3c99b7942e7
Reviewed-on: https://review.typo3.org/55440
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Henning Liebe <h.liebe@neusta.de>
Reviewed-by: Andreas Fernandez <a.fernandez@scripting-base.de>
Tested-by: Andreas Fernandez <a.fernandez@scripting-base.de>
typo3/sysext/redirects/Classes/Evaluation/SourceHost.php

index 0156ff2..193388a 100644 (file)
@@ -16,7 +16,8 @@ namespace TYPO3\CMS\Redirects\Evaluation;
  */
 
 /**
- * Class SourceHost - Used for validation / sanitation of domain values
+ * Class SourceHost
+ * Triggered from DataHandler as TCA formevals hook for validation / sanitation of domain values.
  */
 class SourceHost
 {
@@ -24,12 +25,10 @@ class SourceHost
      * Server-side removing of protocol on save
      *
      * @param string $value The field value to be evaluated
-     * @param string $is_in The "is_in" value of the field configuration from TCA
-     * @param bool $set Boolean defining if the value is written to the database or not.
      * @return string Evaluated field value
      */
-    public function evaluateFieldValue($value, $isIn, &$set)
+    public function evaluateFieldValue(string $value): string
     {
-        return preg_replace('#(.*?:\/\/)#', '', $value);
+        return preg_replace('#(.*?:\/\/)#', '', $value) ?? '';
     }
 }