Passed
Push — master ( 27fb46...dad31c )
by rakekniven
15:02 queued 12s
created
apps/encryption/lib/Exceptions/PublicKeyMissingException.php 1 patch
Indentation   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -25,13 +25,13 @@
 block discarded – undo
25 25
 
26 26
 class PublicKeyMissingException extends GenericEncryptionException {
27 27
 
28
-	/**
29
-	 * @param string $userId
30
-	 */
31
-	public function __construct($userId) {
32
-		if (empty($userId)) {
33
-			$userId = "<no-user-id-given>";
34
-		}
35
-		parent::__construct("Public Key missing for user: $userId");
36
-	}
28
+    /**
29
+     * @param string $userId
30
+     */
31
+    public function __construct($userId) {
32
+        if (empty($userId)) {
33
+            $userId = "<no-user-id-given>";
34
+        }
35
+        parent::__construct("Public Key missing for user: $userId");
36
+    }
37 37
 }
Please login to merge, or discard this patch.
apps/encryption/lib/Settings/Personal.php 1 patch
Indentation   +52 added lines, -52 removed lines patch added patch discarded remove patch
@@ -32,63 +32,63 @@
 block discarded – undo
32 32
 
33 33
 class Personal implements ISettings {
34 34
 
35
-	/** @var IConfig */
36
-	private $config;
37
-	/** @var Session */
38
-	private $session;
39
-	/** @var Util */
40
-	private $util;
41
-	/** @var IUserSession */
42
-	private $userSession;
35
+    /** @var IConfig */
36
+    private $config;
37
+    /** @var Session */
38
+    private $session;
39
+    /** @var Util */
40
+    private $util;
41
+    /** @var IUserSession */
42
+    private $userSession;
43 43
 
44
-	public function __construct(IConfig $config, Session $session, Util $util, IUserSession $userSession) {
45
-		$this->config = $config;
46
-		$this->session = $session;
47
-		$this->util = $util;
48
-		$this->userSession = $userSession;
49
-	}
44
+    public function __construct(IConfig $config, Session $session, Util $util, IUserSession $userSession) {
45
+        $this->config = $config;
46
+        $this->session = $session;
47
+        $this->util = $util;
48
+        $this->userSession = $userSession;
49
+    }
50 50
 
51
-	/**
52
-	 * @return TemplateResponse returns the instance with all parameters set, ready to be rendered
53
-	 * @since 9.1
54
-	 */
55
-	public function getForm() {
56
-		$recoveryAdminEnabled = $this->config->getAppValue('encryption', 'recoveryAdminEnabled');
57
-		$privateKeySet = $this->session->isPrivateKeySet();
51
+    /**
52
+     * @return TemplateResponse returns the instance with all parameters set, ready to be rendered
53
+     * @since 9.1
54
+     */
55
+    public function getForm() {
56
+        $recoveryAdminEnabled = $this->config->getAppValue('encryption', 'recoveryAdminEnabled');
57
+        $privateKeySet = $this->session->isPrivateKeySet();
58 58
 
59
-		if (!$recoveryAdminEnabled && $privateKeySet) {
60
-			return new TemplateResponse('settings', 'settings/empty', [], '');
61
-		}
59
+        if (!$recoveryAdminEnabled && $privateKeySet) {
60
+            return new TemplateResponse('settings', 'settings/empty', [], '');
61
+        }
62 62
 
63
-		$userId = $this->userSession->getUser()->getUID();
64
-		$recoveryEnabledForUser = $this->util->isRecoveryEnabledForUser($userId);
63
+        $userId = $this->userSession->getUser()->getUID();
64
+        $recoveryEnabledForUser = $this->util->isRecoveryEnabledForUser($userId);
65 65
 
66
-		$parameters = [
67
-			'recoveryEnabled' => $recoveryAdminEnabled,
68
-			'recoveryEnabledForUser' => $recoveryEnabledForUser,
69
-			'privateKeySet' => $privateKeySet,
70
-			'initialized' => $this->session->getStatus(),
71
-		];
72
-		return new TemplateResponse('encryption', 'settings-personal', $parameters, '');
73
-	}
66
+        $parameters = [
67
+            'recoveryEnabled' => $recoveryAdminEnabled,
68
+            'recoveryEnabledForUser' => $recoveryEnabledForUser,
69
+            'privateKeySet' => $privateKeySet,
70
+            'initialized' => $this->session->getStatus(),
71
+        ];
72
+        return new TemplateResponse('encryption', 'settings-personal', $parameters, '');
73
+    }
74 74
 
75
-	/**
76
-	 * @return string the section ID, e.g. 'sharing'
77
-	 * @since 9.1
78
-	 */
79
-	public function getSection() {
80
-		return 'security';
81
-	}
75
+    /**
76
+     * @return string the section ID, e.g. 'sharing'
77
+     * @since 9.1
78
+     */
79
+    public function getSection() {
80
+        return 'security';
81
+    }
82 82
 
83
-	/**
84
-	 * @return int whether the form should be rather on the top or bottom of
85
-	 * the admin section. The forms are arranged in ascending order of the
86
-	 * priority values. It is required to return a value between 0 and 100.
87
-	 *
88
-	 * E.g.: 70
89
-	 * @since 9.1
90
-	 */
91
-	public function getPriority() {
92
-		return 80;
93
-	}
83
+    /**
84
+     * @return int whether the form should be rather on the top or bottom of
85
+     * the admin section. The forms are arranged in ascending order of the
86
+     * priority values. It is required to return a value between 0 and 100.
87
+     *
88
+     * E.g.: 70
89
+     * @since 9.1
90
+     */
91
+    public function getPriority() {
92
+        return 80;
93
+    }
94 94
 }
Please login to merge, or discard this patch.
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
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.
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.
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/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/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   +80 added lines, -80 removed lines patch added patch discarded remove patch
@@ -37,94 +37,94 @@
 block discarded – undo
37 37
 
38 38
 class ExpireVersions extends Command {
39 39
 
40
-	/**
41
-	 * @var Expiration
42
-	 */
43
-	private $expiration;
40
+    /**
41
+     * @var Expiration
42
+     */
43
+    private $expiration;
44 44
 
45
-	/**
46
-	 * @var IUserManager
47
-	 */
48
-	private $userManager;
45
+    /**
46
+     * @var IUserManager
47
+     */
48
+    private $userManager;
49 49
 
50
-	/**
51
-	 * @param IUserManager $userManager
52
-	 * @param Expiration $expiration
53
-	 */
54
-	public function __construct(IUserManager $userManager,
55
-								Expiration $expiration) {
56
-		parent::__construct();
50
+    /**
51
+     * @param IUserManager $userManager
52
+     * @param Expiration $expiration
53
+     */
54
+    public function __construct(IUserManager $userManager,
55
+                                Expiration $expiration) {
56
+        parent::__construct();
57 57
 
58
-		$this->userManager = $userManager;
59
-		$this->expiration = $expiration;
60
-	}
58
+        $this->userManager = $userManager;
59
+        $this->expiration = $expiration;
60
+    }
61 61
 
62
-	protected function configure() {
63
-		$this
64
-			->setName('versions:expire')
65
-			->setDescription('Expires the users file versions')
66
-			->addArgument(
67
-				'user_id',
68
-				InputArgument::OPTIONAL | InputArgument::IS_ARRAY,
69
-				'expire file versions of the given user(s), if no user is given file versions for all users will be expired.'
70
-			);
71
-	}
62
+    protected function configure() {
63
+        $this
64
+            ->setName('versions:expire')
65
+            ->setDescription('Expires the users file versions')
66
+            ->addArgument(
67
+                'user_id',
68
+                InputArgument::OPTIONAL | InputArgument::IS_ARRAY,
69
+                'expire file versions of the given user(s), if no user is given file versions for all users will be expired.'
70
+            );
71
+    }
72 72
 
73
-	protected function execute(InputInterface $input, OutputInterface $output): int {
74
-		$maxAge = $this->expiration->getMaxAgeAsTimestamp();
75
-		if (!$maxAge) {
76
-			$output->writeln("No expiry configured.");
77
-			return 1;
78
-		}
73
+    protected function execute(InputInterface $input, OutputInterface $output): int {
74
+        $maxAge = $this->expiration->getMaxAgeAsTimestamp();
75
+        if (!$maxAge) {
76
+            $output->writeln("No expiry configured.");
77
+            return 1;
78
+        }
79 79
 
80
-		$users = $input->getArgument('user_id');
81
-		if (!empty($users)) {
82
-			foreach ($users as $user) {
83
-				if ($this->userManager->userExists($user)) {
84
-					$output->writeln("Remove deleted files of   <info>$user</info>");
85
-					$userObject = $this->userManager->get($user);
86
-					$this->expireVersionsForUser($userObject);
87
-				} else {
88
-					$output->writeln("<error>Unknown user $user</error>");
89
-					return 1;
90
-				}
91
-			}
92
-		} else {
93
-			$p = new ProgressBar($output);
94
-			$p->start();
95
-			$this->userManager->callForSeenUsers(function (IUser $user) use ($p) {
96
-				$p->advance();
97
-				$this->expireVersionsForUser($user);
98
-			});
99
-			$p->finish();
100
-			$output->writeln('');
101
-		}
102
-		return 0;
103
-	}
80
+        $users = $input->getArgument('user_id');
81
+        if (!empty($users)) {
82
+            foreach ($users as $user) {
83
+                if ($this->userManager->userExists($user)) {
84
+                    $output->writeln("Remove deleted files of   <info>$user</info>");
85
+                    $userObject = $this->userManager->get($user);
86
+                    $this->expireVersionsForUser($userObject);
87
+                } else {
88
+                    $output->writeln("<error>Unknown user $user</error>");
89
+                    return 1;
90
+                }
91
+            }
92
+        } else {
93
+            $p = new ProgressBar($output);
94
+            $p->start();
95
+            $this->userManager->callForSeenUsers(function (IUser $user) use ($p) {
96
+                $p->advance();
97
+                $this->expireVersionsForUser($user);
98
+            });
99
+            $p->finish();
100
+            $output->writeln('');
101
+        }
102
+        return 0;
103
+    }
104 104
 
105
-	public function expireVersionsForUser(IUser $user) {
106
-		$uid = $user->getUID();
107
-		if (!$this->setupFS($uid)) {
108
-			return;
109
-		}
110
-		Storage::expireOlderThanMaxForUser($uid);
111
-	}
105
+    public function expireVersionsForUser(IUser $user) {
106
+        $uid = $user->getUID();
107
+        if (!$this->setupFS($uid)) {
108
+            return;
109
+        }
110
+        Storage::expireOlderThanMaxForUser($uid);
111
+    }
112 112
 
113
-	/**
114
-	 * Act on behalf on versions item owner
115
-	 * @param string $user
116
-	 * @return boolean
117
-	 */
118
-	protected function setupFS($user) {
119
-		\OC_Util::tearDownFS();
120
-		\OC_Util::setupFS($user);
113
+    /**
114
+     * Act on behalf on versions item owner
115
+     * @param string $user
116
+     * @return boolean
117
+     */
118
+    protected function setupFS($user) {
119
+        \OC_Util::tearDownFS();
120
+        \OC_Util::setupFS($user);
121 121
 
122
-		// Check if this user has a version directory
123
-		$view = new \OC\Files\View('/' . $user);
124
-		if (!$view->is_dir('/files_versions')) {
125
-			return false;
126
-		}
122
+        // Check if this user has a version directory
123
+        $view = new \OC\Files\View('/' . $user);
124
+        if (!$view->is_dir('/files_versions')) {
125
+            return false;
126
+        }
127 127
 
128
-		return true;
129
-	}
128
+        return true;
129
+    }
130 130
 }
Please login to merge, or discard this patch.
apps/files_versions/lib/Command/CleanUp.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -105,8 +105,8 @@
 block discarded – undo
105 105
 	protected function deleteVersions($user) {
106 106
 		\OC_Util::tearDownFS();
107 107
 		\OC_Util::setupFS($user);
108
-		if ($this->rootFolder->nodeExists('/' . $user . '/files_versions')) {
109
-			$this->rootFolder->get('/' . $user . '/files_versions')->delete();
108
+		if ($this->rootFolder->nodeExists('/'.$user.'/files_versions')) {
109
+			$this->rootFolder->get('/'.$user.'/files_versions')->delete();
110 110
 		}
111 111
 	}
112 112
 }
Please login to merge, or discard this patch.
Indentation   +69 added lines, -69 removed lines patch added patch discarded remove patch
@@ -33,83 +33,83 @@
 block discarded – undo
33 33
 
34 34
 class CleanUp extends Command {
35 35
 
36
-	/** @var IUserManager */
37
-	protected $userManager;
36
+    /** @var IUserManager */
37
+    protected $userManager;
38 38
 
39
-	/** @var IRootFolder */
40
-	protected $rootFolder;
39
+    /** @var IRootFolder */
40
+    protected $rootFolder;
41 41
 
42
-	/**
43
-	 * @param IRootFolder $rootFolder
44
-	 * @param IUserManager $userManager
45
-	 */
46
-	public function __construct(IRootFolder $rootFolder, IUserManager $userManager) {
47
-		parent::__construct();
48
-		$this->userManager = $userManager;
49
-		$this->rootFolder = $rootFolder;
50
-	}
42
+    /**
43
+     * @param IRootFolder $rootFolder
44
+     * @param IUserManager $userManager
45
+     */
46
+    public function __construct(IRootFolder $rootFolder, IUserManager $userManager) {
47
+        parent::__construct();
48
+        $this->userManager = $userManager;
49
+        $this->rootFolder = $rootFolder;
50
+    }
51 51
 
52
-	protected function configure() {
53
-		$this
54
-			->setName('versions:cleanup')
55
-			->setDescription('Delete versions')
56
-			->addArgument(
57
-				'user_id',
58
-				InputArgument::OPTIONAL | InputArgument::IS_ARRAY,
59
-				'delete versions of the given user(s), if no user is given all versions will be deleted'
60
-			);
61
-	}
52
+    protected function configure() {
53
+        $this
54
+            ->setName('versions:cleanup')
55
+            ->setDescription('Delete versions')
56
+            ->addArgument(
57
+                'user_id',
58
+                InputArgument::OPTIONAL | InputArgument::IS_ARRAY,
59
+                'delete versions of the given user(s), if no user is given all versions will be deleted'
60
+            );
61
+    }
62 62
 
63 63
 
64
-	protected function execute(InputInterface $input, OutputInterface $output): int {
65
-		$users = $input->getArgument('user_id');
66
-		if (!empty($users)) {
67
-			foreach ($users as $user) {
68
-				if ($this->userManager->userExists($user)) {
69
-					$output->writeln("Delete versions of   <info>$user</info>");
70
-					$this->deleteVersions($user);
71
-				} else {
72
-					$output->writeln("<error>Unknown user $user</error>");
73
-					return 1;
74
-				}
75
-			}
76
-		} else {
77
-			$output->writeln('Delete all versions');
78
-			foreach ($this->userManager->getBackends() as $backend) {
79
-				$name = get_class($backend);
64
+    protected function execute(InputInterface $input, OutputInterface $output): int {
65
+        $users = $input->getArgument('user_id');
66
+        if (!empty($users)) {
67
+            foreach ($users as $user) {
68
+                if ($this->userManager->userExists($user)) {
69
+                    $output->writeln("Delete versions of   <info>$user</info>");
70
+                    $this->deleteVersions($user);
71
+                } else {
72
+                    $output->writeln("<error>Unknown user $user</error>");
73
+                    return 1;
74
+                }
75
+            }
76
+        } else {
77
+            $output->writeln('Delete all versions');
78
+            foreach ($this->userManager->getBackends() as $backend) {
79
+                $name = get_class($backend);
80 80
 
81
-				if ($backend instanceof IUserBackend) {
82
-					$name = $backend->getBackendName();
83
-				}
81
+                if ($backend instanceof IUserBackend) {
82
+                    $name = $backend->getBackendName();
83
+                }
84 84
 
85
-				$output->writeln("Delete versions for users on backend <info>$name</info>");
85
+                $output->writeln("Delete versions for users on backend <info>$name</info>");
86 86
 
87
-				$limit = 500;
88
-				$offset = 0;
89
-				do {
90
-					$users = $backend->getUsers('', $limit, $offset);
91
-					foreach ($users as $user) {
92
-						$output->writeln("   <info>$user</info>");
93
-						$this->deleteVersions($user);
94
-					}
95
-					$offset += $limit;
96
-				} while (count($users) >= $limit);
97
-			}
98
-		}
99
-		return 0;
100
-	}
87
+                $limit = 500;
88
+                $offset = 0;
89
+                do {
90
+                    $users = $backend->getUsers('', $limit, $offset);
91
+                    foreach ($users as $user) {
92
+                        $output->writeln("   <info>$user</info>");
93
+                        $this->deleteVersions($user);
94
+                    }
95
+                    $offset += $limit;
96
+                } while (count($users) >= $limit);
97
+            }
98
+        }
99
+        return 0;
100
+    }
101 101
 
102 102
 
103
-	/**
104
-	 * delete versions for the given user
105
-	 *
106
-	 * @param string $user
107
-	 */
108
-	protected function deleteVersions($user) {
109
-		\OC_Util::tearDownFS();
110
-		\OC_Util::setupFS($user);
111
-		if ($this->rootFolder->nodeExists('/' . $user . '/files_versions')) {
112
-			$this->rootFolder->get('/' . $user . '/files_versions')->delete();
113
-		}
114
-	}
103
+    /**
104
+     * delete versions for the given user
105
+     *
106
+     * @param string $user
107
+     */
108
+    protected function deleteVersions($user) {
109
+        \OC_Util::tearDownFS();
110
+        \OC_Util::setupFS($user);
111
+        if ($this->rootFolder->nodeExists('/' . $user . '/files_versions')) {
112
+            $this->rootFolder->get('/' . $user . '/files_versions')->delete();
113
+        }
114
+    }
115 115
 }
Please login to merge, or discard this patch.