Article::doEdit() opens and commits transactions of its own, so the transaction we started earlier will already be committed and cannot be rolled back at this point unless one of two particular failure modes hits (ArticleSave hook aborts or mLatest isn't set). A more general fix needs to be devised...
* (bug 16726) siprop=namespacealiases should also list localized aliases
* (bug 16730) Added apprfiltercascade parameter to list=allpages to filter
cascade-protected pages
-* (bug 15430) Cancel the upload process and rollback database changes in case
- no article associated with the file can be created
=== Languages updated in 1.14 ===
} else {
// New file; create the description page.
// There's already a log entry, so don't make a second RC entry
- $result = $article->doEdit( $pageText, $comment, EDIT_NEW | EDIT_SUPPRESS_RC );
- if ( !$result->isOK() ) {
- $dbw->rollback( __METHOD__ );
- return false;
- }
+ $article->doEdit( $pageText, $comment, EDIT_NEW | EDIT_SUPPRESS_RC );
}
# Hooks, hooks, the magic of hooks...