deblan/php-censor
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Fixed worker in fails. Issue #68.

Browse source
This commit is contained in:
Dmitry Khomutov 2017-05-23 22:48:03 +07:00
parent 520e453a16
commit 2af20dc00c
No known key found for this signature in database
GPG key ID: 7EB36C9576F9ECB9
1 changed files with 2 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

19
src/PHPCensor/Worker/BuildWorker.php
View file

@ -109,23 +109,6 @@ class BuildWorker
try {
$builder = new Builder($build, $this->logger);
$builder->execute();
} catch (BuilderException $ex) {
$this->logger->addError($ex->getMessage());
switch($ex->getCode()) {
case BuilderException::FAIL_START:
// non fatal
$this->pheanstalk->release($job);
unset($job);
break;
default:
$build->setStatus(Build::STATUS_FAILED);
$build->setFinished(new \DateTime());
$build->setLog($build->getLog() . PHP_EOL . PHP_EOL . $ex->getMessage());
$buildStore->save($build);
$build->sendStatusPostback();
break;
}
} catch (\PDOException $ex) {
// If we've caught a PDO Exception, it is probably not the fault of the build, but of a failed
// connection or similar. Release the job and kill the worker.
@ -133,6 +116,8 @@ class BuildWorker
$this->pheanstalk->release($job);
unset($job);
} catch (\Exception $ex) {
$this->logger->addError($ex->getMessage());
$build->setStatus(Build::STATUS_FAILED);
$build->setFinished(new \DateTime());
$build->setLog($build->getLog() . PHP_EOL . PHP_EOL . $ex->getMessage());