jenkins-bot has submitted this change and it was merged. ( 
https://gerrit.wikimedia.org/r/390338 )

Change subject: Updated behaviour when queue is populated from dump file to 
detect whether additional info source headers are present and if not inject them
......................................................................


Updated behaviour when queue is populated from dump file to detect whether 
additional info source headers are present and if not inject them

Change-Id: I7a0cca729fc2bfc772c8aba1ccaff4aed8fb6816
---
M Maintenance/PopulateQueueFromDump.php
1 file changed, 9 insertions(+), 3 deletions(-)

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



diff --git a/Maintenance/PopulateQueueFromDump.php 
b/Maintenance/PopulateQueueFromDump.php
index 73551b0..0aa6ea8 100644
--- a/Maintenance/PopulateQueueFromDump.php
+++ b/Maintenance/PopulateQueueFromDump.php
@@ -53,9 +53,15 @@
                                continue;
                        }
 
-                       // push message directly to queue, bypassing 
QueueWrapper's adding
-                       // source fields.
-                       $datastore->push( $message );
+                       // if $message SourceFields headers are not set then we 
send it through QueueWrapper::push()
+                       if ( !array_key_exists( 'source_enqueued_time', 
$message ) ) {
+                               // QueueWrapper::push() injects additional 
useful properties
+                               // useful properties declared here 
\SmashPig\CrmLink\Messages\SourceFields::addToMessage()
+                               QueueWrapper::push( $this->getOption( 'queue' 
), $message );
+                       } else {
+
+                               $datastore->push( $message );
+                       }
 
                        $messageCount++;
                        if ( $messageCount % 1000 == 0 ) {

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7a0cca729fc2bfc772c8aba1ccaff4aed8fb6816
Gerrit-PatchSet: 2
Gerrit-Project: wikimedia/fundraising/SmashPig
Gerrit-Branch: master
Gerrit-Owner: Jgleeson <jglee...@wikimedia.org>
Gerrit-Reviewer: AndyRussG <andrew.green...@gmail.com>
Gerrit-Reviewer: Cdentinger <cdentin...@wikimedia.org>
Gerrit-Reviewer: Eileen <emcnaugh...@wikimedia.org>
Gerrit-Reviewer: Ejegg <ej...@ejegg.com>
Gerrit-Reviewer: Katie Horn <kh...@wikimedia.org>
Gerrit-Reviewer: Mepps <me...@wikimedia.org>
Gerrit-Reviewer: XenoRyet <dkozlow...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to