nextcloud / circles

lib/Events/UserEvents.php

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
 		try {
 			$this->membersService->onUserRemoved($userId);
 		} catch (\Exception $e) {
-			$this->miscService->log('exception while onUserDeleted ' . $e->getMessage());
+			$this->miscService->log('exception while onUserDeleted '.$e->getMessage());
 		}
 	}
 

lib/GlobalScale/GSMount/MountProvider.php

Spacing +3 added lines, -3 removed lines

@@ -148,13 +148,13 @@
 		}
 
 		$info = pathinfo($share->getMountPoint());
-		$filename = $this->get('dirname', $info) . '/' . $this->get('filename', $info);
+		$filename = $this->get('dirname', $info).'/'.$this->get('filename', $info);
 		$extension = $this->get('extension', $info);
-		$extension = ($extension === '') ? '' : '.' . $extension;
+		$extension = ($extension === '') ? '' : '.'.$extension;
 
 		$n = 2;
 		while (true) {
-			$path = $filename . " ($n)" . $extension;
+			$path = $filename." ($n)".$extension;
 			try {
 				$fs->get($path);
 			} catch (NotFoundException $e) {

lib/GlobalScale/FileShare.php

Doc Comments +3 added lines, -3 removed lines

@@ -257,7 +257,7 @@ 
 	 * @param $fileName
 	 * @param string $link
 	 * @param string $author
-	 * @param $circleName
+	 * @param string $circleName
 	 * @param string $email
 	 *
 	 * @throws Exception
@@ -294,7 +294,7 @@ 
 	 * @param string $circleName
 	 * @param string $email
 	 *
-	 * @param $password
+	 * @param string $password
 	 *
 	 * @throws NotFoundException
 	 * @throws Exception
@@ -371,7 +371,7 @@ 
 	 * @param $subject
 	 * @param $text
 	 * @param $fileName
-	 * @param $link
+	 * @param string $link
 	 * @param string $author
 	 * @param string $circleName
 	 *

Spacing +8 added lines, -8 removed lines

@@ -105,14 +105,14 @@ 
 			$node = $share->getNode();
 			$filename = $node->getName();
 		} catch (NotFoundException $e) {
-			$this->miscService->log('issue while FileShare: ' . $e->getMessage());
+			$this->miscService->log('issue while FileShare: '.$e->getMessage());
 
 			return;
 		}
 
 		$event->getData()
 			  ->s('gs_federated', $share->getToken())
-			  ->s('gs_filename', '/' . $filename);
+			  ->s('gs_filename', '/'.$filename);
 	}
 
 
@@ -301,7 +301,7 @@ 
 			);
 		} catch (Exception $e) {
 			OC::$server->getLogger()
-					   ->log(1, 'Circles::sharedByMail - mail were not sent: ' . $e->getMessage());
+					   ->log(1, 'Circles::sharedByMail - mail were not sent: '.$e->getMessage());
 		}
 	}
 
@@ -319,8 +319,8 @@ 
 		$message = $this->mailer->createMessage();
 
 		$this->miscService->log(
-			"Sending mail to circle '" . $circleName . "': " . $email . ' file: ' . $fileName
-			. ' - link: ' . $link, 0
+			"Sending mail to circle '".$circleName."': ".$email.' file: '.$fileName
+			. ' - link: '.$link, 0
 		);
 
 		$subject = $this->l10n->t('%s shared »%s« with you.', [$author, $fileName]);
@@ -359,7 +359,7 @@ 
 
 		$message = $this->mailer->createMessage();
 
-		$this->miscService->log("Sending password mail to circle '" . $circleName . "': " . $email, 0);
+		$this->miscService->log("Sending password mail to circle '".$circleName."': ".$email, 0);
 
 		$filename = $share->getNode()
 						  ->getName();
@@ -414,7 +414,7 @@ 
 		$message->setFrom([\OCP\Util::getDefaultEmailAddress($instanceName) => $senderName]);
 		if ($initiatorEmailAddress !== null) {
 			$message->setReplyTo([$initiatorEmailAddress => $initiatorDisplayName]);
-			$emailTemplate->addFooter($instanceName . ' - ' . $this->defaults->getSlogan());
+			$emailTemplate->addFooter($instanceName.' - '.$this->defaults->getSlogan());
 		} else {
 			$emailTemplate->addFooter();
 		}
@@ -449,7 +449,7 @@ 
 		$emailTemplate->addHeader();
 		$emailTemplate->addHeading($subject, false);
 		$emailTemplate->addBodyText(
-			htmlspecialchars($text) . '<br>' . htmlspecialchars(
+			htmlspecialchars($text).'<br>'.htmlspecialchars(
 				$this->l10n->t('Click the button below to open it.')
 			), $text
 		);

lib/Service/GSDownstreamService.php

Spacing +1 added lines, -1 removed lines

@@ -155,7 +155,7 @@
 			$gs = $this->globalScaleService->getGlobalScaleEvent($event);
 			$gs->manage($event);
 		} catch (Exception $e) {
-			$this->miscService->log('issue onNewEvent: ' . json_encode($event) . ' - ' . $e->getMessage());
+			$this->miscService->log('issue onNewEvent: '.json_encode($event).' - '.$e->getMessage());
 		}
 	}
 

lib/Command/SyncContact.php

Spacing +5 added lines, -5 removed lines

@@ -119,9 +119,9 @@ 
 		$tCards = $tBooks = 0;
 		$knownBooks = [];
 		foreach ($users as $user) {
-			$books = $this->cardDavBackend->getAddressBooksForUser('principals/users/' . $user->getUID());
+			$books = $this->cardDavBackend->getAddressBooksForUser('principals/users/'.$user->getUID());
 			$output->writeln(
-				'- User <info>' . $user->getUID() . '</info> have ' . sizeof($books) . ' address books:'
+				'- User <info>'.$user->getUID().'</info> have '.sizeof($books).' address books:'
 			);
 
 			$tBooks += sizeof($books);
@@ -137,11 +137,11 @@ 
 
 				$shared = '';
 				if ($owner !== $user->getUID()) {
-					$shared = ' (shared by <info>' . $owner . '</info>)';
+					$shared = ' (shared by <info>'.$owner.'</info>)';
 				}
 
 				$output->writeln(
-					'  <comment>*</comment> book #' . $bookId . $shared . ' contains '
+					'  <comment>*</comment> book #'.$bookId.$shared.' contains '
 					. sizeof($cards)
 					. ' entries'
 				);
@@ -151,7 +151,7 @@ 
 		}
 
 		$output->writeln('');
-		$output->writeln('with a total of ' . $tBooks . ' address books and ' . $tCards . ' contact entries');
+		$output->writeln('with a total of '.$tBooks.' address books and '.$tCards.' contact entries');
 	}
 
 

lib/Search/LocalUsers.php

Doc Comments +1 added lines, -1 removed lines

@@ -83,7 +83,7 @@
 
 
 	/**
-	 * @param $search
+	 * @param string $search
 	 *
 	 * @return array
 	 */

lib/Db/CirclesRequest.php

Spacing +1 added lines, -1 removed lines

@@ -203,7 +203,7 @@
 		$cursor->closeCursor();
 
 		if ($data === false) {
-			throw new CircleDoesNotExistException($this->l10n->t('Circle not found ' . $circleUniqueId));
+			throw new CircleDoesNotExistException($this->l10n->t('Circle not found '.$circleUniqueId));
 		}
 
 		$circle = $this->parseCirclesSelectSql($data);

lib/Command/CirclesTest.php

Spacing +1 added lines, -1 removed lines

@@ -92,7 +92,7 @@
 	 */
 	protected function execute(InputInterface $input, OutputInterface $output): int {
 		$instances = $this->globalScaleService->getInstances(true);
-		$output->writeln('<info>Instances: </info>' . json_encode($instances));
+		$output->writeln('<info>Instances: </info>'.json_encode($instances));
 
 		return 0;
 	}

lib/Db/CircleProviderRequestBuilder.php

Doc Comments +1 added lines, -1 removed lines

@@ -114,7 +114,7 @@
 	 * limit the request to the children of a share
 	 *
 	 * @param IQueryBuilder $qb
-	 * @param $userId
+	 * @param string $userId
 	 * @param int $parentId
 	 */
 	protected function limitToShareChildren(IQueryBuilder $qb, $userId, $parentId = -1) {

Spacing +18 added lines, -18 removed lines

@@ -77,7 +77,7 @@ 
 		$pf = ($qb->getType() === QueryBuilder::SELECT) ? 's.' : '';
 		$qb->andWhere(
 			$expr->in(
-				$pf . 'share_with',
+				$pf.'share_with',
 				$qb->createNamedParameter($circleUniqueIds, IQueryBuilder::PARAM_STR_ARRAY)
 			)
 		);
@@ -94,7 +94,7 @@ 
 		$expr = $qb->expr();
 		$pf = ($qb->getType() === QueryBuilder::SELECT) ? 's.' : '';
 
-		$qb->andWhere($expr->eq($pf . 'id', $qb->createNamedParameter($shareId)));
+		$qb->andWhere($expr->eq($pf.'id', $qb->createNamedParameter($shareId)));
 	}
 
 
@@ -142,8 +142,8 @@ 
 
 		$qb->andWhere(
 			$expr->orX(
-				$expr->eq($pf . 'parent', $qb->createNamedParameter($circleId)),
-				$expr->eq($pf . 'id', $qb->createNamedParameter($circleId))
+				$expr->eq($pf.'parent', $qb->createNamedParameter($circleId)),
+				$expr->eq($pf.'id', $qb->createNamedParameter($circleId))
 			)
 		);
 	}
@@ -165,7 +165,7 @@ 
 		$pf = ($qb->getType() === QueryBuilder::SELECT) ? 's.' : '';
 		$qb->andWhere(
 			$expr->in(
-				$pf . 'file_source',
+				$pf.'file_source',
 				$qb->createNamedParameter($files, IQueryBuilder::PARAM_INT_ARRAY)
 			)
 		);
@@ -198,12 +198,12 @@ 
 		$pf = ($qb->getType() === QueryBuilder::SELECT) ? 's.' : '';
 
 		if ($reShares === false) {
-			$qb->andWhere($expr->eq($pf . 'uid_initiator', $qb->createNamedParameter($userId)));
+			$qb->andWhere($expr->eq($pf.'uid_initiator', $qb->createNamedParameter($userId)));
 		} else {
 			$qb->andWhere(
 				$expr->orX(
-					$expr->eq($pf . 'uid_owner', $qb->createNamedParameter($userId)),
-					$expr->eq($pf . 'uid_initiator', $qb->createNamedParameter($userId))
+					$expr->eq($pf.'uid_owner', $qb->createNamedParameter($userId)),
+					$expr->eq($pf.'uid_initiator', $qb->createNamedParameter($userId))
 				)
 			);
 		}
@@ -298,11 +298,11 @@ 
 		$expr = $qb->expr();
 		$andX = $expr->andX();
 
-		$andX->add($expr->eq($aliasM . '.user_id', $qb->createNamedParameter($userId)));
-		$andX->add($expr->eq($aliasM . '.circle_id', $aliasC . '.unique_id'));
-		$andX->add($expr->gte($aliasM . '.level', $qb->createNamedParameter(Member::LEVEL_MEMBER)));
-		$andX->add($expr->eq($aliasM . '.instance', $qb->createNamedParameter('')));
-		$andX->add($expr->eq($aliasM . '.user_type', $qb->createNamedParameter(Member::TYPE_USER)));
+		$andX->add($expr->eq($aliasM.'.user_id', $qb->createNamedParameter($userId)));
+		$andX->add($expr->eq($aliasM.'.circle_id', $aliasC.'.unique_id'));
+		$andX->add($expr->gte($aliasM.'.level', $qb->createNamedParameter(Member::LEVEL_MEMBER)));
+		$andX->add($expr->eq($aliasM.'.instance', $qb->createNamedParameter('')));
+		$andX->add($expr->eq($aliasM.'.user_type', $qb->createNamedParameter(Member::TYPE_USER)));
 
 		return $andX;
 	}
@@ -330,11 +330,11 @@ 
 
 		$andX = $expr->andX();
 
-		$andX->add($expr->eq($aliasM . '.user_id', 'ncgu.gid'));
-		$andX->add($expr->eq($aliasM . '.user_type', $qb->createNamedParameter(Member::TYPE_GROUP)));
-		$andX->add($expr->eq($aliasM . '.instance', $qb->createNamedParameter('')));
-		$andX->add($expr->eq($aliasM . '.circle_id', $aliasC . '.unique_id'));
-		$andX->add($expr->gte($aliasM . '.level', $qb->createNamedParameter(Member::LEVEL_MEMBER)));
+		$andX->add($expr->eq($aliasM.'.user_id', 'ncgu.gid'));
+		$andX->add($expr->eq($aliasM.'.user_type', $qb->createNamedParameter(Member::TYPE_GROUP)));
+		$andX->add($expr->eq($aliasM.'.instance', $qb->createNamedParameter('')));
+		$andX->add($expr->eq($aliasM.'.circle_id', $aliasC.'.unique_id'));
+		$andX->add($expr->gte($aliasM.'.level', $qb->createNamedParameter(Member::LEVEL_MEMBER)));
 
 		return $andX;
 	}