Fix MissingMandatoryParametersException for share notifications

The routing system is non-forgiving

diff --git a/lib/private/Share/MailNotifications.php b/lib/private/Share/MailNotifications.php
index aaecd5353e5930bb33b24e2809e1f299d8dcece9..3d4301a2bdd570b31f4fe9adf114081db1875c2b 100644 (file)
--- a/lib/private/Share/MailNotifications.php
+++ b/lib/private/Share/MailNotifications.php
@@ -123,7 +123,7 @@ class MailNotifications {
 
                        $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
index 0c951d11f88e8d4a85605a234393ddd54e0375da..d1c2a17cb9ede13f0074b40e33ee20c3b148b448 100644 (file)
--- a/tests/lib/Share/MailNotificationsTest.php
+++ b/tests/lib/Share/MailNotificationsTest.php
@@ -241,10 +241,8 @@ class MailNotificationsTest extends \Test\TestCase {
                $this->urlGenerator->expects($this->once())
                        ->method('linkToRouteAbsolute')
                        ->with(
-                               $this->equalTo('files.viewcontroller.showFile'),
-                               $this->equalTo([
-                                       'fileId' => 123,
-                               ])
+                               'files.viewcontroller.showFile',
+                               ['fileid' => 123]
                        );
 
                $recipientList = [$recipient];