Deprecate setStatus method for Responses with a fixed status code

Signed-off-by: jld3103 <[email protected]>
nextcloud · Jun 26, 2023 · e838f84 · e838f84
1 parent 0da5eb3
commit e838f84
Show file tree

Hide file tree

Showing 3 changed files with 30 additions and 0 deletions.
diff --git a/lib/public/AppFramework/Http/NotFoundResponse.php b/lib/public/AppFramework/Http/NotFoundResponse.php
@@ -43,4 +43,14 @@ public function __construct() {
 		$this->setContentSecurityPolicy(new ContentSecurityPolicy());
 		$this->setStatus(Http::STATUS_NOT_FOUND);
 	}
+
+	/**
+	 * @inheritDoc
+	 * @deprecated Do not use this method. It modifies the status code which you are not supposed to do on a NotFoundResponse
+	 * @since 6.0.0 - return value was added in 7.0.0
+	 */
+	public function setStatus($status): static {
+		parent::setStatus($status);
+		return $this;
+	}
 }
diff --git a/lib/public/AppFramework/Http/RedirectResponse.php b/lib/public/AppFramework/Http/RedirectResponse.php
@@ -58,4 +58,14 @@ public function __construct($redirectURL) {
 	public function getRedirectURL() {
 		return $this->redirectURL;
 	}
+
+	/**
+	 * @inheritDoc
+	 * @deprecated Do not use this method. It modifies the status code which you are not supposed to do on a RedirectResponse
+	 * @since 6.0.0 - return value was added in 7.0.0
+	 */
+	public function setStatus($status): static {
+		parent::setStatus($status);
+		return $this;
+	}
 }
diff --git a/lib/public/AppFramework/Http/TooManyRequestsResponse.php b/lib/public/AppFramework/Http/TooManyRequestsResponse.php
@@ -54,4 +54,14 @@ public function render() {
 		$template = new Template('core', '429', 'blank');
 		return $template->fetchPage();
 	}
+
+	/**
+	 * @inheritDoc
+	 * @deprecated Do not use this method. It modifies the status code which you are not supposed to do on a TooManyRequestsResponse
+	 * @since 6.0.0 - return value was added in 7.0.0
+	 */
+	public function setStatus($status): static {
+		parent::setStatus($status);
+		return $this;
+	}
 }