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

Change subject: removing initial delay
......................................................................


removing initial delay

test to see if removing the initial job delay will take care of the issue
where the gwtoolset jobs are not picked-up.

Bug: 58692
Change-Id: I88933229f7bc6bc7a11549291924cf9a742039b1
---
M includes/Handlers/Forms/MetadataMappingHandler.php
1 file changed, 12 insertions(+), 8 deletions(-)

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



diff --git a/includes/Handlers/Forms/MetadataMappingHandler.php 
b/includes/Handlers/Forms/MetadataMappingHandler.php
index 70762cf..3c48f9d 100644
--- a/includes/Handlers/Forms/MetadataMappingHandler.php
+++ b/includes/Handlers/Forms/MetadataMappingHandler.php
@@ -112,15 +112,19 @@
                        )
                );
 
-               $delayed_enabled =
-                       JobQueueGroup::singleton()
-                       ->get( 'gwtoolsetUploadMetadataJob' )
-                       ->delayedJobsEnabled();
+               if ( $this->_whitelisted_post['gwtoolset-record-begin'] >
+                       ( Config::$preview_throttle + 1 )
+               ) {
+                       $delayed_enabled =
+                               JobQueueGroup::singleton()
+                               ->get( 'gwtoolsetUploadMetadataJob' )
+                               ->delayedJobsEnabled();
 
-               if ( $delayed_enabled ) {
-                       $job->params['jobReleaseTimestamp'] = strtotime(
-                               '+' . Utils::sanitizeString( 
Config::$metadata_job_delay )
-                       );
+                       if ( $delayed_enabled ) {
+                               $job->params['jobReleaseTimestamp'] = strtotime(
+                                       '+' . Utils::sanitizeString( 
Config::$metadata_job_delay )
+                               );
+                       }
                }
 
                $result = JobQueueGroup::singleton()->push( $job );

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I88933229f7bc6bc7a11549291924cf9a742039b1
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/GWToolset
Gerrit-Branch: master
Gerrit-Owner: Dan-nl <[email protected]>
Gerrit-Reviewer: Dan-nl <[email protected]>
Gerrit-Reviewer: GergÅ‘ Tisza <[email protected]>
Gerrit-Reviewer: Hashar <[email protected]>
Gerrit-Reviewer: jenkins-bot

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

Reply via email to