RefreshLinksJob parsed with incorrect revision variables

It is necessary to pass the revision id to the parser output to get the
revision variables like REVISIONTIMESTAMP correct. Doing this self
in the job prevents from calling Title::getLatestRevId in
Content::getSecondaryDataUpdates or other places.

This behaviour was lost in I551a90b0

Change-Id: I4bf0e8199edacaa703699f45ec8e65ca9e31e765

diff --git a/includes/job/jobs/RefreshLinksJob.php b/includes/job/jobs/RefreshLinksJob.php
index 384244f..86c808b 100644 (file)
--- a/includes/job/jobs/RefreshLinksJob.php
+++ b/includes/job/jobs/RefreshLinksJob.php
@@ -78,7 +78,10 @@ class RefreshLinksJob extends Job {
                        $content = $revision->getContentHandler()->makeEmptyContent();
                }
 
-               $updates = $content->getSecondaryDataUpdates( $title, null, false );
+               // Revision ID must be passed to the parser output to get revision variables correct
+               $parserOutput = $content->getParserOutput( $title, $revision->getId(), null, false );
+
+               $updates = $content->getSecondaryDataUpdates( $title, null, false, $parserOutput );
                DataUpdate::runUpdates( $updates );
                wfProfileOut( $fname );
        }