Waca\Pages\PageDomainSwitch::main()

Complexity

Conditions	5
Paths	5

Size

Total Lines	39
Code Lines	21

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	30

Importance

Changes	1
Bugs	0	Features	0

<?php
    protected function main()
    {
        if (!WebRequest::wasPosted()) {
            $this->redirect('/');

            return;
        }

        $database = $this->getDatabase();
        $currentUser = User::getCurrent($database);

        /** @var Domain|false $newDomain */
        $newDomain = Domain::getById(WebRequest::postInt('newdomain'), $database);

        if ($newDomain === false) {

The condition $newDomain === false is always false.

            $this->redirect('/');

            return;
        }

        try {
            $this->getDomainAccessManager()->switchDomain($currentUser, $newDomain);
        }
        catch(DomainSwitchNotAllowedException $ex){
            throw new AccessDeniedException($this->getSecurityManager(), $this->getDomainAccessManager());
        }

        // try to stay on the same page if possible.
        // This only checks basic ACLs and not domain privileges, so this may still result in a 403.

        $referrer = WebRequest::postString('referrer');
        $priorPath = explode('/', $referrer);

It seems like $referrer can also be of type null; however, parameter $string of explode() does only seem to accept string, maybe add an additional type check?

        $router = new RequestRouter();
        $route = $router->getRouteFromPath($priorPath);

        if ($this->barrierTest($route[1], $currentUser, $route[0])) {
            $this->redirect('/' . $referrer);
        } else {
            $this->redirect('/');
        }
}