Skip to content

Commit

Permalink
Merge pull request #5794 from nextcloud/fix/5793
Browse files Browse the repository at this point in the history
fix: Avoid optional before required parameter
  • Loading branch information
juliusknorr authored Apr 30, 2024
2 parents 72478f0 + cf89d4d commit dae9748
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/Service/ConfigService.php
Original file line number Diff line number Diff line change
Expand Up @@ -231,7 +231,7 @@ public function getAttachmentFolder(?string $userId = null): string {
return $this->config->getUserValue($userId ?? $this->getUserId(), 'deck', 'attachment_folder', '/Deck');
}

public function setAttachmentFolder(?string $userId = null, string $path): void {
public function setAttachmentFolder(?string $userId, string $path): void {
if ($userId === null && $this->getUserId() === null) {
throw new NoPermissionException('Must be logged in get the attachment folder');
}
Expand Down

0 comments on commit dae9748

Please sign in to comment.