Merge pull request #15745 from nextcloud/backport/15738/stable16

[stable16] Don't notify admins if no potentially over exposing links found

Author: MorrisJobke

lib/private/Repair/RemoveLinkShares.php

@@ -201,8 +201,7 @@ private function sendNotification(): void {
 		}
 	}
 
-	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 @@ private function repair(IOutput $output): void {
 	}
 
 	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');
 	}
 }