jenkins-bot has submitted this change and it was merged.
Change subject: Make use of newer useTransactionalTimeLimit()
......................................................................
Make use of newer useTransactionalTimeLimit()
Change-Id: I7d9993839140bb93e2331d0b58d17f332837d011
---
M RenameuserSQL.php
M specials/SpecialRenameuser.php
2 files changed, 2 insertions(+), 7 deletions(-)
Approvals:
Legoktm: Looks good to me, approved
jenkins-bot: Verified
diff --git a/RenameuserSQL.php b/RenameuserSQL.php
index 1d289c1..9fbacda 100755
--- a/RenameuserSQL.php
+++ b/RenameuserSQL.php
@@ -227,13 +227,6 @@
);
}
- // Increase time limit (like CheckUser); this can take a
while...
- if ( $this->tablesJob ) {
- wfSuppressWarnings();
- set_time_limit( 120 );
- wfRestoreWarnings();
- }
-
$jobs = array(); // jobs for all tables
// Construct jobqueue updates...
// FIXME: if a bureaucrat renames a user in error, he/she
diff --git a/specials/SpecialRenameuser.php b/specials/SpecialRenameuser.php
index f33c391..e4819c4 100644
--- a/specials/SpecialRenameuser.php
+++ b/specials/SpecialRenameuser.php
@@ -38,6 +38,8 @@
throw new UserBlockedError( $this->getUser()->mBlock );
}
+ $this->useTransactionalTimeLimit();
+
$request = $this->getRequest();
$showBlockLog = $request->getBool( 'submit-showBlockLog' );
$usernames = explode( '/', $par, 2 ); // this works as "/" is
not valid in usernames
--
To view, visit https://gerrit.wikimedia.org/r/248605
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I7d9993839140bb93e2331d0b58d17f332837d011
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Renameuser
Gerrit-Branch: master
Gerrit-Owner: Aaron Schulz <[email protected]>
Gerrit-Reviewer: Legoktm <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
