@@ -32,8 +32,7 @@ |
||
32 | 32 | use OCP\IURLGenerator; |
33 | 33 | use OCP\Settings\ISettings; |
34 | 34 | |
35 | -class Admin implements ISettings |
|
36 | -{ |
|
35 | +class Admin implements ISettings { |
|
37 | 36 | /** @var IURLGenerator */ |
38 | 37 | private $urlGenerator; |
39 | 38 |
@@ -61,7 +61,7 @@ |
||
61 | 61 | $exampleProxyUrl = $this->urlGenerator->getBaseUrl() . '/sites/' . urlencode($exampleSite); |
62 | 62 | $exampleFullUrl = $this->urlGenerator->linkToRouteAbsolute( |
63 | 63 | Application::APP_NAME . '.Pico.getPage', |
64 | - [ 'site' => $exampleSite, 'page' => '' ] |
|
64 | + ['site' => $exampleSite, 'page' => ''] |
|
65 | 65 | ); |
66 | 66 | |
67 | 67 | $internalBaseUrl = $this->urlGenerator->getBaseUrl() . '/index.php/apps/' . Application::APP_NAME; |
@@ -52,8 +52,7 @@ |
||
52 | 52 | use OCP\IL10N; |
53 | 53 | use OCP\IURLGenerator; |
54 | 54 | |
55 | -class Website extends WebsiteCore |
|
56 | -{ |
|
55 | +class Website extends WebsiteCore { |
|
57 | 56 | /** @var int */ |
58 | 57 | const SITE_LENGTH_MIN = 3; |
59 | 58 |
@@ -233,7 +233,7 @@ discard block |
||
233 | 233 | |
234 | 234 | /** @var OCFolder $viewerOCFolder */ |
235 | 235 | $viewerOCFolder = $this->rootFolder->getUserFolder($this->getViewer()); |
236 | - $viewerAccessClosure = function (OCNode $node) use ($viewerOCFolder) { |
|
236 | + $viewerAccessClosure = function(OCNode $node) use ($viewerOCFolder) { |
|
237 | 237 | $nodeId = $node->getId(); |
238 | 238 | |
239 | 239 | $viewerNodes = $viewerOCFolder->getById($nodeId); |
@@ -424,7 +424,7 @@ discard block |
||
424 | 424 | { |
425 | 425 | if (!$this->getProxyRequest()) { |
426 | 426 | $route = Application::APP_NAME . '.Pico.getPage'; |
427 | - $parameters = [ 'site' => $this->getSite(), 'page' => '' ]; |
|
427 | + $parameters = ['site' => $this->getSite(), 'page' => '']; |
|
428 | 428 | return $this->urlGenerator->linkToRoute($route, $parameters) . '/'; |
429 | 429 | } else { |
430 | 430 | return \OC::$WEBROOT . '/sites/' . urlencode($this->getSite()) . '/'; |
@@ -188,7 +188,7 @@ discard block |
||
188 | 188 | */ |
189 | 189 | public function setType(int $type): self |
190 | 190 | { |
191 | - if (!in_array($type, [ self::TYPE_PUBLIC, self::TYPE_PRIVATE ], true)) { |
|
191 | + if (!in_array($type, [self::TYPE_PUBLIC, self::TYPE_PRIVATE], true)) { |
|
192 | 192 | throw new \UnexpectedValueException(); |
193 | 193 | } |
194 | 194 | |
@@ -411,15 +411,15 @@ discard block |
||
411 | 411 | |
412 | 412 | $creation = 0; |
413 | 413 | if (!empty($data['creation'])) { |
414 | - $creation = is_numeric($data['creation']) ? (int) $data['creation'] : strtotime($data['creation']); |
|
414 | + $creation = is_numeric($data['creation']) ? (int)$data['creation'] : strtotime($data['creation']); |
|
415 | 415 | } |
416 | 416 | |
417 | - $this->setId(isset($data['id']) ? (int) $data['id'] : 0) |
|
417 | + $this->setId(isset($data['id']) ? (int)$data['id'] : 0) |
|
418 | 418 | ->setUserId($data['user_id']) |
419 | 419 | ->setName($data['name']) |
420 | 420 | ->setSite($data['site']) |
421 | 421 | ->setTheme($data['theme'] ?? 'default') |
422 | - ->setType(isset($data['type']) ? (int) $data['type'] : self::TYPE_PUBLIC) |
|
422 | + ->setType(isset($data['type']) ? (int)$data['type'] : self::TYPE_PUBLIC) |
|
423 | 423 | ->setOptions($options) |
424 | 424 | ->setPath($data['path']) |
425 | 425 | ->setCreation($creation) |
@@ -25,8 +25,7 @@ |
||
25 | 25 | |
26 | 26 | namespace OCA\CMSPico\Model; |
27 | 27 | |
28 | -class WebsiteCore implements \JsonSerializable |
|
29 | -{ |
|
28 | +class WebsiteCore implements \JsonSerializable { |
|
30 | 29 | /** @var int */ |
31 | 30 | const TYPE_PUBLIC = 1; |
32 | 31 |
@@ -27,8 +27,7 @@ |
||
27 | 27 | |
28 | 28 | use OCA\CMSPico\Files\MemoryFile; |
29 | 29 | |
30 | -class TemplateFile extends MemoryFile |
|
31 | -{ |
|
30 | +class TemplateFile extends MemoryFile { |
|
32 | 31 | /** |
33 | 32 | * @param array<string,string> $data |
34 | 33 | */ |
@@ -56,8 +56,7 @@ |
||
56 | 56 | use OCP\IL10N; |
57 | 57 | use OCP\IRequest; |
58 | 58 | |
59 | -class PicoController extends Controller |
|
60 | -{ |
|
59 | +class PicoController extends Controller { |
|
61 | 60 | /** @var string|null */ |
62 | 61 | private $userId; |
63 | 62 |
@@ -150,7 +150,7 @@ |
||
150 | 150 | try { |
151 | 151 | $picoAsset = $this->websitesService->getAsset($site, $asset, $this->userId); |
152 | 152 | |
153 | - $response = new PicoAssetResponse($picoAsset, (bool) $assetsETag); |
|
153 | + $response = new PicoAssetResponse($picoAsset, (bool)$assetsETag); |
|
154 | 154 | |
155 | 155 | $assetETag = $picoAsset->getETag(); |
156 | 156 | $clientETag = $this->request->getHeader('If-None-Match'); |
@@ -45,8 +45,7 @@ |
||
45 | 45 | use OCP\ILogger; |
46 | 46 | use OCP\IRequest; |
47 | 47 | |
48 | -class SettingsController extends Controller |
|
49 | -{ |
|
48 | +class SettingsController extends Controller { |
|
50 | 49 | /** @var string */ |
51 | 50 | private $userId; |
52 | 51 |
@@ -115,7 +115,7 @@ discard block |
||
115 | 115 | */ |
116 | 116 | public function getPersonalWebsites(): DataResponse |
117 | 117 | { |
118 | - $data = [ 'websites' => $this->websitesService->getWebsitesFromUser($this->userId) ]; |
|
118 | + $data = ['websites' => $this->websitesService->getWebsitesFromUser($this->userId)]; |
|
119 | 119 | return new DataResponse($data, Http::STATUS_OK); |
120 | 120 | } |
121 | 121 | |
@@ -143,15 +143,15 @@ discard block |
||
143 | 143 | } catch (\Exception $e) { |
144 | 144 | $data = []; |
145 | 145 | if ($e instanceof WebsiteExistsException) { |
146 | - $data['form_error'] = [ 'field' => 'site', 'message' => $this->l10n->t('Website exists.') ]; |
|
146 | + $data['form_error'] = ['field' => 'site', 'message' => $this->l10n->t('Website exists.')]; |
|
147 | 147 | } elseif (($e instanceof WebsiteInvalidDataException) && $e->getField()) { |
148 | - $data['form_error'] = [ 'field' => $e->getField(), 'message' => $e->getMessage() ]; |
|
148 | + $data['form_error'] = ['field' => $e->getField(), 'message' => $e->getMessage()]; |
|
149 | 149 | } elseif ($e instanceof ThemeNotFoundException) { |
150 | - $data['form_error'] = [ 'field' => 'theme', 'message' => $this->l10n->t('Theme not found.') ]; |
|
150 | + $data['form_error'] = ['field' => 'theme', 'message' => $this->l10n->t('Theme not found.')]; |
|
151 | 151 | } elseif ($e instanceof ThemeNotCompatibleException) { |
152 | - $data['form_error'] = [ 'field' => 'theme', 'message' => $this->l10n->t($e->getReason()) ]; |
|
152 | + $data['form_error'] = ['field' => 'theme', 'message' => $this->l10n->t($e->getReason())]; |
|
153 | 153 | } elseif ($e instanceof TemplateNotFoundException) { |
154 | - $data['form_error'] = [ 'field' => 'template', 'message' => $this->l10n->t('Template not found.') ]; |
|
154 | + $data['form_error'] = ['field' => 'template', 'message' => $this->l10n->t('Template not found.')]; |
|
155 | 155 | } |
156 | 156 | |
157 | 157 | return $this->createErrorResponse($e, $data); |
@@ -176,7 +176,7 @@ discard block |
||
176 | 176 | foreach ($data as $key => $value) { |
177 | 177 | switch ($key) { |
178 | 178 | case 'type': |
179 | - $website->setType((int) $value); |
|
179 | + $website->setType((int)$value); |
|
180 | 180 | break; |
181 | 181 | |
182 | 182 | case 'theme': |
@@ -194,15 +194,15 @@ discard block |
||
194 | 194 | } catch (\Exception $e) { |
195 | 195 | $data = []; |
196 | 196 | if (($e instanceof WebsiteNotFoundException) || ($e instanceof WebsiteForeignOwnerException)) { |
197 | - $data['form_error'] = [ 'field' => 'identifier', 'message' => $this->l10n->t('Website not found.') ]; |
|
197 | + $data['form_error'] = ['field' => 'identifier', 'message' => $this->l10n->t('Website not found.')]; |
|
198 | 198 | } elseif ($e instanceof WebsiteInvalidDataException) { |
199 | - $data['form_error'] = [ 'field' => $e->getField(), 'message' => $e->getMessage() ]; |
|
199 | + $data['form_error'] = ['field' => $e->getField(), 'message' => $e->getMessage()]; |
|
200 | 200 | } elseif ($e instanceof ThemeNotFoundException) { |
201 | - $data['form_error'] = [ 'field' => 'theme', 'message' => $this->l10n->t('Theme not found.') ]; |
|
201 | + $data['form_error'] = ['field' => 'theme', 'message' => $this->l10n->t('Theme not found.')]; |
|
202 | 202 | } elseif ($e instanceof ThemeNotCompatibleException) { |
203 | - $data['form_error'] = [ 'field' => 'theme', 'message' => $this->l10n->t($e->getReason()) ]; |
|
203 | + $data['form_error'] = ['field' => 'theme', 'message' => $this->l10n->t($e->getReason())]; |
|
204 | 204 | } elseif ($e instanceof TemplateNotFoundException) { |
205 | - $data['form_error'] = [ 'field' => 'template', 'message' => $this->l10n->t('Template not found.') ]; |
|
205 | + $data['form_error'] = ['field' => 'template', 'message' => $this->l10n->t('Template not found.')]; |
|
206 | 206 | } |
207 | 207 | |
208 | 208 | return $this->createErrorResponse($e, $data); |
@@ -429,7 +429,7 @@ discard block |
||
429 | 429 | throw new \UnexpectedValueException(); |
430 | 430 | } |
431 | 431 | |
432 | - $this->websitesService->setLinkMode((int) $data['link_mode']); |
|
432 | + $this->websitesService->setLinkMode((int)$data['link_mode']); |
|
433 | 433 | |
434 | 434 | return new DataResponse([], Http::STATUS_CREATED); |
435 | 435 | } catch (\Exception $e) { |
@@ -445,7 +445,7 @@ discard block |
||
445 | 445 | */ |
446 | 446 | private function createErrorResponse(\Exception $exception, array $data = []): DataResponse |
447 | 447 | { |
448 | - $this->logger->logException($exception, [ 'app' => Application::APP_NAME, 'level' => 2 ]); |
|
448 | + $this->logger->logException($exception, ['app' => Application::APP_NAME, 'level' => 2]); |
|
449 | 449 | |
450 | 450 | $data['status'] = 0; |
451 | 451 | if (\OC::$server->getSystemConfig()->getValue('debug', false)) { |
@@ -26,8 +26,7 @@ |
||
26 | 26 | |
27 | 27 | use OCP\AppFramework\Http\EmptyContentSecurityPolicy; |
28 | 28 | |
29 | -class PicoContentSecurityPolicy extends EmptyContentSecurityPolicy |
|
30 | -{ |
|
29 | +class PicoContentSecurityPolicy extends EmptyContentSecurityPolicy { |
|
31 | 30 | /** @var bool Whether inline JS snippets are allowed */ |
32 | 31 | protected $inlineScriptAllowed = true; |
33 | 32 |
@@ -27,8 +27,7 @@ |
||
27 | 27 | use OCP\AppFramework\Http; |
28 | 28 | use OCP\AppFramework\Http\Response; |
29 | 29 | |
30 | -class NotModifiedResponse extends Response |
|
31 | -{ |
|
30 | +class NotModifiedResponse extends Response { |
|
32 | 31 | /** @var Response */ |
33 | 32 | private $originalResponse; |
34 | 33 |
@@ -28,8 +28,7 @@ |
||
28 | 28 | use OCP\AppFramework\Http\EmptyContentSecurityPolicy; |
29 | 29 | use OCP\AppFramework\Http\Response; |
30 | 30 | |
31 | -class PicoPageResponse extends Response |
|
32 | -{ |
|
31 | +class PicoPageResponse extends Response { |
|
33 | 32 | /** @var PicoPage */ |
34 | 33 | private $page; |
35 | 34 |