Merge pull request #3307 from nextcloud/backport/3299/stable1.4

nextcloud · Sep 8, 2021 · 40967a4 · 40967a4
2 parents 82e3400 + bfe9b05
commit 40967a4
Showing 1 changed file with 9 additions and 5 deletions.
diff --git a/lib/Service/ConfigService.php b/lib/Service/ConfigService.php
@@ -73,16 +73,20 @@ public function get($key) {
 				if (!$this->groupManager->isAdmin($this->userId)) {
 					throw new NoPermissionException('You must be admin to get the group limit');
 				}
-				$result = $this->getGroupLimit();
-				break;
+				return $this->getGroupLimit();
 			case 'calendar':
-				$result = (bool)$this->config->getUserValue($this->userId, Application::APP_ID, 'calendar', true);
-				break;
+				if ($this->userId === null) {
+					return false;
+				}
+				return (bool)$this->config->getUserValue($this->userId, Application::APP_ID, 'calendar', true);
 		}
-		return $result;
 	}
 
 	public function isCalendarEnabled(int $boardId = null): bool {
+		if ($this->userId === null) {
+			return false;
+		}
+
 		$defaultState = (bool)$this->config->getUserValue($this->userId, Application::APP_ID, 'calendar', true);
 		if ($boardId === null) {
 			return $defaultState;