Delete CORS domain always as asked

owncloud · Dec 1, 2017 · 5e9412b · 5e9412b
1 parent 20ebcea
commit 5e9412b
Showing 1 changed file with 5 additions and 7 deletions.
diff --git a/settings/Controller/CorsController.php b/settings/Controller/CorsController.php
@@ -137,13 +137,11 @@ public function addDomain($domain) {
 	public function removeDomain($id) {
 		$userId = $this->userId;
 		$domains = json_decode($this->config->getUserValue($userId, 'core', 'domains'), true);
-		if ($id >= 0 && $id < count($domains)) {
-			unset($domains[$id]);
-			if (count($domains)) {
-				$this->config->setUserValue($userId, 'core', 'domains', json_encode($domains));
-			} else {
-				$this->config->deleteUserValue($userId, 'core', 'domains');
-			}
+		unset($domains[$id]);
+		if (count($domains)) {
+			$this->config->setUserValue($userId, 'core', 'domains', json_encode($domains));
+		} else {
+			$this->config->deleteUserValue($userId, 'core', 'domains');
 		}
 
 		return $this->getRedirectResponse();