Allow the expiration date to be set to null

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
This commit is contained in:
Morris Jobke 2017-08-26 11:47:35 +02:00
parent 485dbc8d6c
commit d0bb5c298d
No known key found for this signature in database
GPG Key ID: FE03C3A163FEDE68
2 changed files with 4 additions and 2 deletions

View File

@ -369,14 +369,14 @@ class ShareByMailProvider implements IShareProvider {
* @param string $link
* @param string $initiator
* @param string $shareWith
* @param \DateTime $expiration
* @param \DateTime|null $expiration
* @throws \Exception If mail couldn't be sent
*/
protected function sendMailNotification($filename,
$link,
$initiator,
$shareWith,
\DateTime $expiration) {
$expiration) {
$initiatorUser = $this->userManager->get($initiator);
$initiatorDisplayName = ($initiatorUser instanceof IUser) ? $initiatorUser->getDisplayName() : $initiator;
$subject = (string)$this->l->t('%s shared »%s« with you', array($initiatorDisplayName, $filename));

View File

@ -868,6 +868,7 @@ class ShareByMailProviderTest extends TestCase {
'https://example.com/file.txt',
'OwnerUser',
'john@doe.com',
null,
]);
}
@ -968,6 +969,7 @@ class ShareByMailProviderTest extends TestCase {
'https://example.com/file.txt',
'InitiatorUser',
'john@doe.com',
null,
]);
}
}