Added important comment from @nupplaphil@github.com
This commit is contained in:
parent
f38ad168f5
commit
8092188991
1 changed files with 12 additions and 1 deletions
|
@ -179,7 +179,18 @@ class StaticDatabase extends Database
|
|||
self::$staticConnection = @new ExtendedPDO($connect, $user, $pass);
|
||||
self::$staticConnection->setAttribute(PDO::ATTR_AUTOCOMMIT,0);
|
||||
} catch (PDOException $e) {
|
||||
/// @TODO Try to find a way to log this exception as it contains valueable information
|
||||
/*
|
||||
* @TODO Try to find a way to log this exception as it contains valueable information
|
||||
* @nupplaphil@github.com comment:
|
||||
*
|
||||
* There is no easy possibility to add a logger here, that's why
|
||||
* there isn't any yet and instead a placeholder.. This execution
|
||||
* point is a critical state during a testrun, and tbh I'd like to
|
||||
* leave here no further logic (yet) because I spent hours debugging
|
||||
* cases, where transactions weren't fully closed and
|
||||
* strange/unpredictable errors occur (sometimes -mainly during
|
||||
* debugging other errors :) ...)
|
||||
*/
|
||||
}
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in a new issue