Rossmitchell\Twofactor\Observer\Controller\Frontend\Postdispatch

Complexity

Total Complexity

15

Size/Duplication

Total Lines	142
Duplicated Lines	34.51 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

100%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	execute()	17	17	4
A	isTwoFactorEnabled()	0	4	1
A	hasTwoFactorBeenChecked()	0	7	1
A	redirectToTwoFactorCheck()	0	6	1
A	__construct()	19	19	1
A	shouldTheCustomerBeRedirected()	0	17	4
A	areWeOnAnAllowedPage()	13	13	3

<?php
/**
 * A two factor authentication module that protects both the admin and customer logins
 * Copyright (C) 2017  Ross Mitchell
 *
 * This file is part of Rossmitchell/Twofactor.
 *
 * Rossmitchell/Twofactor is free software: you can redistribute it and/or modify
 * it under the terms of the GNU General Public License as published by
 * the Free Software Foundation, either version 3 of the License, or
 * (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License
 * along with this program. If not, see <http://www.gnu.org/licenses/>.
 */

namespace Rossmitchell\Twofactor\Observer\Controller\Frontend;

use Magento\Framework\App\Action\Action;
use Magento\Framework\App\ResponseFactory;
use Magento\Framework\Event\Observer;
use Magento\Framework\Event\ObserverInterface;
use Magento\Framework\UrlInterface;
use Rossmitchell\Twofactor\Model\Config\Customer as CustomerAdmin;
use Rossmitchell\Twofactor\Model\Customer\Attribute\IsUsingTwoFactor;
use Rossmitchell\Twofactor\Model\Customer\Customer;
use Rossmitchell\Twofactor\Model\Customer\Session;
use Rossmitchell\Twofactor\Model\Verification\IsVerified;
use Rossmitchell\Twofactor\Model\TwoFactorUrls;

class Postdispatch implements ObserverInterface
{
    /**
     * @var ResponseFactory
     */
    private $responseFactory;
    /**
     * @var UrlInterface
     */
    private $url;
    /**
     * @var Customer
     */
    private $customerGetter;
    /**
     * @var IsUsingTwoFactor
     */
    private $isUsingTwoFactor;
    /**
     * @var IsVerified
     */
    private $isVerified;
    /**
     * @var TwoFactorUrls
     */
    private $twoFactorUrls;
    /**
     * @var Session
     */
    private $customerSession;
    /**
     * @var CustomerAdmin
     */
    private $customerAdmin;

    /**
     * Predispatch constructor.
     *
     * @param ResponseFactory  $responseFactory
     * @param UrlInterface     $url
     * @param Customer         $customerGetter
     * @param IsVerified       $isVerified
     * @param Session          $customerSession
     * @param IsUsingTwoFactor $isUsingTwoFactor
     * @param TwoFactorUrls    $twoFactorUrls
     * @param CustomerAdmin    $customerAdmin
     */
    public function __construct(

This method seems to be duplicated in your project.

        ResponseFactory $responseFactory,
        UrlInterface $url,
        Customer $customerGetter,
        IsVerified $isVerified,
        Session $customerSession,
        IsUsingTwoFactor $isUsingTwoFactor,
        TwoFactorUrls $twoFactorUrls,
        CustomerAdmin $customerAdmin
    ) {
        $this->responseFactory  = $responseFactory;
        $this->url              = $url;
        $this->customerGetter   = $customerGetter;
        $this->isUsingTwoFactor = $isUsingTwoFactor;
        $this->isVerified       = $isVerified;
        $this->twoFactorUrls    = $twoFactorUrls;
        $this->customerSession  = $customerSession;
        $this->customerAdmin    = $customerAdmin;
    }

    /**
     * @param Observer $observer
     *
     * @return void
     */
    public function execute(Observer $observer)

This method seems to be duplicated in your project.

    {
        if ($this->isTwoFactorEnabled() === false) {
            return;
        }

        if ($this->shouldTheCustomerBeRedirected() === false) {
            return;
        }

        if ($this->hasTwoFactorBeenChecked() === true) {
            return;
        }

        $controller = $observer->getEvent()->getData('response');
        $this->redirectToTwoFactorCheck($controller);
    }

    private function isTwoFactorEnabled()
    {
        return ($this->customerAdmin->isTwoFactorEnabled() == true);
    }

    private function shouldTheCustomerBeRedirected()
    {
        if ($this->areWeOnAnAllowedPage() === true) {
            return false;
        }

        $customer = $this->customerGetter->getCustomer();
        if ($customer === false) {
            return false;
        }
        $usingTwoFactor = $this->isUsingTwoFactor->getValue($customer);
        if ($usingTwoFactor === false) {
            return false;
        }

        return true;
    }

    private function areWeOnAnAllowedPage()

This method seems to be duplicated in your project.

    {
        $twoFactorUrls = $this->twoFactorUrls;
        if ($twoFactorUrls->areWeOnTheAuthenticationPage(false) === true) {
            return true;
        }

        if ($twoFactorUrls->areWeOnTheVerificationPage(false) === true) {
            return true;
        }

        return false;
    }

    private function hasTwoFactorBeenChecked()
    {
        $session = $this->customerSession;
        $checked = $this->isVerified->isVerified($session);

        return ($checked === true);
    }

    private function redirectToTwoFactorCheck($response)
    {
        $twoFactorCheckUrl = $this->twoFactorUrls->getAuthenticationUrl(false);

        $response->setRedirect($twoFactorCheckUrl);
    }
}