Passed
Push — master ( 447277...a1c1b3 )
by Roeland
23:12 queued 12:05
created
apps/encryption/lib/Hooks/Contracts/IHook.php 1 patch
Indentation   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -23,10 +23,10 @@
 block discarded – undo
23 23
 namespace OCA\Encryption\Hooks\Contracts;
24 24
 
25 25
 interface IHook {
26
-	/**
27
-	 * Connects Hooks
28
-	 *
29
-	 * @return null
30
-	 */
31
-	public function addHooks();
26
+    /**
27
+     * Connects Hooks
28
+     *
29
+     * @return null
30
+     */
31
+    public function addHooks();
32 32
 }
Please login to merge, or discard this patch.
apps/encryption/templates/altmail.php 2 patches
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -4,8 +4,8 @@
 block discarded – undo
4 4
 
5 5
 print_unescaped($l->t("Hey there,\n\nthe admin enabled server-side-encryption. Your files were encrypted using the password '%s'.\n\nPlease login to the web interface, go to the section 'basic encryption module' of your personal settings and update your encryption password by entering this password into the 'old log-in password' field and your current login-password.\n\n", [$_['password']]));
6 6
 if (isset($_['expiration'])) {
7
-	print_unescaped($l->t("The share will expire on %s.", [$_['expiration']]));
8
-	print_unescaped("\n\n");
7
+    print_unescaped($l->t("The share will expire on %s.", [$_['expiration']]));
8
+    print_unescaped("\n\n");
9 9
 }
10 10
 // TRANSLATORS term at the end of a mail
11 11
 p($l->t("Cheers!"));
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -6,5 +6,5 @@
 block discarded – undo
6 6
 ?>
7 7
 
8 8
 	--
9
-<?php p($theme->getName() . ' - ' . $theme->getSlogan()); ?>
9
+<?php p($theme->getName().' - '.$theme->getSlogan()); ?>
10 10
 <?php print_unescaped("\n".$theme->getBaseUrl());
Please login to merge, or discard this patch.
apps/encryption/templates/settings-admin.php 3 patches
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -13,7 +13,7 @@  discard block
 block discarded – undo
13 13
 		<p id="encryptHomeStorageSetting">
14 14
 			<input type="checkbox" class="checkbox" name="encrypt_home_storage" id="encryptHomeStorage"
15 15
 				   value="1" <?php if ($_['encryptHomeStorage']) {
16
-	print_unescaped('checked="checked"');
16
+    print_unescaped('checked="checked"');
17 17
 } ?> />
18 18
 			<label for="encryptHomeStorage"><?php p($l->t('Encrypt the home storage'));?></label></br>
19 19
 			<em><?php p($l->t("Enabling this option encrypts all files stored on the main storage, otherwise only files on external storage will be encrypted")); ?></em>
@@ -45,7 +45,7 @@  discard block
 block discarded – undo
45 45
 			<br/><br/>
46 46
 
47 47
 			<p name="changeRecoveryPasswordBlock" id="encryptionChangeRecoveryKey" <?php if ($_['recoveryEnabled'] === '0') {
48
-	print_unescaped('class="hidden"');
48
+    print_unescaped('class="hidden"');
49 49
 }?>>
50 50
 				<?php p($l->t("Change recovery key password:")); ?>
51 51
 				<span class="msg"></span>
Please login to merge, or discard this patch.
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -9,11 +9,13 @@
 block discarded – undo
9 9
 	<h3><?php p($l->t("Default encryption module")); ?></h3>
10 10
 	<?php if (!$_["initStatus"] && $_['masterKeyEnabled'] === false): ?>
11 11
 		<?php p($l->t("Encryption app is enabled but your keys are not initialized, please log-out and log-in again")); ?>
12
-	<?php else: ?>
12
+	<?php else {
13
+    : ?>
13 14
 		<p id="encryptHomeStorageSetting">
14 15
 			<input type="checkbox" class="checkbox" name="encrypt_home_storage" id="encryptHomeStorage"
15 16
 				   value="1" <?php if ($_['encryptHomeStorage']) {
16 17
 	print_unescaped('checked="checked"');
18
+}
17 19
 } ?> />
18 20
 			<label for="encryptHomeStorage"><?php p($l->t('Encrypt the home storage'));?></label></br>
19 21
 			<em><?php p($l->t("Enabling this option encrypts all files stored on the main storage, otherwise only files on external storage will be encrypted")); ?></em>
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -14,7 +14,7 @@
 block discarded – undo
14 14
 				   value="1" <?php if ($_['encryptHomeStorage']) {
15 15
 	print_unescaped('checked="checked"');
16 16
 } ?> />
17
-			<label for="encryptHomeStorage"><?php p($l->t('Encrypt the home storage'));?></label></br>
17
+			<label for="encryptHomeStorage"><?php p($l->t('Encrypt the home storage')); ?></label></br>
18 18
 			<em><?php p($l->t("Enabling this option encrypts all files stored on the main storage, otherwise only files on external storage will be encrypted")); ?></em>
19 19
 		</p>
20 20
 		<br />
Please login to merge, or discard this patch.
apps/workflowengine/lib/Check/RequestURL.php 2 patches
Indentation   +55 added lines, -55 removed lines patch added patch discarded remove patch
@@ -26,66 +26,66 @@
 block discarded – undo
26 26
 
27 27
 class RequestURL extends AbstractStringCheck {
28 28
 
29
-	/** @var string */
30
-	protected $url;
29
+    /** @var string */
30
+    protected $url;
31 31
 
32
-	/** @var IRequest */
33
-	protected $request;
32
+    /** @var IRequest */
33
+    protected $request;
34 34
 
35
-	/**
36
-	 * @param IL10N $l
37
-	 * @param IRequest $request
38
-	 */
39
-	public function __construct(IL10N $l, IRequest $request) {
40
-		parent::__construct($l);
41
-		$this->request = $request;
42
-	}
35
+    /**
36
+     * @param IL10N $l
37
+     * @param IRequest $request
38
+     */
39
+    public function __construct(IL10N $l, IRequest $request) {
40
+        parent::__construct($l);
41
+        $this->request = $request;
42
+    }
43 43
 
44
-	/**
45
-	 * @param string $operator
46
-	 * @param string $value
47
-	 * @return bool
48
-	 */
49
-	public function executeCheck($operator, $value) {
50
-		$actualValue = $this->getActualValue();
51
-		if (in_array($operator, ['is', '!is'])) {
52
-			switch ($value) {
53
-				case 'webdav':
54
-					if ($operator === 'is') {
55
-						return $this->isWebDAVRequest();
56
-					} else {
57
-						return !$this->isWebDAVRequest();
58
-					}
59
-			}
60
-		}
61
-		return $this->executeStringCheck($operator, $value, $actualValue);
62
-	}
44
+    /**
45
+     * @param string $operator
46
+     * @param string $value
47
+     * @return bool
48
+     */
49
+    public function executeCheck($operator, $value) {
50
+        $actualValue = $this->getActualValue();
51
+        if (in_array($operator, ['is', '!is'])) {
52
+            switch ($value) {
53
+                case 'webdav':
54
+                    if ($operator === 'is') {
55
+                        return $this->isWebDAVRequest();
56
+                    } else {
57
+                        return !$this->isWebDAVRequest();
58
+                    }
59
+            }
60
+        }
61
+        return $this->executeStringCheck($operator, $value, $actualValue);
62
+    }
63 63
 
64
-	/**
65
-	 * @return string
66
-	 */
67
-	protected function getActualValue() {
68
-		if ($this->url !== null) {
69
-			return $this->url;
70
-		}
64
+    /**
65
+     * @return string
66
+     */
67
+    protected function getActualValue() {
68
+        if ($this->url !== null) {
69
+            return $this->url;
70
+        }
71 71
 
72
-		$this->url = $this->request->getServerProtocol() . '://';// E.g. http(s) + ://
73
-		$this->url .= $this->request->getServerHost();// E.g. localhost
74
-		$this->url .= $this->request->getScriptName();// E.g. /nextcloud/index.php
75
-		$this->url .= $this->request->getPathInfo();// E.g. /apps/files_texteditor/ajax/loadfile
72
+        $this->url = $this->request->getServerProtocol() . '://';// E.g. http(s) + ://
73
+        $this->url .= $this->request->getServerHost();// E.g. localhost
74
+        $this->url .= $this->request->getScriptName();// E.g. /nextcloud/index.php
75
+        $this->url .= $this->request->getPathInfo();// E.g. /apps/files_texteditor/ajax/loadfile
76 76
 
77
-		return $this->url; // E.g. https://localhost/nextcloud/index.php/apps/files_texteditor/ajax/loadfile
78
-	}
77
+        return $this->url; // E.g. https://localhost/nextcloud/index.php/apps/files_texteditor/ajax/loadfile
78
+    }
79 79
 
80
-	/**
81
-	 * @return bool
82
-	 */
83
-	protected function isWebDAVRequest() {
84
-		return substr($this->request->getScriptName(), 0 - strlen('/remote.php')) === '/remote.php' && (
85
-			$this->request->getPathInfo() === '/webdav' ||
86
-			strpos($this->request->getPathInfo(), '/webdav/') === 0 ||
87
-			$this->request->getPathInfo() === '/dav/files' ||
88
-			strpos($this->request->getPathInfo(), '/dav/files/') === 0
89
-		);
90
-	}
80
+    /**
81
+     * @return bool
82
+     */
83
+    protected function isWebDAVRequest() {
84
+        return substr($this->request->getScriptName(), 0 - strlen('/remote.php')) === '/remote.php' && (
85
+            $this->request->getPathInfo() === '/webdav' ||
86
+            strpos($this->request->getPathInfo(), '/webdav/') === 0 ||
87
+            $this->request->getPathInfo() === '/dav/files' ||
88
+            strpos($this->request->getPathInfo(), '/dav/files/') === 0
89
+        );
90
+    }
91 91
 }
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -69,10 +69,10 @@
 block discarded – undo
69 69
 			return $this->url;
70 70
 		}
71 71
 
72
-		$this->url = $this->request->getServerProtocol() . '://';// E.g. http(s) + ://
73
-		$this->url .= $this->request->getServerHost();// E.g. localhost
74
-		$this->url .= $this->request->getScriptName();// E.g. /nextcloud/index.php
75
-		$this->url .= $this->request->getPathInfo();// E.g. /apps/files_texteditor/ajax/loadfile
72
+		$this->url = $this->request->getServerProtocol().'://'; // E.g. http(s) + ://
73
+		$this->url .= $this->request->getServerHost(); // E.g. localhost
74
+		$this->url .= $this->request->getScriptName(); // E.g. /nextcloud/index.php
75
+		$this->url .= $this->request->getPathInfo(); // E.g. /apps/files_texteditor/ajax/loadfile
76 76
 
77 77
 		return $this->url; // E.g. https://localhost/nextcloud/index.php/apps/files_texteditor/ajax/loadfile
78 78
 	}
Please login to merge, or discard this patch.
apps/sharebymail/lib/Settings/Admin.php 1 patch
Indentation   +31 added lines, -31 removed lines patch added patch discarded remove patch
@@ -27,40 +27,40 @@
 block discarded – undo
27 27
 
28 28
 class Admin implements ISettings {
29 29
 
30
-	/** @var SettingsManager */
31
-	private $settingsManager;
30
+    /** @var SettingsManager */
31
+    private $settingsManager;
32 32
 
33
-	public function __construct(SettingsManager $settingsManager) {
34
-		$this->settingsManager = $settingsManager;
35
-	}
33
+    public function __construct(SettingsManager $settingsManager) {
34
+        $this->settingsManager = $settingsManager;
35
+    }
36 36
 
37
-	/**
38
-	 * @return TemplateResponse
39
-	 */
40
-	public function getForm() {
41
-		$parameters = [
42
-			'sendPasswordMail' => $this->settingsManager->sendPasswordByMail(),
43
-			'enforcePasswordProtection' => $this->settingsManager->enforcePasswordProtection()
44
-		];
37
+    /**
38
+     * @return TemplateResponse
39
+     */
40
+    public function getForm() {
41
+        $parameters = [
42
+            'sendPasswordMail' => $this->settingsManager->sendPasswordByMail(),
43
+            'enforcePasswordProtection' => $this->settingsManager->enforcePasswordProtection()
44
+        ];
45 45
 
46
-		return new TemplateResponse('sharebymail', 'settings-admin', $parameters, '');
47
-	}
46
+        return new TemplateResponse('sharebymail', 'settings-admin', $parameters, '');
47
+    }
48 48
 
49
-	/**
50
-	 * @return string the section ID, e.g. 'sharing'
51
-	 */
52
-	public function getSection() {
53
-		return 'sharing';
54
-	}
49
+    /**
50
+     * @return string the section ID, e.g. 'sharing'
51
+     */
52
+    public function getSection() {
53
+        return 'sharing';
54
+    }
55 55
 
56
-	/**
57
-	 * @return int whether the form should be rather on the top or bottom of
58
-	 * the admin section. The forms are arranged in ascending order of the
59
-	 * priority values. It is required to return a value between 0 and 100.
60
-	 *
61
-	 * E.g.: 70
62
-	 */
63
-	public function getPriority() {
64
-		return 40;
65
-	}
56
+    /**
57
+     * @return int whether the form should be rather on the top or bottom of
58
+     * the admin section. The forms are arranged in ascending order of the
59
+     * priority values. It is required to return a value between 0 and 100.
60
+     *
61
+     * E.g.: 70
62
+     */
63
+    public function getPriority() {
64
+        return 40;
65
+    }
66 66
 }
Please login to merge, or discard this patch.
apps/sharebymail/templates/settings-admin.php 1 patch
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -12,11 +12,11 @@
 block discarded – undo
12 12
 
13 13
 	<p>
14 14
 		<input id="sendPasswordMail" type="checkbox" class="checkbox" <?php if ($_['sendPasswordMail']) {
15
-	p('checked');
15
+    p('checked');
16 16
 } ?> />
17 17
 		<label for="sendPasswordMail"><?php p($l->t('Send password by mail')); ?></label><br/>
18 18
 		<input id="enforcePasswordProtection" type="checkbox" class="checkbox" <?php if ($_['enforcePasswordProtection']) {
19
-	p('checked');
19
+    p('checked');
20 20
 } ?> />
21 21
 		<label for="enforcePasswordProtection"><?php p($l->t('Enforce password protection')); ?></label>
22 22
 	</p>
Please login to merge, or discard this patch.
apps/provisioning_api/lib/Middleware/Exceptions/NotSubAdminException.php 1 patch
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -5,7 +5,7 @@
 block discarded – undo
5 5
 use OCP\AppFramework\Http;
6 6
 
7 7
 class NotSubAdminException extends \Exception {
8
-	public function __construct() {
9
-		parent::__construct('Logged in user must be at least a sub admin', Http::STATUS_FORBIDDEN);
10
-	}
8
+    public function __construct() {
9
+        parent::__construct('Logged in user must be at least a sub admin', Http::STATUS_FORBIDDEN);
10
+    }
11 11
 }
Please login to merge, or discard this patch.
apps/provisioning_api/lib/AppInfo/Application.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -17,7 +17,7 @@  discard block
 block discarded – undo
17 17
 		$container = $this->getContainer();
18 18
 		$server = $container->getServer();
19 19
 
20
-		$container->registerService(NewUserMailHelper::class, function (SimpleContainer $c) use ($server) {
20
+		$container->registerService(NewUserMailHelper::class, function(SimpleContainer $c) use ($server) {
21 21
 			return new NewUserMailHelper(
22 22
 				$server->query(Defaults::class),
23 23
 				$server->getURLGenerator(),
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
 				Util::getDefaultEmailAddress('no-reply')
31 31
 			);
32 32
 		});
33
-		$container->registerService('ProvisioningApiMiddleware', function (SimpleContainer $c) use ($server) {
33
+		$container->registerService('ProvisioningApiMiddleware', function(SimpleContainer $c) use ($server) {
34 34
 			$user = $server->getUserManager()->get($c['UserId']);
35 35
 			$isAdmin = $user !== null ? $server->getGroupManager()->isAdmin($user->getUID()) : false;
36 36
 			$isSubAdmin = $user !== null ? $server->getGroupManager()->getSubAdmin()->isSubAdmin($user) : false;
Please login to merge, or discard this patch.
Indentation   +29 added lines, -29 removed lines patch added patch discarded remove patch
@@ -37,35 +37,35 @@
 block discarded – undo
37 37
 use OCP\Util;
38 38
 
39 39
 class Application extends App {
40
-	public function __construct(array $urlParams = []) {
41
-		parent::__construct('provisioning_api', $urlParams);
40
+    public function __construct(array $urlParams = []) {
41
+        parent::__construct('provisioning_api', $urlParams);
42 42
 
43
-		$container = $this->getContainer();
44
-		$server = $container->getServer();
43
+        $container = $this->getContainer();
44
+        $server = $container->getServer();
45 45
 
46
-		$container->registerService(NewUserMailHelper::class, function (SimpleContainer $c) use ($server) {
47
-			return new NewUserMailHelper(
48
-				$server->query(Defaults::class),
49
-				$server->getURLGenerator(),
50
-				$server->getL10NFactory(),
51
-				$server->getMailer(),
52
-				$server->getSecureRandom(),
53
-				new TimeFactory(),
54
-				$server->getConfig(),
55
-				$server->getCrypto(),
56
-				Util::getDefaultEmailAddress('no-reply')
57
-			);
58
-		});
59
-		$container->registerService('ProvisioningApiMiddleware', function (SimpleContainer $c) use ($server) {
60
-			$user = $server->getUserManager()->get($c['UserId']);
61
-			$isAdmin = $user !== null ? $server->getGroupManager()->isAdmin($user->getUID()) : false;
62
-			$isSubAdmin = $user !== null ? $server->getGroupManager()->getSubAdmin()->isSubAdmin($user) : false;
63
-			return new ProvisioningApiMiddleware(
64
-				$c->query(IControllerMethodReflector::class),
65
-				$isAdmin,
66
-				$isSubAdmin
67
-			);
68
-		});
69
-		$container->registerMiddleWare('ProvisioningApiMiddleware');
70
-	}
46
+        $container->registerService(NewUserMailHelper::class, function (SimpleContainer $c) use ($server) {
47
+            return new NewUserMailHelper(
48
+                $server->query(Defaults::class),
49
+                $server->getURLGenerator(),
50
+                $server->getL10NFactory(),
51
+                $server->getMailer(),
52
+                $server->getSecureRandom(),
53
+                new TimeFactory(),
54
+                $server->getConfig(),
55
+                $server->getCrypto(),
56
+                Util::getDefaultEmailAddress('no-reply')
57
+            );
58
+        });
59
+        $container->registerService('ProvisioningApiMiddleware', function (SimpleContainer $c) use ($server) {
60
+            $user = $server->getUserManager()->get($c['UserId']);
61
+            $isAdmin = $user !== null ? $server->getGroupManager()->isAdmin($user->getUID()) : false;
62
+            $isSubAdmin = $user !== null ? $server->getGroupManager()->getSubAdmin()->isSubAdmin($user) : false;
63
+            return new ProvisioningApiMiddleware(
64
+                $c->query(IControllerMethodReflector::class),
65
+                $isAdmin,
66
+                $isSubAdmin
67
+            );
68
+        });
69
+        $container->registerMiddleWare('ProvisioningApiMiddleware');
70
+    }
71 71
 }
Please login to merge, or discard this patch.
apps/files_versions/lib/Command/ExpireVersions.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -89,7 +89,7 @@  discard block
 block discarded – undo
89 89
 		} else {
90 90
 			$p = new ProgressBar($output);
91 91
 			$p->start();
92
-			$this->userManager->callForSeenUsers(function (IUser $user) use ($p) {
92
+			$this->userManager->callForSeenUsers(function(IUser $user) use ($p) {
93 93
 				$p->advance();
94 94
 				$this->expireVersionsForUser($user);
95 95
 			});
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
 		\OC_Util::setupFS($user);
117 117
 
118 118
 		// Check if this user has a version directory
119
-		$view = new \OC\Files\View('/' . $user);
119
+		$view = new \OC\Files\View('/'.$user);
120 120
 		if (!$view->is_dir('/files_versions')) {
121 121
 			return false;
122 122
 		}
Please login to merge, or discard this patch.
Indentation   +78 added lines, -78 removed lines patch added patch discarded remove patch
@@ -36,92 +36,92 @@
 block discarded – undo
36 36
 
37 37
 class ExpireVersions extends Command {
38 38
 
39
-	/**
40
-	 * @var Expiration
41
-	 */
42
-	private $expiration;
39
+    /**
40
+     * @var Expiration
41
+     */
42
+    private $expiration;
43 43
 	
44
-	/**
45
-	 * @var IUserManager
46
-	 */
47
-	private $userManager;
44
+    /**
45
+     * @var IUserManager
46
+     */
47
+    private $userManager;
48 48
 
49
-	/**
50
-	 * @param IUserManager $userManager
51
-	 * @param Expiration $expiration
52
-	 */
53
-	public function __construct(IUserManager $userManager,
54
-								Expiration $expiration) {
55
-		parent::__construct();
49
+    /**
50
+     * @param IUserManager $userManager
51
+     * @param Expiration $expiration
52
+     */
53
+    public function __construct(IUserManager $userManager,
54
+                                Expiration $expiration) {
55
+        parent::__construct();
56 56
 
57
-		$this->userManager = $userManager;
58
-		$this->expiration = $expiration;
59
-	}
57
+        $this->userManager = $userManager;
58
+        $this->expiration = $expiration;
59
+    }
60 60
 
61
-	protected function configure() {
62
-		$this
63
-			->setName('versions:expire')
64
-			->setDescription('Expires the users file versions')
65
-			->addArgument(
66
-				'user_id',
67
-				InputArgument::OPTIONAL | InputArgument::IS_ARRAY,
68
-				'expire file versions of the given user(s), if no user is given file versions for all users will be expired.'
69
-			);
70
-	}
61
+    protected function configure() {
62
+        $this
63
+            ->setName('versions:expire')
64
+            ->setDescription('Expires the users file versions')
65
+            ->addArgument(
66
+                'user_id',
67
+                InputArgument::OPTIONAL | InputArgument::IS_ARRAY,
68
+                'expire file versions of the given user(s), if no user is given file versions for all users will be expired.'
69
+            );
70
+    }
71 71
 
72
-	protected function execute(InputInterface $input, OutputInterface $output) {
73
-		$maxAge = $this->expiration->getMaxAgeAsTimestamp();
74
-		if (!$maxAge) {
75
-			$output->writeln("No expiry configured.");
76
-			return;
77
-		}
72
+    protected function execute(InputInterface $input, OutputInterface $output) {
73
+        $maxAge = $this->expiration->getMaxAgeAsTimestamp();
74
+        if (!$maxAge) {
75
+            $output->writeln("No expiry configured.");
76
+            return;
77
+        }
78 78
 
79
-		$users = $input->getArgument('user_id');
80
-		if (!empty($users)) {
81
-			foreach ($users as $user) {
82
-				if ($this->userManager->userExists($user)) {
83
-					$output->writeln("Remove deleted files of   <info>$user</info>");
84
-					$userObject = $this->userManager->get($user);
85
-					$this->expireVersionsForUser($userObject);
86
-				} else {
87
-					$output->writeln("<error>Unknown user $user</error>");
88
-				}
89
-			}
90
-		} else {
91
-			$p = new ProgressBar($output);
92
-			$p->start();
93
-			$this->userManager->callForSeenUsers(function (IUser $user) use ($p) {
94
-				$p->advance();
95
-				$this->expireVersionsForUser($user);
96
-			});
97
-			$p->finish();
98
-			$output->writeln('');
99
-		}
100
-	}
79
+        $users = $input->getArgument('user_id');
80
+        if (!empty($users)) {
81
+            foreach ($users as $user) {
82
+                if ($this->userManager->userExists($user)) {
83
+                    $output->writeln("Remove deleted files of   <info>$user</info>");
84
+                    $userObject = $this->userManager->get($user);
85
+                    $this->expireVersionsForUser($userObject);
86
+                } else {
87
+                    $output->writeln("<error>Unknown user $user</error>");
88
+                }
89
+            }
90
+        } else {
91
+            $p = new ProgressBar($output);
92
+            $p->start();
93
+            $this->userManager->callForSeenUsers(function (IUser $user) use ($p) {
94
+                $p->advance();
95
+                $this->expireVersionsForUser($user);
96
+            });
97
+            $p->finish();
98
+            $output->writeln('');
99
+        }
100
+    }
101 101
 
102
-	public function expireVersionsForUser(IUser $user) {
103
-		$uid = $user->getUID();
104
-		if (!$this->setupFS($uid)) {
105
-			return;
106
-		}
107
-		Storage::expireOlderThanMaxForUser($uid);
108
-	}
102
+    public function expireVersionsForUser(IUser $user) {
103
+        $uid = $user->getUID();
104
+        if (!$this->setupFS($uid)) {
105
+            return;
106
+        }
107
+        Storage::expireOlderThanMaxForUser($uid);
108
+    }
109 109
 
110
-	/**
111
-	 * Act on behalf on versions item owner
112
-	 * @param string $user
113
-	 * @return boolean
114
-	 */
115
-	protected function setupFS($user) {
116
-		\OC_Util::tearDownFS();
117
-		\OC_Util::setupFS($user);
110
+    /**
111
+     * Act on behalf on versions item owner
112
+     * @param string $user
113
+     * @return boolean
114
+     */
115
+    protected function setupFS($user) {
116
+        \OC_Util::tearDownFS();
117
+        \OC_Util::setupFS($user);
118 118
 
119
-		// Check if this user has a version directory
120
-		$view = new \OC\Files\View('/' . $user);
121
-		if (!$view->is_dir('/files_versions')) {
122
-			return false;
123
-		}
119
+        // Check if this user has a version directory
120
+        $view = new \OC\Files\View('/' . $user);
121
+        if (!$view->is_dir('/files_versions')) {
122
+            return false;
123
+        }
124 124
 
125
-		return true;
126
-	}
125
+        return true;
126
+    }
127 127
 }
Please login to merge, or discard this patch.