jenkins-bot has submitted this change and it was merged. Change subject: forceRenameUsers: Add --reason option to set a log summary ......................................................................
forceRenameUsers: Add --reason option to set a log summary Change-Id: I52706f7ec3f42d76905175d26a4ab1a0f65ba278 --- M maintenance/forceRenameUsers.php 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Chad: Looks good to me, approved jenkins-bot: Verified diff --git a/maintenance/forceRenameUsers.php b/maintenance/forceRenameUsers.php index e597db7..7803f4d 100644 --- a/maintenance/forceRenameUsers.php +++ b/maintenance/forceRenameUsers.php @@ -19,6 +19,7 @@ public function __construct() { $this->mDescription = 'Forcibly renames and migrates unattached accounts to global ones'; + $this->addOption( 'reason', 'Reason to use for log summaries', true, true ); $this->setBatchSize( 10 ); } @@ -92,7 +93,6 @@ $this->log( "Set renameuser_status for {$newCAUser->getName()}." ); - // @todo set a reason for log entries $job = new LocalRenameUserJob( Title::newFromText( 'Global rename job' ), array( @@ -102,6 +102,7 @@ 'movepages' => true, 'suppressredirects' => true, 'promotetoglobal' => true, + 'reason' => $this->getOption( 'reason' ), ) ); -- To view, visit https://gerrit.wikimedia.org/r/204221 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I52706f7ec3f42d76905175d26a4ab1a0f65ba278 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/CentralAuth Gerrit-Branch: master Gerrit-Owner: Legoktm <legoktm.wikipe...@gmail.com> Gerrit-Reviewer: Chad <ch...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits