Merge pull request #25998 from nextcloud/techdept/psalm/personalsettings

Fix some types in the Settings Personal pages
This commit is contained in:
Roeland Jago Douma 2021-03-09 09:41:18 +01:00 committed by GitHub
commit f5da1ec51d
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 8 additions and 8 deletions

View File

@ -33,7 +33,7 @@ class Additional implements ISettings {
* @return TemplateResponse returns the instance with all parameters set, ready to be rendered * @return TemplateResponse returns the instance with all parameters set, ready to be rendered
* @since 9.1 * @since 9.1
*/ */
public function getForm() { public function getForm(): TemplateResponse {
return new TemplateResponse('settings', 'settings/empty'); return new TemplateResponse('settings', 'settings/empty');
} }
@ -41,7 +41,7 @@ class Additional implements ISettings {
* @return string the section ID, e.g. 'sharing' * @return string the section ID, e.g. 'sharing'
* @since 9.1 * @since 9.1
*/ */
public function getSection() { public function getSection(): string {
return 'additional'; return 'additional';
} }
@ -53,7 +53,7 @@ class Additional implements ISettings {
* E.g.: 70 * E.g.: 70
* @since 9.1 * @since 9.1
*/ */
public function getPriority() { public function getPriority(): int {
return '5'; return 5;
} }
} }

View File

@ -74,7 +74,7 @@ class ServerDevNotice implements ISettings {
/** /**
* @return TemplateResponse * @return TemplateResponse
*/ */
public function getForm() { public function getForm(): TemplateResponse {
$userFolder = $this->rootFolder->getUserFolder($this->userSession->getUser()->getUID()); $userFolder = $this->rootFolder->getUserFolder($this->userSession->getUser()->getUID());
$hasInitialState = false; $hasInitialState = false;
@ -98,9 +98,9 @@ class ServerDevNotice implements ISettings {
} }
/** /**
* @return string the section ID, e.g. 'sharing' * @return string|null the section ID, e.g. 'sharing'
*/ */
public function getSection() { public function getSection(): ?string {
if ($this->registry->delegateHasValidSubscription()) { if ($this->registry->delegateHasValidSubscription()) {
return null; return null;
} }
@ -115,7 +115,7 @@ class ServerDevNotice implements ISettings {
* *
* E.g.: 70 * E.g.: 70
*/ */
public function getPriority() { public function getPriority(): int {
return 1000; return 1000;
} }
} }