nextcloud / circles

lib/Model/Federated/FederatedEvent.php

Doc Comments +1 added lines, -1 removed lines

@@ -111,7 +111,7 @@
 	}
 
 	/**
-	 * @param mixed $class
+	 * @param string $class
 	 *
 	 * @return self
 	 */

lib/Model/FederatedUser.php

Spacing +4 added lines, -4 removed lines

@@ -224,10 +224,10 @@
 	 * @return INC21QueryRow
 	 */
 	public function importFromDatabase(array $data, string $prefix = ''): INC21QueryRow {
-		$this->setSingleId($this->get($prefix . 'member_id', $data));
-		$this->setUserId($this->get($prefix . 'user_id', $data));
-		$this->setUserType($this->getInt($prefix . 'user_type', $data));
-		$this->setInstance($this->get($prefix . 'instance', $data));
+		$this->setSingleId($this->get($prefix.'member_id', $data));
+		$this->setUserId($this->get($prefix.'user_id', $data));
+		$this->setUserType($this->getInt($prefix.'user_type', $data));
+		$this->setInstance($this->get($prefix.'instance', $data));
 
 		return $this;
 	}

lib/Command/CirclesCreate.php

Spacing +1 added lines, -1 removed lines

@@ -117,7 +117,7 @@
 		$owner = $this->federatedUserService->createLocalFederatedUser($ownerId);
 		$circle = $this->circleService->create($name, $owner);
 
-		echo json_encode($circle, JSON_PRETTY_PRINT) . "\n";
+		echo json_encode($circle, JSON_PRETTY_PRINT)."\n";
 
 		return 0;
 	}

lib/FederatedItems/MemberAdd.php

Spacing +3 added lines, -3 removed lines

@@ -177,7 +177,7 @@ 
 			$this->sendMailExistingShares($template, $author, $recipient);
 			$this->sendPasswordExistingShares($author, $recipient, $password);
 		} catch (Exception $e) {
-			$this->miscService->log('Failed to send mail about existing share ' . $e->getMessage());
+			$this->miscService->log('Failed to send mail about existing share '.$e->getMessage());
 		}
 	}
 
@@ -327,7 +327,7 @@ 
 		$authorName = ($authorUser instanceof IUser) ? $authorUser->getDisplayName() : $author;
 		$authorEmail = ($authorUser instanceof IUser) ? $authorUser->getEMailAddress() : null;
 
-		$this->miscService->log("Sending password mail about existing files to '" . $email . "'", 0);
+		$this->miscService->log("Sending password mail about existing files to '".$email."'", 0);
 
 		$plainBodyPart = $this->l10n->t(
 			"%1\$s shared multiple files with you.\nYou should have already received a separate mail with a link to access them.\n",
@@ -369,7 +369,7 @@ 
 		$message->setFrom([\OCP\Util::getDefaultEmailAddress($instanceName) => $senderName]);
 		if ($authorEmail !== null) {
 			$message->setReplyTo([$authorEmail => $authorName]);
-			$emailTemplate->addFooter($instanceName . ' - ' . $this->defaults->getSlogan());
+			$emailTemplate->addFooter($instanceName.' - '.$this->defaults->getSlogan());
 		} else {
 			$emailTemplate->addFooter();
 		}

lib/Service/FederatedEventService.php

Spacing +3 added lines, -3 removed lines

@@ -238,16 +238,16 @@
 		try {
 			$test = new ReflectionClass($class);
 		} catch (ReflectionException $e) {
-			throw new FederatedEventException('ReflectionException with ' . $class . ': ' . $e->getMessage());
+			throw new FederatedEventException('ReflectionException with '.$class.': '.$e->getMessage());
 		}
 
 		if (!in_array(IFederatedItem::class, $test->getInterfaceNames())) {
-			throw new FederatedEventException($class . ' does not implements IFederatedItem');
+			throw new FederatedEventException($class.' does not implements IFederatedItem');
 		}
 
 		$item = OC::$server->get($class);
 		if (!($item instanceof IFederatedItem)) {
-			throw new FederatedEventException($class . ' not an IFederatedItem');
+			throw new FederatedEventException($class.' not an IFederatedItem');
 		}
 
 		$this->setFederatedEventBypass($event, $item);

lib/Command/MembersAdd.php

Spacing +3 added lines, -3 removed lines

@@ -119,12 +119,12 @@ 
 		$userType = $input->getOption('type');
 
 		//
-		$this->federatedUserService->commandLineInitiator($input->getOption('initiator'), $circleId,  false);
+		$this->federatedUserService->commandLineInitiator($input->getOption('initiator'), $circleId, false);
 
 		$member = $this->federatedUserService->createFederatedUser($userId, (int)$userType);
 		$outcome = $this->memberService->addMember($circleId, $member);
 
-		echo json_encode($outcome, JSON_PRETTY_PRINT) . "\n";
+		echo json_encode($outcome, JSON_PRETTY_PRINT)."\n";
 
 		return 0;
 	}
@@ -175,7 +175,7 @@ 
 				$instance = $host;
 			}
 
-			$result[] = $user['userid']['value'] . ' <info>@' . $host . '</info>';
+			$result[] = $user['userid']['value'].' <info>@'.$host.'</info>';
 		}
 
 //		if ($userId === '') {

lib/Model/Helpers/MemberHelper.php

Spacing +3 added lines, -3 removed lines

@@ -104,7 +104,7 @@ 
 		try {
 			$level = Member::parseLevelString($levelString);
 		} catch (MemberLevelException $e) {
-			throw new MemberHelperException('method ' . $levelString . ' not found');
+			throw new MemberHelperException('method '.$levelString.' not found');
 		}
 
 		$this->mustHaveLevelEqualOrAbove($level);
@@ -122,11 +122,11 @@ 
 		try {
 			$level = Member::parseLevelString($levelString);
 		} catch (MemberLevelException $e) {
-			throw new MemberHelperException('method ' . $levelString . ' not found');
+			throw new MemberHelperException('method '.$levelString.' not found');
 		}
 
 		if ($this->member->getLevel() >= $level) {
-			throw new MemberLevelException('Member cannot be ' . $levelString);
+			throw new MemberLevelException('Member cannot be '.$levelString);
 		}
 	}
 

lib/Command/MembersLevel.php

Spacing +1 added lines, -1 removed lines

@@ -108,7 +108,7 @@
 		$level = Member::parseLevelString($input->getArgument('level'));
 		$outcome = $this->memberService->memberLevel($memberId, $level);
 
-		echo json_encode($outcome, JSON_PRETTY_PRINT) . "\n";
+		echo json_encode($outcome, JSON_PRETTY_PRINT)."\n";
 
 		return 0;
 	}

lib/Command/MembersRemove.php

Spacing +1 added lines, -1 removed lines

@@ -106,7 +106,7 @@
 
 		$outcome = $this->memberService->removeMember($memberId);
 
-		echo json_encode($outcome, JSON_PRETTY_PRINT) . "\n";
+		echo json_encode($outcome, JSON_PRETTY_PRINT)."\n";
 
 		return 0;
 	}