ENH PHP 8.1 compatibility

silverstripe · Apr 13, 2022 · 2b2bb8b · 2b2bb8b
1 parent 11a563c
commit 2b2bb8b
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 7 deletions.
diff --git a/src/RegisterHandler.php b/src/RegisterHandler.php
@@ -78,7 +78,7 @@ public function start(StoreInterface $store): array
     protected function generateSecret(): string
     {
         $length = $this->config()->get('secret_length');
-        return substr(trim(Base32::encodeUpper(random_bytes(64)), '='), 0, $length);
+        return substr(trim(Base32::encodeUpper(random_bytes(64)) ?? '', '='), 0, $length);
     }
 
     /**
@@ -92,7 +92,7 @@ protected function generateSecret(): string
      */
     public function register(HTTPRequest $request, StoreInterface $store): Result
     {
-        $data = json_decode($request->getBody(), true);
+        $data = json_decode($request->getBody() ?? '', true);
         $result = $this->getTotp($store)->verify($data['code'] ?? '');
         if (!$result) {
             return Result::create(false, _t(__CLASS__ . '.INVALID_CODE', 'Provided code was not valid'));

diff --git a/src/VerifyHandler.php b/src/VerifyHandler.php
@@ -75,7 +75,7 @@ public function start(StoreInterface $store, RegisteredMethod $method): array
 
     public function verify(HTTPRequest $request, StoreInterface $store, RegisteredMethod $registeredMethod): Result
     {
-        $data = json_decode($request->getBody(), true);
+        $data = json_decode($request->getBody() ?? '', true);
         if (!$this->getTotp($store)->verify($data['code'] ?? '')) {
             return Result::create(false, _t(__CLASS__ . '.INVALID_CODE', 'Invalid code'));
         }

diff --git a/tests/behat/src/FeatureContext.php b/tests/behat/src/FeatureContext.php
@@ -44,8 +44,8 @@ public function iPressTheBackupCodesButton($button)
      */
     private function pressMfaButton($section, $button)
     {
-        $section = str_replace("'", "\\'", $section);
-        $button = str_replace("'", "\\'", $button);
+        $section = str_replace("'", "\\'", $section ?? '');
+        $button = str_replace("'", "\\'", $button ?? '');
         $js = <<<JS
             document.querySelectorAll('.registered-method-list-item').forEach(el => {
                 if (!el.innerHTML.includes('{$section}')) {

diff --git a/tests/php/RegisterHandlerTest.php b/tests/php/RegisterHandlerTest.php
@@ -49,12 +49,12 @@ public function testStart()
 
         $this->assertTrue($result['enabled'], 'Method should be enabled');
         $this->assertStringContainsString(
-            rawurlencode(SiteConfig::current_site_config()->Title),
+            rawurlencode(SiteConfig::current_site_config()->Title ?? ''),
             $result['uri'],
             'Site name should be stored in provisioning URI'
         );
         $this->assertStringContainsString(
-            rawurlencode($this->member->Email),
+            rawurlencode($this->member->Email ?? ''),
             $result['uri'],
             'Provisioning URI should contain user email'
         );