also looking at this
Exceptions
#27
5 files
+168
-14
Review threads live on the whole diff, not on one commit, so none are shown here - a thread's line means something in the branch's final form, and painting it into an intermediate step would put it on code it is not about.
| @@ -223,7 +223,7 @@ class Media | ||
| 223 | 223 | * |
| 224 | 224 | * @throws InvalidFormatException |
| 225 | 225 | */ |
| 226 | public function makePrivate($id): array | |
| 226 | public function makePrivate($id) | |
| 227 | 227 | { |
| 228 | 228 | try { |
| 229 | 229 | if (! is_int($id)) { |
| @@ -267,7 +267,7 @@ class Media | ||
| 267 | 267 | * |
| 268 | 268 | * @throws InvalidFormatException |
| 269 | 269 | */ |
| 270 | public function duplicate($id): array | |
| 270 | public function duplicate($id) | |
| 271 | 271 | { |
| 272 | 272 | try { |
| 273 | 273 | if (! is_int($id)) { |