Waca\ConsoleTasks\MigrateToRoles::execute()

Complexity

Conditions	7
Paths	17

Size

Total Lines	51
Code Lines	33

Duplication

Lines	7
Ratio	13.73 %

Code Coverage

Tests	0
CRAP Score	56

Importance

Changes

0

<?php
    public function execute()
    {
        $communityUser = User::getCommunity();

        $database = $this->getDatabase();
        $statement = $database->query('SELECT id, status, checkuser FROM user;');
        $update = $database->prepare("UPDATE user SET status = 'Active' WHERE id = :id;");

        $users = $statement->fetchAll(PDO::FETCH_ASSOC);

        foreach ($users as $user) {
            $toAdd = array('user');

            if($user['status'] === 'Admin'){
                $toAdd[] = 'admin';
            }

            if($user['checkuser'] == 1){
                $toAdd[] = 'checkuser';
            }

            foreach ($toAdd as $x) {

This code seems to be duplicated across your project.

                $a = new UserRole();
                $a->setUser($user['id']);
                $a->setRole($x);
                $a->setDatabase($database);
                $a->save();
            }

            $logData = serialize(array(
                'added' => $toAdd,
                'removed' => array(),
                'reason' => 'Initial migration'
            ));

            $log = new Log();
            $log->setDatabase($database);
            $log->setAction('RoleChange');
            $log->setObjectId($user['id']);
            $log->setObjectType('User');
            $log->setUser($communityUser);
            $log->setComment($logData);
            $log->save();

            if($user['status'] === 'Admin' || $user['status'] === 'User'){
                $update->execute(array('id' => $user['id']));
            }
        }

        $database->exec("UPDATE schemaversion SET version = 25;");
    }
}