Firesphere\HaveIBeenPwned\Controllers\HaveIBeenPwnedPageController

Complexity

Total Complexity

2

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	checkEmail()	0	17	2

<?php

namespace Firesphere\HaveIBeenPwned\Controllers;

use Firesphere\HaveIBeenPwned\Models\HaveIBeenPwnedPage;
use Firesphere\HaveIBeenPwned\Services\HaveIBeenPwnedService;
use PageController;
use SilverStripe\Control\HTTPRequest;
use SilverStripe\Core\Injector\Injector;
use SilverStripe\Security\Member;
use SilverStripe\Security\Security;

/**
 * Class \Firesphere\HaveIBeenPwned\Controllers\HaveIBeenPwnedPageController
 *
 */
class HaveIBeenPwnedPageController extends PageController
{
    /**
     * @var array
     */
    private static $allowed_actions = [

The private property $allowed_actions is not used, and could be removed.

        'checkEmail',
    ];

    /**
     * @var array
     */
    private static $url_handlers = [

The private property $url_handlers is not used, and could be removed.

        'check-email' => 'checkEmail',
    ];

    /**
     * @param HTTPRequest|null $request
     * @param array $params
     * @return $this
     * @throws \GuzzleHttp\Exception\GuzzleException
     */
    public function checkEmail($request = null, $params = [])

The parameter $request is not used and could be removed.

    {
        /** @var Member|null $user */
        $user = Security::getCurrentUser();

        if ($user) {

$user is of type SilverStripe\Security\Member, thus it always evaluated to true.

            /** @var HaveIBeenPwnedService $service */
            $service = Injector::inst()->createWithArgs(HaveIBeenPwnedService::class, [$params]);

            $breachedEmails = $service->checkPwndEmail($user);

            $contentText = str_replace("\r\n", '<br />', $breachedEmails);

            $this->dataRecord->Content .= '<p>' . $contentText . '</p>';
        }

        return $this;
    }
}