wbrawner/server
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Use DI for the objects where possible

Browse source
This commit is contained in:
Joas Schilling 2015-03-25 18:21:30 +01:00
parent dd535968e8
commit e365ea7ec5
2 changed files with 27 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
apps/files/appinfo/app.php
View file

@ -59,6 +59,9 @@ $templateManager->registerTemplate('application/vnd.oasis.opendocument.spreadshe
\OC::$server->getActivityManager()->registerExtension(function() {
return new \OCA\Files\Activity(
\OC::$server->query('L10NFactory'),
\OC::$server->getURLGenerator()
\OC::$server->getURLGenerator(),
\OC::$server->getActivityManager(),
\OC::$server->getTagManager(),
\OC::$server->getConfig()
);
});

29
apps/files/lib/activity.php
View file

@ -24,7 +24,10 @@ namespace OCA\Files;
use OC\L10N\Factory;
use OCP\Activity\IExtension;
use OCP\Activity\IManager;
use OCP\IConfig;
use OCP\IL10N;
use OCP\ITagManager;
use OCP\IURLGenerator;
class Activity implements IExtension {
@ -46,14 +49,29 @@ class Activity implements IExtension {
/** @var IURLGenerator */
protected $URLGenerator;
/** @var \OCP\Activity\IManager */
protected $activityManager;
/** @var \OCP\IConfig */
protected $config;
/** @var \OCP\ITagManager */
protected $tagManager;
/**
* @param Factory $languageFactory
* @param IURLGenerator $URLGenerator
* @param IManager $activityManager
* @param ITagManager $tagManager
* @param IConfig $config
*/
public function __construct(Factory $languageFactory, IURLGenerator $URLGenerator) {
public function __construct(Factory $languageFactory, IURLGenerator $URLGenerator, IManager $activityManager, ITagManager $tagManager, IConfig $config) {
$this->languageFactory = $languageFactory;
$this->URLGenerator = $URLGenerator;
$this->l = $this->getL10N();
$this->activityManager = $activityManager;
$this->tagManager = $tagManager;
$this->config = $config;
}
/**
@ -288,7 +306,7 @@ class Activity implements IExtension {
* @return array|false
*/
public function getQueryForFilter($filter) {
$user = \OC::$server->getActivityManager()->getCurrentUserId();
$user = $this->activityManager->getCurrentUserId();
// Display actions from all files
if ($filter === self::FILTER_FILES) {
return ['`app` = ?', ['files']];
@ -301,9 +319,7 @@ class Activity implements IExtension {
// Display actions from favorites only
if ($filter === self::FILTER_FAVORITES || $filter === 'all' && $this->userSettingFavoritesOnly($user)) {
$tagManager = \OC::$server->getTagManager();
$tags = $tagManager->load('files', [], false, $user);
$tags = $this->tagManager->load('files', [], false, $user);
$favorites = $tags->getFavorites();
if (isset($favorites[50])) {
@ -311,6 +327,7 @@ class Activity implements IExtension {
return ['`app` = ?', ['files']];
}
// Can not DI because the user is not known on instantiation
$rootFolder = \OC::$server->getUserFolder($user);
$parameters = $fileQueryList = [];
foreach ($favorites as $favorite) {
@ -350,6 +367,6 @@ class Activity implements IExtension {
* @return bool
*/
protected function userSettingFavoritesOnly($user) {
return (bool) \OC::$server->getConfig()->getUserValue($user, 'activity', 'notify_stream_' . self::TYPE_FAVORITES, false);
return (bool) $this->config->getUserValue($user, 'activity', 'notify_stream_' . self::TYPE_FAVORITES, false);
}
}