| @@ -282,7 +282,7 @@ | ||
| 282 | 282 | * generateEvent() | 
| 283 | 283 | * Create an Activity Event with the basic settings for the app. | 
| 284 | 284 | * | 
| 285 | - * @param $type | |
| 285 | + * @param string $type | |
| 286 | 286 | * | 
| 287 | 287 | * @return \OCP\Activity\IEvent | 
| 288 | 288 | */ | 
| @@ -284,7 +284,7 @@ | ||
| 284 | 284 | ob_start(); | 
| 285 | 285 | echo(json_encode($result)); | 
| 286 | 286 | $size = ob_get_length(); | 
| 287 | -		header('Content-Length: ' . $size); | |
| 287 | +		header('Content-Length: '.$size); | |
| 288 | 288 | ob_end_flush(); | 
| 289 | 289 | flush(); | 
| 290 | 290 | } | 
| @@ -66,6 +66,9 @@ discard block | ||
| 66 | 66 | $this->userManager = $userManager; | 
| 67 | 67 | } | 
| 68 | 68 | |
| 69 | + /** | |
| 70 | + * @param string $message | |
| 71 | + */ | |
| 69 | 72 |  	public function log($message, $level = 4) { | 
| 70 | 73 | $data = array( | 
| 71 | 74 | 'app' => $this->appName, | 
| @@ -86,7 +89,7 @@ discard block | ||
| 86 | 89 | |
| 87 | 90 | /** | 
| 88 | 91 | * @param $arr | 
| 89 | - * @param $k | |
| 92 | + * @param string $k | |
| 90 | 93 | * | 
| 91 | 94 | * @param string $default | 
| 92 | 95 | * | 
| @@ -257,7 +260,7 @@ discard block | ||
| 257 | 260 | |
| 258 | 261 | |
| 259 | 262 | /** | 
| 260 | - * @param $ident | |
| 263 | + * @param string $ident | |
| 261 | 264 | * | 
| 262 | 265 | * @return mixed|string | 
| 263 | 266 | * @deprecated | 
| @@ -323,7 +323,7 @@ | ||
| 323 | 323 | return; | 
| 324 | 324 | } | 
| 325 | 325 | |
| 326 | - $tmpContact = $this->userId . ':' . $ident; | |
| 326 | + $tmpContact = $this->userId.':'.$ident; | |
| 327 | 327 | $result = MiscService::getContactData($tmpContact); | 
| 328 | 328 |  		if (empty($result)) { | 
| 329 | 329 |  			throw new NoUserException($this->l10n->t("This contact is not available")); | 
| @@ -349,8 +349,8 @@ discard block | ||
| 349 | 349 | /** | 
| 350 | 350 | * Verify if a local account is valid. | 
| 351 | 351 | * | 
| 352 | - * @param $ident | |
| 353 | - * @param $type | |
| 352 | + * @param string $ident | |
| 353 | + * @param integer $type | |
| 354 | 354 | * | 
| 355 | 355 | * @param string $instance | 
| 356 | 356 | * | 
| @@ -401,8 +401,8 @@ discard block | ||
| 401 | 401 | /** | 
| 402 | 402 | * Verify if a contact exist in current user address books. | 
| 403 | 403 | * | 
| 404 | - * @param $ident | |
| 405 | - * @param $type | |
| 404 | + * @param string $ident | |
| 405 | + * @param integer $type | |
| 406 | 406 | * | 
| 407 | 407 | * @throws NoUserException | 
| 408 | 408 | * @throws EmailAccountInvalidFormatException | 
| @@ -494,8 +494,8 @@ discard block | ||
| 494 | 494 | * Will return any data of a user related to a circle (as a Member). User can be a 'non-member' | 
| 495 | 495 | * Viewer needs to be at least Member of the Circle | 
| 496 | 496 | * | 
| 497 | - * @param $circleId | |
| 498 | - * @param $userId | |
| 497 | + * @param string $circleId | |
| 498 | + * @param string $userId | |
| 499 | 499 | * @param $type | 
| 500 | 500 | * @param bool $forceAll | 
| 501 | 501 | * | 
| @@ -551,7 +551,7 @@ discard block | ||
| 551 | 551 | * @param int $level | 
| 552 | 552 | * @param bool $force | 
| 553 | 553 | * | 
| 554 | - * @return array | |
| 554 | + * @return DeprecatedMember[] | |
| 555 | 555 | * @throws CircleDoesNotExistException | 
| 556 | 556 | * @throws CircleTypeNotValidException | 
| 557 | 557 | * @throws ConfigNoCircleAvailableException | 
| @@ -146,7 +146,7 @@ | ||
| 146 | 146 | * @return string | 
| 147 | 147 | */ | 
| 148 | 148 |  	public function getUniqueId(): string { | 
| 149 | - return $this->getAddressBookId() . '/' . $this->getCardUri(); | |
| 149 | + return $this->getAddressBookId().'/'.$this->getCardUri(); | |
| 150 | 150 | } | 
| 151 | 151 | |
| 152 | 152 | |
| @@ -94,7 +94,7 @@ | ||
| 94 | 94 | } | 
| 95 | 95 | |
| 96 | 96 | /** | 
| 97 | - * @param mixed $type | |
| 97 | + * @param string $type | |
| 98 | 98 | * | 
| 99 | 99 | * @return GSEvent | 
| 100 | 100 | */ | 
| @@ -109,7 +109,7 @@ | ||
| 109 | 109 | * @return string | 
| 110 | 110 | */ | 
| 111 | 111 |  	protected function stripPath($path) { | 
| 112 | - $prefix = '/' . $this->userId . '/files'; | |
| 112 | + $prefix = '/'.$this->userId.'/files'; | |
| 113 | 113 | |
| 114 | 114 | return rtrim(substr($path, strlen($prefix)), '/'); | 
| 115 | 115 | } | 
| @@ -154,7 +154,7 @@ discard block | ||
| 154 | 154 | return $mountPoint; | 
| 155 | 155 | } | 
| 156 | 156 | |
| 157 | - return '/' . $userId . '/files/' . ltrim($mountPoint, '/'); | |
| 157 | + return '/'.$userId.'/files/'.ltrim($mountPoint, '/'); | |
| 158 | 158 | } | 
| 159 | 159 | |
| 160 | 160 | /** | 
| @@ -293,7 +293,7 @@ discard block | ||
| 293 | 293 |  	public function toMount(string $userId, string $protocol = 'https'): array { | 
| 294 | 294 | return [ | 
| 295 | 295 | 'owner' => $this->getOwner(), | 
| 296 | - 'remote' => $protocol . '://' . $this->getInstance(), | |
| 296 | + 'remote' => $protocol.'://'.$this->getInstance(), | |
| 297 | 297 | 'token' => $this->getToken(), | 
| 298 | 298 | 'share_token' => $this->getToken(), | 
| 299 | 299 | 'password' => $this->getPassword(), | 
| @@ -121,11 +121,11 @@ | ||
| 121 | 121 | |
| 122 | 122 |  	private function leftJoinMountPoint(IQueryBuilder $qb, string $userId) { | 
| 123 | 123 | $expr = $qb->expr(); | 
| 124 | - $pf = '' . $this->default_select_alias . '.'; | |
| 124 | + $pf = ''.$this->default_select_alias.'.'; | |
| 125 | 125 | |
| 126 | 126 | $on = $expr->andX(); | 
| 127 | 127 |  		$on->add($expr->eq('mp.user_id', $qb->createNamedParameter($userId))); | 
| 128 | -		$on->add($expr->eq('mp.share_id', $pf . 'id')); | |
| 128 | +		$on->add($expr->eq('mp.share_id', $pf.'id')); | |
| 129 | 129 | |
| 130 | 130 | /** @noinspection PhpMethodParametersCountMismatchInspection */ | 
| 131 | 131 |  		$qb->selectAlias('mp.mountPoint', 'gsshares_mountpoint') | 
| @@ -148,13 +148,13 @@ | ||
| 148 | 148 | } | 
| 149 | 149 | |
| 150 | 150 | $info = pathinfo($share->getMountPoint()); | 
| 151 | -		$filename = $this->get('dirname', $info) . '/' . $this->get('filename', $info); | |
| 151 | +		$filename = $this->get('dirname', $info).'/'.$this->get('filename', $info); | |
| 152 | 152 |  		$extension = $this->get('extension', $info); | 
| 153 | - $extension = ($extension === '') ? '' : '.' . $extension; | |
| 153 | + $extension = ($extension === '') ? '' : '.'.$extension; | |
| 154 | 154 | |
| 155 | 155 | $n = 2; | 
| 156 | 156 |  		while (true) { | 
| 157 | - $path = $filename . " ($n)" . $extension; | |
| 157 | + $path = $filename." ($n)".$extension; | |
| 158 | 158 |  			try { | 
| 159 | 159 | $fs->get($path); | 
| 160 | 160 |  			} catch (NotFoundException $e) { |