promoted readonly properties

Author: dg

src/Application/Application.php

@@ -48,22 +48,14 @@ class Application
 	/** @var Request[] */
 	private array $requests = [];
 	private ?IPresenter $presenter = null;
-	private Nette\Http\IRequest $httpRequest;
-	private Nette\Http\IResponse $httpResponse;
-	private IPresenterFactory $presenterFactory;
-	private Router $router;
 
 
 	public function __construct(
-		IPresenterFactory $presenterFactory,
-		Router $router,
-		Nette\Http\IRequest $httpRequest,
-		Nette\Http\IResponse $httpResponse,
+		private readonly IPresenterFactory $presenterFactory,
+		private readonly Router $router,
+		private readonly Nette\Http\IRequest $httpRequest,
+		private readonly Nette\Http\IResponse $httpResponse,
 	) {
-		$this->httpRequest = $httpRequest;
-		$this->httpResponse = $httpResponse;
-		$this->presenterFactory = $presenterFactory;
-		$this->router = $router;
 	}
 
 

src/Application/Responses/FileResponse.php

@@ -19,26 +19,20 @@
 final class FileResponse implements Nette\Application\Response
 {
 	public bool $resuming = true;
-	private string $file;
-	private string $contentType;
-	private string $name;
-	private bool $forceDownload;
+	private readonly string $name;
 
 
 	public function __construct(
-		string $file,
+		private readonly string $file,
 		?string $name = null,
-		?string $contentType = null,
-		bool $forceDownload = true,
+		private readonly string $contentType = 'application/octet-stream',
+		private readonly bool $forceDownload = true,
 	) {
 		if (!is_file($file) || !is_readable($file)) {
 			throw new Nette\Application\BadRequestException("File '$file' doesn't exist or is not readable.");
 		}
 
-		$this->file = $file;
 		$this->name = $name ?? basename($file);
-		$this->contentType = $contentType ?? 'application/octet-stream';
-		$this->forceDownload = $forceDownload;
 	}
 
 

src/Application/Responses/ForwardResponse.php

@@ -17,12 +17,9 @@
  */
 final class ForwardResponse implements Nette\Application\Response
 {
-	private Nette\Application\Request $request;
-
-
-	public function __construct(Nette\Application\Request $request)
-	{
-		$this->request = $request;
+	public function __construct(
+		private readonly Nette\Application\Request $request,
+	) {
 	}
 
 

src/Application/Responses/JsonResponse.php

@@ -17,14 +17,10 @@
  */
 final class JsonResponse implements Nette\Application\Response
 {
-	private mixed $payload;
-	private string $contentType;
-
-
-	public function __construct(mixed $payload, ?string $contentType = null)
-	{
-		$this->payload = $payload;
-		$this->contentType = $contentType ?? 'application/json';
+	public function __construct(
+		private readonly mixed $payload,
+		private readonly string $contentType = 'application/json',
+	) {
 	}
 
 

src/Application/Responses/RedirectResponse.php

@@ -18,14 +18,10 @@
  */
 final class RedirectResponse implements Nette\Application\Response
 {
-	private string $url;
-	private int $httpCode;
-
-
-	public function __construct(string $url, int $httpCode = Http\IResponse::S302_Found)
-	{
-		$this->url = $url;
-		$this->httpCode = $httpCode;
+	public function __construct(
+		private readonly string $url,
+		private readonly int $httpCode = Http\IResponse::S302_Found,
+	) {
 	}
 
 

src/Application/Responses/TextResponse.php

@@ -17,12 +17,9 @@
  */
 final class TextResponse implements Nette\Application\Response
 {
-	private mixed $source;
-
-
-	public function __construct(mixed $source)
-	{
-		$this->source = $source;
+	public function __construct(
+		private readonly mixed $source,
+	) {
 	}
 
 

src/Application/Routers/CliRouter.php

@@ -20,12 +20,10 @@ final class CliRouter implements Nette\Routing\Router
 {
 	private const PresenterKey = 'action';
 
-	private array $defaults;
 
-
-	public function __construct(array $defaults = [])
-	{
-		$this->defaults = $defaults;
+	public function __construct(
+		private readonly array $defaults = [],
+	) {
 	}