chore: Use iterator in ExpireTrash command

Signed-off-by: Louis Chemineau <louis@chmn.me>
This commit is contained in:
Louis Chemineau 2025-04-02 12:44:54 +02:00
parent 4d1e0cd6fb
commit d25a563d97
No known key found for this signature in database

View file

@ -33,7 +33,8 @@ class ExpireTrash extends Command {
* @param IUserManager|null $userManager
* @param Expiration|null $expiration
*/
public function __construct(?IUserManager $userManager = null,
public function __construct(private LoggerInterface $logger,
?IUserManager $userManager = null,
?Expiration $expiration = null) {
parent::__construct();
@ -74,10 +75,12 @@ class ExpireTrash extends Command {
} else {
$p = new ProgressBar($output);
$p->start();
$this->userManager->callForSeenUsers(function (IUser $user) use ($p) {
$users = $this->userManager->getSeenUsers();
foreach ($users as $user) {
$p->advance();
$this->expireTrashForUser($user);
});
}
$p->finish();
$output->writeln('');
}