jenkins-bot has submitted this change and it was merged.

Change subject: Removed useless root job params.
......................................................................


Removed useless root job params.

* This would not be effective and does not seem to be used anyway.

Change-Id: If36de24fb4591e26413b04118e38486c0d52ebb8
---
M lib/includes/ChangeNotificationJob.php
1 file changed, 0 insertions(+), 7 deletions(-)

Approvals:
  Aude: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/lib/includes/ChangeNotificationJob.php 
b/lib/includes/ChangeNotificationJob.php
index 6feadfb..30560cb 100644
--- a/lib/includes/ChangeNotificationJob.php
+++ b/lib/includes/ChangeNotificationJob.php
@@ -71,13 +71,6 @@
                $params['repo'] = $repo;
                $params['changeIds'] = $changeIds;
 
-               $params = array_merge(
-                       \Job::newRootJobParams(
-                               $repo . ':' . implode( ',', $changeIds )
-                       ),
-                       $params
-               );
-
                $job = new ChangeNotificationJob( $dummyTitle, $params );
 
                wfProfileOut( __METHOD__ );

-- 
To view, visit https://gerrit.wikimedia.org/r/64599
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If36de24fb4591e26413b04118e38486c0d52ebb8
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Aaron Schulz <[email protected]>
Gerrit-Reviewer: Aude <[email protected]>
Gerrit-Reviewer: Daniel Kinzler <[email protected]>
Gerrit-Reviewer: Jeroen De Dauw <[email protected]>
Gerrit-Reviewer: jenkins-bot

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to