Passed
Push — master ( 13846a...151523 )
by Joas
15:13 queued 14s
created
apps/workflowengine/lib/Service/Logger.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -52,8 +52,8 @@
 block discarded – undo
52 52
 	}
53 53
 
54 54
 	protected function initLogger() {
55
-		$default = $this->config->getSystemValue('datadirectory', \OC::$SERVERROOT . '/data') . '/flow.log';
56
-		$logFile = trim((string)$this->config->getAppValue(Application::APP_ID, 'logfile', $default));
55
+		$default = $this->config->getSystemValue('datadirectory', \OC::$SERVERROOT.'/data').'/flow.log';
56
+		$logFile = trim((string) $this->config->getAppValue(Application::APP_ID, 'logfile', $default));
57 57
 		if ($logFile !== '') {
58 58
 			$this->flowLogger = $this->logFactory->getCustomPsrLogger($logFile);
59 59
 		}
Please login to merge, or discard this patch.
lib/private/Contacts/ContactsMenu/Entry.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -117,7 +117,7 @@  discard block
 block discarded – undo
117 117
 	 * sort the actions by priority and name
118 118
 	 */
119 119
 	private function sortActions(): void {
120
-		usort($this->actions, function (IAction $action1, IAction $action2) {
120
+		usort($this->actions, function(IAction $action1, IAction $action2) {
121 121
 			$prio1 = $action1->getPriority();
122 122
 			$prio2 = $action2->getPriority();
123 123
 
@@ -154,7 +154,7 @@  discard block
 block discarded – undo
154 154
 	 */
155 155
 	public function jsonSerialize(): array {
156 156
 		$topAction = !empty($this->actions) ? $this->actions[0]->jsonSerialize() : null;
157
-		$otherActions = array_map(function (IAction $action) {
157
+		$otherActions = array_map(function(IAction $action) {
158 158
 			return $action->jsonSerialize();
159 159
 		}, array_slice($this->actions, 1));
160 160
 
Please login to merge, or discard this patch.
apps/settings/lib/AppInfo/Application.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -85,11 +85,11 @@  discard block
 block discarded – undo
85 85
 		 * Core class wrappers
86 86
 		 */
87 87
 		/** FIXME: Remove once OC_User is non-static and mockable */
88
-		$context->registerService('isAdmin', function () {
88
+		$context->registerService('isAdmin', function() {
89 89
 			return \OC_User::isAdminUser(\OC_User::getUser());
90 90
 		});
91 91
 		/** FIXME: Remove once OC_SubAdmin is non-static and mockable */
92
-		$context->registerService('isSubAdmin', function () {
92
+		$context->registerService('isSubAdmin', function() {
93 93
 			$userObject = \OC::$server->getUserSession()->getUser();
94 94
 			$isSubAdmin = false;
95 95
 			if ($userObject !== null) {
@@ -97,18 +97,18 @@  discard block
 block discarded – undo
97 97
 			}
98 98
 			return $isSubAdmin;
99 99
 		});
100
-		$context->registerService(IProvider::class, function (IAppContainer $appContainer) {
100
+		$context->registerService(IProvider::class, function(IAppContainer $appContainer) {
101 101
 			/** @var IServerContainer $serverContainer */
102 102
 			$serverContainer = $appContainer->query(IServerContainer::class);
103 103
 			return $serverContainer->query(IProvider::class);
104 104
 		});
105
-		$context->registerService(IManager::class, function (IAppContainer $appContainer) {
105
+		$context->registerService(IManager::class, function(IAppContainer $appContainer) {
106 106
 			/** @var IServerContainer $serverContainer */
107 107
 			$serverContainer = $appContainer->query(IServerContainer::class);
108 108
 			return $serverContainer->getSettingsManager();
109 109
 		});
110 110
 
111
-		$context->registerService(NewUserMailHelper::class, function (IAppContainer $appContainer) {
111
+		$context->registerService(NewUserMailHelper::class, function(IAppContainer $appContainer) {
112 112
 			/** @var Server $server */
113 113
 			$server = $appContainer->query(IServerContainer::class);
114 114
 			/** @var Defaults $defaults */
Please login to merge, or discard this patch.
apps/admin_audit/lib/Actions/UserManagement.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -62,8 +62,8 @@  discard block
 block discarded – undo
62 62
 	public function assign(string $uid): void {
63 63
 		$this->log(
64 64
 		'UserID assigned: "%s"',
65
-			[ 'uid' => $uid ],
66
-			[ 'uid' ]
65
+			['uid' => $uid],
66
+			['uid']
67 67
 		);
68 68
 	}
69 69
 
@@ -90,8 +90,8 @@  discard block
 block discarded – undo
90 90
 	public function unassign(string $uid): void {
91 91
 		$this->log(
92 92
 			'UserID unassigned: "%s"',
93
-			[ 'uid' => $uid ],
94
-			[ 'uid' ]
93
+			['uid' => $uid],
94
+			['uid']
95 95
 		);
96 96
 	}
97 97
 
Please login to merge, or discard this patch.
apps/provisioning_api/lib/AppInfo/Application.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
 	public function register(IRegistrationContext $context): void {
61 61
 		$context->registerEventListener(UserDeletedEvent::class, UserDeletedListener::class);
62 62
 
63
-		$context->registerService(NewUserMailHelper::class, function (ContainerInterface $c) {
63
+		$context->registerService(NewUserMailHelper::class, function(ContainerInterface $c) {
64 64
 			return new NewUserMailHelper(
65 65
 				$c->get(Defaults::class),
66 66
 				$c->get(IURLGenerator::class),
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
 				Util::getDefaultEmailAddress('no-reply')
74 74
 			);
75 75
 		});
76
-		$context->registerService(ProvisioningApiMiddleware::class, function (ContainerInterface $c) {
76
+		$context->registerService(ProvisioningApiMiddleware::class, function(ContainerInterface $c) {
77 77
 			$user = $c->get(IUserManager::class)->get($c->get('UserId'));
78 78
 			$isAdmin = false;
79 79
 			$isSubAdmin = false;
Please login to merge, or discard this patch.
apps/user_ldap/lib/Command/ResetUser.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -103,7 +103,7 @@
 block discarded – undo
103 103
 			if (isset($pluginManagerSuppressed)) {
104 104
 				$this->pluginManager->setSuppressDeletion($pluginManagerSuppressed);
105 105
 			}
106
-			$output->writeln('<error>' . $e->getMessage() . '</error>');
106
+			$output->writeln('<error>'.$e->getMessage().'</error>');
107 107
 			return 1;
108 108
 		}
109 109
 		$output->writeln('<error>Error while resetting user</error>');
Please login to merge, or discard this patch.
lib/private/Collaboration/Collaborators/UserPlugin.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -124,7 +124,7 @@  discard block
 block discarded – undo
124 124
 						}
125 125
 					}
126 126
 
127
-					uasort($users, function ($a, $b) {
127
+					uasort($users, function($a, $b) {
128 128
 						/**
129 129
 						 * @var \OC\User\User $a
130 130
 						 * @var \OC\User\User $b
@@ -169,7 +169,7 @@  discard block
 block discarded – undo
169 169
 					'message' => $userStatus->getMessage(),
170 170
 					'icon' => $userStatus->getIcon(),
171 171
 					'clearAt' => $userStatus->getClearAt()
172
-						? (int)$userStatus->getClearAt()->format('U')
172
+						? (int) $userStatus->getClearAt()->format('U')
173 173
 						: null,
174 174
 				];
175 175
 			}
@@ -253,7 +253,7 @@  discard block
 block discarded – undo
253 253
 							'message' => $userStatus->getMessage(),
254 254
 							'icon' => $userStatus->getIcon(),
255 255
 							'clearAt' => $userStatus->getClearAt()
256
-								? (int)$userStatus->getClearAt()->format('U')
256
+								? (int) $userStatus->getClearAt()->format('U')
257 257
 								: null,
258 258
 						];
259 259
 					}
Please login to merge, or discard this patch.
apps/files_sharing/lib/Command/CleanupRemoteStorages.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -69,11 +69,11 @@  discard block
 block discarded – undo
69 69
 	public function execute(InputInterface $input, OutputInterface $output): int {
70 70
 		$remoteStorages = $this->getRemoteStorages();
71 71
 
72
-		$output->writeln(count($remoteStorages) . ' remote storage(s) need(s) to be checked');
72
+		$output->writeln(count($remoteStorages).' remote storage(s) need(s) to be checked');
73 73
 
74 74
 		$remoteShareIds = $this->getRemoteShareIds();
75 75
 
76
-		$output->writeln(count($remoteShareIds) . ' remote share(s) exist');
76
+		$output->writeln(count($remoteShareIds).' remote share(s) exist');
77 77
 
78 78
 		foreach ($remoteShareIds as $id => $remoteShareId) {
79 79
 			if (isset($remoteStorages[$remoteShareId])) {
@@ -151,13 +151,13 @@  discard block
 block discarded – undo
151 151
 			->where($queryBuilder->expr()->like(
152 152
 				'id',
153 153
 				// match all 'shared::' + 32 characters storages
154
-				$queryBuilder->createNamedParameter($this->connection->escapeLikeParameter('shared::') . str_repeat('_', 32)),
154
+				$queryBuilder->createNamedParameter($this->connection->escapeLikeParameter('shared::').str_repeat('_', 32)),
155 155
 				IQueryBuilder::PARAM_STR)
156 156
 			)
157 157
 			->andWhere($queryBuilder->expr()->notLike(
158 158
 				'id',
159 159
 				// but not the ones starting with a '/', they are for normal shares
160
-				$queryBuilder->createNamedParameter($this->connection->escapeLikeParameter('shared::/') . '%'),
160
+				$queryBuilder->createNamedParameter($this->connection->escapeLikeParameter('shared::/').'%'),
161 161
 				IQueryBuilder::PARAM_STR)
162 162
 			)->orderBy('numeric_id');
163 163
 		$query = $queryBuilder->execute();
@@ -183,7 +183,7 @@  discard block
 block discarded – undo
183 183
 			$cloudId = $this->cloudIdManager->getCloudId($row['owner'], $row['remote']);
184 184
 			$remote = $cloudId->getRemote();
185 185
 
186
-			$remoteShareIds[$row['id']] = 'shared::' . md5($row['share_token'] . '@' . $remote);
186
+			$remoteShareIds[$row['id']] = 'shared::'.md5($row['share_token'].'@'.$remote);
187 187
 		}
188 188
 
189 189
 		return $remoteShareIds;
Please login to merge, or discard this patch.
lib/private/Encryption/Update.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
 			if ($params['itemType'] === 'file' || $params['itemType'] === 'folder') {
91 91
 				$path = Filesystem::getPath($params['fileSource']);
92 92
 				[$owner, $ownerPath] = $this->getOwnerPath($path);
93
-				$absPath = '/' . $owner . '/files/' . $ownerPath;
93
+				$absPath = '/'.$owner.'/files/'.$ownerPath;
94 94
 				$this->update($absPath);
95 95
 			}
96 96
 		}
@@ -106,7 +106,7 @@  discard block
 block discarded – undo
106 106
 			if ($params['itemType'] === 'file' || $params['itemType'] === 'folder') {
107 107
 				$path = Filesystem::getPath($params['fileSource']);
108 108
 				[$owner, $ownerPath] = $this->getOwnerPath($path);
109
-				$absPath = '/' . $owner . '/files/' . $ownerPath;
109
+				$absPath = '/'.$owner.'/files/'.$ownerPath;
110 110
 				$this->update($absPath);
111 111
 			}
112 112
 		}
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
 	 */
121 121
 	public function postRestore($params) {
122 122
 		if ($this->encryptionManager->isEnabled()) {
123
-			$path = Filesystem::normalizePath('/' . $this->uid . '/files/' . $params['filePath']);
123
+			$path = Filesystem::normalizePath('/'.$this->uid.'/files/'.$params['filePath']);
124 124
 			$this->update($path);
125 125
 		}
126 126
 	}
@@ -139,7 +139,7 @@  discard block
 block discarded – undo
139 139
 			dirname($source) !== dirname($target)
140 140
 		) {
141 141
 			[$owner, $ownerPath] = $this->getOwnerPath($target);
142
-			$absPath = '/' . $owner . '/files/' . $ownerPath;
142
+			$absPath = '/'.$owner.'/files/'.$ownerPath;
143 143
 			$this->update($absPath);
144 144
 		}
145 145
 	}
@@ -154,10 +154,10 @@  discard block
 block discarded – undo
154 154
 	protected function getOwnerPath($path) {
155 155
 		$info = Filesystem::getFileInfo($path);
156 156
 		$owner = Filesystem::getOwner($path);
157
-		$view = new View('/' . $owner . '/files');
157
+		$view = new View('/'.$owner.'/files');
158 158
 		$path = $view->getPath($info->getId());
159 159
 		if ($path === null) {
160
-			throw new InvalidArgumentException('No file found for ' . $info->getId());
160
+			throw new InvalidArgumentException('No file found for '.$info->getId());
161 161
 		}
162 162
 
163 163
 		return [$owner, $path];
@@ -193,7 +193,7 @@  discard block
 block discarded – undo
193 193
 				$encryptionModule->update($file, $this->uid, $usersSharing);
194 194
 			} catch (GenericEncryptionException $e) {
195 195
 				// If the update of an individual file fails e.g. due to a corrupt key we should continue the operation and just log the failure
196
-				$this->logger->error('Failed to update encryption module for ' . $this->uid . ' ' . $file, [ 'exception' => $e ]);
196
+				$this->logger->error('Failed to update encryption module for '.$this->uid.' '.$file, ['exception' => $e]);
197 197
 			}
198 198
 		}
199 199
 	}
Please login to merge, or discard this patch.