Merge "Some fixes for Ia2782d64 (2f60cd2)"
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Sat, 5 Jan 2013 22:04:17 +0000 (22:04 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Sat, 5 Jan 2013 22:04:17 +0000 (22:04 +0000)
includes/logging/LogFormatter.php

index 3029925..4f699f1 100644 (file)
@@ -891,9 +891,11 @@ class DeleteLogFormatter extends LogFormatter {
                $params = parent::getMessageParameters();
                $subtype = $this->entry->getSubtype();
                if ( in_array( $subtype, array( 'event', 'revision' ) ) ) {
+                       // $params[3] here is 'revision' for page revisions, 'oldimage' for file versions, or a comma-separated list of log_ids for log entries.
+                       // $subtype here is 'revision' for page revisions and file versions, or 'event' for log entries.
                        if (
-                               ($subtype === 'event' && count( $params ) === 6 ) ||
-                               ($subtype === 'revision' && isset( $params[3] ) && $params[3] === 'revision' )
+                               ( $subtype === 'event' && count( $params ) === 6 ) ||
+                               ( $subtype === 'revision' && isset( $params[3] ) && ( $params[3] === 'revision' || $params[3] === 'oldimage' ) )
                        ) {
                                $paramStart = $subtype === 'revision' ? 4 : 3;