Merge pull request #23029 from nextcloud/backport/23024/stable20

[stable20] Add occ command to set theming values
This commit is contained in:
Roeland Jago Douma 2020-09-28 12:24:23 +02:00 committed by GitHub
commit 15ff980583
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
8 changed files with 302 additions and 163 deletions

View File

@ -25,4 +25,7 @@
<admin>OCA\Theming\Settings\Admin</admin> <admin>OCA\Theming\Settings\Admin</admin>
<admin-section>OCA\Theming\Settings\Section</admin-section> <admin-section>OCA\Theming\Settings\Section</admin-section>
</settings> </settings>
<commands>
<command>OCA\Theming\Command\UpdateConfig</command>
</commands>
</info> </info>

View File

@ -0,0 +1,135 @@
<?php
/**
* @copyright Copyright (c) 2020 Julius Härtl <jus@bitgrid.net>
*
* @author Julius Härtl <jus@bitgrid.net>
*
* @license GNU AGPL version 3 or any later version
*
* This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU Affero General Public License as
* published by the Free Software Foundation, either version 3 of the
* License, or (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU Affero General Public License for more details.
*
* You should have received a copy of the GNU Affero General Public License
* along with this program. If not, see <http://www.gnu.org/licenses/>.
*
*/
namespace OCA\Theming\Command;
use OCA\Theming\ImageManager;
use OCA\Theming\ThemingDefaults;
use OCP\IConfig;
use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;
class UpdateConfig extends Command {
public const SUPPORTED_KEYS = [
'name', 'url', 'imprintUrl', 'privacyUrl', 'slogan', 'color'
];
public const SUPPORTED_IMAGE_KEYS = [
'background', 'logo', 'favicon', 'logoheader'
];
private $themingDefaults;
private $imageManager;
private $config;
public function __construct(ThemingDefaults $themingDefaults, ImageManager $imageManager, IConfig $config) {
parent::__construct();
$this->themingDefaults = $themingDefaults;
$this->imageManager = $imageManager;
$this->config = $config;
}
protected function configure() {
$this
->setName('theming:config')
->setDescription('Set theming app config values')
->addArgument(
'key',
InputArgument::OPTIONAL,
'Key to update the theming app configuration (leave empty to get a list of all configured values)' . PHP_EOL .
'One of: ' . implode(', ', self::SUPPORTED_KEYS)
)
->addArgument(
'value',
InputArgument::OPTIONAL,
'Value to set (leave empty to obtain the current value)'
)
->addOption(
'reset',
'r',
InputOption::VALUE_NONE,
'Reset the given config key to default'
);
}
protected function execute(InputInterface $input, OutputInterface $output): int {
$key = $input->getArgument('key');
$value = $input->getArgument('value');
if ($key === null) {
$output->writeln('Current theming config:');
foreach (self::SUPPORTED_KEYS as $key) {
$value = $this->config->getAppValue('theming', $key, '');
$output->writeln('- ' . $key . ': ' . $value . '');
}
foreach (self::SUPPORTED_IMAGE_KEYS as $key) {
$value = $this->config->getAppValue('theming', $key . 'Mime', '');
$output->writeln('- ' . $key . ': ' . $value . '');
}
return 0;
}
if (!in_array($key, self::SUPPORTED_KEYS, true)) {
$output->writeln('<error>Invalid config key provided</error>');
return 1;
}
if ($input->getOption('reset')) {
$defaultValue = $this->themingDefaults->undo($key);
$output->writeln('<info>Reset ' . $key . ' to ' . $defaultValue . '</info>');
return 0;
}
if ($value === null) {
$value = $this->config->getAppValue('theming', $key, '');
if ($value !== '') {
$output->writeln('<info>' . $key . ' is currently set to ' . $value . '</info>');
} else {
$output->writeln('<info>' . $key . ' is currently not set</info>');
}
return 0;
}
if (in_array($key, self::SUPPORTED_IMAGE_KEYS, true)) {
if (file_exists(__DIR__ . $value)) {
$value = __DIR__ . $value;
}
if (!file_exists($value)) {
$output->writeln('<error>File could not be found: ' . $value . '</error>');
return 1;
}
$value = $this->imageManager->updateImage($key, $value);
$key = $key . 'Mime';
}
$this->themingDefaults->set($key, $value);
$output->writeln('<info>Updated ' . $key . ' to ' . $value . '</info>');
return 0;
}
}

View File

@ -214,9 +214,6 @@ class ThemingController extends Controller {
* @throws NotPermittedException * @throws NotPermittedException
*/ */
public function uploadImage(): DataResponse { public function uploadImage(): DataResponse {
// logo / background
// new: favicon logo-header
//
$key = $this->request->getParam('key'); $key = $this->request->getParam('key');
$image = $this->request->getUploadedFile('image'); $image = $this->request->getUploadedFile('image');
$error = null; $error = null;
@ -249,23 +246,14 @@ class ThemingController extends Controller {
); );
} }
$name = '';
try { try {
$folder = $this->appData->getFolder('images'); $mime = $this->imageManager->updateImage($key, $image['tmp_name']);
} catch (NotFoundException $e) { $this->themingDefaults->set($key . 'Mime', $mime);
$folder = $this->appData->newFolder('images'); } catch (\Exception $e) {
}
$this->imageManager->delete($key);
$target = $folder->newFile($key);
$supportedFormats = $this->getSupportedUploadImageFormats($key);
$detectedMimeType = mime_content_type($image['tmp_name']);
if (!in_array($image['type'], $supportedFormats) || !in_array($detectedMimeType, $supportedFormats)) {
return new DataResponse( return new DataResponse(
[ [
'data' => [ 'data' => [
'message' => $this->l10n->t('Unsupported image type'), 'message' => $e->getMessage()
], ],
'status' => 'failure', 'status' => 'failure',
], ],
@ -273,28 +261,7 @@ class ThemingController extends Controller {
); );
} }
if ($key === 'background' && strpos($detectedMimeType, 'image/svg') === false) {
// Optimize the image since some people may upload images that will be
// either to big or are not progressive rendering.
$newImage = @imagecreatefromstring(file_get_contents($image['tmp_name'], 'r'));
$tmpFile = $this->tempManager->getTemporaryFile();
$newWidth = imagesx($newImage) < 4096 ? imagesx($newImage) : 4096;
$newHeight = imagesy($newImage) / (imagesx($newImage) / $newWidth);
$outputImage = imagescale($newImage, $newWidth, $newHeight);
imageinterlace($outputImage, 1);
imagejpeg($outputImage, $tmpFile, 75);
imagedestroy($outputImage);
$target->putContent(file_get_contents($tmpFile, 'r'));
} else {
$target->putContent(file_get_contents($image['tmp_name'], 'r'));
}
$name = $image['name']; $name = $image['name'];
$this->themingDefaults->set($key.'Mime', $image['type']);
$cssCached = $this->scssCacher->process(\OC::$SERVERROOT, 'core/css/css-variables.scss', 'core'); $cssCached = $this->scssCacher->process(\OC::$SERVERROOT, 'core/css/css-variables.scss', 'core');
return new DataResponse( return new DataResponse(
@ -311,24 +278,6 @@ class ThemingController extends Controller {
); );
} }
/**
* Returns a list of supported mime types for image uploads.
* "favicon" images are only allowed to be SVG when imagemagick with SVG support is available.
*
* @param string $key The image key, e.g. "favicon"
* @return array
*/
private function getSupportedUploadImageFormats(string $key): array {
$supportedFormats = ['image/jpeg', 'image/png', 'image/gif',];
if ($key !== 'favicon' || $this->imageManager->shouldReplaceIcons() === true) {
$supportedFormats[] = 'image/svg+xml';
$supportedFormats[] = 'image/svg';
}
return $supportedFormats;
}
/** /**
* Revert setting to default value * Revert setting to default value
* *
@ -341,11 +290,6 @@ class ThemingController extends Controller {
// reprocess server scss for preview // reprocess server scss for preview
$cssCached = $this->scssCacher->process(\OC::$SERVERROOT, 'core/css/css-variables.scss', 'core'); $cssCached = $this->scssCacher->process(\OC::$SERVERROOT, 'core/css/css-variables.scss', 'core');
if (strpos($setting, 'Mime') !== -1) {
$imageKey = str_replace('Mime', '', $setting);
$this->imageManager->delete($imageKey);
}
return new DataResponse( return new DataResponse(
[ [
'data' => 'data' =>

View File

@ -36,6 +36,7 @@ use OCP\Files\SimpleFS\ISimpleFolder;
use OCP\ICacheFactory; use OCP\ICacheFactory;
use OCP\IConfig; use OCP\IConfig;
use OCP\ILogger; use OCP\ILogger;
use OCP\ITempManager;
use OCP\IURLGenerator; use OCP\IURLGenerator;
class ImageManager { class ImageManager {
@ -52,27 +53,22 @@ class ImageManager {
private $cacheFactory; private $cacheFactory;
/** @var ILogger */ /** @var ILogger */
private $logger; private $logger;
/** @var ITempManager */
private $tempManager;
/**
* ImageManager constructor.
*
* @param IConfig $config
* @param IAppData $appData
* @param IURLGenerator $urlGenerator
* @param ICacheFactory $cacheFactory
* @param ILogger $logger
*/
public function __construct(IConfig $config, public function __construct(IConfig $config,
IAppData $appData, IAppData $appData,
IURLGenerator $urlGenerator, IURLGenerator $urlGenerator,
ICacheFactory $cacheFactory, ICacheFactory $cacheFactory,
ILogger $logger ILogger $logger,
ITempManager $tempManager
) { ) {
$this->config = $config; $this->config = $config;
$this->appData = $appData; $this->appData = $appData;
$this->urlGenerator = $urlGenerator; $this->urlGenerator = $urlGenerator;
$this->cacheFactory = $cacheFactory; $this->cacheFactory = $cacheFactory;
$this->logger = $logger; $this->logger = $logger;
$this->tempManager = $tempManager;
} }
public function getImageUrl(string $key, bool $useSvg = true): string { public function getImageUrl(string $key, bool $useSvg = true): string {
@ -211,6 +207,62 @@ class ImageManager {
} }
} }
public function updateImage(string $key, string $tmpFile) {
$this->delete($key);
try {
$folder = $this->appData->getFolder('images');
} catch (NotFoundException $e) {
$folder = $this->appData->newFolder('images');
}
$target = $folder->newFile($key);
$supportedFormats = $this->getSupportedUploadImageFormats($key);
$detectedMimeType = mime_content_type($tmpFile);
if (!in_array($detectedMimeType, $supportedFormats, true)) {
throw new \Exception('Unsupported image type');
}
if ($key === 'background' && strpos($detectedMimeType, 'image/svg') === false) {
// Optimize the image since some people may upload images that will be
// either to big or are not progressive rendering.
$newImage = @imagecreatefromstring(file_get_contents($tmpFile));
$tmpFile = $this->tempManager->getTemporaryFile();
$newWidth = (int)(imagesx($newImage) < 4096 ? imagesx($newImage) : 4096);
$newHeight = (int)(imagesy($newImage) / (imagesx($newImage) / $newWidth));
$outputImage = imagescale($newImage, $newWidth, $newHeight);
imageinterlace($outputImage, 1);
imagejpeg($outputImage, $tmpFile, 75);
imagedestroy($outputImage);
$target->putContent(file_get_contents($tmpFile));
} else {
$target->putContent(file_get_contents($tmpFile));
}
return $detectedMimeType;
}
/**
* Returns a list of supported mime types for image uploads.
* "favicon" images are only allowed to be SVG when imagemagick with SVG support is available.
*
* @param string $key The image key, e.g. "favicon"
* @return array
*/
private function getSupportedUploadImageFormats(string $key): array {
$supportedFormats = ['image/jpeg', 'image/png', 'image/gif'];
if ($key !== 'favicon' || $this->shouldReplaceIcons() === true) {
$supportedFormats[] = 'image/svg+xml';
$supportedFormats[] = 'image/svg';
}
return $supportedFormats;
}
/** /**
* remove cached files that are not required any longer * remove cached files that are not required any longer
* *

View File

@ -398,6 +398,7 @@ class ThemingDefaults extends \OC_Defaults {
$this->config->deleteAppValue('theming', $setting); $this->config->deleteAppValue('theming', $setting);
$this->increaseCacheBuster(); $this->increaseCacheBuster();
$returnValue = '';
switch ($setting) { switch ($setting) {
case 'name': case 'name':
$returnValue = $this->getEntity(); $returnValue = $this->getEntity();
@ -411,8 +412,11 @@ class ThemingDefaults extends \OC_Defaults {
case 'color': case 'color':
$returnValue = $this->getColorPrimary(); $returnValue = $this->getColorPrimary();
break; break;
default: case 'logo':
$returnValue = ''; case 'logoheader':
case 'background':
case 'favicon':
$this->imageManager->delete($setting);
break; break;
} }

View File

@ -47,34 +47,32 @@ use OCP\AppFramework\Utility\ITimeFactory;
use OCP\Files\IAppData; use OCP\Files\IAppData;
use OCP\Files\NotFoundException; use OCP\Files\NotFoundException;
use OCP\Files\SimpleFS\ISimpleFile; use OCP\Files\SimpleFS\ISimpleFile;
use OCP\Files\SimpleFS\ISimpleFolder;
use OCP\IConfig; use OCP\IConfig;
use OCP\IL10N; use OCP\IL10N;
use OCP\IRequest; use OCP\IRequest;
use OCP\ITempManager; use OCP\ITempManager;
use OCP\IURLGenerator; use OCP\IURLGenerator;
use PHPUnit\Framework\MockObject\MockObject;
use Test\TestCase; use Test\TestCase;
class ThemingControllerTest extends TestCase { class ThemingControllerTest extends TestCase {
/** @var IRequest|\PHPUnit\Framework\MockObject\MockObject */ /** @var IRequest|MockObject */
private $request; private $request;
/** @var IConfig|\PHPUnit\Framework\MockObject\MockObject */ /** @var IConfig|MockObject */
private $config; private $config;
/** @var ThemingDefaults|\PHPUnit\Framework\MockObject\MockObject */ /** @var ThemingDefaults|MockObject */
private $themingDefaults; private $themingDefaults;
/** @var \OCP\AppFramework\Utility\ITimeFactory */ /** @var IL10N|MockObject */
private $timeFactory;
/** @var IL10N|\PHPUnit\Framework\MockObject\MockObject */
private $l10n; private $l10n;
/** @var ThemingController */ /** @var ThemingController */
private $themingController; private $themingController;
/** @var ITempManager */ /** @var ITempManager */
private $tempManager; private $tempManager;
/** @var IAppManager|\PHPUnit\Framework\MockObject\MockObject */ /** @var IAppManager|MockObject */
private $appManager; private $appManager;
/** @var IAppData|\PHPUnit\Framework\MockObject\MockObject */ /** @var IAppData|MockObject */
private $appData; private $appData;
/** @var ImageManager|\PHPUnit\Framework\MockObject\MockObject */ /** @var ImageManager|MockObject */
private $imageManager; private $imageManager;
/** @var SCSSCacher */ /** @var SCSSCacher */
private $scssCacher; private $scssCacher;
@ -93,12 +91,12 @@ class ThemingControllerTest extends TestCase {
$this->urlGenerator = $this->createMock(IURLGenerator::class); $this->urlGenerator = $this->createMock(IURLGenerator::class);
$this->imageManager = $this->createMock(ImageManager::class); $this->imageManager = $this->createMock(ImageManager::class);
$this->timeFactory = $this->createMock(ITimeFactory::class); $timeFactory = $this->createMock(ITimeFactory::class);
$this->timeFactory->expects($this->any()) $timeFactory->expects($this->any())
->method('getTime') ->method('getTime')
->willReturn(123); ->willReturn(123);
$this->overwriteService(ITimeFactory::class, $this->timeFactory); $this->overwriteService(ITimeFactory::class, $timeFactory);
$this->themingController = new ThemingController( $this->themingController = new ThemingController(
'theming', 'theming',
@ -287,12 +285,9 @@ class ThemingControllerTest extends TestCase {
return $str; return $str;
}); });
$folder = $this->createMock(ISimpleFolder::class); $this->imageManager->expects($this->once())
$this->appData ->method('updateImage')
->expects($this->once()) ->willThrowException(new \Exception('Unsupported image type'));
->method('getFolder')
->with('images')
->willReturn($folder);
$expected = new DataResponse( $expected = new DataResponse(
[ [
@ -331,12 +326,9 @@ class ThemingControllerTest extends TestCase {
return $str; return $str;
}); });
$folder = $this->createMock(ISimpleFolder::class); $this->imageManager->expects($this->once())
$this->appData ->method('updateImage')
->expects($this->once()) ->willThrowException(new \Exception('Unsupported image type'));
->method('getFolder')
->with('images')
->willReturn($folder);
$expected = new DataResponse( $expected = new DataResponse(
[ [
@ -392,31 +384,6 @@ class ThemingControllerTest extends TestCase {
return $str; return $str;
}); });
$file = $this->createMock(ISimpleFile::class);
$folder = $this->createMock(ISimpleFolder::class);
if ($folderExists) {
$this->appData
->expects($this->once())
->method('getFolder')
->with('images')
->willReturn($folder);
} else {
$this->appData
->expects($this->at(0))
->method('getFolder')
->with('images')
->willThrowException(new NotFoundException());
$this->appData
->expects($this->at(1))
->method('newFolder')
->with('images')
->willReturn($folder);
}
$folder->expects($this->once())
->method('newFile')
->with('logo')
->willReturn($file);
$this->urlGenerator->expects($this->once()) $this->urlGenerator->expects($this->once())
->method('linkTo') ->method('linkTo')
->willReturn('serverCss'); ->willReturn('serverCss');
@ -424,6 +391,10 @@ class ThemingControllerTest extends TestCase {
->method('getImageUrl') ->method('getImageUrl')
->with('logo') ->with('logo')
->willReturn('imageUrl'); ->willReturn('imageUrl');
$this->imageManager->expects($this->once())
->method('updateImage');
$expected = new DataResponse( $expected = new DataResponse(
[ [
'data' => 'data' =>
@ -468,30 +439,8 @@ class ThemingControllerTest extends TestCase {
return $str; return $str;
}); });
$file = $this->createMock(ISimpleFile::class); $this->imageManager->expects($this->once())
$folder = $this->createMock(ISimpleFolder::class); ->method('updateImage');
if ($folderExists) {
$this->appData
->expects($this->once())
->method('getFolder')
->with('images')
->willReturn($folder);
} else {
$this->appData
->expects($this->at(0))
->method('getFolder')
->with('images')
->willThrowException(new NotFoundException());
$this->appData
->expects($this->at(1))
->method('newFolder')
->with('images')
->willReturn($folder);
}
$folder->expects($this->once())
->method('newFile')
->with('background')
->willReturn($file);
$this->urlGenerator->expects($this->once()) $this->urlGenerator->expects($this->once())
->method('linkTo') ->method('linkTo')
@ -542,12 +491,9 @@ class ThemingControllerTest extends TestCase {
return $str; return $str;
}); });
$folder = $this->createMock(ISimpleFolder::class); $this->imageManager->expects($this->once())
$this->appData ->method('updateImage')
->expects($this->once()) ->willThrowException(new \Exception('Unsupported image type'));
->method('getFolder')
->with('images')
->willReturn($folder);
$expected = new DataResponse( $expected = new DataResponse(
[ [
@ -717,9 +663,6 @@ class ThemingControllerTest extends TestCase {
->method('linkTo') ->method('linkTo')
->with('', '/core/css/someHash-css-variables.scss') ->with('', '/core/css/someHash-css-variables.scss')
->willReturn('/nextcloudWebroot/core/css/someHash-css-variables.scss'); ->willReturn('/nextcloudWebroot/core/css/someHash-css-variables.scss');
$this->imageManager->expects($this->once())
->method('delete')
->with($filename);
$expected = new DataResponse( $expected = new DataResponse(
[ [

View File

@ -36,23 +36,27 @@ use OCP\Files\SimpleFS\ISimpleFolder;
use OCP\ICacheFactory; use OCP\ICacheFactory;
use OCP\IConfig; use OCP\IConfig;
use OCP\ILogger; use OCP\ILogger;
use OCP\ITempManager;
use OCP\IURLGenerator; use OCP\IURLGenerator;
use PHPUnit\Framework\MockObject\MockObject;
use Test\TestCase; use Test\TestCase;
class ImageManagerTest extends TestCase { class ImageManagerTest extends TestCase {
/** @var IConfig|\PHPUnit\Framework\MockObject\MockObject */ /** @var IConfig|MockObject */
protected $config; protected $config;
/** @var IAppData|\PHPUnit\Framework\MockObject\MockObject */ /** @var IAppData|MockObject */
protected $appData; protected $appData;
/** @var ImageManager */ /** @var ImageManager */
protected $imageManager; protected $imageManager;
/** @var IURLGenerator|\PHPUnit\Framework\MockObject\MockObject */ /** @var IURLGenerator|MockObject */
private $urlGenerator; private $urlGenerator;
/** @var ICacheFactory|\PHPUnit\Framework\MockObject\MockObject */ /** @var ICacheFactory|MockObject */
private $cacheFactory; private $cacheFactory;
/** @var ILogger|\PHPUnit\Framework\MockObject\MockObject */ /** @var ILogger|MockObject */
private $logger; private $logger;
/** @var ITempManager|MockObject */
private $tempManager;
protected function setUp(): void { protected function setUp(): void {
parent::setUp(); parent::setUp();
@ -61,12 +65,14 @@ class ImageManagerTest extends TestCase {
$this->urlGenerator = $this->createMock(IURLGenerator::class); $this->urlGenerator = $this->createMock(IURLGenerator::class);
$this->cacheFactory = $this->createMock(ICacheFactory::class); $this->cacheFactory = $this->createMock(ICacheFactory::class);
$this->logger = $this->createMock(ILogger::class); $this->logger = $this->createMock(ILogger::class);
$this->tempManager = $this->createMock(ITempManager::class);
$this->imageManager = new ImageManager( $this->imageManager = new ImageManager(
$this->config, $this->config,
$this->appData, $this->appData,
$this->urlGenerator, $this->urlGenerator,
$this->cacheFactory, $this->cacheFactory,
$this->logger $this->logger,
$this->tempManager
); );
} }
@ -84,7 +90,7 @@ class ImageManagerTest extends TestCase {
} }
public function mockGetImage($key, $file) { public function mockGetImage($key, $file) {
/** @var \PHPUnit\Framework\MockObject\MockObject $folder */ /** @var MockObject $folder */
$folder = $this->createMock(ISimpleFolder::class); $folder = $this->createMock(ISimpleFolder::class);
if ($file === null) { if ($file === null) {
$folder->expects($this->once()) $folder->expects($this->once())
@ -327,4 +333,56 @@ class ImageManagerTest extends TestCase {
->willReturn($folders[2]); ->willReturn($folders[2]);
$this->imageManager->cleanup(); $this->imageManager->cleanup();
} }
public function dataUpdateImage() {
return [
['background', __DIR__ . '/../../../tests/data/testimage.png', true, true],
['background', __DIR__ . '/../../../tests/data/testimage.png', false, true],
['background', __DIR__ . '/../../../tests/data/testimage.jpg', true, true],
['logo', __DIR__ . '/../../../tests/data/testimagelarge.svg', true, false],
];
}
/**
* @dataProvider dataUpdateImage
*/
public function testUpdateImage($key, $tmpFile, $folderExists, $shouldConvert) {
$file = $this->createMock(ISimpleFile::class);
$folder = $this->createMock(ISimpleFolder::class);
$oldFile = $this->createMock(ISimpleFile::class);
$folder->expects($this->any())
->method('getFile')
->willReturn($oldFile);
if ($folderExists) {
$this->appData
->expects($this->any())
->method('getFolder')
->with('images')
->willReturn($folder);
} else {
$this->appData
->expects($this->any())
->method('getFolder')
->with('images')
->willThrowException(new NotFoundException());
$this->appData
->expects($this->any())
->method('newFolder')
->with('images')
->willReturn($folder);
}
$folder->expects($this->once())
->method('newFile')
->with($key)
->willReturn($file);
if ($shouldConvert) {
$this->tempManager->expects($this->once())
->method('getTemporaryFile')
->willReturn('/tmp/randomtempfile-theming');
}
$this->imageManager->updateImage($key, $tmpFile);
}
} }

View File

@ -1121,7 +1121,7 @@ class Server extends ServerContainer implements IServerContainer {
$c->getURLGenerator(), $c->getURLGenerator(),
$c->getMemCacheFactory(), $c->getMemCacheFactory(),
new Util($c->getConfig(), $this->getAppManager(), $c->getAppDataDir('theming')), new Util($c->getConfig(), $this->getAppManager(), $c->getAppDataDir('theming')),
new ImageManager($c->getConfig(), $c->getAppDataDir('theming'), $c->getURLGenerator(), $this->getMemCacheFactory(), $this->getLogger()), new ImageManager($c->getConfig(), $c->getAppDataDir('theming'), $c->getURLGenerator(), $this->getMemCacheFactory(), $this->getLogger(), $this->getTempManager()),
$c->getAppManager(), $c->getAppManager(),
$c->getNavigationManager() $c->getNavigationManager()
); );