App\Controllers\Ajax\User

Complexity

Total Complexity

6

Size/Duplication

Total Lines	55
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	isEmailUnique()	0	16	3
A	toggleActivation()	0	21	2

<?php

namespace App\Controllers\Ajax;

use App\Models\UserModel;
use Core\AjaxController;
use Core\Container;
use Core\JsonException;
use Core\Traits\StringFunctions;

class User  extends AjaxController{

    use StringFunctions;

The trait Core\Traits\StringFunctions requires the property $childNodes which is not provided by App\Controllers\Ajax\User.

    private $userModel;

    public function __construct(Container $container)
    {
        parent::__construct($container);
        $this->userModel = new UserModel($this->container);
    }

    public function isEmailUnique($get)
    {
        //the router needs a parameter with get functions else throsw a wobbly
        //we pass a get variable and call the /controller/function/get?bla
        //for better use and security, we must pass "get" as the parameter
        if(!$this->startsWith(strtolower($get),"get"))
        {
            throw new JsonException("invalid call");
        }
        $email = $this->request->getData("email");
        try {
            $return = !$this->userModel->isEmailUsed($email);

It seems like $email can also be of type null; however, parameter $email of App\Models\UserModel::isEmailUsed() does only seem to accept string, maybe add an additional type check?

        } catch (\Exception $e) {
            throw new JsonException($e->getMessage());
        }
        echo json_encode($return);

    }

    /**
     * @throws JsonException
     */
    public function toggleActivation()
    {
        $this->onlyAdmin();
        $this->onlyPost();
        $state = (bool)($this->request->getData("state") === 'true');
        $userId = (int)$this->request->getData("userId");

        $result = array();
        $result["success"] = false;
        $result["state"] = $state;
        $result["userId"] = $userId;

        // we can not update the Original Admin activation state
        if($userId !== 1)
        {
            $result["success"] = $this->userModel->activateUser(!$state, $userId);
            $result["state"] = !$state;
            $result["userId"] = $userId;
        }

        echo json_encode($result);
    }
}