From a7fb71f908f081d941981a62ed80e8bf58c15c07 Mon Sep 17 00:00:00 2001 From: Roeland Jago Douma Date: Fri, 22 Mar 2019 09:46:41 +0100 Subject: [PATCH] Fetch proper translations Fixes #14793 This is caused by the mess we have with OC\Settings mapping to settings and lib/private/Settings. Anyway this is the quick fix. Moving stuff around for 17 seems better. Signed-off-by: Roeland Jago Douma --- lib/private/Settings/Admin/Sharing.php | 5 +++-- tests/lib/Settings/Admin/SharingTest.php | 8 +++++++- 2 files changed, 10 insertions(+), 3 deletions(-) diff --git a/lib/private/Settings/Admin/Sharing.php b/lib/private/Settings/Admin/Sharing.php index ee6a64c85b..a044d314b0 100644 --- a/lib/private/Settings/Admin/Sharing.php +++ b/lib/private/Settings/Admin/Sharing.php @@ -31,6 +31,7 @@ use OCP\AppFramework\Http\TemplateResponse; use OCP\Constants; use OCP\IConfig; use OCP\IL10N; +use OCP\L10N\IFactory; use OCP\Settings\ISettings; use OCP\Share\IManager; use OCP\Util; @@ -48,9 +49,9 @@ class Sharing implements ISettings { /** * @param IConfig $config */ - public function __construct(IConfig $config, IL10N $l, IManager $shareManager) { + public function __construct(IConfig $config, IFactory $l, IManager $shareManager) { $this->config = $config; - $this->l = $l; + $this->l = $l->get('lib'); $this->shareManager = $shareManager; } diff --git a/tests/lib/Settings/Admin/SharingTest.php b/tests/lib/Settings/Admin/SharingTest.php index c97f22e54b..c40ff98a86 100644 --- a/tests/lib/Settings/Admin/SharingTest.php +++ b/tests/lib/Settings/Admin/SharingTest.php @@ -28,6 +28,7 @@ use OCP\AppFramework\Http\TemplateResponse; use OCP\Constants; use OCP\IConfig; use OCP\IL10N; +use OCP\L10N\IFactory; use OCP\Share\IManager; use Test\TestCase; @@ -45,11 +46,16 @@ class SharingTest extends TestCase { parent::setUp(); $this->config = $this->getMockBuilder(IConfig::class)->getMock(); $this->l10n = $this->getMockBuilder(IL10N::class)->getMock(); + + $l10Factory = $this->createMock(IFactory::class); + $l10Factory->method('get') + ->willReturn($this->l10n); + $this->shareManager = $this->getMockBuilder(IManager::class)->getMock(); $this->admin = new Sharing( $this->config, - $this->l10n, + $l10Factory, $this->shareManager ); }