diff options
| author | Joas Schilling <coding@schilljs.com> | 2017-06-13 18:44:14 +0200 |
|---|---|---|
| committer | Joas Schilling <coding@schilljs.com> | 2017-06-13 18:53:52 +0200 |
| commit | 49c71bdbcb0d10affec4066c822a619f29fa608a (patch) | |
| tree | faf90bfdb9f109ce0973b101d14ecc7bcfa20f68 | |
| parent | 4c56414cabc195031cf5dbec00e2b28a67237f85 (diff) | |
| download | nextcloud-server-49c71bdbcb0d10affec4066c822a619f29fa608a.tar.gz nextcloud-server-49c71bdbcb0d10affec4066c822a619f29fa608a.zip | |
Fix unknown share token
Signed-off-by: Joas Schilling <coding@schilljs.com>
| -rw-r--r-- | lib/private/Share20/Manager.php | 4 | ||||
| -rw-r--r-- | 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 06d03f2ec7c..57ba39d41f6 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 13556285b61..79af2f60ce8 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(); |