Merge pull request #1394 from nextcloud/fix-share-notification-10

Fix MissingMandatoryParametersException for share notifications
nextcloud · Sep 13, 2016 · 43bff15 · 43bff15
2 parents 14ea159 + 107b418
commit 43bff15
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/lib/private/Share/MailNotifications.php b/lib/private/Share/MailNotifications.php
@@ -123,7 +123,7 @@ public function sendInternalShareMail($recipientList, $itemSource, $itemType) {
 
 			$link = $this->urlGenerator->linkToRouteAbsolute(
 				'files.viewcontroller.showFile',
-				['fileId' => $items[0]['item_source']]
+				['fileid' => $items[0]['item_source']]
 			);
 
 			list($htmlBody, $textBody) = $this->createMailBody($filename, $link, $expiration, 'internal');

diff --git a/tests/lib/Share/MailNotificationsTest.php b/tests/lib/Share/MailNotificationsTest.php
@@ -241,10 +241,8 @@ public function testSendInternalShareMail() {
 		$this->urlGenerator->expects($this->once())
 			->method('linkToRouteAbsolute')
 			->with(
-				$this->equalTo('files.viewcontroller.showFile'),
-				$this->equalTo([
-					'fileId' => 123,
-				])
+				'files.viewcontroller.showFile',
+				['fileid' => 123]
 			);
 
 		$recipientList = [$recipient];