From 0f8c1b13a39541fecdb149f41e9904d89a60ea64 Mon Sep 17 00:00:00 2001
From: Joas Schilling <coding@schilljs.com>
Date: Tue, 13 Jun 2017 18:44:14 +0200
Subject: [PATCH] Fix unknown share token

Signed-off-by: Joas Schilling <coding@schilljs.com>
---
 lib/private/Share20/Manager.php   | 4 ++--
 tests/lib/Share20/ManagerTest.php | 5 +++++
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/lib/private/Share20/Manager.php b/lib/private/Share20/Manager.php
index 06d03f2ec7..57ba39d41f 100644
--- a/lib/private/Share20/Manager.php
+++ b/lib/private/Share20/Manager.php
@@ -1090,7 +1090,7 @@ class Manager implements IManager {
 		}
 
 		if ($share === null) {
-			throw new ShareNotFound();
+			throw new ShareNotFound($this->l->t('The requested share does not exist anymore'));
 		}
 
 		$this->checkExpireDate($share);
@@ -1110,7 +1110,7 @@ class Manager implements IManager {
 		if ($share->getExpirationDate() !== null &&
 			$share->getExpirationDate() <= new \DateTime()) {
 			$this->deleteShare($share);
-			throw new ShareNotFound();
+			throw new ShareNotFound($this->l->t('The requested share does not exist anymore'));
 		}
 
 	}
diff --git a/tests/lib/Share20/ManagerTest.php b/tests/lib/Share20/ManagerTest.php
index 13556285b6..79af2f60ce 100644
--- a/tests/lib/Share20/ManagerTest.php
+++ b/tests/lib/Share20/ManagerTest.php
@@ -2142,6 +2142,7 @@ class ManagerTest extends \Test\TestCase {
 
 	/**
 	 * @expectedException \OCP\Share\Exceptions\ShareNotFound
+	 * @expectedExceptionMessage The requested share does not exist anymore
 	 */
 	public function testGetShareByTokenExpired() {
 		$this->config
@@ -2150,6 +2151,10 @@ class ManagerTest extends \Test\TestCase {
 			->with('core', 'shareapi_allow_links', 'yes')
 			->willReturn('yes');
 
+		$this->l->expects($this->once())
+			->method('t')
+			->willReturnArgument(0);
+
 		$manager = $this->createManagerMock()
 			->setMethods(['deleteShare'])
 			->getMock();