Follow-up r81536: Fix misleading comment in Status and add a comment to LocalFileRest...

diff --git a/includes/Status.php b/includes/Status.php
index 4fce6e9..6bd9456 100644 (file)
--- a/includes/Status.php
+++ b/includes/Status.php
@@ -18,7 +18,7 @@ class Status {
 
        /** Counters for batch operations */
        public $successCount = 0, $failCount = 0;
-       /** Array to indicate which items of the batch operations failed */
+       /** Array to indicate which items of the batch operations were successful */
        public $success = array();
 
        /*semi-private*/ var $errors = array();

diff --git a/includes/filerepo/LocalFile.php b/includes/filerepo/LocalFile.php
index 7ea694a..f86c5dd 100644 (file)
--- a/includes/filerepo/LocalFile.php
+++ b/includes/filerepo/LocalFile.php
@@ -1947,6 +1947,13 @@ class LocalFileRestoreBatch {
                return $status;
        }
        
+       /**
+        * Cleanup a failed batch. The batch was only partially successful, so 
+        * rollback by removing all items that were succesfully copied.
+        * 
+        * @param Status $storeStatus
+        * @param array $storeBatch
+        */
        function cleanupFailedBatch( $storeStatus, $storeBatch ) {
                $cleanupBatch = array();