diff --git a/apps/files_versions/lib/AppInfo/Application.php b/apps/files_versions/lib/AppInfo/Application.php index 4431fbf51bfeebe5a75af55daffb878dc311a763..be80f7d853b129f4104eec12a35f6a91b70d726e 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 d69e5bb87ce25b33bb25f42001d5c1fe9f888da6..47a1903a535991124fe1aa5f747314264b93d634 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 09de0ad6d90c214d3d96217ed8d730f36d7dd90c..7fdaa27afd9622584e2c18a1f4c4e444983a5836 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 52afdfdaa5aaa06d1e2da0f143fff6a2bbec9c75..217bf02fd39a5516b3f967220c3484831eb3010c 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); } }