diff options
-rw-r--r-- | apps/files_trashbin/appinfo/app.php | 1 | ||||
-rw-r--r-- | apps/files_trashbin/appinfo/install.php | 23 | ||||
-rw-r--r-- | apps/files_trashbin/appinfo/update.php | 3 | ||||
-rw-r--r-- | apps/files_trashbin/lib/backgroundjob/expiretrash.php | 104 | ||||
-rw-r--r-- | apps/files_trashbin/lib/expiration.php | 9 | ||||
-rw-r--r-- | apps/files_trashbin/lib/trashbin.php | 2 | ||||
-rw-r--r-- | apps/files_trashbin/tests/backgroundjob/expiretrash.php | 37 | ||||
-rw-r--r-- | apps/files_trashbin/tests/expiration.php | 34 |
8 files changed, 212 insertions, 1 deletions
diff --git a/apps/files_trashbin/appinfo/app.php b/apps/files_trashbin/appinfo/app.php index 8f079fe6120..4805f9eeafd 100644 --- a/apps/files_trashbin/appinfo/app.php +++ b/apps/files_trashbin/appinfo/app.php @@ -23,6 +23,7 @@ * along with this program. If not, see <http://www.gnu.org/licenses/> * */ + $l = \OC::$server->getL10N('files_trashbin'); // register hooks diff --git a/apps/files_trashbin/appinfo/install.php b/apps/files_trashbin/appinfo/install.php new file mode 100644 index 00000000000..dc4c2847c22 --- /dev/null +++ b/apps/files_trashbin/appinfo/install.php @@ -0,0 +1,23 @@ +<?php +/** + * @author Victor Dubiniuk <dubiniuk@owncloud.com> + * + * @copyright Copyright (c) 2015, ownCloud, Inc. + * @license AGPL-3.0 + * + * This code is free software: you can redistribute it and/or modify + * it under the terms of the GNU Affero General Public License, version 3, + * as published by the Free Software Foundation. + * + * 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, version 3, + * along with this program. If not, see <http://www.gnu.org/licenses/> + * + */ + + // Cron job for deleting expired trash items +\OC::$server->getJobList()->add('OCA\Files_Trashbin\BackgroundJob\ExpireTrash'); diff --git a/apps/files_trashbin/appinfo/update.php b/apps/files_trashbin/appinfo/update.php index b77210ae4c0..ae018a9da5d 100644 --- a/apps/files_trashbin/appinfo/update.php +++ b/apps/files_trashbin/appinfo/update.php @@ -46,3 +46,6 @@ if (version_compare($installedVersion, '0.6.4', '<')) { $config->setSystemValue('trashbin_retention_obligation', $newObligation); $config->deleteSystemValue('trashbin_auto_expire'); } + +// Cron job for deleting expired trash items +\OC::$server->getJobList()->add('OCA\Files_Trashbin\BackgroundJob\ExpireTrash'); diff --git a/apps/files_trashbin/lib/backgroundjob/expiretrash.php b/apps/files_trashbin/lib/backgroundjob/expiretrash.php new file mode 100644 index 00000000000..350628444cf --- /dev/null +++ b/apps/files_trashbin/lib/backgroundjob/expiretrash.php @@ -0,0 +1,104 @@ +<?php +/** + * @author Victor Dubiniuk <dubiniuk@owncloud.com> + * + * @copyright Copyright (c) 2015, ownCloud, Inc. + * @license AGPL-3.0 + * + * This code is free software: you can redistribute it and/or modify + * it under the terms of the GNU Affero General Public License, version 3, + * as published by the Free Software Foundation. + * + * 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, version 3, + * along with this program. If not, see <http://www.gnu.org/licenses/> + * + */ + +namespace OCA\Files_Trashbin\BackgroundJob; + +use OCP\IDBConnection; +use OCP\IUserManager; +use OCA\Files_Trashbin\AppInfo\Application; +use OCA\Files_Trashbin\Expiration; +use OCA\Files_Trashbin\Helper; +use OCA\Files_Trashbin\Trashbin; + +class ExpireTrash extends \OC\BackgroundJob\TimedJob { + + const ITEMS_PER_SESSION = 1000; + + /** + * @var Expiration + */ + private $expiration; + + /** + * @var IDBConnection + */ + private $dbConnection; + + /** + * @var IUserManager + */ + private $userManager; + + public function __construct(IDBConnection $dbConnection = null, IUserManager $userManager = null, Expiration $expiration = null) { + // Run once per 30 minutes + $this->setInterval(60 * 30); + + if (is_null($expiration) || is_null($userManager) || is_null($dbConnection)) { + $this->fixDIForJobs(); + } else { + $this->dbConnection = $dbConnection; + $this->userManager = $userManager; + $this->expiration = $expiration; + } + } + + protected function fixDIForJobs() { + $application = new Application(); + $this->dbConnection = \OC::$server->getDatabaseConnection(); + $this->userManager = \OC::$server->getUserManager(); + $this->expiration = $application->getContainer()->query('Expiration'); + } + + protected function run($argument) { + $maxAge = $this->expiration->getMaxAgeAsTimestamp(); + if (!$maxAge) { + return; + } + $users = $this->userManager->search(''); + foreach ($users as $user) { + $uid = $user->getUID(); + if (!$this->setupFS($uid)) { + continue; + } + $dirContent = Helper::getTrashFiles('/', $uid, 'mtime'); + var_dump($dirContent); + Trashbin::deleteExpiredFiles($dirContent, $uid); + } + + \OC_Util::tearDownFS(); + } + + /** + * Act on behalf on trash item owner + * @param string $user + * @return boolean + */ + private function setupFS($user){ + if (!$this->userManager->userExists($user)) { + return false; + } + + \OC_Util::tearDownFS(); + \OC_Util::setupFS($user); + + return true; + } +} diff --git a/apps/files_trashbin/lib/expiration.php b/apps/files_trashbin/lib/expiration.php index 138540febf8..c8a6abb627b 100644 --- a/apps/files_trashbin/lib/expiration.php +++ b/apps/files_trashbin/lib/expiration.php @@ -105,6 +105,15 @@ class Expiration { return $isOlderThanMax || $isMinReached; } + public function getMaxAgeAsTimestamp(){ + $maxAge = false; + if ($this->isEnabled() && $this->maxAge !== self::NO_OBLIGATION) { + $time = $this->timeFactory->getTime(); + $maxAge = $time - ($this->maxAge * 86400); + } + return $maxAge; + } + private function parseRetentionObligation(){ $splitValues = explode(',', $this->retentionObligation); if (!isset($splitValues[0])) { diff --git a/apps/files_trashbin/lib/trashbin.php b/apps/files_trashbin/lib/trashbin.php index 2719eece2a8..784a8e45b21 100644 --- a/apps/files_trashbin/lib/trashbin.php +++ b/apps/files_trashbin/lib/trashbin.php @@ -689,7 +689,7 @@ class Trashbin { * @param string $user * @return array size of deleted files and number of deleted files */ - protected static function deleteExpiredFiles($files, $user) { + public static function deleteExpiredFiles($files, $user) { $application = new Application(); $expiration = $application->getContainer()->query('Expiration'); $size = 0; diff --git a/apps/files_trashbin/tests/backgroundjob/expiretrash.php b/apps/files_trashbin/tests/backgroundjob/expiretrash.php new file mode 100644 index 00000000000..f9f939c40ea --- /dev/null +++ b/apps/files_trashbin/tests/backgroundjob/expiretrash.php @@ -0,0 +1,37 @@ +<?php +/** + * @author Victor Dubiniuk <dubiniuk@owncloud.com> + * + * @copyright Copyright (c) 2015, ownCloud, Inc. + * @license AGPL-3.0 + * + * This code is free software: you can redistribute it and/or modify + * it under the terms of the GNU Affero General Public License, version 3, + * as published by the Free Software Foundation. + * + * 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, version 3, + * along with this program. If not, see <http://www.gnu.org/licenses/> + * + */ + +use OCA\Files_Trashbin\BackgroundJob\ExpireTrash; + +class ExpireTrash_Test extends \PHPUnit_Framework_TestCase { + public function testConstructAndRun() { + $backgroundJob = new OCA\Files_Trashbin\BackgroundJob\ExpireTrash( + $this->getMock('OCP\IDBConnection'), + $this->getMockBuilder('OCA\Files_Trashbin\Expiration')->disableOriginalConstructor()->getMock() + ); + + $jobList = $this->getMock('\OCP\BackgroundJob\IJobList'); + + /** @var \OC\BackgroundJob\JobList $jobList */ + $backgroundJob->execute($jobList); + $this->assertTrue(true); + } +} diff --git a/apps/files_trashbin/tests/expiration.php b/apps/files_trashbin/tests/expiration.php index 7bd51dccddd..b3c6fcd95af 100644 --- a/apps/files_trashbin/tests/expiration.php +++ b/apps/files_trashbin/tests/expiration.php @@ -24,6 +24,8 @@ use \OCA\Files_Trashbin\Expiration; class Expiration_Test extends \PHPUnit_Framework_TestCase { const SECONDS_PER_DAY = 86400; //60*60*24 + const FAKE_TIME_NOW = 1000000; + public function expirationData(){ $today = 100*self::SECONDS_PER_DAY; $back10Days = (100-10)*self::SECONDS_PER_DAY; @@ -142,6 +144,38 @@ class Expiration_Test extends \PHPUnit_Framework_TestCase { $this->assertAttributeEquals($expectedCanPurgeToSaveSpace, 'canPurgeToSaveSpace', $expiration); } + + public function timestampTestData(){ + return [ + [ 'disabled', false], + [ 'auto', false ], + [ 'auto,auto', false ], + [ 'auto, auto', false ], + [ 'auto, 3', self::FAKE_TIME_NOW - (3*self::SECONDS_PER_DAY) ], + [ '5, auto', false ], + [ '3, 5', self::FAKE_TIME_NOW - (5*self::SECONDS_PER_DAY) ], + [ '10, 3', self::FAKE_TIME_NOW - (10*self::SECONDS_PER_DAY) ], + ]; + } + + + /** + * @dataProvider timestampTestData + * + * @param string $configValue + * @param int $expectedMaxAgeTimestamp + */ + public function testGetMaxAgeAsTimestamp($configValue, $expectedMaxAgeTimestamp){ + $mockedConfig = $this->getMockedConfig($configValue); + $mockedTimeFactory = $this->getMockedTimeFactory( + self::FAKE_TIME_NOW + ); + + $expiration = new Expiration($mockedConfig, $mockedTimeFactory); + $actualTimestamp = $expiration->getMaxAgeAsTimestamp(); + $this->assertEquals($expectedMaxAgeTimestamp, $actualTimestamp); + } + /** * * @param int $time |