https://www.mediawiki.org/wiki/Special:Code/MediaWiki/113195
Revision: 113195
Author: aaron
Date: 2012-03-06 22:59:59 +0000 (Tue, 06 Mar 2012)
Log Message:
-----------
Fixed FileOp::attemptBatch() behavior when unexpected failures occur with the
'force' option. It can't continue on since the ops assumed that $predicates was
correct, which it won't be. It now simply halts operations.
Modified Paths:
--------------
trunk/phase3/includes/filerepo/backend/FileOp.php
Modified: trunk/phase3/includes/filerepo/backend/FileOp.php
===================================================================
--- trunk/phase3/includes/filerepo/backend/FileOp.php 2012-03-06 22:59:41 UTC
(rev 113194)
+++ trunk/phase3/includes/filerepo/backend/FileOp.php 2012-03-06 22:59:59 UTC
(rev 113195)
@@ -134,13 +134,12 @@
} else {
$status->success[$index] = false;
++$status->failCount;
- if ( !$ignoreErrors ) {
- // Log remaining ops as failed for
recovery...
- for ( $i = ($index + 1); $i < count(
$performOps ); $i++ ) {
- $performOps[$i]->logFailure(
'attempt_aborted' );
- }
- return $status; // bail out
+ // We can't continue (even with $ignoreErrors)
as $predicates is wrong.
+ // Log the remaining ops as failed for
recovery...
+ for ( $i = ($index + 1); $i < count(
$performOps ); $i++ ) {
+ $performOps[$i]->logFailure(
'attempt_aborted' );
}
+ return $status; // bail out
}
}
_______________________________________________
MediaWiki-CVS mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-cvs
