Http: Review exception classes

- Rename:
  - `StreamDetachedException` -> `StreamClosedException`
  - `StreamInvalidRequestException` -> `InvalidStreamRequestException`
  - `UploadedFileException` -> `UploadFailedException`

Author: lkrms

src/Toolkit/Http/Exception/HttpServerException.php

@@ -2,4 +2,7 @@
 
 namespace Salient\Http\Exception;
 
+/**
+ * @api
+ */
 class HttpServerException extends HttpException {}

src/Toolkit/Http/Exception/InvalidStreamRequestException.php

@@ -0,0 +1,8 @@
+<?php declare(strict_types=1);
+
+namespace Salient\Http\Exception;
+
+/**
+ * @api
+ */
+class InvalidStreamRequestException extends StreamException {}

src/Toolkit/Http/Exception/StreamClosedException.php

@@ -0,0 +1,8 @@
+<?php declare(strict_types=1);
+
+namespace Salient\Http\Exception;
+
+/**
+ * @api
+ */
+class StreamClosedException extends StreamException {}

src/Toolkit/Http/Exception/StreamDetachedException.php

@@ -0,0 +1,8 @@
+<?php declare(strict_types=1);
+
+namespace Salient\Http\Exception;
+
+/**
+ * @api
+ */
+class UploadFailedException extends HttpException {}

src/Toolkit/Http/Exception/StreamInvalidRequestException.php

@@ -5,8 +5,8 @@
 use Psr\Http\Message\StreamInterface as PsrStreamInterface;
 use Salient\Contract\Http\Message\MultipartStreamInterface;
 use Salient\Contract\Http\Message\StreamPartInterface;
-use Salient\Http\Exception\StreamDetachedException;
-use Salient\Http\Exception\StreamInvalidRequestException;
+use Salient\Http\Exception\InvalidStreamRequestException;
+use Salient\Http\Exception\StreamClosedException;
 use Salient\Http\Headers;
 use Salient\Http\HttpUtil;
 use Salient\Utility\Regex;
@@ -276,7 +276,7 @@ public function write(string $string): int
     {
         $this->assertIsOpen();
 
-        throw new StreamInvalidRequestException('Stream is not writable');
+        throw new InvalidStreamRequestException('Stream is not writable');
     }
 
     /**
@@ -287,7 +287,7 @@ public function seek(int $offset, int $whence = \SEEK_SET): void
         $this->assertIsOpen();
 
         if (!$this->IsSeekable) {
-            throw new StreamInvalidRequestException('Stream is not seekable');
+            throw new InvalidStreamRequestException('Stream is not seekable');
         }
 
         switch ($whence) {
@@ -380,7 +380,7 @@ private function doClose(): void
     private function assertIsOpen(): void
     {
         if (!$this->IsOpen) {
-            throw new StreamDetachedException('Stream is closed or detached');
+            throw new StreamClosedException('Stream is closed or detached');
         }
     }
 }

src/Toolkit/Http/Exception/UploadFailedException.php

@@ -4,7 +4,7 @@
 
 use Psr\Http\Message\StreamInterface as PsrStreamInterface;
 use Psr\Http\Message\UploadedFileInterface as PsrUploadedFileInterface;
-use Salient\Http\Exception\UploadedFileException;
+use Salient\Http\Exception\UploadFailedException;
 use Salient\Http\HttpUtil;
 use Salient\Utility\Exception\InvalidArgumentTypeException;
 use Salient\Utility\File;
@@ -94,7 +94,7 @@ public function moveTo(string $targetPath): void
                 : @move_uploaded_file($this->File, $targetPath);
             if ($result === false) {
                 $error = error_get_last();
-                throw new UploadedFileException($error['message'] ?? sprintf(
+                throw new UploadFailedException($error['message'] ?? sprintf(
                     'Error moving %s to %s',
                     $this->File,
                     $targetPath,
@@ -143,15 +143,15 @@ public function getClientMediaType(): ?string
     private function assertIsValid(): void
     {
         if ($this->Error !== \UPLOAD_ERR_OK) {
-            throw new UploadedFileException(sprintf(
+            throw new UploadFailedException(sprintf(
                 'Upload failed (%d: %s)',
                 $this->Error,
                 static::ERROR_MESSAGE[$this->Error] ?? '',
             ));
         }
 
         if ($this->IsMoved) {
-            throw new UploadedFileException('Upload already moved');
+            throw new UploadFailedException('Upload already moved');
         }
     }
 }

src/Toolkit/Http/Exception/UploadedFileException.php

@@ -6,9 +6,9 @@
 use Salient\Contract\Http\Message\StreamInterface;
 use Salient\Contract\Http\Message\StreamPartInterface;
 use Salient\Contract\Http\HasFormDataFlag;
-use Salient\Http\Exception\StreamDetachedException;
+use Salient\Http\Exception\InvalidStreamRequestException;
+use Salient\Http\Exception\StreamClosedException;
 use Salient\Http\Exception\StreamEncapsulationException;
-use Salient\Http\Exception\StreamInvalidRequestException;
 use Salient\Http\Internal\FormDataEncoder;
 use Salient\Utility\Exception\InvalidArgumentTypeException;
 use Salient\Utility\File;
@@ -245,7 +245,7 @@ public function write(string $string): int
         $this->assertHasStream();
 
         if (!$this->IsWritable) {
-            throw new StreamInvalidRequestException('Stream is not writable');
+            throw new InvalidStreamRequestException('Stream is not writable');
         }
 
         return File::write($this->Stream, $string, null, $this->Uri);
@@ -304,7 +304,7 @@ private function assertIsSeekable(): void
         $this->assertHasStream();
 
         if (!$this->IsSeekable) {
-            throw new StreamInvalidRequestException('Stream is not seekable');
+            throw new InvalidStreamRequestException('Stream is not seekable');
         }
     }
 
@@ -316,7 +316,7 @@ private function assertIsReadable(): void
         $this->assertHasStream();
 
         if (!$this->IsReadable) {
-            throw new StreamInvalidRequestException('Stream is not readable');
+            throw new InvalidStreamRequestException('Stream is not readable');
         }
     }
 
@@ -326,7 +326,7 @@ private function assertIsReadable(): void
     private function assertHasStream(): void
     {
         if (!$this->Stream) {
-            throw new StreamDetachedException('Stream is closed or detached');
+            throw new StreamClosedException('Stream is closed or detached');
         }
     }
 }