From: Eddie Greiner-Petter <git@eddie-sh.de>
Date: Fri, 6 Oct 2017 19:35:29 +0000 (+0200)
Subject: Clarify purgeExpiredUserrights maintenance script
X-Git-Tag: 1.31.0-rc.0~1854
X-Git-Url: https://git.cyclocoop.org/%7B%24admin_url%7Dmembres/cotisations/rappels.php?a=commitdiff_plain;h=4c186cbbdf93b5f6580b029f2a2f61e23c8e3550;p=lhc%2Fweb%2Fwiklou.git

Clarify purgeExpiredUserrights maintenance script

This moves an @since tag where it belongs (class-level instead of
file-level comment) and removes a wrong message (purges are run directly
after execute() ended and will have immediate effect).

Follows-up on fdb4557.

Change-Id: I26fdb1c147b9b44690a3dd0f7c836aca85430c5e
---

diff --git a/maintenance/purgeExpiredUserrights.php b/maintenance/purgeExpiredUserrights.php
index 7554d4d0f2..f5a085d999 100644
--- a/maintenance/purgeExpiredUserrights.php
+++ b/maintenance/purgeExpiredUserrights.php
@@ -18,11 +18,16 @@
  * @copyright GPLv2 http://www.gnu.org/copyleft/gpl.html
  * @author Eddie Greiner-Petter <wikimedia.org at eddie-sh.de>
  * @ingroup Maintenance
- * @since 1.31
  */
 
 require_once __DIR__ . '/Maintenance.php';
 
+/*
+ * Maintenance script to move expired userrights to user_former_groups
+ *
+ * @since 1.31
+ */
+
 class PurgeExpiredUserrights extends Maintenance {
 	public function __construct() {
 		parent::__construct();
@@ -32,9 +37,7 @@ class PurgeExpiredUserrights extends Maintenance {
 	public function execute() {
 		$this->output( "Purging expired user rights...\n" );
 		UserGroupMembership::purgeExpired();
-		$this->output(
-			"Purge requests submitted. (These requests are deferred and may not have immediate effect.)\n"
-		);
+		$this->output( "Purge requests submitted.\n" );
 	}
 }