[TASK] Replace gettype calls type-specific checks 68/57468/4
authorWouter Wolters <typo3@wouterwolters.nl>
Wed, 4 Jul 2018 18:45:58 +0000 (20:45 +0200)
committerAndreas Fernandez <a.fernandez@scripting-base.de>
Fri, 20 Jul 2018 06:42:41 +0000 (08:42 +0200)
Resolves: #85479
Releases: master
Change-Id: I576ab5fb6a7937016497a93d3b4dd025478e81bd
Reviewed-on: https://review.typo3.org/57468
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Jörg Bösche <typo3@joergboesche.de>
Tested-by: Jörg Bösche <typo3@joergboesche.de>
Reviewed-by: Andreas Fernandez <a.fernandez@scripting-base.de>
Tested-by: Andreas Fernandez <a.fernandez@scripting-base.de>
typo3/sysext/frontend/Classes/Controller/ErrorController.php
typo3/sysext/frontend/Classes/Controller/TypoScriptFrontendController.php
typo3/sysext/saltedpasswords/Tests/Unit/Salt/Argon2iSaltTest.php
typo3/sysext/saltedpasswords/Tests/Unit/Salt/BcryptSaltTest.php

index 669b6a9..8b8aea1 100644 (file)
@@ -144,7 +144,7 @@ class ErrorController
         $response = null;
         $content = '';
         // Simply boolean; Just shows TYPO3 error page with reason:
-        if (gettype($errorHandler) === 'boolean' || strtolower($errorHandler) === 'true' || (string)$errorHandler === '1') {
+        if (is_bool($errorHandler) || strtolower($errorHandler) === 'true' || (string)$errorHandler === '1') {
             $content = GeneralUtility::makeInstance(ErrorPageController::class)->errorAction(
                 'Page Not Found',
                 'The page did not exist or was inaccessible.' . ($reason ? ' Reason: ' . $reason : '')
index 9160cc9..30340ca 100644 (file)
@@ -1975,7 +1975,7 @@ class TypoScriptFrontendController implements LoggerAwareInterface
         }
         // Create response:
         // Simply boolean; Just shows TYPO3 error page with reason:
-        if (strtolower($code) === 'true' || (string)$code === '1' || gettype($code) === 'boolean') {
+        if (strtolower($code) === 'true' || (string)$code === '1' || is_bool($code)) {
             echo GeneralUtility::makeInstance(ErrorPageController::class)->errorAction(
                 'Page Not Found',
                 'The page did not exist or was inaccessible.' . ($reason ? ' Reason: ' . $reason : '')
index 5768517..9debc7c 100644 (file)
@@ -101,7 +101,7 @@ class Argon2iSaltTest extends UnitTestCase
     {
         $hash = $this->subject->getHashedPassword('password');
         $this->assertNotNull($hash);
-        $this->assertEquals('string', gettype($hash));
+        $this->assertTrue(is_string($hash));
     }
 
     /**
index c82b5b1..3153ea6 100644 (file)
@@ -77,7 +77,7 @@ class BcryptSaltTest extends UnitTestCase
     {
         $hash = $this->subject->getHashedPassword('password');
         $this->assertNotNull($hash);
-        $this->assertEquals('string', gettype($hash));
+        $this->assertTrue(is_string($hash));
     }
 
     /**