PUGX / PUGXMultiUserBundle

An extension for FOSUserBundle to handle users of different types. Compatible with Doctrine ORM.
163 stars 96 forks source link

findUsers in usermanager doesn't work #10

Closed mlueft closed 11 years ago

mlueft commented 11 years ago

The original version of the function "findUsers" doesn't work.

public function findUsers()
{
    $classes = $this->userDiscriminator->getClasses();

    $users = array();
    foreach ($classes as $class) {
        $repo = $this->om->getRepository($class);

        // in this line all already retrieved users get deleted from the array
        $users = $repo->findAll();

        if ($users) {
            $users = array_merge($users, $users);
        }
    }

    return $users;
}

This function works

public function findUsers()
{
    $classes = $this->userDiscriminator->getClasses();

    $usersAll = array();  // $usersAll
    foreach ($classes as $class) {
        $repo = $this->om->getRepository($class);

        $users = $repo->findAll();

        if ($users) {
            $usersAll = array_merge($usersAll, $users); // $usersAll
        }
    }

    return $usersAll; // $usersAll
}

regards,

Michael

leopro commented 11 years ago

https://github.com/PUGX/PUGXMultiUserBundle/commit/73c0e610a70709576c6ded98c19ad1a97670fa92