From cd53498256c8d600821a55180c75564bcc63a364 Mon Sep 17 00:00:00 2001
From: Roeland Jago Douma <roeland@famdouma.nl>
Date: Fri, 11 May 2018 22:25:07 +0200
Subject: [PATCH] Use more magic DI in files_versions

There is no need to register this. The system can figure out by itself
what it needs to do :)

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
---
 apps/files_versions/lib/AppInfo/Application.php  | 10 ----------
 .../lib/BackgroundJob/ExpireVersions.php         | 16 +++-------------
 .../lib/Command/ExpireVersions.php               |  8 ++++----
 apps/files_versions/lib/Storage.php              |  2 +-
 4 files changed, 8 insertions(+), 28 deletions(-)

diff --git a/apps/files_versions/lib/AppInfo/Application.php b/apps/files_versions/lib/AppInfo/Application.php
index 4431fbf51b..be80f7d853 100644
--- a/apps/files_versions/lib/AppInfo/Application.php
+++ b/apps/files_versions/lib/AppInfo/Application.php
@@ -40,16 +40,6 @@ class Application extends App {
 		 */
 		$container->registerCapability(Capabilities::class);
 
-		/*
-		 * Register expiration
-		 */
-		$container->registerService('Expiration', function($c) {
-			return  new Expiration(
-				$c->query('ServerContainer')->getConfig(),
-				$c->query(ITimeFactory::class)
-			);
-		});
-
 		/*
 		 * Register $principalBackend for the DAV collection
 		 */
diff --git a/apps/files_versions/lib/BackgroundJob/ExpireVersions.php b/apps/files_versions/lib/BackgroundJob/ExpireVersions.php
index d69e5bb87c..47a1903a53 100644
--- a/apps/files_versions/lib/BackgroundJob/ExpireVersions.php
+++ b/apps/files_versions/lib/BackgroundJob/ExpireVersions.php
@@ -44,22 +44,12 @@ class ExpireVersions extends \OC\BackgroundJob\TimedJob {
 	 */
 	private $userManager;
 
-	public function __construct(IUserManager $userManager = null, Expiration $expiration = null) {
+	public function __construct(IUserManager $userManager, Expiration $expiration) {
 		// Run once per 30 minutes
 		$this->setInterval(60 * 30);
 
-		if (is_null($expiration) || is_null($userManager)) {
-			$this->fixDIForJobs();
-		} else {
-			$this->expiration = $expiration;
-			$this->userManager = $userManager;
-		}
-	}
-
-	protected function fixDIForJobs() {
-		$application = new Application();
-		$this->expiration = $application->getContainer()->query('Expiration');
-		$this->userManager = \OC::$server->getUserManager();
+		$this->expiration = $expiration;
+		$this->userManager = $userManager;
 	}
 
 	protected function run($argument) {
diff --git a/apps/files_versions/lib/Command/ExpireVersions.php b/apps/files_versions/lib/Command/ExpireVersions.php
index 09de0ad6d9..7fdaa27afd 100644
--- a/apps/files_versions/lib/Command/ExpireVersions.php
+++ b/apps/files_versions/lib/Command/ExpireVersions.php
@@ -46,11 +46,11 @@ class ExpireVersions extends Command {
 	private $userManager;
 
 	/**
-	 * @param IUserManager|null $userManager
-	 * @param Expiration|null $expiration
+	 * @param IUserManager $userManager
+	 * @param Expiration $expiration
 	 */
-	public function __construct(IUserManager $userManager = null,
-								Expiration $expiration = null) {
+	public function __construct(IUserManager $userManager,
+								Expiration $expiration) {
 		parent::__construct();
 
 		$this->userManager = $userManager;
diff --git a/apps/files_versions/lib/Storage.php b/apps/files_versions/lib/Storage.php
index 52afdfdaa5..217bf02fd3 100644
--- a/apps/files_versions/lib/Storage.php
+++ b/apps/files_versions/lib/Storage.php
@@ -845,7 +845,7 @@ class Storage {
 		if (is_null(self::$application)) {
 			self::$application = new Application();
 		}
-		return self::$application->getContainer()->query('Expiration');
+		return self::$application->getContainer()->query(Expiration::class);
 	}
 
 }