@@ -26,11 +26,11 @@ |
||
26 | 26 | |
27 | 27 | // Migration OC8.2 -> OC9 |
28 | 28 | if (version_compare($installedVersion, '0.9.1', '<')) { |
29 | - $m = new Migration(\OC::$server->getDatabaseConnection(), \OC::$server->getConfig()); |
|
30 | - $m->removeReShares(); |
|
31 | - $m->updateInitiatorInfo(); |
|
29 | + $m = new Migration(\OC::$server->getDatabaseConnection(), \OC::$server->getConfig()); |
|
30 | + $m->removeReShares(); |
|
31 | + $m->updateInitiatorInfo(); |
|
32 | 32 | } |
33 | 33 | |
34 | 34 | if (version_compare($installedVersion, '1.1.1', '<')) { |
35 | - $m = new Migration(\OC::$server->getDatabaseConnection(), \OC::$server->getConfig()); |
|
35 | + $m = new Migration(\OC::$server->getDatabaseConnection(), \OC::$server->getConfig()); |
|
36 | 36 | } |
@@ -30,105 +30,105 @@ |
||
30 | 30 | |
31 | 31 | $application = new \OCA\Files_Sharing\AppInfo\Application(); |
32 | 32 | $application->registerRoutes($this, [ |
33 | - 'resources' => [ |
|
34 | - 'ExternalShares' => ['url' => '/api/externalShares'], |
|
35 | - ], |
|
36 | - 'routes' => [ |
|
37 | - [ |
|
38 | - 'name' => 'externalShares#testRemote', |
|
39 | - 'url' => '/testremote', |
|
40 | - 'verb' => 'GET' |
|
41 | - ], |
|
42 | - [ |
|
43 | - 'name' => 'PublicPreview#getPreview', |
|
44 | - 'url' => '/publicpreview', |
|
45 | - 'verb' => 'GET', |
|
46 | - ], |
|
33 | + 'resources' => [ |
|
34 | + 'ExternalShares' => ['url' => '/api/externalShares'], |
|
35 | + ], |
|
36 | + 'routes' => [ |
|
37 | + [ |
|
38 | + 'name' => 'externalShares#testRemote', |
|
39 | + 'url' => '/testremote', |
|
40 | + 'verb' => 'GET' |
|
41 | + ], |
|
42 | + [ |
|
43 | + 'name' => 'PublicPreview#getPreview', |
|
44 | + 'url' => '/publicpreview', |
|
45 | + 'verb' => 'GET', |
|
46 | + ], |
|
47 | 47 | |
48 | - [ |
|
49 | - 'name' => 'PublicPreview#getPreview', |
|
50 | - 'url' => '/ajax/publicpreview.php', |
|
51 | - 'verb' => 'GET', |
|
52 | - ], |
|
53 | - ], |
|
54 | - 'ocs' => [ |
|
55 | - /* |
|
48 | + [ |
|
49 | + 'name' => 'PublicPreview#getPreview', |
|
50 | + 'url' => '/ajax/publicpreview.php', |
|
51 | + 'verb' => 'GET', |
|
52 | + ], |
|
53 | + ], |
|
54 | + 'ocs' => [ |
|
55 | + /* |
|
56 | 56 | * OCS Share API |
57 | 57 | */ |
58 | - [ |
|
59 | - 'name' => 'ShareAPI#getShares', |
|
60 | - 'url' => '/api/v1/shares', |
|
61 | - 'verb' => 'GET', |
|
62 | - ], |
|
63 | - [ |
|
64 | - 'name' => 'ShareAPI#createShare', |
|
65 | - 'url' => '/api/v1/shares', |
|
66 | - 'verb' => 'POST', |
|
67 | - ], |
|
68 | - [ |
|
69 | - 'name' => 'ShareAPI#getShare', |
|
70 | - 'url' => '/api/v1/shares/{id}', |
|
71 | - 'verb' => 'GET', |
|
72 | - ], |
|
73 | - [ |
|
74 | - 'name' => 'ShareAPI#updateShare', |
|
75 | - 'url' => '/api/v1/shares/{id}', |
|
76 | - 'verb' => 'PUT', |
|
77 | - ], |
|
78 | - [ |
|
79 | - 'name' => 'ShareAPI#deleteShare', |
|
80 | - 'url' => '/api/v1/shares/{id}', |
|
81 | - 'verb' => 'DELETE', |
|
82 | - ], |
|
83 | - /* |
|
58 | + [ |
|
59 | + 'name' => 'ShareAPI#getShares', |
|
60 | + 'url' => '/api/v1/shares', |
|
61 | + 'verb' => 'GET', |
|
62 | + ], |
|
63 | + [ |
|
64 | + 'name' => 'ShareAPI#createShare', |
|
65 | + 'url' => '/api/v1/shares', |
|
66 | + 'verb' => 'POST', |
|
67 | + ], |
|
68 | + [ |
|
69 | + 'name' => 'ShareAPI#getShare', |
|
70 | + 'url' => '/api/v1/shares/{id}', |
|
71 | + 'verb' => 'GET', |
|
72 | + ], |
|
73 | + [ |
|
74 | + 'name' => 'ShareAPI#updateShare', |
|
75 | + 'url' => '/api/v1/shares/{id}', |
|
76 | + 'verb' => 'PUT', |
|
77 | + ], |
|
78 | + [ |
|
79 | + 'name' => 'ShareAPI#deleteShare', |
|
80 | + 'url' => '/api/v1/shares/{id}', |
|
81 | + 'verb' => 'DELETE', |
|
82 | + ], |
|
83 | + /* |
|
84 | 84 | * OCS Sharee API |
85 | 85 | */ |
86 | - [ |
|
87 | - 'name' => 'ShareesAPI#search', |
|
88 | - 'url' => '/api/v1/sharees', |
|
89 | - 'verb' => 'GET', |
|
90 | - ], |
|
91 | - /* |
|
86 | + [ |
|
87 | + 'name' => 'ShareesAPI#search', |
|
88 | + 'url' => '/api/v1/sharees', |
|
89 | + 'verb' => 'GET', |
|
90 | + ], |
|
91 | + /* |
|
92 | 92 | * Remote Shares |
93 | 93 | */ |
94 | - [ |
|
95 | - 'name' => 'Remote#getShares', |
|
96 | - 'url' => '/api/v1/remote_shares', |
|
97 | - 'verb' => 'GET', |
|
98 | - ], |
|
99 | - [ |
|
100 | - 'name' => 'Remote#getOpenShares', |
|
101 | - 'url' => '/api/v1/remote_shares/pending', |
|
102 | - 'verb' => 'GET', |
|
103 | - ], |
|
104 | - [ |
|
105 | - 'name' => 'Remote#acceptShare', |
|
106 | - 'url' => '/api/v1/remote_shares/pending/{id}', |
|
107 | - 'verb' => 'POST', |
|
108 | - ], |
|
109 | - [ |
|
110 | - 'name' => 'Remote#declineShare', |
|
111 | - 'url' => '/api/v1/remote_shares/pending/{id}', |
|
112 | - 'verb' => 'DELETE', |
|
113 | - ], |
|
114 | - [ |
|
115 | - 'name' => 'Remote#getShare', |
|
116 | - 'url' => '/api/v1/remote_shares/{id}', |
|
117 | - 'verb' => 'GET', |
|
118 | - ], |
|
119 | - [ |
|
120 | - 'name' => 'Remote#unshare', |
|
121 | - 'url' => '/api/v1/remote_shares/{id}', |
|
122 | - 'verb' => 'DELETE', |
|
123 | - ], |
|
124 | - ], |
|
94 | + [ |
|
95 | + 'name' => 'Remote#getShares', |
|
96 | + 'url' => '/api/v1/remote_shares', |
|
97 | + 'verb' => 'GET', |
|
98 | + ], |
|
99 | + [ |
|
100 | + 'name' => 'Remote#getOpenShares', |
|
101 | + 'url' => '/api/v1/remote_shares/pending', |
|
102 | + 'verb' => 'GET', |
|
103 | + ], |
|
104 | + [ |
|
105 | + 'name' => 'Remote#acceptShare', |
|
106 | + 'url' => '/api/v1/remote_shares/pending/{id}', |
|
107 | + 'verb' => 'POST', |
|
108 | + ], |
|
109 | + [ |
|
110 | + 'name' => 'Remote#declineShare', |
|
111 | + 'url' => '/api/v1/remote_shares/pending/{id}', |
|
112 | + 'verb' => 'DELETE', |
|
113 | + ], |
|
114 | + [ |
|
115 | + 'name' => 'Remote#getShare', |
|
116 | + 'url' => '/api/v1/remote_shares/{id}', |
|
117 | + 'verb' => 'GET', |
|
118 | + ], |
|
119 | + [ |
|
120 | + 'name' => 'Remote#unshare', |
|
121 | + 'url' => '/api/v1/remote_shares/{id}', |
|
122 | + 'verb' => 'DELETE', |
|
123 | + ], |
|
124 | + ], |
|
125 | 125 | ]); |
126 | 126 | |
127 | 127 | /** @var $this \OCP\Route\IRouter */ |
128 | 128 | $this->create('files_sharing_ajax_list', 'ajax/list.php') |
129 | - ->actionInclude('files_sharing/ajax/list.php'); |
|
129 | + ->actionInclude('files_sharing/ajax/list.php'); |
|
130 | 130 | $this->create('sharing_external_shareinfo', '/shareinfo') |
131 | - ->actionInclude('files_sharing/ajax/shareinfo.php'); |
|
131 | + ->actionInclude('files_sharing/ajax/shareinfo.php'); |
|
132 | 132 | |
133 | 133 | // OCS API |
134 | 134 |
@@ -31,127 +31,127 @@ |
||
31 | 31 | |
32 | 32 | class AppConfigController extends OCSController { |
33 | 33 | |
34 | - /** @var IConfig */ |
|
35 | - protected $config; |
|
36 | - |
|
37 | - /** @var IAppConfig */ |
|
38 | - protected $appConfig; |
|
39 | - |
|
40 | - /** |
|
41 | - * @param string $appName |
|
42 | - * @param IRequest $request |
|
43 | - * @param IConfig $config |
|
44 | - * @param IAppConfig $appConfig |
|
45 | - */ |
|
46 | - public function __construct($appName, |
|
47 | - IRequest $request, |
|
48 | - IConfig $config, |
|
49 | - IAppConfig $appConfig) { |
|
50 | - parent::__construct($appName, $request); |
|
51 | - $this->config = $config; |
|
52 | - $this->appConfig = $appConfig; |
|
53 | - } |
|
54 | - |
|
55 | - /** |
|
56 | - * @return DataResponse |
|
57 | - */ |
|
58 | - public function getApps() { |
|
59 | - return new DataResponse([ |
|
60 | - 'data' => $this->appConfig->getApps(), |
|
61 | - ]); |
|
62 | - } |
|
63 | - |
|
64 | - /** |
|
65 | - * @param string $app |
|
66 | - * @return DataResponse |
|
67 | - */ |
|
68 | - public function getKeys($app) { |
|
69 | - try { |
|
70 | - $this->verifyAppId($app); |
|
71 | - } catch (\InvalidArgumentException $e) { |
|
72 | - return new DataResponse(['data' => ['message' => $e->getMessage()]], Http::STATUS_FORBIDDEN); |
|
73 | - } |
|
74 | - return new DataResponse([ |
|
75 | - 'data' => $this->config->getAppKeys($app), |
|
76 | - ]); |
|
77 | - } |
|
78 | - |
|
79 | - /** |
|
80 | - * @param string $app |
|
81 | - * @param string $key |
|
82 | - * @param string $defaultValue |
|
83 | - * @return DataResponse |
|
84 | - */ |
|
85 | - public function getValue($app, $key, $defaultValue = '') { |
|
86 | - try { |
|
87 | - $this->verifyAppId($app); |
|
88 | - } catch (\InvalidArgumentException $e) { |
|
89 | - return new DataResponse(['data' => ['message' => $e->getMessage()]], Http::STATUS_FORBIDDEN); |
|
90 | - } |
|
91 | - return new DataResponse([ |
|
92 | - 'data' => $this->config->getAppValue($app, $key, $defaultValue), |
|
93 | - ]); |
|
94 | - } |
|
95 | - |
|
96 | - /** |
|
97 | - * @PasswordConfirmationRequired |
|
98 | - * @param string $app |
|
99 | - * @param string $key |
|
100 | - * @param string $value |
|
101 | - * @return DataResponse |
|
102 | - */ |
|
103 | - public function setValue($app, $key, $value) { |
|
104 | - try { |
|
105 | - $this->verifyAppId($app); |
|
106 | - $this->verifyConfigKey($app, $key); |
|
107 | - } catch (\InvalidArgumentException $e) { |
|
108 | - return new DataResponse(['data' => ['message' => $e->getMessage()]], Http::STATUS_FORBIDDEN); |
|
109 | - } |
|
110 | - |
|
111 | - $this->config->setAppValue($app, $key, $value); |
|
112 | - return new DataResponse(); |
|
113 | - } |
|
114 | - |
|
115 | - /** |
|
116 | - * @PasswordConfirmationRequired |
|
117 | - * @param string $app |
|
118 | - * @param string $key |
|
119 | - * @return DataResponse |
|
120 | - */ |
|
121 | - public function deleteKey($app, $key) { |
|
122 | - try { |
|
123 | - $this->verifyAppId($app); |
|
124 | - $this->verifyConfigKey($app, $key); |
|
125 | - } catch (\InvalidArgumentException $e) { |
|
126 | - return new DataResponse(['data' => ['message' => $e->getMessage()]], Http::STATUS_FORBIDDEN); |
|
127 | - } |
|
128 | - |
|
129 | - $this->config->deleteAppValue($app, $key); |
|
130 | - return new DataResponse(); |
|
131 | - } |
|
132 | - |
|
133 | - /** |
|
134 | - * @param string $app |
|
135 | - * @throws \InvalidArgumentException |
|
136 | - */ |
|
137 | - protected function verifyAppId($app) { |
|
138 | - if (\OC_App::cleanAppId($app) !== $app) { |
|
139 | - throw new \InvalidArgumentException('Invalid app id given'); |
|
140 | - } |
|
141 | - } |
|
142 | - |
|
143 | - /** |
|
144 | - * @param string $app |
|
145 | - * @param string $key |
|
146 | - * @throws \InvalidArgumentException |
|
147 | - */ |
|
148 | - protected function verifyConfigKey($app, $key) { |
|
149 | - if (in_array($key, ['installed_version', 'enabled', 'types'])) { |
|
150 | - throw new \InvalidArgumentException('The given key can not be set'); |
|
151 | - } |
|
152 | - |
|
153 | - if ($app === 'core' && (strpos($key, 'public_') === 0 || strpos($key, 'remote_') === 0)) { |
|
154 | - throw new \InvalidArgumentException('The given key can not be set'); |
|
155 | - } |
|
156 | - } |
|
34 | + /** @var IConfig */ |
|
35 | + protected $config; |
|
36 | + |
|
37 | + /** @var IAppConfig */ |
|
38 | + protected $appConfig; |
|
39 | + |
|
40 | + /** |
|
41 | + * @param string $appName |
|
42 | + * @param IRequest $request |
|
43 | + * @param IConfig $config |
|
44 | + * @param IAppConfig $appConfig |
|
45 | + */ |
|
46 | + public function __construct($appName, |
|
47 | + IRequest $request, |
|
48 | + IConfig $config, |
|
49 | + IAppConfig $appConfig) { |
|
50 | + parent::__construct($appName, $request); |
|
51 | + $this->config = $config; |
|
52 | + $this->appConfig = $appConfig; |
|
53 | + } |
|
54 | + |
|
55 | + /** |
|
56 | + * @return DataResponse |
|
57 | + */ |
|
58 | + public function getApps() { |
|
59 | + return new DataResponse([ |
|
60 | + 'data' => $this->appConfig->getApps(), |
|
61 | + ]); |
|
62 | + } |
|
63 | + |
|
64 | + /** |
|
65 | + * @param string $app |
|
66 | + * @return DataResponse |
|
67 | + */ |
|
68 | + public function getKeys($app) { |
|
69 | + try { |
|
70 | + $this->verifyAppId($app); |
|
71 | + } catch (\InvalidArgumentException $e) { |
|
72 | + return new DataResponse(['data' => ['message' => $e->getMessage()]], Http::STATUS_FORBIDDEN); |
|
73 | + } |
|
74 | + return new DataResponse([ |
|
75 | + 'data' => $this->config->getAppKeys($app), |
|
76 | + ]); |
|
77 | + } |
|
78 | + |
|
79 | + /** |
|
80 | + * @param string $app |
|
81 | + * @param string $key |
|
82 | + * @param string $defaultValue |
|
83 | + * @return DataResponse |
|
84 | + */ |
|
85 | + public function getValue($app, $key, $defaultValue = '') { |
|
86 | + try { |
|
87 | + $this->verifyAppId($app); |
|
88 | + } catch (\InvalidArgumentException $e) { |
|
89 | + return new DataResponse(['data' => ['message' => $e->getMessage()]], Http::STATUS_FORBIDDEN); |
|
90 | + } |
|
91 | + return new DataResponse([ |
|
92 | + 'data' => $this->config->getAppValue($app, $key, $defaultValue), |
|
93 | + ]); |
|
94 | + } |
|
95 | + |
|
96 | + /** |
|
97 | + * @PasswordConfirmationRequired |
|
98 | + * @param string $app |
|
99 | + * @param string $key |
|
100 | + * @param string $value |
|
101 | + * @return DataResponse |
|
102 | + */ |
|
103 | + public function setValue($app, $key, $value) { |
|
104 | + try { |
|
105 | + $this->verifyAppId($app); |
|
106 | + $this->verifyConfigKey($app, $key); |
|
107 | + } catch (\InvalidArgumentException $e) { |
|
108 | + return new DataResponse(['data' => ['message' => $e->getMessage()]], Http::STATUS_FORBIDDEN); |
|
109 | + } |
|
110 | + |
|
111 | + $this->config->setAppValue($app, $key, $value); |
|
112 | + return new DataResponse(); |
|
113 | + } |
|
114 | + |
|
115 | + /** |
|
116 | + * @PasswordConfirmationRequired |
|
117 | + * @param string $app |
|
118 | + * @param string $key |
|
119 | + * @return DataResponse |
|
120 | + */ |
|
121 | + public function deleteKey($app, $key) { |
|
122 | + try { |
|
123 | + $this->verifyAppId($app); |
|
124 | + $this->verifyConfigKey($app, $key); |
|
125 | + } catch (\InvalidArgumentException $e) { |
|
126 | + return new DataResponse(['data' => ['message' => $e->getMessage()]], Http::STATUS_FORBIDDEN); |
|
127 | + } |
|
128 | + |
|
129 | + $this->config->deleteAppValue($app, $key); |
|
130 | + return new DataResponse(); |
|
131 | + } |
|
132 | + |
|
133 | + /** |
|
134 | + * @param string $app |
|
135 | + * @throws \InvalidArgumentException |
|
136 | + */ |
|
137 | + protected function verifyAppId($app) { |
|
138 | + if (\OC_App::cleanAppId($app) !== $app) { |
|
139 | + throw new \InvalidArgumentException('Invalid app id given'); |
|
140 | + } |
|
141 | + } |
|
142 | + |
|
143 | + /** |
|
144 | + * @param string $app |
|
145 | + * @param string $key |
|
146 | + * @throws \InvalidArgumentException |
|
147 | + */ |
|
148 | + protected function verifyConfigKey($app, $key) { |
|
149 | + if (in_array($key, ['installed_version', 'enabled', 'types'])) { |
|
150 | + throw new \InvalidArgumentException('The given key can not be set'); |
|
151 | + } |
|
152 | + |
|
153 | + if ($app === 'core' && (strpos($key, 'public_') === 0 || strpos($key, 'remote_') === 0)) { |
|
154 | + throw new \InvalidArgumentException('The given key can not be set'); |
|
155 | + } |
|
156 | + } |
|
157 | 157 | } |
@@ -7,22 +7,22 @@ |
||
7 | 7 | use OCP\AppFramework\App; |
8 | 8 | |
9 | 9 | class Application extends App { |
10 | - public function __construct(array $urlParams = array()) { |
|
11 | - parent::__construct('provisioning_api', $urlParams); |
|
10 | + public function __construct(array $urlParams = array()) { |
|
11 | + parent::__construct('provisioning_api', $urlParams); |
|
12 | 12 | |
13 | - $container = $this->getContainer(); |
|
14 | - $server = $container->getServer(); |
|
13 | + $container = $this->getContainer(); |
|
14 | + $server = $container->getServer(); |
|
15 | 15 | |
16 | - $container->registerService('ProvisioningApiMiddleware', function(SimpleContainer $c) use ($server) { |
|
17 | - $user = $server->getUserManager()->get($c['UserId']); |
|
18 | - $isAdmin = $user !== null ? $server->getGroupManager()->isAdmin($user->getUID()) : false; |
|
19 | - $isSubAdmin = $user !== null ? $server->getGroupManager()->getSubAdmin()->isSubAdmin($user) : false; |
|
20 | - return new ProvisioningApiMiddleware( |
|
21 | - $c['ControllerMethodReflector'], |
|
22 | - $isAdmin, |
|
23 | - $isSubAdmin |
|
24 | - ); |
|
25 | - }); |
|
26 | - $container->registerMiddleWare('ProvisioningApiMiddleware'); |
|
27 | - } |
|
16 | + $container->registerService('ProvisioningApiMiddleware', function(SimpleContainer $c) use ($server) { |
|
17 | + $user = $server->getUserManager()->get($c['UserId']); |
|
18 | + $isAdmin = $user !== null ? $server->getGroupManager()->isAdmin($user->getUID()) : false; |
|
19 | + $isSubAdmin = $user !== null ? $server->getGroupManager()->getSubAdmin()->isSubAdmin($user) : false; |
|
20 | + return new ProvisioningApiMiddleware( |
|
21 | + $c['ControllerMethodReflector'], |
|
22 | + $isAdmin, |
|
23 | + $isSubAdmin |
|
24 | + ); |
|
25 | + }); |
|
26 | + $container->registerMiddleWare('ProvisioningApiMiddleware'); |
|
27 | + } |
|
28 | 28 | } |
@@ -5,7 +5,7 @@ |
||
5 | 5 | use OCP\AppFramework\Http; |
6 | 6 | |
7 | 7 | class NotSubAdminException extends \Exception { |
8 | - public function __construct() { |
|
9 | - parent::__construct('Logged in user must be at least a sub admin', Http::STATUS_FORBIDDEN); |
|
10 | - } |
|
8 | + public function __construct() { |
|
9 | + parent::__construct('Logged in user must be at least a sub admin', Http::STATUS_FORBIDDEN); |
|
10 | + } |
|
11 | 11 | } |
12 | 12 | \ No newline at end of file |
@@ -10,55 +10,55 @@ |
||
10 | 10 | |
11 | 11 | class ProvisioningApiMiddleware extends Middleware { |
12 | 12 | |
13 | - /** @var IControllerMethodReflector */ |
|
14 | - private $reflector; |
|
13 | + /** @var IControllerMethodReflector */ |
|
14 | + private $reflector; |
|
15 | 15 | |
16 | - /** @var bool */ |
|
17 | - private $isAdmin; |
|
16 | + /** @var bool */ |
|
17 | + private $isAdmin; |
|
18 | 18 | |
19 | - /** @var bool */ |
|
20 | - private $isSubAdmin; |
|
19 | + /** @var bool */ |
|
20 | + private $isSubAdmin; |
|
21 | 21 | |
22 | - /** |
|
23 | - * ProvisioningApiMiddleware constructor. |
|
24 | - * |
|
25 | - * @param IControllerMethodReflector $reflector |
|
26 | - * @param bool $isAdmin |
|
27 | - * @param bool $isSubAdmin |
|
28 | - */ |
|
29 | - public function __construct( |
|
30 | - IControllerMethodReflector $reflector, |
|
31 | - $isAdmin, |
|
32 | - $isSubAdmin) { |
|
33 | - $this->reflector = $reflector; |
|
34 | - $this->isAdmin = $isAdmin; |
|
35 | - $this->isSubAdmin = $isSubAdmin; |
|
36 | - } |
|
22 | + /** |
|
23 | + * ProvisioningApiMiddleware constructor. |
|
24 | + * |
|
25 | + * @param IControllerMethodReflector $reflector |
|
26 | + * @param bool $isAdmin |
|
27 | + * @param bool $isSubAdmin |
|
28 | + */ |
|
29 | + public function __construct( |
|
30 | + IControllerMethodReflector $reflector, |
|
31 | + $isAdmin, |
|
32 | + $isSubAdmin) { |
|
33 | + $this->reflector = $reflector; |
|
34 | + $this->isAdmin = $isAdmin; |
|
35 | + $this->isSubAdmin = $isSubAdmin; |
|
36 | + } |
|
37 | 37 | |
38 | - /** |
|
39 | - * @param \OCP\AppFramework\Controller $controller |
|
40 | - * @param string $methodName |
|
41 | - * |
|
42 | - * @throws NotSubAdminException |
|
43 | - */ |
|
44 | - public function beforeController($controller, $methodName) { |
|
45 | - if (!$this->isAdmin && !$this->reflector->hasAnnotation('NoSubAdminRequired') && !$this->isSubAdmin) { |
|
46 | - throw new NotSubAdminException(); |
|
47 | - } |
|
48 | - } |
|
38 | + /** |
|
39 | + * @param \OCP\AppFramework\Controller $controller |
|
40 | + * @param string $methodName |
|
41 | + * |
|
42 | + * @throws NotSubAdminException |
|
43 | + */ |
|
44 | + public function beforeController($controller, $methodName) { |
|
45 | + if (!$this->isAdmin && !$this->reflector->hasAnnotation('NoSubAdminRequired') && !$this->isSubAdmin) { |
|
46 | + throw new NotSubAdminException(); |
|
47 | + } |
|
48 | + } |
|
49 | 49 | |
50 | - /** |
|
51 | - * @param \OCP\AppFramework\Controller $controller |
|
52 | - * @param string $methodName |
|
53 | - * @param \Exception $exception |
|
54 | - * @throws \Exception |
|
55 | - * @return Response |
|
56 | - */ |
|
57 | - public function afterException($controller, $methodName, \Exception $exception) { |
|
58 | - if ($exception instanceof NotSubAdminException) { |
|
59 | - throw new OCSException($exception->getMessage(), \OCP\API::RESPOND_UNAUTHORISED); |
|
60 | - } |
|
50 | + /** |
|
51 | + * @param \OCP\AppFramework\Controller $controller |
|
52 | + * @param string $methodName |
|
53 | + * @param \Exception $exception |
|
54 | + * @throws \Exception |
|
55 | + * @return Response |
|
56 | + */ |
|
57 | + public function afterException($controller, $methodName, \Exception $exception) { |
|
58 | + if ($exception instanceof NotSubAdminException) { |
|
59 | + throw new OCSException($exception->getMessage(), \OCP\API::RESPOND_UNAUTHORISED); |
|
60 | + } |
|
61 | 61 | |
62 | - throw $exception; |
|
63 | - } |
|
62 | + throw $exception; |
|
63 | + } |
|
64 | 64 | } |
65 | 65 | \ No newline at end of file |
@@ -1,17 +1,17 @@ |
||
1 | 1 | <?php |
2 | - script('updatenotification', 'admin'); |
|
2 | + script('updatenotification', 'admin'); |
|
3 | 3 | |
4 | - /** @var array $_ */ |
|
5 | - /** @var bool $isNewVersionAvailable */ |
|
6 | - $isNewVersionAvailable = $_['isNewVersionAvailable']; |
|
7 | - /** @var string $newVersionString */ |
|
8 | - $newVersionString = $_['newVersionString']; |
|
9 | - /** @var string $lastCheckedDate */ |
|
10 | - $lastCheckedDate = $_['lastChecked']; |
|
11 | - /** @var array $channels */ |
|
12 | - $channels = $_['channels']; |
|
13 | - /** @var string $currentChannel */ |
|
14 | - $currentChannel = $_['currentChannel']; |
|
4 | + /** @var array $_ */ |
|
5 | + /** @var bool $isNewVersionAvailable */ |
|
6 | + $isNewVersionAvailable = $_['isNewVersionAvailable']; |
|
7 | + /** @var string $newVersionString */ |
|
8 | + $newVersionString = $_['newVersionString']; |
|
9 | + /** @var string $lastCheckedDate */ |
|
10 | + $lastCheckedDate = $_['lastChecked']; |
|
11 | + /** @var array $channels */ |
|
12 | + $channels = $_['channels']; |
|
13 | + /** @var string $currentChannel */ |
|
14 | + $currentChannel = $_['currentChannel']; |
|
15 | 15 | ?> |
16 | 16 | <form id="oca_updatenotification_section" class="followupsection"> |
17 | 17 | <?php if($isNewVersionAvailable === true) { ?> |
@@ -37,139 +37,139 @@ |
||
37 | 37 | use OCP\Settings\ISettings; |
38 | 38 | |
39 | 39 | class AdminController extends Controller implements ISettings { |
40 | - /** @var IJobList */ |
|
41 | - private $jobList; |
|
42 | - /** @var ISecureRandom */ |
|
43 | - private $secureRandom; |
|
44 | - /** @var IConfig */ |
|
45 | - private $config; |
|
46 | - /** @var ITimeFactory */ |
|
47 | - private $timeFactory; |
|
48 | - /** @var UpdateChecker */ |
|
49 | - private $updateChecker; |
|
50 | - /** @var IL10N */ |
|
51 | - private $l10n; |
|
52 | - /** @var IDateTimeFormatter */ |
|
53 | - private $dateTimeFormatter; |
|
54 | - |
|
55 | - /** |
|
56 | - * @param string $appName |
|
57 | - * @param IRequest $request |
|
58 | - * @param IJobList $jobList |
|
59 | - * @param ISecureRandom $secureRandom |
|
60 | - * @param IConfig $config |
|
61 | - * @param ITimeFactory $timeFactory |
|
62 | - * @param IL10N $l10n |
|
63 | - * @param UpdateChecker $updateChecker |
|
64 | - * @param IDateTimeFormatter $dateTimeFormatter |
|
65 | - */ |
|
66 | - public function __construct($appName, |
|
67 | - IRequest $request, |
|
68 | - IJobList $jobList, |
|
69 | - ISecureRandom $secureRandom, |
|
70 | - IConfig $config, |
|
71 | - ITimeFactory $timeFactory, |
|
72 | - IL10N $l10n, |
|
73 | - UpdateChecker $updateChecker, |
|
74 | - IDateTimeFormatter $dateTimeFormatter) { |
|
75 | - parent::__construct($appName, $request); |
|
76 | - $this->jobList = $jobList; |
|
77 | - $this->secureRandom = $secureRandom; |
|
78 | - $this->config = $config; |
|
79 | - $this->timeFactory = $timeFactory; |
|
80 | - $this->l10n = $l10n; |
|
81 | - $this->updateChecker = $updateChecker; |
|
82 | - $this->dateTimeFormatter = $dateTimeFormatter; |
|
83 | - } |
|
84 | - |
|
85 | - /** |
|
86 | - * @return TemplateResponse |
|
87 | - */ |
|
88 | - public function displayPanel() { |
|
89 | - $lastUpdateCheck = $this->dateTimeFormatter->formatDateTime( |
|
90 | - $this->config->getAppValue('core', 'lastupdatedat') |
|
91 | - ); |
|
92 | - |
|
93 | - $channels = [ |
|
94 | - 'daily', |
|
95 | - 'beta', |
|
96 | - 'stable', |
|
97 | - 'production', |
|
98 | - ]; |
|
99 | - $currentChannel = \OCP\Util::getChannel(); |
|
100 | - |
|
101 | - // Remove the currently used channel from the channels list |
|
102 | - if(($key = array_search($currentChannel, $channels)) !== false) { |
|
103 | - unset($channels[$key]); |
|
104 | - } |
|
105 | - $updateState = $this->updateChecker->getUpdateState(); |
|
106 | - |
|
107 | - $notifyGroups = json_decode($this->config->getAppValue('updatenotification', 'notify_groups', '["admin"]'), true); |
|
108 | - |
|
109 | - $params = [ |
|
110 | - 'isNewVersionAvailable' => !empty($updateState['updateAvailable']), |
|
111 | - 'lastChecked' => $lastUpdateCheck, |
|
112 | - 'currentChannel' => $currentChannel, |
|
113 | - 'channels' => $channels, |
|
114 | - 'newVersionString' => (empty($updateState['updateVersion'])) ? '' : $updateState['updateVersion'], |
|
115 | - 'downloadLink' => (empty($updateState['downloadLink'])) ? '' : $updateState['downloadLink'], |
|
116 | - 'updaterEnabled' => (empty($updateState['updaterEnabled'])) ? false : $updateState['updaterEnabled'], |
|
117 | - |
|
118 | - 'notify_groups' => implode('|', $notifyGroups), |
|
119 | - ]; |
|
120 | - |
|
121 | - return new TemplateResponse($this->appName, 'admin', $params, ''); |
|
122 | - } |
|
123 | - |
|
124 | - /** |
|
125 | - * @UseSession |
|
126 | - * |
|
127 | - * @param string $channel |
|
128 | - * @return DataResponse |
|
129 | - */ |
|
130 | - public function setChannel($channel) { |
|
131 | - \OCP\Util::setChannel($channel); |
|
132 | - $this->config->setAppValue('core', 'lastupdatedat', 0); |
|
133 | - return new DataResponse(['status' => 'success', 'data' => ['message' => $this->l10n->t('Channel updated')]]); |
|
134 | - } |
|
135 | - |
|
136 | - /** |
|
137 | - * @return DataResponse |
|
138 | - */ |
|
139 | - public function createCredentials() { |
|
140 | - // Create a new job and store the creation date |
|
141 | - $this->jobList->add('OCA\UpdateNotification\ResetTokenBackgroundJob'); |
|
142 | - $this->config->setAppValue('core', 'updater.secret.created', $this->timeFactory->getTime()); |
|
143 | - |
|
144 | - // Create a new token |
|
145 | - $newToken = $this->secureRandom->generate(64); |
|
146 | - $this->config->setSystemValue('updater.secret', password_hash($newToken, PASSWORD_DEFAULT)); |
|
147 | - |
|
148 | - return new DataResponse($newToken); |
|
149 | - } |
|
150 | - |
|
151 | - /** |
|
152 | - * @return TemplateResponse returns the instance with all parameters set, ready to be rendered |
|
153 | - */ |
|
154 | - public function getForm() { |
|
155 | - return $this->displayPanel(); |
|
156 | - } |
|
157 | - |
|
158 | - /** |
|
159 | - * @return string the section ID, e.g. 'sharing' |
|
160 | - */ |
|
161 | - public function getSection() { |
|
162 | - return 'server'; |
|
163 | - } |
|
164 | - |
|
165 | - /** |
|
166 | - * @return int whether the form should be rather on the top or bottom of |
|
167 | - * the admin section. The forms are arranged in ascending order of the |
|
168 | - * priority values. It is required to return a value between 0 and 100. |
|
169 | - * |
|
170 | - * E.g.: 70 |
|
171 | - */ |
|
172 | - public function getPriority() { |
|
173 | - return 1; |
|
174 | - } |
|
40 | + /** @var IJobList */ |
|
41 | + private $jobList; |
|
42 | + /** @var ISecureRandom */ |
|
43 | + private $secureRandom; |
|
44 | + /** @var IConfig */ |
|
45 | + private $config; |
|
46 | + /** @var ITimeFactory */ |
|
47 | + private $timeFactory; |
|
48 | + /** @var UpdateChecker */ |
|
49 | + private $updateChecker; |
|
50 | + /** @var IL10N */ |
|
51 | + private $l10n; |
|
52 | + /** @var IDateTimeFormatter */ |
|
53 | + private $dateTimeFormatter; |
|
54 | + |
|
55 | + /** |
|
56 | + * @param string $appName |
|
57 | + * @param IRequest $request |
|
58 | + * @param IJobList $jobList |
|
59 | + * @param ISecureRandom $secureRandom |
|
60 | + * @param IConfig $config |
|
61 | + * @param ITimeFactory $timeFactory |
|
62 | + * @param IL10N $l10n |
|
63 | + * @param UpdateChecker $updateChecker |
|
64 | + * @param IDateTimeFormatter $dateTimeFormatter |
|
65 | + */ |
|
66 | + public function __construct($appName, |
|
67 | + IRequest $request, |
|
68 | + IJobList $jobList, |
|
69 | + ISecureRandom $secureRandom, |
|
70 | + IConfig $config, |
|
71 | + ITimeFactory $timeFactory, |
|
72 | + IL10N $l10n, |
|
73 | + UpdateChecker $updateChecker, |
|
74 | + IDateTimeFormatter $dateTimeFormatter) { |
|
75 | + parent::__construct($appName, $request); |
|
76 | + $this->jobList = $jobList; |
|
77 | + $this->secureRandom = $secureRandom; |
|
78 | + $this->config = $config; |
|
79 | + $this->timeFactory = $timeFactory; |
|
80 | + $this->l10n = $l10n; |
|
81 | + $this->updateChecker = $updateChecker; |
|
82 | + $this->dateTimeFormatter = $dateTimeFormatter; |
|
83 | + } |
|
84 | + |
|
85 | + /** |
|
86 | + * @return TemplateResponse |
|
87 | + */ |
|
88 | + public function displayPanel() { |
|
89 | + $lastUpdateCheck = $this->dateTimeFormatter->formatDateTime( |
|
90 | + $this->config->getAppValue('core', 'lastupdatedat') |
|
91 | + ); |
|
92 | + |
|
93 | + $channels = [ |
|
94 | + 'daily', |
|
95 | + 'beta', |
|
96 | + 'stable', |
|
97 | + 'production', |
|
98 | + ]; |
|
99 | + $currentChannel = \OCP\Util::getChannel(); |
|
100 | + |
|
101 | + // Remove the currently used channel from the channels list |
|
102 | + if(($key = array_search($currentChannel, $channels)) !== false) { |
|
103 | + unset($channels[$key]); |
|
104 | + } |
|
105 | + $updateState = $this->updateChecker->getUpdateState(); |
|
106 | + |
|
107 | + $notifyGroups = json_decode($this->config->getAppValue('updatenotification', 'notify_groups', '["admin"]'), true); |
|
108 | + |
|
109 | + $params = [ |
|
110 | + 'isNewVersionAvailable' => !empty($updateState['updateAvailable']), |
|
111 | + 'lastChecked' => $lastUpdateCheck, |
|
112 | + 'currentChannel' => $currentChannel, |
|
113 | + 'channels' => $channels, |
|
114 | + 'newVersionString' => (empty($updateState['updateVersion'])) ? '' : $updateState['updateVersion'], |
|
115 | + 'downloadLink' => (empty($updateState['downloadLink'])) ? '' : $updateState['downloadLink'], |
|
116 | + 'updaterEnabled' => (empty($updateState['updaterEnabled'])) ? false : $updateState['updaterEnabled'], |
|
117 | + |
|
118 | + 'notify_groups' => implode('|', $notifyGroups), |
|
119 | + ]; |
|
120 | + |
|
121 | + return new TemplateResponse($this->appName, 'admin', $params, ''); |
|
122 | + } |
|
123 | + |
|
124 | + /** |
|
125 | + * @UseSession |
|
126 | + * |
|
127 | + * @param string $channel |
|
128 | + * @return DataResponse |
|
129 | + */ |
|
130 | + public function setChannel($channel) { |
|
131 | + \OCP\Util::setChannel($channel); |
|
132 | + $this->config->setAppValue('core', 'lastupdatedat', 0); |
|
133 | + return new DataResponse(['status' => 'success', 'data' => ['message' => $this->l10n->t('Channel updated')]]); |
|
134 | + } |
|
135 | + |
|
136 | + /** |
|
137 | + * @return DataResponse |
|
138 | + */ |
|
139 | + public function createCredentials() { |
|
140 | + // Create a new job and store the creation date |
|
141 | + $this->jobList->add('OCA\UpdateNotification\ResetTokenBackgroundJob'); |
|
142 | + $this->config->setAppValue('core', 'updater.secret.created', $this->timeFactory->getTime()); |
|
143 | + |
|
144 | + // Create a new token |
|
145 | + $newToken = $this->secureRandom->generate(64); |
|
146 | + $this->config->setSystemValue('updater.secret', password_hash($newToken, PASSWORD_DEFAULT)); |
|
147 | + |
|
148 | + return new DataResponse($newToken); |
|
149 | + } |
|
150 | + |
|
151 | + /** |
|
152 | + * @return TemplateResponse returns the instance with all parameters set, ready to be rendered |
|
153 | + */ |
|
154 | + public function getForm() { |
|
155 | + return $this->displayPanel(); |
|
156 | + } |
|
157 | + |
|
158 | + /** |
|
159 | + * @return string the section ID, e.g. 'sharing' |
|
160 | + */ |
|
161 | + public function getSection() { |
|
162 | + return 'server'; |
|
163 | + } |
|
164 | + |
|
165 | + /** |
|
166 | + * @return int whether the form should be rather on the top or bottom of |
|
167 | + * the admin section. The forms are arranged in ascending order of the |
|
168 | + * priority values. It is required to return a value between 0 and 100. |
|
169 | + * |
|
170 | + * E.g.: 70 |
|
171 | + */ |
|
172 | + public function getPriority() { |
|
173 | + return 1; |
|
174 | + } |
|
175 | 175 | } |
@@ -26,48 +26,48 @@ |
||
26 | 26 | use OC\Updater\VersionCheck; |
27 | 27 | |
28 | 28 | class UpdateChecker { |
29 | - /** @var VersionCheck */ |
|
30 | - private $updater; |
|
29 | + /** @var VersionCheck */ |
|
30 | + private $updater; |
|
31 | 31 | |
32 | - /** |
|
33 | - * @param VersionCheck $updater |
|
34 | - */ |
|
35 | - public function __construct(VersionCheck $updater) { |
|
36 | - $this->updater = $updater; |
|
37 | - } |
|
32 | + /** |
|
33 | + * @param VersionCheck $updater |
|
34 | + */ |
|
35 | + public function __construct(VersionCheck $updater) { |
|
36 | + $this->updater = $updater; |
|
37 | + } |
|
38 | 38 | |
39 | - /** |
|
40 | - * @return array |
|
41 | - */ |
|
42 | - public function getUpdateState() { |
|
43 | - $data = $this->updater->check(); |
|
44 | - $result = []; |
|
39 | + /** |
|
40 | + * @return array |
|
41 | + */ |
|
42 | + public function getUpdateState() { |
|
43 | + $data = $this->updater->check(); |
|
44 | + $result = []; |
|
45 | 45 | |
46 | - if(isset($data['version']) && $data['version'] !== '' && $data['version'] !== []) { |
|
47 | - $result['updateAvailable'] = true; |
|
48 | - $result['updateVersion'] = $data['versionstring']; |
|
49 | - $result['updaterEnabled'] = $data['autoupdater'] === '1'; |
|
50 | - if(substr($data['web'], 0, 8) === 'https://') { |
|
51 | - $result['updateLink'] = $data['web']; |
|
52 | - } |
|
53 | - if(substr($data['url'], 0, 8) === 'https://') { |
|
54 | - $result['downloadLink'] = $data['url']; |
|
55 | - } |
|
46 | + if(isset($data['version']) && $data['version'] !== '' && $data['version'] !== []) { |
|
47 | + $result['updateAvailable'] = true; |
|
48 | + $result['updateVersion'] = $data['versionstring']; |
|
49 | + $result['updaterEnabled'] = $data['autoupdater'] === '1'; |
|
50 | + if(substr($data['web'], 0, 8) === 'https://') { |
|
51 | + $result['updateLink'] = $data['web']; |
|
52 | + } |
|
53 | + if(substr($data['url'], 0, 8) === 'https://') { |
|
54 | + $result['downloadLink'] = $data['url']; |
|
55 | + } |
|
56 | 56 | |
57 | - return $result; |
|
58 | - } |
|
57 | + return $result; |
|
58 | + } |
|
59 | 59 | |
60 | - return []; |
|
61 | - } |
|
60 | + return []; |
|
61 | + } |
|
62 | 62 | |
63 | - /** |
|
64 | - * @param array $data |
|
65 | - */ |
|
66 | - public function getJavaScript(array $data) { |
|
67 | - $data['array']['oc_updateState'] = json_encode([ |
|
68 | - 'updateAvailable' => true, |
|
69 | - 'updateVersion' => $this->getUpdateState()['updateVersion'], |
|
70 | - 'updateLink' => isset($this->getUpdateState()['updateLink']) ? $this->getUpdateState()['updateLink'] : '', |
|
71 | - ]); |
|
72 | - } |
|
63 | + /** |
|
64 | + * @param array $data |
|
65 | + */ |
|
66 | + public function getJavaScript(array $data) { |
|
67 | + $data['array']['oc_updateState'] = json_encode([ |
|
68 | + 'updateAvailable' => true, |
|
69 | + 'updateVersion' => $this->getUpdateState()['updateVersion'], |
|
70 | + 'updateLink' => isset($this->getUpdateState()['updateLink']) ? $this->getUpdateState()['updateLink'] : '', |
|
71 | + ]); |
|
72 | + } |
|
73 | 73 | } |