Passed
Push — master ( 0da822...e25a62 )
by Morris
13:45 queued 10s
created
core/Command/User/Setting.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -127,7 +127,7 @@  discard block
 block discarded – undo
127 127
 	protected function checkInput(InputInterface $input) {
128 128
 		$uid = $input->getArgument('uid');
129 129
 		if (!$input->getOption('ignore-missing-user') && !$this->userManager->userExists($uid)) {
130
-			throw new \InvalidArgumentException('The user "' . $uid . '" does not exist.');
130
+			throw new \InvalidArgumentException('The user "'.$uid.'" does not exist.');
131 131
 		}
132 132
 
133 133
 		if ($input->getArgument('key') === '' && $input->hasParameterOption('--default-value')) {
@@ -162,7 +162,7 @@  discard block
 block discarded – undo
162 162
 		try {
163 163
 			$this->checkInput($input);
164 164
 		} catch (\InvalidArgumentException $e) {
165
-			$output->writeln('<error>' . $e->getMessage() . '</error>');
165
+			$output->writeln('<error>'.$e->getMessage().'</error>');
166 166
 			return 1;
167 167
 		}
168 168
 
@@ -174,11 +174,11 @@  discard block
 block discarded – undo
174 174
 			$value = $this->config->getUserValue($uid, $app, $key, null);
175 175
 			if ($input->getArgument('value') !== null) {
176 176
 				if ($input->hasParameterOption('--update-only') && $value === null) {
177
-					$output->writeln('<error>The setting does not exist for user "' . $uid . '".</error>');
177
+					$output->writeln('<error>The setting does not exist for user "'.$uid.'".</error>');
178 178
 					return 1;
179 179
 				}
180 180
 
181
-				if ($app === 'settings' && in_array($key , ['email', 'display_name'])) {
181
+				if ($app === 'settings' && in_array($key, ['email', 'display_name'])) {
182 182
 					$user = $this->userManager->get($uid);
183 183
 					if ($user instanceof IUser) {
184 184
 						if ($key === 'email') {
@@ -204,11 +204,11 @@  discard block
 block discarded – undo
204 204
 				return 0;
205 205
 			} elseif ($input->hasParameterOption('--delete')) {
206 206
 				if ($input->hasParameterOption('--error-if-not-exists') && $value === null) {
207
-					$output->writeln('<error>The setting does not exist for user "' . $uid . '".</error>');
207
+					$output->writeln('<error>The setting does not exist for user "'.$uid.'".</error>');
208 208
 					return 1;
209 209
 				}
210 210
 
211
-				if ($app === 'settings' && in_array($key , ['email', 'display_name'])) {
211
+				if ($app === 'settings' && in_array($key, ['email', 'display_name'])) {
212 212
 					$user = $this->userManager->get($uid);
213 213
 					if ($user instanceof IUser) {
214 214
 						if ($key === 'email') {
@@ -236,7 +236,7 @@  discard block
 block discarded – undo
236 236
 					$output->writeln($user->getDisplayName());
237 237
 					return 0;
238 238
 				}
239
-				$output->writeln('<error>The setting does not exist for user "' . $uid . '".</error>');
239
+				$output->writeln('<error>The setting does not exist for user "'.$uid.'".</error>');
240 240
 				return 1;
241 241
 			}
242 242
 		} else {
Please login to merge, or discard this patch.