diff options
| author | Adam Wight <adam.wight@wikimedia.de> | 2019-12-13 10:58:24 +0100 |
|---|---|---|
| committer | Bartosz Dziewoński <dziewonski@fastmail.fm> | 2024-09-17 22:20:58 +0200 |
| commit | 188d2cbbb00d870c20cafeafac56e20c8f520291 (patch) | |
| tree | 3b04e7011f80769af889bd4350f72c967bc412a1 /includes/jobqueue/Job.php | |
| parent | 8c803338a5ddeea69467540d28c795ef9ff16107 (diff) | |
| download | mediawikicore-188d2cbbb00d870c20cafeafac56e20c8f520291.tar.gz mediawikicore-188d2cbbb00d870c20cafeafac56e20c8f520291.zip | |
Remove unchecked exception annotations
Callers should not catch an unchecked exception, so it doesn't belong
in a function signature. Unchecked exceptions indicate a coding error,
which by definition the code will not be able to handle correctly.
If any of these exceptions were supposed to be in response to an edge
case, user input, or initial conditions, then they should be changed
to a runtime error. If the exception class cannot be changed, then
the annotation should include a comment explaining its purpose and
prognosis.
Bug: T240672
Change-Id: I2e640b9737cb68090a8e1cb70067d1b74037d647
Diffstat (limited to 'includes/jobqueue/Job.php')
| -rw-r--r-- | includes/jobqueue/Job.php | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/includes/jobqueue/Job.php b/includes/jobqueue/Job.php index e4a904114079..2bbec7491e4b 100644 --- a/includes/jobqueue/Job.php +++ b/includes/jobqueue/Job.php @@ -70,7 +70,6 @@ abstract class Job implements RunnableJob { * * @param string $command Job command * @param array|PageReference $params Job parameters - * @throws InvalidArgumentException * @return Job */ public static function factory( $command, $params = [] ) { |