[TASK] Make GeneralUtility tests notice free 26/57926/3
authorChristian Kuhn <lolli@schwarzbu.ch>
Thu, 16 Aug 2018 10:38:59 +0000 (12:38 +0200)
committerWouter Wolters <typo3@wouterwolters.nl>
Thu, 16 Aug 2018 10:54:02 +0000 (12:54 +0200)
Resolves: #85871
Releases: master
Change-Id: Idca3e0bef85849eff4de818e0c736ac9aec281c2
Reviewed-on: https://review.typo3.org/57926
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Andreas Fernandez <a.fernandez@scripting-base.de>
Tested-by: Andreas Fernandez <a.fernandez@scripting-base.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
typo3/sysext/core/Classes/Utility/GeneralUtility.php
typo3/sysext/core/Tests/Unit/Utility/GeneralUtilityTest.php

index 127fb13..789ee77 100644 (file)
@@ -189,7 +189,9 @@ class GeneralUtility
      */
     public static function _GET($var = null)
     {
-        $value = $var === null ? $_GET : (empty($var) ? null : $_GET[$var]);
+        $value = $var === null
+            ? $_GET
+            : (empty($var) ? null : ($_GET[$var] ?? null));
         // This is there for backwards-compatibility, in order to avoid NULL
         if (isset($value) && !is_array($value)) {
             $value = (string)$value;
@@ -814,7 +816,9 @@ class GeneralUtility
      */
     public static function isFirstPartOfStr($str, $partStr)
     {
-        return $partStr != '' && strpos((string)$str, (string)$partStr, 0) === 0;
+        $str = is_array($str) ? '' : (string)$str;
+        $partStr = is_array($partStr) ? '' : (string)$partStr;
+        return $partStr !== '' && strpos($str, $partStr, 0) === 0;
     }
 
     /**
index 9bea3c8..4ecaec0 100644 (file)
@@ -48,11 +48,6 @@ class GeneralUtilityTest extends UnitTestCase
     const NO_FIX_PERMISSIONS_ON_WINDOWS = 'fixPermissions() not available on Windows (method does nothing)';
 
     /**
-     * Subject is not notice free, disable E_NOTICES
-     */
-    protected static $suppressNotices = true;
-
-    /**
      * @var bool Reset singletons created by subject
      */
     protected $resetSingletonInstances = true;