PhpEarth\Stats\Collection\UserCollection::getTopUsers()

Complexity

Conditions	4
Paths	6

Size

Total Lines	18
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function getTopUsers($limit = null, array $ignoredUsers = [])
    {
        if (sizeof($this->topUsers) == 0) {
            $users = $this->data;
            usort($users, [$this, 'sortUsers']);
            $users = array_reverse($users, true);

            $this->topUsers = $users;
        }

        $topUsers = [];
        foreach ($this->topUsers as $user) {
            if (!in_array($user->getName(), $ignoredUsers)) {
                $topUsers[] = $user;
            }
        }

        return array_slice($topUsers, 0, $limit);

It seems like $limit can also be of type string; however, parameter $length of array_slice() does only seem to accept integer, maybe add an additional type check?

    }