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>

2 files changed, 19 insertions, 14 deletions

diff --git a/core/Application.php b/core/Application.php
index ec93c50b445..dad7546dcb8 100644
--- a/core/Application.php
+++ b/core/Application.php
@@ -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)
+			);
 		});
 	}
 }
diff --git a/core/Controller/CssController.php b/core/Controller/CssController.php
index eb5e8d55b26..e547987902d 100644
--- a/core/Controller/CssController.php
+++ b/core/Controller/CssController.php
@@ -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();
 		}