jenkins-bot has submitted this change and it was merged.
Change subject: Fixed regression from 97b2a1dfd with URL specified edit tags
......................................................................
Fixed regression from 97b2a1dfd with URL specified edit tags
Bug: T100248
Change-Id: I2688356cd5f628dca395d1caaa82b9a5b21c025e
---
M includes/EditPage.php
1 file changed, 7 insertions(+), 5 deletions(-)
Approvals:
Ori.livneh: Looks good to me, approved
jenkins-bot: Verified
diff --git a/includes/EditPage.php b/includes/EditPage.php
index b0da562..e88baaf 100644
--- a/includes/EditPage.php
+++ b/includes/EditPage.php
@@ -1958,11 +1958,13 @@
if ( $this->changeTags && isset(
$doEditStatus->value['revision'] ) ) {
// If a revision was created, apply any change tags
that were requested
- ChangeTags::addTags(
- $this->changeTags,
- isset( $doEditStatus->value['rc'] ) ?
$doEditStatus->value['rc']->mAttribs['rc_id'] : null,
- $doEditStatus->value['revision']->getId()
- );
+ $addTags = $this->changeTags;
+ $revId = $doEditStatus->value['revision']->getId();
+ // Defer this both for performance and so that
addTags() sees the rc_id
+ // since the recentchange entry addition is deferred
first (bug T100248)
+ DeferredUpdates::addCallableUpdate( function() use (
$addTags, $revId ) {
+ ChangeTags::addTags( $addTags, null, $revId );
+ } );
}
return $status;
--
To view, visit https://gerrit.wikimedia.org/r/213602
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I2688356cd5f628dca395d1caaa82b9a5b21c025e
Gerrit-PatchSet: 3
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Aaron Schulz <[email protected]>
Gerrit-Reviewer: Jackmcbarn <[email protected]>
Gerrit-Reviewer: Ori.livneh <[email protected]>
Gerrit-Reviewer: Tpt <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
