Merge "JobQueueMemory: Actually return values in MappedIterator callback"
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Thu, 14 Sep 2017 02:33:56 +0000 (02:33 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Thu, 14 Sep 2017 02:33:57 +0000 (02:33 +0000)
includes/jobqueue/JobQueueMemory.php

index 649e2af..f9e2c3d 100644 (file)
@@ -177,7 +177,7 @@ class JobQueueMemory extends JobQueue {
                return new MappedIterator(
                        $unclaimed,
                        function ( $value ) {
-                               $this->jobFromSpecInternal( $value );
+                               return $this->jobFromSpecInternal( $value );
                        }
                );
        }
@@ -196,7 +196,7 @@ class JobQueueMemory extends JobQueue {
                return new MappedIterator(
                        $claimed,
                        function ( $value ) {
-                               $this->jobFromSpecInternal( $value );
+                               return $this->jobFromSpecInternal( $value );
                        }
                );
        }