mirror of
https://github.com/nextcloud/server.git
synced 2026-06-08 16:26:59 -04:00
Uses early returns.
To improve code readability. Signed-off-by: Faraz Samapoor <fsa@adlas.at>
This commit is contained in:
parent
ab79706253
commit
90652fd30f
2 changed files with 43 additions and 40 deletions
|
|
@ -75,37 +75,39 @@ class CleanUp extends Command {
|
|||
|
||||
if (!empty($users)) {
|
||||
foreach ($users as $user) {
|
||||
if ($this->userManager->userExists($user)) {
|
||||
$output->writeln("Delete versions of <info>$user</info>");
|
||||
$this->deleteVersions($user, $path);
|
||||
} else {
|
||||
if (!$this->userManager->userExists($user)) {
|
||||
$output->writeln("<error>Unknown user $user</error>");
|
||||
return self::FAILURE;
|
||||
}
|
||||
|
||||
$output->writeln("Delete versions of <info>$user</info>");
|
||||
$this->deleteVersions($user, $path);
|
||||
}
|
||||
} else {
|
||||
$output->writeln('Delete all versions');
|
||||
foreach ($this->userManager->getBackends() as $backend) {
|
||||
$name = get_class($backend);
|
||||
|
||||
if ($backend instanceof IUserBackend) {
|
||||
$name = $backend->getBackendName();
|
||||
}
|
||||
|
||||
$output->writeln("Delete versions for users on backend <info>$name</info>");
|
||||
|
||||
$limit = 500;
|
||||
$offset = 0;
|
||||
do {
|
||||
$users = $backend->getUsers('', $limit, $offset);
|
||||
foreach ($users as $user) {
|
||||
$output->writeln(" <info>$user</info>");
|
||||
$this->deleteVersions($user);
|
||||
}
|
||||
$offset += $limit;
|
||||
} while (count($users) >= $limit);
|
||||
}
|
||||
return self::SUCCESS;
|
||||
}
|
||||
|
||||
$output->writeln('Delete all versions');
|
||||
foreach ($this->userManager->getBackends() as $backend) {
|
||||
$name = get_class($backend);
|
||||
|
||||
if ($backend instanceof IUserBackend) {
|
||||
$name = $backend->getBackendName();
|
||||
}
|
||||
|
||||
$output->writeln("Delete versions for users on backend <info>$name</info>");
|
||||
|
||||
$limit = 500;
|
||||
$offset = 0;
|
||||
do {
|
||||
$users = $backend->getUsers('', $limit, $offset);
|
||||
foreach ($users as $user) {
|
||||
$output->writeln(" <info>$user</info>");
|
||||
$this->deleteVersions($user);
|
||||
}
|
||||
$offset += $limit;
|
||||
} while (count($users) >= $limit);
|
||||
}
|
||||
|
||||
return self::SUCCESS;
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -64,25 +64,26 @@ class ExpireVersions extends Command {
|
|||
$users = $input->getArgument('user_id');
|
||||
if (!empty($users)) {
|
||||
foreach ($users as $user) {
|
||||
if ($this->userManager->userExists($user)) {
|
||||
$output->writeln("Remove deleted files of <info>$user</info>");
|
||||
$userObject = $this->userManager->get($user);
|
||||
$this->expireVersionsForUser($userObject);
|
||||
} else {
|
||||
if (!$this->userManager->userExists($user)) {
|
||||
$output->writeln("<error>Unknown user $user</error>");
|
||||
return self::FAILURE;
|
||||
}
|
||||
|
||||
$output->writeln("Remove deleted files of <info>$user</info>");
|
||||
$userObject = $this->userManager->get($user);
|
||||
$this->expireVersionsForUser($userObject);
|
||||
}
|
||||
} else {
|
||||
$p = new ProgressBar($output);
|
||||
$p->start();
|
||||
$this->userManager->callForSeenUsers(function (IUser $user) use ($p) {
|
||||
$p->advance();
|
||||
$this->expireVersionsForUser($user);
|
||||
});
|
||||
$p->finish();
|
||||
$output->writeln('');
|
||||
return self::SUCCESS;
|
||||
}
|
||||
|
||||
$p = new ProgressBar($output);
|
||||
$p->start();
|
||||
$this->userManager->callForSeenUsers(function (IUser $user) use ($p) {
|
||||
$p->advance();
|
||||
$this->expireVersionsForUser($user);
|
||||
});
|
||||
$p->finish();
|
||||
$output->writeln('');
|
||||
return self::SUCCESS;
|
||||
}
|
||||
|
||||
|
|
|
|||
Loading…
Reference in a new issue