| @@ -8,9 +8,9 @@ | ||
| 8 | 8 | use Dudlewebs\WPMCS\s3\Psr\Http\Message\UploadedFileInterface; |
| 9 | 9 | use RuntimeException; |
| 10 | 10 | class UploadedFile implements UploadedFileInterface |
| 11 | 11 | { |
| 12 | - private const ERRORS = [\UPLOAD_ERR_OK, \UPLOAD_ERR_INI_SIZE, \UPLOAD_ERR_FORM_SIZE, \UPLOAD_ERR_PARTIAL, \UPLOAD_ERR_NO_FILE, \UPLOAD_ERR_NO_TMP_DIR, \UPLOAD_ERR_CANT_WRITE, \UPLOAD_ERR_EXTENSION]; | |
| 12 | + private const ERROR_MAP = [\UPLOAD_ERR_OK => 'UPLOAD_ERR_OK', \UPLOAD_ERR_INI_SIZE => 'UPLOAD_ERR_INI_SIZE', \UPLOAD_ERR_FORM_SIZE => 'UPLOAD_ERR_FORM_SIZE', \UPLOAD_ERR_PARTIAL => 'UPLOAD_ERR_PARTIAL', \UPLOAD_ERR_NO_FILE => 'UPLOAD_ERR_NO_FILE', \UPLOAD_ERR_NO_TMP_DIR => 'UPLOAD_ERR_NO_TMP_DIR', \UPLOAD_ERR_CANT_WRITE => 'UPLOAD_ERR_CANT_WRITE', \UPLOAD_ERR_EXTENSION => 'UPLOAD_ERR_EXTENSION']; | |
| 13 | 13 | /** |
| 14 | 14 | * @var string|null |
| 15 | 15 | */ |
| 16 | 16 | private $clientFilename; |
| @@ -74,9 +74,9 @@ | ||
| 74 | 74 | * @throws InvalidArgumentException |
| 75 | 75 | */ |
| 76 | 76 | private function setError(int $error) : void |
| 77 | 77 | { |
| 78 | - if (\false === \in_array($error, UploadedFile::ERRORS, \true)) { | |
| 78 | + if (!isset(UploadedFile::ERROR_MAP[$error])) { | |
| 79 | 79 | throw new InvalidArgumentException('Invalid error status for UploadedFile'); |
| 80 | 80 | } |
| 81 | 81 | $this->error = $error; |
| 82 | 82 | } |
| @@ -100,9 +100,9 @@ | ||
| 100 | 100 | */ |
| 101 | 101 | private function validateActive() : void |
| 102 | 102 | { |
| 103 | 103 | if (\false === $this->isOk()) { |
| 104 | - throw new RuntimeException('Cannot retrieve stream due to upload error'); | |
| 104 | + throw new RuntimeException(\sprintf('Cannot retrieve stream due to upload error (%s)', self::ERROR_MAP[$this->error])); | |
| 105 | 105 | } |
| 106 | 106 | if ($this->isMoved()) { |
| 107 | 107 | throw new RuntimeException('Cannot retrieve stream after it has already been moved'); |
| 108 | 108 | } |