MWExceptionHandler::rollbackMasterChangesAndLog() only logged exceptions if there were already master changes. This is extremely problematic when debugging, especially in situations like DeferredUpdates where they were silently being swallowed. This makes it log exceptions in all paths, erring on the side of logging the same exception twice (theoretically it's possible I suppose) instead of not at all. Also make the method able to handle DBError exceptions, which most of the callers seemed to be assuming. ApiMain was handling this explicitly. Bug: T168347 Change-Id: I8739051f824a455ba669344184c3b11ac95cb561 |
||
|---|---|---|
| .. | ||
| BadRequestError.php | ||
| BadTitleError.php | ||
| ErrorPageError.php | ||
| FatalError.php | ||
| HttpError.php | ||
| LocalizedException.php | ||
| MWContentSerializationException.php | ||
| MWException.php | ||
| MWExceptionHandler.php | ||
| MWExceptionRenderer.php | ||
| MWUnknownContentModelException.php | ||
| PermissionsError.php | ||
| ReadOnlyError.php | ||
| ThrottledError.php | ||
| UserBlockedError.php | ||
| UserNotLoggedIn.php | ||