always specify limit, except you do like questionable surprises

always specify limit, except you do like questionable surprises, part 2
This commit is contained in:
Arthur Schiwon 2015-02-06 17:37:05 +01:00
parent 80e38b84dc
commit 15b2d33599
2 changed files with 18 additions and 7 deletions

View File

@ -62,11 +62,17 @@ class MigrateKeys extends Command {
} }
$output->writeln("Migrating keys for users on backend <info>$name</info>"); $output->writeln("Migrating keys for users on backend <info>$name</info>");
$users = $backend->getUsers();
foreach ($users as $user) { $limit = 500;
$offset = 0;
do {
$users = $backend->getUsers('', $limit, $offset);
foreach ($users as $user) {
$output->writeln(" <info>$user</info>"); $output->writeln(" <info>$user</info>");
$migration->reorganizeFolderStructureForUser($user); $migration->reorganizeFolderStructureForUser($user);
} }
$offset += $limit;
} while(count($users) >= $limit);
} }
} }

View File

@ -42,10 +42,15 @@ class Migration {
public function reorganizeFolderStructure() { public function reorganizeFolderStructure() {
$this->reorganizeSystemFolderStructure(); $this->reorganizeSystemFolderStructure();
$users = \OCP\User::getUsers(); $limit = 500;
foreach ($users as $user) { $offset = 0;
$this->reorganizeFolderStructureForUser($user); do {
} $users = \OCP\User::getUsers('', $limit, $offset);
foreach ($users as $user) {
$this->reorganizeFolderStructureForUser($user);
}
$offset += $limit;
} while(count($users) >= $limit);
} }
public function reorganizeSystemFolderStructure() { public function reorganizeSystemFolderStructure() {