Nosial/ncc
2
0
Fork 0
Code Issues 1 Pull requests Projects Releases 11 Packages Wiki Activity Actions

Fixed issue where error states are not properly handled in ExecutionPointerManager issue [#44](https://git.n64.cc/nosial/ncc/-/issues/44)

Browse source
This commit is contained in:
Netkas 2023-02-24 22:32:28 -05:00
parent 32a4a8c2cf
commit 712020b129
2 changed files with 13 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
CHANGELOG.md
View file

@ -7,6 +7,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
## [1.0.2] - Unreleased
### Fixed
- Fixed issue where error states are not properly handled in `ExecutionPointerManager` issue [#44](https://git.n64.cc/nosial/ncc/-/issues/44)
### Changed
- Updated `Symfony\Filesystem` to version 6.2.5
- Updated `Symfony\polyfill-ctype` to version 1.27.0

14
src/ncc/Managers/ExecutionPointerManager.php
View file

@ -437,11 +437,17 @@
}
catch(Exception $e)
{
unset($e);
$this->handleExit($package, $version, $unit->ExecutionPolicy->ExitHandlers->Error);
}
if($unit->ExecutionPolicy->ExitHandlers !== null && $unit->ExecutionPolicy->ExitHandlers->Error !== null)
{
$this->handleExit($package, $version, $unit->ExecutionPolicy->ExitHandlers->Error);
}
Console::outDebug(sprintf('exit_code=%s', $process->getExitCode()));
Console::outException(sprintf('An error occurred while executing the unit \'%s\' for \'%s\' (exit code %s)', $unit->ExecutionPolicy->Name, $package, $process->getExitCode()), $e);
}
finally
{
Console::outDebug(sprintf('exit_code=%s', $process->getExitCode()));
}
if($unit->ExecutionPolicy->ExitHandlers !== null)
{