mbk-lab
/
nextcloud
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #15738 from nextcloud/enh/dont-notify-admin-if-no-exposing-share-links-found 

Don't notify admins if no potentially over exposing links found
This commit is contained in:
Roeland Jago Douma 2019-05-27 11:50:43 +02:00 committed by GitHub
parent cdfd2d8dd1 d00566b750
commit 8b6d8ed423
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
lib/private/Repair/RemoveLinkShares.php
View File

@ -201,8 +201,7 @@ class RemoveLinkShares implements IRepairStep {
}
}
private function repair(IOutput $output): void {
$total = $this->getTotal();
private function repair(IOutput $output, int $total): void {
$output->startProgress($total);
$shareCursor = $this->getShares();
@ -225,12 +224,13 @@ class RemoveLinkShares implements IRepairStep {
}
public function run(IOutput $output): void {
if ($this->shouldRun()) {
$output->info('Removing potentially over exposing link shares');
$this->repair($output);
$output->info('Removed potentially over exposing link shares');
} else {
if ($this->shouldRun() === false || ($total = $this->getTotal()) === 0) {
$output->info('No need to remove link shares.');
return;
}
$output->info('Removing potentially over exposing link shares');
$this->repair($output, $total);
$output->info('Removed potentially over exposing link shares');
}
}