@@ -29,13 +29,13 @@ |
||
29 | 29 | \OCA\Files_Trashbin\Trashbin::registerHooks(); |
30 | 30 | |
31 | 31 | \OCA\Files\App::getNavigationManager()->add(function () { |
32 | - $l = \OC::$server->getL10N('files_trashbin'); |
|
33 | - return [ |
|
34 | - 'id' => 'trashbin', |
|
35 | - 'appname' => 'files_trashbin', |
|
36 | - 'script' => 'list.php', |
|
37 | - 'order' => 50, |
|
38 | - 'name' => $l->t('Deleted files'), |
|
39 | - 'classes' => 'pinned', |
|
40 | - ]; |
|
32 | + $l = \OC::$server->getL10N('files_trashbin'); |
|
33 | + return [ |
|
34 | + 'id' => 'trashbin', |
|
35 | + 'appname' => 'files_trashbin', |
|
36 | + 'script' => 'list.php', |
|
37 | + 'order' => 50, |
|
38 | + 'name' => $l->t('Deleted files'), |
|
39 | + 'classes' => 'pinned', |
|
40 | + ]; |
|
41 | 41 | }); |
@@ -27,173 +27,173 @@ |
||
27 | 27 | namespace OC\Template; |
28 | 28 | |
29 | 29 | abstract class ResourceLocator { |
30 | - protected $theme; |
|
31 | - |
|
32 | - protected $mapping; |
|
33 | - protected $serverroot; |
|
34 | - protected $thirdpartyroot; |
|
35 | - protected $webroot; |
|
36 | - |
|
37 | - protected $resources = array(); |
|
38 | - |
|
39 | - /** @var \OCP\ILogger */ |
|
40 | - protected $logger; |
|
41 | - |
|
42 | - /** |
|
43 | - * @param \OCP\ILogger $logger |
|
44 | - * @param string $theme |
|
45 | - * @param array $core_map |
|
46 | - * @param array $party_map |
|
47 | - */ |
|
48 | - public function __construct(\OCP\ILogger $logger, $theme, $core_map, $party_map) { |
|
49 | - $this->logger = $logger; |
|
50 | - $this->theme = $theme; |
|
51 | - $this->mapping = $core_map + $party_map; |
|
52 | - $this->serverroot = key($core_map); |
|
53 | - $this->thirdpartyroot = key($party_map); |
|
54 | - $this->webroot = $this->mapping[$this->serverroot]; |
|
55 | - } |
|
56 | - |
|
57 | - /** |
|
58 | - * @param string $resource |
|
59 | - */ |
|
60 | - abstract public function doFind($resource); |
|
61 | - |
|
62 | - /** |
|
63 | - * @param string $resource |
|
64 | - */ |
|
65 | - abstract public function doFindTheme($resource); |
|
66 | - |
|
67 | - /** |
|
68 | - * Finds the resources and adds them to the list |
|
69 | - * |
|
70 | - * @param array $resources |
|
71 | - */ |
|
72 | - public function find($resources) { |
|
73 | - foreach ($resources as $resource) { |
|
74 | - try { |
|
75 | - $this->doFind($resource); |
|
76 | - } catch (ResourceNotFoundException $e) { |
|
77 | - $resourceApp = substr($resource, 0, strpos($resource, '/')); |
|
78 | - $this->logger->debug('Could not find resource file "' . $e->getResourcePath() . '"', ['app' => $resourceApp]); |
|
79 | - } |
|
80 | - } |
|
81 | - if (!empty($this->theme)) { |
|
82 | - foreach ($resources as $resource) { |
|
83 | - try { |
|
84 | - $this->doFindTheme($resource); |
|
85 | - } catch (ResourceNotFoundException $e) { |
|
86 | - $resourceApp = substr($resource, 0, strpos($resource, '/')); |
|
87 | - $this->logger->debug('Could not find resource file in theme "' . $e->getResourcePath() . '"', ['app' => $resourceApp]); |
|
88 | - } |
|
89 | - } |
|
90 | - } |
|
91 | - } |
|
92 | - |
|
93 | - /** |
|
94 | - * append the $file resource if exist at $root |
|
95 | - * |
|
96 | - * @param string $root path to check |
|
97 | - * @param string $file the filename |
|
98 | - * @param string|null $webRoot base for path, default map $root to $webRoot |
|
99 | - * @return bool True if the resource was found, false otherwise |
|
100 | - */ |
|
101 | - protected function appendIfExist($root, $file, $webRoot = null) { |
|
102 | - if (is_file($root.'/'.$file)) { |
|
103 | - $this->append($root, $file, $webRoot, false); |
|
104 | - return true; |
|
105 | - } |
|
106 | - return false; |
|
107 | - } |
|
108 | - |
|
109 | - /** |
|
110 | - * Attempt to find the webRoot |
|
111 | - * |
|
112 | - * traverse the potential web roots upwards in the path |
|
113 | - * |
|
114 | - * example: |
|
115 | - * - root: /srv/www/apps/myapp |
|
116 | - * - available mappings: ['/srv/www'] |
|
117 | - * |
|
118 | - * First we check if a mapping for /srv/www/apps/myapp is available, |
|
119 | - * then /srv/www/apps, /srv/www/apps, /srv/www, ... until we find a |
|
120 | - * valid web root |
|
121 | - * |
|
122 | - * @param string $root |
|
123 | - * @return string|null The web root or null on failure |
|
124 | - */ |
|
125 | - protected function findWebRoot($root) { |
|
126 | - $webRoot = null; |
|
127 | - $tmpRoot = $root; |
|
128 | - |
|
129 | - while ($webRoot === null) { |
|
130 | - if (isset($this->mapping[$tmpRoot])) { |
|
131 | - $webRoot = $this->mapping[$tmpRoot]; |
|
132 | - break; |
|
133 | - } |
|
134 | - |
|
135 | - if ($tmpRoot === '/') { |
|
136 | - break; |
|
137 | - } |
|
138 | - |
|
139 | - $tmpRoot = dirname($tmpRoot); |
|
140 | - } |
|
141 | - |
|
142 | - if ($webRoot === null) { |
|
143 | - $realpath = realpath($root); |
|
144 | - |
|
145 | - if ($realpath && ($realpath !== $root)) { |
|
146 | - return $this->findWebRoot($realpath); |
|
147 | - } |
|
148 | - } |
|
149 | - |
|
150 | - return $webRoot; |
|
151 | - } |
|
152 | - |
|
153 | - /** |
|
154 | - * append the $file resource at $root |
|
155 | - * |
|
156 | - * @param string $root path to check |
|
157 | - * @param string $file the filename |
|
158 | - * @param string|null $webRoot base for path, default map $root to $webRoot |
|
159 | - * @param bool $throw Throw an exception, when the route does not exist |
|
160 | - * @throws ResourceNotFoundException Only thrown when $throw is true and the resource is missing |
|
161 | - */ |
|
162 | - protected function append($root, $file, $webRoot = null, $throw = true) { |
|
163 | - |
|
164 | - if (!is_string($root)) { |
|
165 | - if ($throw) { |
|
166 | - throw new ResourceNotFoundException($file, $webRoot); |
|
167 | - } |
|
168 | - return; |
|
169 | - } |
|
170 | - |
|
171 | - if (!$webRoot) { |
|
172 | - $webRoot = $this->findWebRoot($root); |
|
173 | - |
|
174 | - if ($webRoot === null) { |
|
175 | - $webRoot = ''; |
|
176 | - $this->logger->error('ResourceLocator can not find a web root (root: {root}, file: {file}, webRoot: {webRoot}, throw: {throw})', [ |
|
177 | - 'app' => 'lib', |
|
178 | - 'root' => $root, |
|
179 | - 'file' => $file, |
|
180 | - 'webRoot' => $webRoot, |
|
181 | - 'throw' => $throw ? 'true' : 'false' |
|
182 | - ]); |
|
183 | - } |
|
184 | - } |
|
185 | - $this->resources[] = array($root, $webRoot, $file); |
|
186 | - |
|
187 | - if ($throw && !is_file($root . '/' . $file)) { |
|
188 | - throw new ResourceNotFoundException($file, $webRoot); |
|
189 | - } |
|
190 | - } |
|
191 | - |
|
192 | - /** |
|
193 | - * Returns the list of all resources that should be loaded |
|
194 | - * @return array |
|
195 | - */ |
|
196 | - public function getResources() { |
|
197 | - return $this->resources; |
|
198 | - } |
|
30 | + protected $theme; |
|
31 | + |
|
32 | + protected $mapping; |
|
33 | + protected $serverroot; |
|
34 | + protected $thirdpartyroot; |
|
35 | + protected $webroot; |
|
36 | + |
|
37 | + protected $resources = array(); |
|
38 | + |
|
39 | + /** @var \OCP\ILogger */ |
|
40 | + protected $logger; |
|
41 | + |
|
42 | + /** |
|
43 | + * @param \OCP\ILogger $logger |
|
44 | + * @param string $theme |
|
45 | + * @param array $core_map |
|
46 | + * @param array $party_map |
|
47 | + */ |
|
48 | + public function __construct(\OCP\ILogger $logger, $theme, $core_map, $party_map) { |
|
49 | + $this->logger = $logger; |
|
50 | + $this->theme = $theme; |
|
51 | + $this->mapping = $core_map + $party_map; |
|
52 | + $this->serverroot = key($core_map); |
|
53 | + $this->thirdpartyroot = key($party_map); |
|
54 | + $this->webroot = $this->mapping[$this->serverroot]; |
|
55 | + } |
|
56 | + |
|
57 | + /** |
|
58 | + * @param string $resource |
|
59 | + */ |
|
60 | + abstract public function doFind($resource); |
|
61 | + |
|
62 | + /** |
|
63 | + * @param string $resource |
|
64 | + */ |
|
65 | + abstract public function doFindTheme($resource); |
|
66 | + |
|
67 | + /** |
|
68 | + * Finds the resources and adds them to the list |
|
69 | + * |
|
70 | + * @param array $resources |
|
71 | + */ |
|
72 | + public function find($resources) { |
|
73 | + foreach ($resources as $resource) { |
|
74 | + try { |
|
75 | + $this->doFind($resource); |
|
76 | + } catch (ResourceNotFoundException $e) { |
|
77 | + $resourceApp = substr($resource, 0, strpos($resource, '/')); |
|
78 | + $this->logger->debug('Could not find resource file "' . $e->getResourcePath() . '"', ['app' => $resourceApp]); |
|
79 | + } |
|
80 | + } |
|
81 | + if (!empty($this->theme)) { |
|
82 | + foreach ($resources as $resource) { |
|
83 | + try { |
|
84 | + $this->doFindTheme($resource); |
|
85 | + } catch (ResourceNotFoundException $e) { |
|
86 | + $resourceApp = substr($resource, 0, strpos($resource, '/')); |
|
87 | + $this->logger->debug('Could not find resource file in theme "' . $e->getResourcePath() . '"', ['app' => $resourceApp]); |
|
88 | + } |
|
89 | + } |
|
90 | + } |
|
91 | + } |
|
92 | + |
|
93 | + /** |
|
94 | + * append the $file resource if exist at $root |
|
95 | + * |
|
96 | + * @param string $root path to check |
|
97 | + * @param string $file the filename |
|
98 | + * @param string|null $webRoot base for path, default map $root to $webRoot |
|
99 | + * @return bool True if the resource was found, false otherwise |
|
100 | + */ |
|
101 | + protected function appendIfExist($root, $file, $webRoot = null) { |
|
102 | + if (is_file($root.'/'.$file)) { |
|
103 | + $this->append($root, $file, $webRoot, false); |
|
104 | + return true; |
|
105 | + } |
|
106 | + return false; |
|
107 | + } |
|
108 | + |
|
109 | + /** |
|
110 | + * Attempt to find the webRoot |
|
111 | + * |
|
112 | + * traverse the potential web roots upwards in the path |
|
113 | + * |
|
114 | + * example: |
|
115 | + * - root: /srv/www/apps/myapp |
|
116 | + * - available mappings: ['/srv/www'] |
|
117 | + * |
|
118 | + * First we check if a mapping for /srv/www/apps/myapp is available, |
|
119 | + * then /srv/www/apps, /srv/www/apps, /srv/www, ... until we find a |
|
120 | + * valid web root |
|
121 | + * |
|
122 | + * @param string $root |
|
123 | + * @return string|null The web root or null on failure |
|
124 | + */ |
|
125 | + protected function findWebRoot($root) { |
|
126 | + $webRoot = null; |
|
127 | + $tmpRoot = $root; |
|
128 | + |
|
129 | + while ($webRoot === null) { |
|
130 | + if (isset($this->mapping[$tmpRoot])) { |
|
131 | + $webRoot = $this->mapping[$tmpRoot]; |
|
132 | + break; |
|
133 | + } |
|
134 | + |
|
135 | + if ($tmpRoot === '/') { |
|
136 | + break; |
|
137 | + } |
|
138 | + |
|
139 | + $tmpRoot = dirname($tmpRoot); |
|
140 | + } |
|
141 | + |
|
142 | + if ($webRoot === null) { |
|
143 | + $realpath = realpath($root); |
|
144 | + |
|
145 | + if ($realpath && ($realpath !== $root)) { |
|
146 | + return $this->findWebRoot($realpath); |
|
147 | + } |
|
148 | + } |
|
149 | + |
|
150 | + return $webRoot; |
|
151 | + } |
|
152 | + |
|
153 | + /** |
|
154 | + * append the $file resource at $root |
|
155 | + * |
|
156 | + * @param string $root path to check |
|
157 | + * @param string $file the filename |
|
158 | + * @param string|null $webRoot base for path, default map $root to $webRoot |
|
159 | + * @param bool $throw Throw an exception, when the route does not exist |
|
160 | + * @throws ResourceNotFoundException Only thrown when $throw is true and the resource is missing |
|
161 | + */ |
|
162 | + protected function append($root, $file, $webRoot = null, $throw = true) { |
|
163 | + |
|
164 | + if (!is_string($root)) { |
|
165 | + if ($throw) { |
|
166 | + throw new ResourceNotFoundException($file, $webRoot); |
|
167 | + } |
|
168 | + return; |
|
169 | + } |
|
170 | + |
|
171 | + if (!$webRoot) { |
|
172 | + $webRoot = $this->findWebRoot($root); |
|
173 | + |
|
174 | + if ($webRoot === null) { |
|
175 | + $webRoot = ''; |
|
176 | + $this->logger->error('ResourceLocator can not find a web root (root: {root}, file: {file}, webRoot: {webRoot}, throw: {throw})', [ |
|
177 | + 'app' => 'lib', |
|
178 | + 'root' => $root, |
|
179 | + 'file' => $file, |
|
180 | + 'webRoot' => $webRoot, |
|
181 | + 'throw' => $throw ? 'true' : 'false' |
|
182 | + ]); |
|
183 | + } |
|
184 | + } |
|
185 | + $this->resources[] = array($root, $webRoot, $file); |
|
186 | + |
|
187 | + if ($throw && !is_file($root . '/' . $file)) { |
|
188 | + throw new ResourceNotFoundException($file, $webRoot); |
|
189 | + } |
|
190 | + } |
|
191 | + |
|
192 | + /** |
|
193 | + * Returns the list of all resources that should be loaded |
|
194 | + * @return array |
|
195 | + */ |
|
196 | + public function getResources() { |
|
197 | + return $this->resources; |
|
198 | + } |
|
199 | 199 | } |
@@ -31,76 +31,76 @@ |
||
31 | 31 | |
32 | 32 | class SameSiteCookieMiddleware extends Middleware { |
33 | 33 | |
34 | - /** @var Request */ |
|
35 | - private $request; |
|
36 | - |
|
37 | - /** @var ControllerMethodReflector */ |
|
38 | - private $reflector; |
|
39 | - |
|
40 | - public function __construct(Request $request, |
|
41 | - ControllerMethodReflector $reflector) { |
|
42 | - $this->request = $request; |
|
43 | - $this->reflector = $reflector; |
|
44 | - } |
|
45 | - |
|
46 | - public function beforeController($controller, $methodName) { |
|
47 | - $requestUri = $this->request->getScriptName(); |
|
48 | - $processingScript = explode('/', $requestUri); |
|
49 | - $processingScript = $processingScript[count($processingScript)-1]; |
|
50 | - |
|
51 | - if ($processingScript !== 'index.php') { |
|
52 | - return; |
|
53 | - } |
|
54 | - |
|
55 | - $noSSC = $this->reflector->hasAnnotation('NoSameSiteCookieRequired'); |
|
56 | - if ($noSSC) { |
|
57 | - return; |
|
58 | - } |
|
59 | - |
|
60 | - if (!$this->request->passesLaxCookieCheck()) { |
|
61 | - throw new LaxSameSiteCookieFailedException(); |
|
62 | - } |
|
63 | - } |
|
64 | - |
|
65 | - public function afterException($controller, $methodName, \Exception $exception) { |
|
66 | - if ($exception instanceof LaxSameSiteCookieFailedException) { |
|
67 | - $respone = new Response(); |
|
68 | - $respone->setStatus(Http::STATUS_FOUND); |
|
69 | - $respone->addHeader('Location', $this->request->getRequestUri()); |
|
70 | - |
|
71 | - $this->setSameSiteCookie(); |
|
72 | - |
|
73 | - return $respone; |
|
74 | - } |
|
75 | - |
|
76 | - throw $exception; |
|
77 | - } |
|
78 | - |
|
79 | - protected function setSameSiteCookie() { |
|
80 | - $cookieParams = $this->request->getCookieParams(); |
|
81 | - $secureCookie = ($cookieParams['secure'] === true) ? 'secure; ' : ''; |
|
82 | - $policies = [ |
|
83 | - 'lax', |
|
84 | - 'strict', |
|
85 | - ]; |
|
86 | - |
|
87 | - // Append __Host to the cookie if it meets the requirements |
|
88 | - $cookiePrefix = ''; |
|
89 | - if($cookieParams['secure'] === true && $cookieParams['path'] === '/') { |
|
90 | - $cookiePrefix = '__Host-'; |
|
91 | - } |
|
92 | - |
|
93 | - foreach($policies as $policy) { |
|
94 | - header( |
|
95 | - sprintf( |
|
96 | - 'Set-Cookie: %snc_sameSiteCookie%s=true; path=%s; httponly;' . $secureCookie . 'expires=Fri, 31-Dec-2100 23:59:59 GMT; SameSite=%s', |
|
97 | - $cookiePrefix, |
|
98 | - $policy, |
|
99 | - $cookieParams['path'], |
|
100 | - $policy |
|
101 | - ), |
|
102 | - false |
|
103 | - ); |
|
104 | - } |
|
105 | - } |
|
34 | + /** @var Request */ |
|
35 | + private $request; |
|
36 | + |
|
37 | + /** @var ControllerMethodReflector */ |
|
38 | + private $reflector; |
|
39 | + |
|
40 | + public function __construct(Request $request, |
|
41 | + ControllerMethodReflector $reflector) { |
|
42 | + $this->request = $request; |
|
43 | + $this->reflector = $reflector; |
|
44 | + } |
|
45 | + |
|
46 | + public function beforeController($controller, $methodName) { |
|
47 | + $requestUri = $this->request->getScriptName(); |
|
48 | + $processingScript = explode('/', $requestUri); |
|
49 | + $processingScript = $processingScript[count($processingScript)-1]; |
|
50 | + |
|
51 | + if ($processingScript !== 'index.php') { |
|
52 | + return; |
|
53 | + } |
|
54 | + |
|
55 | + $noSSC = $this->reflector->hasAnnotation('NoSameSiteCookieRequired'); |
|
56 | + if ($noSSC) { |
|
57 | + return; |
|
58 | + } |
|
59 | + |
|
60 | + if (!$this->request->passesLaxCookieCheck()) { |
|
61 | + throw new LaxSameSiteCookieFailedException(); |
|
62 | + } |
|
63 | + } |
|
64 | + |
|
65 | + public function afterException($controller, $methodName, \Exception $exception) { |
|
66 | + if ($exception instanceof LaxSameSiteCookieFailedException) { |
|
67 | + $respone = new Response(); |
|
68 | + $respone->setStatus(Http::STATUS_FOUND); |
|
69 | + $respone->addHeader('Location', $this->request->getRequestUri()); |
|
70 | + |
|
71 | + $this->setSameSiteCookie(); |
|
72 | + |
|
73 | + return $respone; |
|
74 | + } |
|
75 | + |
|
76 | + throw $exception; |
|
77 | + } |
|
78 | + |
|
79 | + protected function setSameSiteCookie() { |
|
80 | + $cookieParams = $this->request->getCookieParams(); |
|
81 | + $secureCookie = ($cookieParams['secure'] === true) ? 'secure; ' : ''; |
|
82 | + $policies = [ |
|
83 | + 'lax', |
|
84 | + 'strict', |
|
85 | + ]; |
|
86 | + |
|
87 | + // Append __Host to the cookie if it meets the requirements |
|
88 | + $cookiePrefix = ''; |
|
89 | + if($cookieParams['secure'] === true && $cookieParams['path'] === '/') { |
|
90 | + $cookiePrefix = '__Host-'; |
|
91 | + } |
|
92 | + |
|
93 | + foreach($policies as $policy) { |
|
94 | + header( |
|
95 | + sprintf( |
|
96 | + 'Set-Cookie: %snc_sameSiteCookie%s=true; path=%s; httponly;' . $secureCookie . 'expires=Fri, 31-Dec-2100 23:59:59 GMT; SameSite=%s', |
|
97 | + $cookiePrefix, |
|
98 | + $policy, |
|
99 | + $cookieParams['path'], |
|
100 | + $policy |
|
101 | + ), |
|
102 | + false |
|
103 | + ); |
|
104 | + } |
|
105 | + } |
|
106 | 106 | } |
@@ -32,7 +32,7 @@ |
||
32 | 32 | * @package OC\AppFramework\Middleware\Security\Exceptions |
33 | 33 | */ |
34 | 34 | class LaxSameSiteCookieFailedException extends SecurityException { |
35 | - public function __construct() { |
|
36 | - parent::__construct('Lax Same Site Cookie is invalid in request.', Http::STATUS_PRECONDITION_FAILED); |
|
37 | - } |
|
35 | + public function __construct() { |
|
36 | + parent::__construct('Lax Same Site Cookie is invalid in request.', Http::STATUS_PRECONDITION_FAILED); |
|
37 | + } |
|
38 | 38 | } |
@@ -29,60 +29,60 @@ |
||
29 | 29 | use OCP\IURLGenerator; |
30 | 30 | |
31 | 31 | class ContactsManager { |
32 | - /** @var CardDavBackend */ |
|
33 | - private $backend; |
|
32 | + /** @var CardDavBackend */ |
|
33 | + private $backend; |
|
34 | 34 | |
35 | - /** @var IL10N */ |
|
36 | - private $l10n; |
|
35 | + /** @var IL10N */ |
|
36 | + private $l10n; |
|
37 | 37 | |
38 | - /** |
|
39 | - * ContactsManager constructor. |
|
40 | - * |
|
41 | - * @param CardDavBackend $backend |
|
42 | - * @param IL10N $l10n |
|
43 | - */ |
|
44 | - public function __construct(CardDavBackend $backend, IL10N $l10n) { |
|
45 | - $this->backend = $backend; |
|
46 | - $this->l10n = $l10n; |
|
47 | - } |
|
38 | + /** |
|
39 | + * ContactsManager constructor. |
|
40 | + * |
|
41 | + * @param CardDavBackend $backend |
|
42 | + * @param IL10N $l10n |
|
43 | + */ |
|
44 | + public function __construct(CardDavBackend $backend, IL10N $l10n) { |
|
45 | + $this->backend = $backend; |
|
46 | + $this->l10n = $l10n; |
|
47 | + } |
|
48 | 48 | |
49 | - /** |
|
50 | - * @param IManager $cm |
|
51 | - * @param string $userId |
|
52 | - * @param IURLGenerator $urlGenerator |
|
53 | - */ |
|
54 | - public function setupContactsProvider(IManager $cm, $userId, IURLGenerator $urlGenerator) { |
|
55 | - $addressBooks = $this->backend->getAddressBooksForUser("principals/users/$userId"); |
|
56 | - $this->register($cm, $addressBooks, $urlGenerator); |
|
57 | - $this->setupSystemContactsProvider($cm, $urlGenerator); |
|
58 | - } |
|
49 | + /** |
|
50 | + * @param IManager $cm |
|
51 | + * @param string $userId |
|
52 | + * @param IURLGenerator $urlGenerator |
|
53 | + */ |
|
54 | + public function setupContactsProvider(IManager $cm, $userId, IURLGenerator $urlGenerator) { |
|
55 | + $addressBooks = $this->backend->getAddressBooksForUser("principals/users/$userId"); |
|
56 | + $this->register($cm, $addressBooks, $urlGenerator); |
|
57 | + $this->setupSystemContactsProvider($cm, $urlGenerator); |
|
58 | + } |
|
59 | 59 | |
60 | - /** |
|
61 | - * @param IManager $cm |
|
62 | - * @param IURLGenerator $urlGenerator |
|
63 | - */ |
|
64 | - public function setupSystemContactsProvider(IManager $cm, IURLGenerator $urlGenerator) { |
|
65 | - $addressBooks = $this->backend->getAddressBooksForUser("principals/system/system"); |
|
66 | - $this->register($cm, $addressBooks, $urlGenerator); |
|
67 | - } |
|
60 | + /** |
|
61 | + * @param IManager $cm |
|
62 | + * @param IURLGenerator $urlGenerator |
|
63 | + */ |
|
64 | + public function setupSystemContactsProvider(IManager $cm, IURLGenerator $urlGenerator) { |
|
65 | + $addressBooks = $this->backend->getAddressBooksForUser("principals/system/system"); |
|
66 | + $this->register($cm, $addressBooks, $urlGenerator); |
|
67 | + } |
|
68 | 68 | |
69 | - /** |
|
70 | - * @param IManager $cm |
|
71 | - * @param $addressBooks |
|
72 | - * @param IURLGenerator $urlGenerator |
|
73 | - */ |
|
74 | - private function register(IManager $cm, $addressBooks, $urlGenerator) { |
|
75 | - foreach ($addressBooks as $addressBookInfo) { |
|
76 | - $addressBook = new \OCA\DAV\CardDAV\AddressBook($this->backend, $addressBookInfo, $this->l10n); |
|
77 | - $cm->registerAddressBook( |
|
78 | - new AddressBookImpl( |
|
79 | - $addressBook, |
|
80 | - $addressBookInfo, |
|
81 | - $this->backend, |
|
82 | - $urlGenerator |
|
83 | - ) |
|
84 | - ); |
|
85 | - } |
|
86 | - } |
|
69 | + /** |
|
70 | + * @param IManager $cm |
|
71 | + * @param $addressBooks |
|
72 | + * @param IURLGenerator $urlGenerator |
|
73 | + */ |
|
74 | + private function register(IManager $cm, $addressBooks, $urlGenerator) { |
|
75 | + foreach ($addressBooks as $addressBookInfo) { |
|
76 | + $addressBook = new \OCA\DAV\CardDAV\AddressBook($this->backend, $addressBookInfo, $this->l10n); |
|
77 | + $cm->registerAddressBook( |
|
78 | + new AddressBookImpl( |
|
79 | + $addressBook, |
|
80 | + $addressBookInfo, |
|
81 | + $this->backend, |
|
82 | + $urlGenerator |
|
83 | + ) |
|
84 | + ); |
|
85 | + } |
|
86 | + } |
|
87 | 87 | |
88 | 88 | } |
@@ -11,15 +11,15 @@ |
||
11 | 11 | */ |
12 | 12 | class Version1002Date20170926101419 extends BigIntMigration { |
13 | 13 | |
14 | - /** |
|
15 | - * @return array Returns an array with the following structure |
|
16 | - * ['table1' => ['column1', 'column2'], ...] |
|
17 | - * @since 13.0.0 |
|
18 | - */ |
|
19 | - protected function getColumnsByTable() { |
|
20 | - return [ |
|
21 | - 'twofactor_backupcodes' => ['id'], |
|
22 | - ]; |
|
23 | - } |
|
14 | + /** |
|
15 | + * @return array Returns an array with the following structure |
|
16 | + * ['table1' => ['column1', 'column2'], ...] |
|
17 | + * @since 13.0.0 |
|
18 | + */ |
|
19 | + protected function getColumnsByTable() { |
|
20 | + return [ |
|
21 | + 'twofactor_backupcodes' => ['id'], |
|
22 | + ]; |
|
23 | + } |
|
24 | 24 | |
25 | 25 | } |
@@ -11,25 +11,25 @@ |
||
11 | 11 | */ |
12 | 12 | class Version1004Date20170926103422 extends BigIntMigration { |
13 | 13 | |
14 | - /** |
|
15 | - * @return array Returns an array with the following structure |
|
16 | - * ['table1' => ['column1', 'column2'], ...] |
|
17 | - * @since 13.0.0 |
|
18 | - */ |
|
19 | - protected function getColumnsByTable() { |
|
20 | - return [ |
|
21 | - 'addressbooks' => ['id'], |
|
22 | - 'addressbookchanges' => ['id', 'addressbookid'], |
|
23 | - 'calendars' => ['id'], |
|
24 | - 'calendarchanges' => ['id', 'calendarid'], |
|
25 | - 'calendarobjects' => ['id', 'calendarid'], |
|
26 | - 'calendarobjects_props' => ['id', 'calendarid', 'objectid'], |
|
27 | - 'calendarsubscriptions' => ['id'], |
|
28 | - 'cards' => ['id', 'addressbookid'], |
|
29 | - 'cards_properties' => ['id', 'addressbookid', 'cardid'], |
|
30 | - 'dav_shares' => ['id', 'resourceid'], |
|
31 | - 'schedulingobjects' => ['id'], |
|
32 | - ]; |
|
33 | - } |
|
14 | + /** |
|
15 | + * @return array Returns an array with the following structure |
|
16 | + * ['table1' => ['column1', 'column2'], ...] |
|
17 | + * @since 13.0.0 |
|
18 | + */ |
|
19 | + protected function getColumnsByTable() { |
|
20 | + return [ |
|
21 | + 'addressbooks' => ['id'], |
|
22 | + 'addressbookchanges' => ['id', 'addressbookid'], |
|
23 | + 'calendars' => ['id'], |
|
24 | + 'calendarchanges' => ['id', 'calendarid'], |
|
25 | + 'calendarobjects' => ['id', 'calendarid'], |
|
26 | + 'calendarobjects_props' => ['id', 'calendarid', 'objectid'], |
|
27 | + 'calendarsubscriptions' => ['id'], |
|
28 | + 'cards' => ['id', 'addressbookid'], |
|
29 | + 'cards_properties' => ['id', 'addressbookid', 'cardid'], |
|
30 | + 'dav_shares' => ['id', 'resourceid'], |
|
31 | + 'schedulingobjects' => ['id'], |
|
32 | + ]; |
|
33 | + } |
|
34 | 34 | |
35 | 35 | } |
@@ -10,22 +10,22 @@ |
||
10 | 10 | interface IContactsStore { |
11 | 11 | |
12 | 12 | |
13 | - /** |
|
14 | - * @param IUser $user |
|
15 | - * @param $filter |
|
16 | - * @return IEntry[] |
|
17 | - * @since 13.0.0 |
|
18 | - */ |
|
19 | - public function getContacts(IUser $user, $filter); |
|
13 | + /** |
|
14 | + * @param IUser $user |
|
15 | + * @param $filter |
|
16 | + * @return IEntry[] |
|
17 | + * @since 13.0.0 |
|
18 | + */ |
|
19 | + public function getContacts(IUser $user, $filter); |
|
20 | 20 | |
21 | - /** |
|
22 | - * @brief finds a contact by specifying the property to search on ($shareType) and the value ($shareWith) |
|
23 | - * @param IUser $user |
|
24 | - * @param integer $shareType |
|
25 | - * @param string $shareWith |
|
26 | - * @return IEntry|null |
|
27 | - * @since 13.0.0 |
|
28 | - */ |
|
29 | - public function findOne(IUser $user, $shareType, $shareWith); |
|
21 | + /** |
|
22 | + * @brief finds a contact by specifying the property to search on ($shareType) and the value ($shareWith) |
|
23 | + * @param IUser $user |
|
24 | + * @param integer $shareType |
|
25 | + * @param string $shareWith |
|
26 | + * @return IEntry|null |
|
27 | + * @since 13.0.0 |
|
28 | + */ |
|
29 | + public function findOne(IUser $user, $shareType, $shareWith); |
|
30 | 30 | |
31 | 31 | } |
@@ -32,14 +32,14 @@ |
||
32 | 32 | */ |
33 | 33 | class Collection extends \Sabre\CalDAV\Principal\Collection { |
34 | 34 | |
35 | - /** |
|
36 | - * Returns a child object based on principal information |
|
37 | - * |
|
38 | - * @param array $principalInfo |
|
39 | - * @return User |
|
40 | - */ |
|
41 | - function getChildForPrincipal(array $principalInfo) { |
|
42 | - return new User($this->principalBackend, $principalInfo); |
|
43 | - } |
|
35 | + /** |
|
36 | + * Returns a child object based on principal information |
|
37 | + * |
|
38 | + * @param array $principalInfo |
|
39 | + * @return User |
|
40 | + */ |
|
41 | + function getChildForPrincipal(array $principalInfo) { |
|
42 | + return new User($this->principalBackend, $principalInfo); |
|
43 | + } |
|
44 | 44 | |
45 | 45 | } |