@@ -28,40 +28,40 @@ |
||
28 | 28 | namespace OCA\Files; |
29 | 29 | |
30 | 30 | class App { |
31 | - /** |
|
32 | - * @var \OCP\INavigationManager |
|
33 | - */ |
|
34 | - private static $navigationManager; |
|
31 | + /** |
|
32 | + * @var \OCP\INavigationManager |
|
33 | + */ |
|
34 | + private static $navigationManager; |
|
35 | 35 | |
36 | - /** |
|
37 | - * Returns the app's navigation manager |
|
38 | - * |
|
39 | - * @return \OCP\INavigationManager |
|
40 | - */ |
|
41 | - public static function getNavigationManager() { |
|
42 | - // TODO: move this into a service in the Application class |
|
43 | - if (self::$navigationManager === null) { |
|
44 | - self::$navigationManager = new \OC\NavigationManager( |
|
45 | - \OC::$server->getAppManager(), |
|
46 | - \OC::$server->getURLGenerator(), |
|
47 | - \OC::$server->getL10NFactory(), |
|
48 | - \OC::$server->getUserSession(), |
|
49 | - \OC::$server->getGroupManager(), |
|
50 | - \OC::$server->getConfig() |
|
51 | - ); |
|
52 | - self::$navigationManager->clear(false); |
|
53 | - } |
|
54 | - return self::$navigationManager; |
|
55 | - } |
|
36 | + /** |
|
37 | + * Returns the app's navigation manager |
|
38 | + * |
|
39 | + * @return \OCP\INavigationManager |
|
40 | + */ |
|
41 | + public static function getNavigationManager() { |
|
42 | + // TODO: move this into a service in the Application class |
|
43 | + if (self::$navigationManager === null) { |
|
44 | + self::$navigationManager = new \OC\NavigationManager( |
|
45 | + \OC::$server->getAppManager(), |
|
46 | + \OC::$server->getURLGenerator(), |
|
47 | + \OC::$server->getL10NFactory(), |
|
48 | + \OC::$server->getUserSession(), |
|
49 | + \OC::$server->getGroupManager(), |
|
50 | + \OC::$server->getConfig() |
|
51 | + ); |
|
52 | + self::$navigationManager->clear(false); |
|
53 | + } |
|
54 | + return self::$navigationManager; |
|
55 | + } |
|
56 | 56 | |
57 | - public static function extendJsConfig($settings) { |
|
58 | - $appConfig = json_decode($settings['array']['oc_appconfig'], true); |
|
57 | + public static function extendJsConfig($settings) { |
|
58 | + $appConfig = json_decode($settings['array']['oc_appconfig'], true); |
|
59 | 59 | |
60 | - $maxChunkSize = (int)\OC::$server->getConfig()->getAppValue('files', 'max_chunk_size', 10 * 1024 * 1024); |
|
61 | - $appConfig['files'] = [ |
|
62 | - 'max_chunk_size' => $maxChunkSize |
|
63 | - ]; |
|
60 | + $maxChunkSize = (int)\OC::$server->getConfig()->getAppValue('files', 'max_chunk_size', 10 * 1024 * 1024); |
|
61 | + $appConfig['files'] = [ |
|
62 | + 'max_chunk_size' => $maxChunkSize |
|
63 | + ]; |
|
64 | 64 | |
65 | - $settings['array']['oc_appconfig'] = json_encode($appConfig); |
|
66 | - } |
|
65 | + $settings['array']['oc_appconfig'] = json_encode($appConfig); |
|
66 | + } |
|
67 | 67 | } |
@@ -36,51 +36,51 @@ |
||
36 | 36 | */ |
37 | 37 | class RSA extends AuthMechanism { |
38 | 38 | |
39 | - /** @var IConfig */ |
|
40 | - private $config; |
|
39 | + /** @var IConfig */ |
|
40 | + private $config; |
|
41 | 41 | |
42 | - public function __construct(IL10N $l, IConfig $config) { |
|
43 | - $this->config = $config; |
|
42 | + public function __construct(IL10N $l, IConfig $config) { |
|
43 | + $this->config = $config; |
|
44 | 44 | |
45 | - $this |
|
46 | - ->setIdentifier('publickey::rsa') |
|
47 | - ->setScheme(self::SCHEME_PUBLICKEY) |
|
48 | - ->setText($l->t('RSA public key')) |
|
49 | - ->addParameters([ |
|
50 | - new DefinitionParameter('user', $l->t('Username')), |
|
51 | - new DefinitionParameter('public_key', $l->t('Public key')), |
|
52 | - (new DefinitionParameter('private_key', 'private_key')) |
|
53 | - ->setType(DefinitionParameter::VALUE_HIDDEN), |
|
54 | - ]) |
|
55 | - ->addCustomJs('public_key') |
|
56 | - ; |
|
57 | - } |
|
45 | + $this |
|
46 | + ->setIdentifier('publickey::rsa') |
|
47 | + ->setScheme(self::SCHEME_PUBLICKEY) |
|
48 | + ->setText($l->t('RSA public key')) |
|
49 | + ->addParameters([ |
|
50 | + new DefinitionParameter('user', $l->t('Username')), |
|
51 | + new DefinitionParameter('public_key', $l->t('Public key')), |
|
52 | + (new DefinitionParameter('private_key', 'private_key')) |
|
53 | + ->setType(DefinitionParameter::VALUE_HIDDEN), |
|
54 | + ]) |
|
55 | + ->addCustomJs('public_key') |
|
56 | + ; |
|
57 | + } |
|
58 | 58 | |
59 | - public function manipulateStorageConfig(StorageConfig &$storage, IUser $user = null) { |
|
60 | - $auth = new RSACrypt(); |
|
61 | - $auth->setPassword($this->config->getSystemValue('secret', '')); |
|
62 | - if (!$auth->loadKey($storage->getBackendOption('private_key'))) { |
|
63 | - throw new \RuntimeException('unable to load private key'); |
|
64 | - } |
|
65 | - $storage->setBackendOption('public_key_auth', $auth); |
|
66 | - } |
|
59 | + public function manipulateStorageConfig(StorageConfig &$storage, IUser $user = null) { |
|
60 | + $auth = new RSACrypt(); |
|
61 | + $auth->setPassword($this->config->getSystemValue('secret', '')); |
|
62 | + if (!$auth->loadKey($storage->getBackendOption('private_key'))) { |
|
63 | + throw new \RuntimeException('unable to load private key'); |
|
64 | + } |
|
65 | + $storage->setBackendOption('public_key_auth', $auth); |
|
66 | + } |
|
67 | 67 | |
68 | - /** |
|
69 | - * Generate a keypair |
|
70 | - * |
|
71 | - * @param int $keyLenth |
|
72 | - * @return array ['privatekey' => $privateKey, 'publickey' => $publicKey] |
|
73 | - */ |
|
74 | - public function createKey($keyLength) { |
|
75 | - $rsa = new RSACrypt(); |
|
76 | - $rsa->setPublicKeyFormat(RSACrypt::PUBLIC_FORMAT_OPENSSH); |
|
77 | - $rsa->setPassword($this->config->getSystemValue('secret', '')); |
|
68 | + /** |
|
69 | + * Generate a keypair |
|
70 | + * |
|
71 | + * @param int $keyLenth |
|
72 | + * @return array ['privatekey' => $privateKey, 'publickey' => $publicKey] |
|
73 | + */ |
|
74 | + public function createKey($keyLength) { |
|
75 | + $rsa = new RSACrypt(); |
|
76 | + $rsa->setPublicKeyFormat(RSACrypt::PUBLIC_FORMAT_OPENSSH); |
|
77 | + $rsa->setPassword($this->config->getSystemValue('secret', '')); |
|
78 | 78 | |
79 | - if ($keyLength !== 1024 && $keyLength !== 2048 && $keyLength !== 4096) { |
|
80 | - $keyLength = 1024; |
|
81 | - } |
|
79 | + if ($keyLength !== 1024 && $keyLength !== 2048 && $keyLength !== 4096) { |
|
80 | + $keyLength = 1024; |
|
81 | + } |
|
82 | 82 | |
83 | - return $rsa->createKey($keyLength); |
|
84 | - } |
|
83 | + return $rsa->createKey($keyLength); |
|
84 | + } |
|
85 | 85 | |
86 | 86 | } |
@@ -34,57 +34,57 @@ |
||
34 | 34 | use Symfony\Component\Console\Output\OutputInterface; |
35 | 35 | |
36 | 36 | class Info extends Base { |
37 | - /** @var IUserManager */ |
|
38 | - protected $userManager; |
|
39 | - /** @var IGroupManager */ |
|
40 | - protected $groupManager; |
|
37 | + /** @var IUserManager */ |
|
38 | + protected $userManager; |
|
39 | + /** @var IGroupManager */ |
|
40 | + protected $groupManager; |
|
41 | 41 | |
42 | - /** |
|
43 | - * @param IUserManager $userManager |
|
44 | - * @param IGroupManager $groupManager |
|
45 | - */ |
|
46 | - public function __construct(IUserManager $userManager, IGroupManager $groupManager) { |
|
47 | - $this->userManager = $userManager; |
|
48 | - $this->groupManager = $groupManager; |
|
49 | - parent::__construct(); |
|
50 | - } |
|
42 | + /** |
|
43 | + * @param IUserManager $userManager |
|
44 | + * @param IGroupManager $groupManager |
|
45 | + */ |
|
46 | + public function __construct(IUserManager $userManager, IGroupManager $groupManager) { |
|
47 | + $this->userManager = $userManager; |
|
48 | + $this->groupManager = $groupManager; |
|
49 | + parent::__construct(); |
|
50 | + } |
|
51 | 51 | |
52 | - protected function configure() { |
|
53 | - $this |
|
54 | - ->setName('user:info') |
|
55 | - ->setDescription('show user info') |
|
56 | - ->addArgument( |
|
57 | - 'user', |
|
58 | - InputArgument::REQUIRED, |
|
59 | - 'user to show' |
|
60 | - )->addOption( |
|
61 | - 'output', |
|
62 | - null, |
|
63 | - InputOption::VALUE_OPTIONAL, |
|
64 | - 'Output format (plain, json or json_pretty, default is plain)', |
|
65 | - $this->defaultOutputFormat |
|
66 | - ); |
|
67 | - } |
|
52 | + protected function configure() { |
|
53 | + $this |
|
54 | + ->setName('user:info') |
|
55 | + ->setDescription('show user info') |
|
56 | + ->addArgument( |
|
57 | + 'user', |
|
58 | + InputArgument::REQUIRED, |
|
59 | + 'user to show' |
|
60 | + )->addOption( |
|
61 | + 'output', |
|
62 | + null, |
|
63 | + InputOption::VALUE_OPTIONAL, |
|
64 | + 'Output format (plain, json or json_pretty, default is plain)', |
|
65 | + $this->defaultOutputFormat |
|
66 | + ); |
|
67 | + } |
|
68 | 68 | |
69 | - protected function execute(InputInterface $input, OutputInterface $output) { |
|
70 | - $user = $this->userManager->get($input->getArgument('user')); |
|
71 | - if (is_null($user)) { |
|
72 | - $output->writeln('<error>user not found</error>'); |
|
73 | - return 1; |
|
74 | - } |
|
75 | - $groups = $this->groupManager->getUserGroupIds($user); |
|
76 | - $data = [ |
|
77 | - 'user_id' => $user->getUID(), |
|
78 | - 'display_name' => $user->getDisplayName(), |
|
79 | - 'email' => $user->getEMailAddress() ? $user->getEMailAddress() : '', |
|
80 | - 'cloud_id' => $user->getCloudId(), |
|
81 | - 'enabled' => $user->isEnabled(), |
|
82 | - 'groups' => $groups, |
|
83 | - 'quota' => $user->getQuota(), |
|
84 | - 'last_seen' => date(\DateTime::ATOM, $user->getLastLogin()), // ISO-8601 |
|
85 | - 'user_directory' => $user->getHome(), |
|
86 | - 'backend' => $user->getBackendClassName() |
|
87 | - ]; |
|
88 | - $this->writeArrayInOutputFormat($input, $output, $data); |
|
89 | - } |
|
69 | + protected function execute(InputInterface $input, OutputInterface $output) { |
|
70 | + $user = $this->userManager->get($input->getArgument('user')); |
|
71 | + if (is_null($user)) { |
|
72 | + $output->writeln('<error>user not found</error>'); |
|
73 | + return 1; |
|
74 | + } |
|
75 | + $groups = $this->groupManager->getUserGroupIds($user); |
|
76 | + $data = [ |
|
77 | + 'user_id' => $user->getUID(), |
|
78 | + 'display_name' => $user->getDisplayName(), |
|
79 | + 'email' => $user->getEMailAddress() ? $user->getEMailAddress() : '', |
|
80 | + 'cloud_id' => $user->getCloudId(), |
|
81 | + 'enabled' => $user->isEnabled(), |
|
82 | + 'groups' => $groups, |
|
83 | + 'quota' => $user->getQuota(), |
|
84 | + 'last_seen' => date(\DateTime::ATOM, $user->getLastLogin()), // ISO-8601 |
|
85 | + 'user_directory' => $user->getHome(), |
|
86 | + 'backend' => $user->getBackendClassName() |
|
87 | + ]; |
|
88 | + $this->writeArrayInOutputFormat($input, $output, $data); |
|
89 | + } |
|
90 | 90 | } |
@@ -34,35 +34,35 @@ |
||
34 | 34 | * @since 11.0.0 |
35 | 35 | */ |
36 | 36 | interface ISimpleRoot { |
37 | - /** |
|
38 | - * Get the folder with name $name |
|
39 | - * |
|
40 | - * @param string $name |
|
41 | - * @return ISimpleFolder |
|
42 | - * @throws NotFoundException |
|
43 | - * @throws \RuntimeException |
|
44 | - * @since 11.0.0 |
|
45 | - */ |
|
46 | - public function getFolder(string $name): ISimpleFolder; |
|
37 | + /** |
|
38 | + * Get the folder with name $name |
|
39 | + * |
|
40 | + * @param string $name |
|
41 | + * @return ISimpleFolder |
|
42 | + * @throws NotFoundException |
|
43 | + * @throws \RuntimeException |
|
44 | + * @since 11.0.0 |
|
45 | + */ |
|
46 | + public function getFolder(string $name): ISimpleFolder; |
|
47 | 47 | |
48 | - /** |
|
49 | - * Get all the Folders |
|
50 | - * |
|
51 | - * @return ISimpleFolder[] |
|
52 | - * @throws NotFoundException |
|
53 | - * @throws \RuntimeException |
|
54 | - * @since 11.0.0 |
|
55 | - */ |
|
56 | - public function getDirectoryListing(): array; |
|
48 | + /** |
|
49 | + * Get all the Folders |
|
50 | + * |
|
51 | + * @return ISimpleFolder[] |
|
52 | + * @throws NotFoundException |
|
53 | + * @throws \RuntimeException |
|
54 | + * @since 11.0.0 |
|
55 | + */ |
|
56 | + public function getDirectoryListing(): array; |
|
57 | 57 | |
58 | - /** |
|
59 | - * Create a new folder named $name |
|
60 | - * |
|
61 | - * @param string $name |
|
62 | - * @return ISimpleFolder |
|
63 | - * @throws NotPermittedException |
|
64 | - * @throws \RuntimeException |
|
65 | - * @since 11.0.0 |
|
66 | - */ |
|
67 | - public function newFolder(string $name): ISimpleFolder; |
|
58 | + /** |
|
59 | + * Create a new folder named $name |
|
60 | + * |
|
61 | + * @param string $name |
|
62 | + * @return ISimpleFolder |
|
63 | + * @throws NotPermittedException |
|
64 | + * @throws \RuntimeException |
|
65 | + * @since 11.0.0 |
|
66 | + */ |
|
67 | + public function newFolder(string $name): ISimpleFolder; |
|
68 | 68 | } |
@@ -43,62 +43,62 @@ |
||
43 | 43 | |
44 | 44 | class OCJSController extends Controller { |
45 | 45 | |
46 | - /** @var JSConfigHelper */ |
|
47 | - private $helper; |
|
46 | + /** @var JSConfigHelper */ |
|
47 | + private $helper; |
|
48 | 48 | |
49 | - /** |
|
50 | - * OCJSController constructor. |
|
51 | - * |
|
52 | - * @param string $appName |
|
53 | - * @param IRequest $request |
|
54 | - * @param IFactory $l10nFactory |
|
55 | - * @param Defaults $defaults |
|
56 | - * @param IAppManager $appManager |
|
57 | - * @param ISession $session |
|
58 | - * @param IUserSession $userSession |
|
59 | - * @param IConfig $config |
|
60 | - * @param IGroupManager $groupManager |
|
61 | - * @param IniGetWrapper $iniWrapper |
|
62 | - * @param IURLGenerator $urlGenerator |
|
63 | - * @param CapabilitiesManager $capabilitiesManager |
|
64 | - */ |
|
65 | - public function __construct($appName, |
|
66 | - IRequest $request, |
|
67 | - IFactory $l10nFactory, |
|
68 | - Defaults $defaults, |
|
69 | - IAppManager $appManager, |
|
70 | - ISession $session, |
|
71 | - IUserSession $userSession, |
|
72 | - IConfig $config, |
|
73 | - IGroupManager $groupManager, |
|
74 | - IniGetWrapper $iniWrapper, |
|
75 | - IURLGenerator $urlGenerator, |
|
76 | - CapabilitiesManager $capabilitiesManager) { |
|
77 | - parent::__construct($appName, $request); |
|
49 | + /** |
|
50 | + * OCJSController constructor. |
|
51 | + * |
|
52 | + * @param string $appName |
|
53 | + * @param IRequest $request |
|
54 | + * @param IFactory $l10nFactory |
|
55 | + * @param Defaults $defaults |
|
56 | + * @param IAppManager $appManager |
|
57 | + * @param ISession $session |
|
58 | + * @param IUserSession $userSession |
|
59 | + * @param IConfig $config |
|
60 | + * @param IGroupManager $groupManager |
|
61 | + * @param IniGetWrapper $iniWrapper |
|
62 | + * @param IURLGenerator $urlGenerator |
|
63 | + * @param CapabilitiesManager $capabilitiesManager |
|
64 | + */ |
|
65 | + public function __construct($appName, |
|
66 | + IRequest $request, |
|
67 | + IFactory $l10nFactory, |
|
68 | + Defaults $defaults, |
|
69 | + IAppManager $appManager, |
|
70 | + ISession $session, |
|
71 | + IUserSession $userSession, |
|
72 | + IConfig $config, |
|
73 | + IGroupManager $groupManager, |
|
74 | + IniGetWrapper $iniWrapper, |
|
75 | + IURLGenerator $urlGenerator, |
|
76 | + CapabilitiesManager $capabilitiesManager) { |
|
77 | + parent::__construct($appName, $request); |
|
78 | 78 | |
79 | - $this->helper = new JSConfigHelper( |
|
80 | - $l10nFactory->get('lib'), |
|
81 | - $defaults, |
|
82 | - $appManager, |
|
83 | - $session, |
|
84 | - $userSession->getUser(), |
|
85 | - $config, |
|
86 | - $groupManager, |
|
87 | - $iniWrapper, |
|
88 | - $urlGenerator, |
|
89 | - $capabilitiesManager |
|
90 | - ); |
|
91 | - } |
|
79 | + $this->helper = new JSConfigHelper( |
|
80 | + $l10nFactory->get('lib'), |
|
81 | + $defaults, |
|
82 | + $appManager, |
|
83 | + $session, |
|
84 | + $userSession->getUser(), |
|
85 | + $config, |
|
86 | + $groupManager, |
|
87 | + $iniWrapper, |
|
88 | + $urlGenerator, |
|
89 | + $capabilitiesManager |
|
90 | + ); |
|
91 | + } |
|
92 | 92 | |
93 | - /** |
|
94 | - * @NoCSRFRequired |
|
95 | - * @PublicPage |
|
96 | - * |
|
97 | - * @return DataDisplayResponse |
|
98 | - */ |
|
99 | - public function getConfig() { |
|
100 | - $data = $this->helper->getConfig(); |
|
93 | + /** |
|
94 | + * @NoCSRFRequired |
|
95 | + * @PublicPage |
|
96 | + * |
|
97 | + * @return DataDisplayResponse |
|
98 | + */ |
|
99 | + public function getConfig() { |
|
100 | + $data = $this->helper->getConfig(); |
|
101 | 101 | |
102 | - return new DataDisplayResponse($data, Http::STATUS_OK, ['Content-type' => 'text/javascript']); |
|
103 | - } |
|
102 | + return new DataDisplayResponse($data, Http::STATUS_OK, ['Content-type' => 'text/javascript']); |
|
103 | + } |
|
104 | 104 | } |
@@ -34,101 +34,101 @@ |
||
34 | 34 | use OCP\Share\IShareHelper; |
35 | 35 | |
36 | 36 | class Listener { |
37 | - /** @var IManager */ |
|
38 | - protected $activityManager; |
|
39 | - /** @var IUserSession */ |
|
40 | - protected $session; |
|
41 | - /** @var \OCP\App\IAppManager */ |
|
42 | - protected $appManager; |
|
43 | - /** @var \OCP\Files\Config\IMountProviderCollection */ |
|
44 | - protected $mountCollection; |
|
45 | - /** @var \OCP\Files\IRootFolder */ |
|
46 | - protected $rootFolder; |
|
47 | - /** @var IShareHelper */ |
|
48 | - protected $shareHelper; |
|
37 | + /** @var IManager */ |
|
38 | + protected $activityManager; |
|
39 | + /** @var IUserSession */ |
|
40 | + protected $session; |
|
41 | + /** @var \OCP\App\IAppManager */ |
|
42 | + protected $appManager; |
|
43 | + /** @var \OCP\Files\Config\IMountProviderCollection */ |
|
44 | + protected $mountCollection; |
|
45 | + /** @var \OCP\Files\IRootFolder */ |
|
46 | + protected $rootFolder; |
|
47 | + /** @var IShareHelper */ |
|
48 | + protected $shareHelper; |
|
49 | 49 | |
50 | - /** |
|
51 | - * Listener constructor. |
|
52 | - * |
|
53 | - * @param IManager $activityManager |
|
54 | - * @param IUserSession $session |
|
55 | - * @param IAppManager $appManager |
|
56 | - * @param IMountProviderCollection $mountCollection |
|
57 | - * @param IRootFolder $rootFolder |
|
58 | - * @param IShareHelper $shareHelper |
|
59 | - */ |
|
60 | - public function __construct(IManager $activityManager, |
|
61 | - IUserSession $session, |
|
62 | - IAppManager $appManager, |
|
63 | - IMountProviderCollection $mountCollection, |
|
64 | - IRootFolder $rootFolder, |
|
65 | - IShareHelper $shareHelper) { |
|
66 | - $this->activityManager = $activityManager; |
|
67 | - $this->session = $session; |
|
68 | - $this->appManager = $appManager; |
|
69 | - $this->mountCollection = $mountCollection; |
|
70 | - $this->rootFolder = $rootFolder; |
|
71 | - $this->shareHelper = $shareHelper; |
|
72 | - } |
|
50 | + /** |
|
51 | + * Listener constructor. |
|
52 | + * |
|
53 | + * @param IManager $activityManager |
|
54 | + * @param IUserSession $session |
|
55 | + * @param IAppManager $appManager |
|
56 | + * @param IMountProviderCollection $mountCollection |
|
57 | + * @param IRootFolder $rootFolder |
|
58 | + * @param IShareHelper $shareHelper |
|
59 | + */ |
|
60 | + public function __construct(IManager $activityManager, |
|
61 | + IUserSession $session, |
|
62 | + IAppManager $appManager, |
|
63 | + IMountProviderCollection $mountCollection, |
|
64 | + IRootFolder $rootFolder, |
|
65 | + IShareHelper $shareHelper) { |
|
66 | + $this->activityManager = $activityManager; |
|
67 | + $this->session = $session; |
|
68 | + $this->appManager = $appManager; |
|
69 | + $this->mountCollection = $mountCollection; |
|
70 | + $this->rootFolder = $rootFolder; |
|
71 | + $this->shareHelper = $shareHelper; |
|
72 | + } |
|
73 | 73 | |
74 | - /** |
|
75 | - * @param CommentsEvent $event |
|
76 | - */ |
|
77 | - public function commentEvent(CommentsEvent $event) { |
|
78 | - if ($event->getComment()->getObjectType() !== 'files' |
|
79 | - || $event->getEvent() !== CommentsEvent::EVENT_ADD |
|
80 | - || !$this->appManager->isInstalled('activity')) { |
|
81 | - // Comment not for file, not adding a comment or no activity-app enabled (save the energy) |
|
82 | - return; |
|
83 | - } |
|
74 | + /** |
|
75 | + * @param CommentsEvent $event |
|
76 | + */ |
|
77 | + public function commentEvent(CommentsEvent $event) { |
|
78 | + if ($event->getComment()->getObjectType() !== 'files' |
|
79 | + || $event->getEvent() !== CommentsEvent::EVENT_ADD |
|
80 | + || !$this->appManager->isInstalled('activity')) { |
|
81 | + // Comment not for file, not adding a comment or no activity-app enabled (save the energy) |
|
82 | + return; |
|
83 | + } |
|
84 | 84 | |
85 | - // Get all mount point owners |
|
86 | - $cache = $this->mountCollection->getMountCache(); |
|
87 | - $mounts = $cache->getMountsForFileId((int)$event->getComment()->getObjectId()); |
|
88 | - if (empty($mounts)) { |
|
89 | - return; |
|
90 | - } |
|
85 | + // Get all mount point owners |
|
86 | + $cache = $this->mountCollection->getMountCache(); |
|
87 | + $mounts = $cache->getMountsForFileId((int)$event->getComment()->getObjectId()); |
|
88 | + if (empty($mounts)) { |
|
89 | + return; |
|
90 | + } |
|
91 | 91 | |
92 | - $users = []; |
|
93 | - foreach ($mounts as $mount) { |
|
94 | - $owner = $mount->getUser()->getUID(); |
|
95 | - $ownerFolder = $this->rootFolder->getUserFolder($owner); |
|
96 | - $nodes = $ownerFolder->getById((int)$event->getComment()->getObjectId()); |
|
97 | - if (!empty($nodes)) { |
|
98 | - /** @var Node $node */ |
|
99 | - $node = array_shift($nodes); |
|
100 | - $al = $this->shareHelper->getPathsForAccessList($node); |
|
101 | - $users += $al['users']; |
|
102 | - } |
|
103 | - } |
|
92 | + $users = []; |
|
93 | + foreach ($mounts as $mount) { |
|
94 | + $owner = $mount->getUser()->getUID(); |
|
95 | + $ownerFolder = $this->rootFolder->getUserFolder($owner); |
|
96 | + $nodes = $ownerFolder->getById((int)$event->getComment()->getObjectId()); |
|
97 | + if (!empty($nodes)) { |
|
98 | + /** @var Node $node */ |
|
99 | + $node = array_shift($nodes); |
|
100 | + $al = $this->shareHelper->getPathsForAccessList($node); |
|
101 | + $users += $al['users']; |
|
102 | + } |
|
103 | + } |
|
104 | 104 | |
105 | - $actor = $this->session->getUser(); |
|
106 | - if ($actor instanceof IUser) { |
|
107 | - $actor = $actor->getUID(); |
|
108 | - } else { |
|
109 | - $actor = ''; |
|
110 | - } |
|
105 | + $actor = $this->session->getUser(); |
|
106 | + if ($actor instanceof IUser) { |
|
107 | + $actor = $actor->getUID(); |
|
108 | + } else { |
|
109 | + $actor = ''; |
|
110 | + } |
|
111 | 111 | |
112 | - $activity = $this->activityManager->generateEvent(); |
|
113 | - $activity->setApp('comments') |
|
114 | - ->setType('comments') |
|
115 | - ->setAuthor($actor) |
|
116 | - ->setObject($event->getComment()->getObjectType(), (int) $event->getComment()->getObjectId()) |
|
117 | - ->setMessage('add_comment_message', [ |
|
118 | - 'commentId' => $event->getComment()->getId(), |
|
119 | - ]); |
|
112 | + $activity = $this->activityManager->generateEvent(); |
|
113 | + $activity->setApp('comments') |
|
114 | + ->setType('comments') |
|
115 | + ->setAuthor($actor) |
|
116 | + ->setObject($event->getComment()->getObjectType(), (int) $event->getComment()->getObjectId()) |
|
117 | + ->setMessage('add_comment_message', [ |
|
118 | + 'commentId' => $event->getComment()->getId(), |
|
119 | + ]); |
|
120 | 120 | |
121 | - foreach ($users as $user => $path) { |
|
122 | - // numerical user ids end up as integers from array keys, but string |
|
123 | - // is required |
|
124 | - $activity->setAffectedUser((string)$user); |
|
121 | + foreach ($users as $user => $path) { |
|
122 | + // numerical user ids end up as integers from array keys, but string |
|
123 | + // is required |
|
124 | + $activity->setAffectedUser((string)$user); |
|
125 | 125 | |
126 | - $activity->setSubject('add_comment_subject', [ |
|
127 | - 'actor' => $actor, |
|
128 | - 'fileId' => (int) $event->getComment()->getObjectId(), |
|
129 | - 'filePath' => trim($path, '/'), |
|
130 | - ]); |
|
131 | - $this->activityManager->publish($activity); |
|
132 | - } |
|
133 | - } |
|
126 | + $activity->setSubject('add_comment_subject', [ |
|
127 | + 'actor' => $actor, |
|
128 | + 'fileId' => (int) $event->getComment()->getObjectId(), |
|
129 | + 'filePath' => trim($path, '/'), |
|
130 | + ]); |
|
131 | + $this->activityManager->publish($activity); |
|
132 | + } |
|
133 | + } |
|
134 | 134 | } |
@@ -29,82 +29,82 @@ |
||
29 | 29 | |
30 | 30 | class InfoChecker extends BasicEmitter { |
31 | 31 | |
32 | - /** @var string[] */ |
|
33 | - private $shippedApps; |
|
32 | + /** @var string[] */ |
|
33 | + private $shippedApps; |
|
34 | 34 | |
35 | - /** @var string[] */ |
|
36 | - private $alwaysEnabled; |
|
35 | + /** @var string[] */ |
|
36 | + private $alwaysEnabled; |
|
37 | 37 | |
38 | - /** |
|
39 | - * @param string $appId |
|
40 | - * @return array |
|
41 | - * @throws \RuntimeException |
|
42 | - */ |
|
43 | - public function analyse($appId): array { |
|
44 | - $appPath = \OC_App::getAppPath($appId); |
|
45 | - if ($appPath === false) { |
|
46 | - throw new \RuntimeException("No app with given id <$appId> known."); |
|
47 | - } |
|
38 | + /** |
|
39 | + * @param string $appId |
|
40 | + * @return array |
|
41 | + * @throws \RuntimeException |
|
42 | + */ |
|
43 | + public function analyse($appId): array { |
|
44 | + $appPath = \OC_App::getAppPath($appId); |
|
45 | + if ($appPath === false) { |
|
46 | + throw new \RuntimeException("No app with given id <$appId> known."); |
|
47 | + } |
|
48 | 48 | |
49 | - $xml = new \DOMDocument(); |
|
50 | - $xml->load($appPath . '/appinfo/info.xml'); |
|
49 | + $xml = new \DOMDocument(); |
|
50 | + $xml->load($appPath . '/appinfo/info.xml'); |
|
51 | 51 | |
52 | - $schema = \OC::$SERVERROOT . '/resources/app-info.xsd'; |
|
53 | - try { |
|
54 | - if ($this->isShipped($appId)) { |
|
55 | - // Shipped apps are allowed to have the public and default_enabled tags |
|
56 | - $schema = \OC::$SERVERROOT . '/resources/app-info-shipped.xsd'; |
|
57 | - } |
|
58 | - } catch (\Exception $e) { |
|
59 | - // Assume it is not shipped |
|
60 | - } |
|
52 | + $schema = \OC::$SERVERROOT . '/resources/app-info.xsd'; |
|
53 | + try { |
|
54 | + if ($this->isShipped($appId)) { |
|
55 | + // Shipped apps are allowed to have the public and default_enabled tags |
|
56 | + $schema = \OC::$SERVERROOT . '/resources/app-info-shipped.xsd'; |
|
57 | + } |
|
58 | + } catch (\Exception $e) { |
|
59 | + // Assume it is not shipped |
|
60 | + } |
|
61 | 61 | |
62 | - $errors = []; |
|
63 | - if (!$xml->schemaValidate($schema)) { |
|
64 | - foreach (libxml_get_errors() as $error) { |
|
65 | - $errors[] = [ |
|
66 | - 'type' => 'parseError', |
|
67 | - 'field' => $error->message, |
|
68 | - ]; |
|
69 | - $this->emit('InfoChecker', 'parseError', [$error->message]); |
|
70 | - } |
|
71 | - } |
|
62 | + $errors = []; |
|
63 | + if (!$xml->schemaValidate($schema)) { |
|
64 | + foreach (libxml_get_errors() as $error) { |
|
65 | + $errors[] = [ |
|
66 | + 'type' => 'parseError', |
|
67 | + 'field' => $error->message, |
|
68 | + ]; |
|
69 | + $this->emit('InfoChecker', 'parseError', [$error->message]); |
|
70 | + } |
|
71 | + } |
|
72 | 72 | |
73 | - return $errors; |
|
74 | - } |
|
73 | + return $errors; |
|
74 | + } |
|
75 | 75 | |
76 | - /** |
|
77 | - * This is a copy of \OC\App\AppManager::isShipped(), keep both in sync. |
|
78 | - * This method is copied, so the code checker works even when Nextcloud is |
|
79 | - * not installed yet. The AppManager requires a database connection, which |
|
80 | - * fails in that case. |
|
81 | - * |
|
82 | - * @param string $appId |
|
83 | - * @return bool |
|
84 | - * @throws \Exception |
|
85 | - */ |
|
86 | - protected function isShipped(string $appId): bool { |
|
87 | - $this->loadShippedJson(); |
|
88 | - return \in_array($appId, $this->shippedApps, true); |
|
89 | - } |
|
76 | + /** |
|
77 | + * This is a copy of \OC\App\AppManager::isShipped(), keep both in sync. |
|
78 | + * This method is copied, so the code checker works even when Nextcloud is |
|
79 | + * not installed yet. The AppManager requires a database connection, which |
|
80 | + * fails in that case. |
|
81 | + * |
|
82 | + * @param string $appId |
|
83 | + * @return bool |
|
84 | + * @throws \Exception |
|
85 | + */ |
|
86 | + protected function isShipped(string $appId): bool { |
|
87 | + $this->loadShippedJson(); |
|
88 | + return \in_array($appId, $this->shippedApps, true); |
|
89 | + } |
|
90 | 90 | |
91 | - /** |
|
92 | - * This is a copy of \OC\App\AppManager::loadShippedJson(), keep both in sync |
|
93 | - * This method is copied, so the code checker works even when Nextcloud is |
|
94 | - * not installed yet. The AppManager requires a database connection, which |
|
95 | - * fails in that case. |
|
96 | - * |
|
97 | - * @throws \Exception |
|
98 | - */ |
|
99 | - protected function loadShippedJson() { |
|
100 | - if ($this->shippedApps === null) { |
|
101 | - $shippedJson = \OC::$SERVERROOT . '/core/shipped.json'; |
|
102 | - if (!file_exists($shippedJson)) { |
|
103 | - throw new \Exception("File not found: $shippedJson"); |
|
104 | - } |
|
105 | - $content = json_decode(file_get_contents($shippedJson), true); |
|
106 | - $this->shippedApps = $content['shippedApps']; |
|
107 | - $this->alwaysEnabled = $content['alwaysEnabled']; |
|
108 | - } |
|
109 | - } |
|
91 | + /** |
|
92 | + * This is a copy of \OC\App\AppManager::loadShippedJson(), keep both in sync |
|
93 | + * This method is copied, so the code checker works even when Nextcloud is |
|
94 | + * not installed yet. The AppManager requires a database connection, which |
|
95 | + * fails in that case. |
|
96 | + * |
|
97 | + * @throws \Exception |
|
98 | + */ |
|
99 | + protected function loadShippedJson() { |
|
100 | + if ($this->shippedApps === null) { |
|
101 | + $shippedJson = \OC::$SERVERROOT . '/core/shipped.json'; |
|
102 | + if (!file_exists($shippedJson)) { |
|
103 | + throw new \Exception("File not found: $shippedJson"); |
|
104 | + } |
|
105 | + $content = json_decode(file_get_contents($shippedJson), true); |
|
106 | + $this->shippedApps = $content['shippedApps']; |
|
107 | + $this->alwaysEnabled = $content['alwaysEnabled']; |
|
108 | + } |
|
109 | + } |
|
110 | 110 | } |
@@ -32,58 +32,58 @@ |
||
32 | 32 | * Person mount points can be moved by the user |
33 | 33 | */ |
34 | 34 | class PersonalMount extends ExternalMountPoint implements MoveableMount { |
35 | - /** @var UserStoragesService */ |
|
36 | - protected $storagesService; |
|
35 | + /** @var UserStoragesService */ |
|
36 | + protected $storagesService; |
|
37 | 37 | |
38 | - /** @var int */ |
|
39 | - protected $numericStorageId; |
|
38 | + /** @var int */ |
|
39 | + protected $numericStorageId; |
|
40 | 40 | |
41 | - /** |
|
42 | - * @param UserStoragesService $storagesService |
|
43 | - * @param int $storageId |
|
44 | - * @param \OCP\Files\Storage $storage |
|
45 | - * @param string $mountpoint |
|
46 | - * @param array $arguments (optional) configuration for the storage backend |
|
47 | - * @param \OCP\Files\Storage\IStorageFactory $loader |
|
48 | - * @param array $mountOptions mount specific options |
|
49 | - */ |
|
50 | - public function __construct( |
|
51 | - UserStoragesService $storagesService, |
|
52 | - $storageId, |
|
53 | - $storage, |
|
54 | - $mountpoint, |
|
55 | - $arguments = null, |
|
56 | - $loader = null, |
|
57 | - $mountOptions = null |
|
58 | - ) { |
|
59 | - parent::__construct($storage, $mountpoint, $arguments, $loader, $mountOptions); |
|
60 | - $this->storagesService = $storagesService; |
|
61 | - $this->numericStorageId = $storageId; |
|
62 | - } |
|
41 | + /** |
|
42 | + * @param UserStoragesService $storagesService |
|
43 | + * @param int $storageId |
|
44 | + * @param \OCP\Files\Storage $storage |
|
45 | + * @param string $mountpoint |
|
46 | + * @param array $arguments (optional) configuration for the storage backend |
|
47 | + * @param \OCP\Files\Storage\IStorageFactory $loader |
|
48 | + * @param array $mountOptions mount specific options |
|
49 | + */ |
|
50 | + public function __construct( |
|
51 | + UserStoragesService $storagesService, |
|
52 | + $storageId, |
|
53 | + $storage, |
|
54 | + $mountpoint, |
|
55 | + $arguments = null, |
|
56 | + $loader = null, |
|
57 | + $mountOptions = null |
|
58 | + ) { |
|
59 | + parent::__construct($storage, $mountpoint, $arguments, $loader, $mountOptions); |
|
60 | + $this->storagesService = $storagesService; |
|
61 | + $this->numericStorageId = $storageId; |
|
62 | + } |
|
63 | 63 | |
64 | - /** |
|
65 | - * Move the mount point to $target |
|
66 | - * |
|
67 | - * @param string $target the target mount point |
|
68 | - * @return bool |
|
69 | - */ |
|
70 | - public function moveMount($target) { |
|
71 | - $storage = $this->storagesService->getStorage($this->numericStorageId); |
|
72 | - // remove "/$user/files" prefix |
|
73 | - $targetParts = explode('/', trim($target, '/'), 3); |
|
74 | - $storage->setMountPoint($targetParts[2]); |
|
75 | - $this->storagesService->updateStorage($storage); |
|
76 | - $this->setMountPoint($target); |
|
77 | - return true; |
|
78 | - } |
|
64 | + /** |
|
65 | + * Move the mount point to $target |
|
66 | + * |
|
67 | + * @param string $target the target mount point |
|
68 | + * @return bool |
|
69 | + */ |
|
70 | + public function moveMount($target) { |
|
71 | + $storage = $this->storagesService->getStorage($this->numericStorageId); |
|
72 | + // remove "/$user/files" prefix |
|
73 | + $targetParts = explode('/', trim($target, '/'), 3); |
|
74 | + $storage->setMountPoint($targetParts[2]); |
|
75 | + $this->storagesService->updateStorage($storage); |
|
76 | + $this->setMountPoint($target); |
|
77 | + return true; |
|
78 | + } |
|
79 | 79 | |
80 | - /** |
|
81 | - * Remove the mount points |
|
82 | - * |
|
83 | - * @return bool |
|
84 | - */ |
|
85 | - public function removeMount() { |
|
86 | - $this->storagesService->removeStorage($this->numericStorageId); |
|
87 | - return true; |
|
88 | - } |
|
80 | + /** |
|
81 | + * Remove the mount points |
|
82 | + * |
|
83 | + * @return bool |
|
84 | + */ |
|
85 | + public function removeMount() { |
|
86 | + $this->storagesService->removeStorage($this->numericStorageId); |
|
87 | + return true; |
|
88 | + } |
|
89 | 89 | } |
@@ -30,21 +30,21 @@ |
||
30 | 30 | * @package OC\IntegrityCheck\Helpers |
31 | 31 | */ |
32 | 32 | class EnvironmentHelper { |
33 | - /** |
|
34 | - * Provides \OC::$SERVERROOT |
|
35 | - * |
|
36 | - * @return string |
|
37 | - */ |
|
38 | - public function getServerRoot(): string { |
|
39 | - return rtrim(\OC::$SERVERROOT, '/'); |
|
40 | - } |
|
33 | + /** |
|
34 | + * Provides \OC::$SERVERROOT |
|
35 | + * |
|
36 | + * @return string |
|
37 | + */ |
|
38 | + public function getServerRoot(): string { |
|
39 | + return rtrim(\OC::$SERVERROOT, '/'); |
|
40 | + } |
|
41 | 41 | |
42 | - /** |
|
43 | - * Provides \OC_Util::getChannel() |
|
44 | - * |
|
45 | - * @return string |
|
46 | - */ |
|
47 | - public function getChannel(): string { |
|
48 | - return \OC_Util::getChannel(); |
|
49 | - } |
|
42 | + /** |
|
43 | + * Provides \OC_Util::getChannel() |
|
44 | + * |
|
45 | + * @return string |
|
46 | + */ |
|
47 | + public function getChannel(): string { |
|
48 | + return \OC_Util::getChannel(); |
|
49 | + } |
|
50 | 50 | } |