diff options
| author |  Alexandre Alapetite <alexandre@alapetite.fr>
| 2024-01-15 10:36:30 +0100 |
|---|---|---|
| committer |  GitHub <noreply@github.com>
| 2024-01-15 10:36:30 +0100 |
| commit | 314077a457f04cc2f0472e036af029e2676fbf02 (patch) | |
| tree | 1f38bb78761a56b8ee2034caba0dbda3912ef7c1 /app/Models/Entry.php | |
| parent | 52f6c8399b41e0c8be49dd56c89f451843189791 (diff) | |
PHPStan prepare exceptions (#6037)
Take advantage of
https://phpstan.org/blog/bring-your-exceptions-under-control
Minimum changes to pass `tooWideThrowType` and `implicitThrows`.
Revert some mistakes from:
https://github.com/FreshRSS/FreshRSS/pull/5504
Preparation needed before new PRs of the same type:
https://github.com/FreshRSS/FreshRSS/pull/5962
Fix several wrong PHPDocs and catches:
> Method ... has ...Exception in PHPDoc @throws tag but it's not thrown.
> Dead catch - ...Exception is never thrown in the try block.
Diffstat (limited to 'app/Models/Entry.php')
| -rw-r--r-- | app/Models/Entry.php | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/app/Models/Entry.php b/app/Models/Entry.php index 16f01483a..d3cfa07de 100644 --- a/app/Models/Entry.php +++ b/app/Models/Entry.php @@ -691,6 +691,7 @@ HTML; /** * @param array<string,mixed> $attributes + * @throws Minz_Exception */ public static function getContentByParsing(string $url, string $path, array $attributes = [], int $maxRedirs = 3): string { $cachePath = FreshRSS_Feed::cacheFilename($url, $attributes, FreshRSS_Feed::KIND_HTML_XPATH); @@ -741,7 +742,7 @@ HTML; $html = trim(sanitizeHTML($content, $base)); return $html; } else { - throw new Exception(); + throw new Minz_Exception(); } } |