No need for the CssManager

* It is a simple wrapper we can always add it later if needed

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
This commit is contained in:
Roeland Jago Douma 2016-11-30 16:14:00 +01:00
parent bd5bcd2d8d
commit 1e44a15dd1
No known key found for this signature in database
GPG Key ID: F941078878347C0C
4 changed files with 19 additions and 77 deletions

View File

@ -30,11 +30,11 @@
namespace OC\Core;
use OC\AppFramework\Utility\SimpleContainer;
use OC\CssManager;
use OC\Security\IdentityProof\Manager;
use OCP\AppFramework\App;
use OCP\Files\IAppData;
use OC\Core\Controller\CssController;
use OCP\AppFramework\Utility\ITimeFactory;
use OCP\IRequest;
use OCP\Util;
/**
@ -58,8 +58,13 @@ class Application extends App {
\OC::$server->getCrypto()
);
});
$container->registerService(CssManager::class, function () {
return new CssManager(\OC::$server->getAppDataDir('css'));
$container->registerService(CssController::class, function () use ($container) {
return new CssController(
$container->query('appName'),
$container->query(IRequest::class),
\OC::$server->getAppDataDir('css'),
$container->query(ITimeFactory::class)
);
});
}
}

View File

@ -27,30 +27,29 @@ use OCP\AppFramework\Controller;
use OCP\AppFramework\Http;
use OCP\AppFramework\Http\NotFoundResponse;
use OCP\AppFramework\Http\FileDisplayResponse;
use OCP\Files\IAppData;
use OCP\Files\NotFoundException;
use OCP\IRequest;
use OCP\Notification\IApp;
class CssController extends Controller {
/** @var CssManager */
protected $cssManager;
/** @var IAppData */
protected $appData;
/** @var TimeFactory */
protected $timeFactory;
/**
* @param string $appName
* @param IRequest $request
* @param CssManager $cssManager
* @param IAppData $appData
* @param TimeFactory $timeFactory
*/
public function __construct($appName, IRequest $request, CssManager $cssManager, TimeFactory $timeFactory) {
public function __construct($appName, IRequest $request, IAppData $appData, TimeFactory $timeFactory) {
parent::__construct($appName, $request);
$this->cssManager = $cssManager;
$this->appData = $appData;
$this->timeFactory = $timeFactory;
}
@ -64,7 +63,8 @@ class CssController extends Controller {
*/
public function getCss($fileName, $appName) {
try {
$cssFile = $this->cssManager->getCss($fileName, $appName);
$folder = $this->appData->getFolder($appName);
$cssFile = $folder->getFile($fileName);
} catch(NotFoundException $e) {
return new NotFoundResponse();
}

View File

@ -95,10 +95,6 @@ class DIContainer extends SimpleContainer implements IAppContainer {
return $this->getServer()->getAvatarManager();
});
$this->registerService('OCP\\ICssManager', function($c) {
return $this->getServer()->getCssManager();
});
$this->registerService('OCP\\Activity\\IManager', function($c) {
return $this->getServer()->getActivityManager();
});

View File

@ -1,59 +0,0 @@
<?php
/**
* @copyright Copyright (c) 2016, John Molakvoæ (skjnldsv@protonmail.com)
*
* @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 OC;
use OCP\Files\IAppData;
use OCP\Files\NotFoundException;
use OCP\Files\SimpleFS\ISimpleFile;
/**
* This class implements methods to access SCSS cached files
* @since 11.0.0
*/
class CssManager {
/** @var IAppData */
private $appData;
/**
* CssManager constructor.
*
* @param IAppData $appData
*/
public function __construct(IAppData $appData) {
$this->appData = $appData;
}
/**
* Get the css file and return ISimpleFile
*
* @param string $fileName css filename with extension
* @param string $appName css app name
* @return ISimpleFile
* @throws NotFoundException
*/
public function getCss($fileName, $appName) {
$folder = $this->appData->getFolder($appName);
return $folder->getFile($fileName);
}
}