Completed
Pull Request — master (#2050)
by
unknown
34s
created
lib/ShareByCircleProvider.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -287,7 +287,7 @@  discard block
 block discarded – undo
287 287
 		// also the default footer contains a "Do not reply" which needs to be adjusted.
288 288
 		if ($initiatorEmail !== null) {
289 289
 			$message->setReplyTo([$initiatorEmail => $initiatorDisplayName]);
290
-			$emailTemplate->addFooter($instanceName . ($this->defaults->getSlogan() !== '' ? ' - ' . $this->defaults->getSlogan() : ''));
290
+			$emailTemplate->addFooter($instanceName.($this->defaults->getSlogan() !== '' ? ' - '.$this->defaults->getSlogan() : ''));
291 291
 		} else {
292 292
 			$emailTemplate->addFooter();
293 293
 		}
@@ -295,7 +295,7 @@  discard block
 block discarded – undo
295 295
 		$message->useTemplate($emailTemplate);
296 296
 		$failedRecipients = $this->mailer->send($message);
297 297
 		if (!empty($failedRecipients)) {
298
-			$this->logger->error('Share notification mail could not be sent to: ' . implode(', ', $failedRecipients));
298
+			$this->logger->error('Share notification mail could not be sent to: '.implode(', ', $failedRecipients));
299 299
 			return;
300 300
 		}
301 301
 	}
@@ -551,7 +551,7 @@  discard block
 block discarded – undo
551 551
 
552 552
 		return array_filter(
553 553
 			array_map(
554
-				function (ShareWrapper $wrapper) {
554
+				function(ShareWrapper $wrapper) {
555 555
 					return $wrapper->getShare($this->rootFolder, $this->userManager, $this->urlGenerator);
556 556
 				}, $wrappedShares
557 557
 			)
@@ -603,7 +603,7 @@  discard block
 block discarded – undo
603 603
 
604 604
 		return array_filter(
605 605
 			array_map(
606
-				function (ShareWrapper $wrapper) {
606
+				function(ShareWrapper $wrapper) {
607 607
 					return $wrapper->getShare($this->rootFolder, $this->userManager, $this->urlGenerator);
608 608
 				}, $wrappedShares
609 609
 			)
@@ -652,7 +652,7 @@  discard block
 block discarded – undo
652 652
 
653 653
 		return array_filter(
654 654
 			array_map(
655
-				function (ShareWrapper $wrapper) {
655
+				function(ShareWrapper $wrapper) {
656 656
 					return $wrapper->getShare(
657 657
 						$this->rootFolder, $this->userManager, $this->urlGenerator, true
658 658
 					);
@@ -683,7 +683,7 @@  discard block
 block discarded – undo
683 683
 
684 684
 		$share = $wrappedShare->getShare($this->rootFolder, $this->userManager, $this->urlGenerator);
685 685
 		if ($share->getPassword() !== '') {
686
-			$this->logger->notice('share is protected by a password, hash: ' . $share->getPassword());
686
+			$this->logger->notice('share is protected by a password, hash: '.$share->getPassword());
687 687
 		}
688 688
 
689 689
 		return $share;
@@ -793,7 +793,7 @@  discard block
 block discarded – undo
793 793
 							];
794 794
 						} else {
795 795
 							// we only store temp value, as token is unknown at this point
796
-							$remote[$member->getUserid() . '@' . $member->getInstance()] = [
796
+							$remote[$member->getUserid().'@'.$member->getInstance()] = [
797 797
 								'node_id' => $share->getFileSource(),
798 798
 								'shareId' => $share->getId(),
799 799
 								'memberId' => $member->getId(),
Please login to merge, or discard this patch.
tests/unit/lib/ShareByCircleProviderTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -82,7 +82,7 @@  discard block
 block discarded – undo
82 82
 		]);
83 83
 
84 84
 		$userManager = $this->createMock(IUserManager::class);
85
-		$userManager->method('get')->willReturnCallback(function ($uid) use (
85
+		$userManager->method('get')->willReturnCallback(function($uid) use (
86 86
 			$initiatorUser,
87 87
 			$validUser,
88 88
 			$invalidEmailUser,
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
97 97
 		});
98 98
 
99 99
 		$mailer = $this->createMock(IMailer::class);
100
-		$mailer->method('validateMailAddress')->willReturnCallback(function ($email) {
100
+		$mailer->method('validateMailAddress')->willReturnCallback(function($email) {
101 101
 			return $email === '[email protected]';
102 102
 		});
103 103
 
@@ -179,7 +179,7 @@  discard block
 block discarded – undo
179 179
 		]);
180 180
 
181 181
 		$userManager = $this->createMock(IUserManager::class);
182
-		$userManager->method('get')->willReturnCallback(function ($uid) use ($initiatorUser) {
182
+		$userManager->method('get')->willReturnCallback(function($uid) use ($initiatorUser) {
183 183
 			if ($uid === 'initiator') {
184 184
 				return $initiatorUser;
185 185
 			}
Please login to merge, or discard this patch.