Refactor method to throw exception instead of true/false

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
This commit is contained in:
Daniel Kesselberg 2018-10-07 18:12:38 +02:00
parent fbe62e1985
commit 14d802b8f3
No known key found for this signature in database
GPG Key ID: 36E3664E099D0614
1 changed files with 11 additions and 7 deletions

View File

@ -129,15 +129,19 @@ class Base extends Command implements CompletionAwareInterface {
}
/**
* @return bool
* Throw InterruptedException when interrupted by user
*
* @throws InterruptedException
*/
protected function hasBeenInterrupted() {
// return always false if pcntl_signal functions are not accessible
if ($this->php_pcntl_signal) {
pcntl_signal_dispatch();
return $this->interrupted;
} else {
return false;
if ($this->php_pcntl_signal === false) {
return;
}
pcntl_signal_dispatch();
if ($this->interrupted === true) {
throw new InterruptedException('Command interrupted by user');
}
}