Skip to content

Commit

Permalink
BUGFIX: Pass named arguments to executeCommand (#35)
Browse files Browse the repository at this point in the history
  • Loading branch information
kdambekalns authored and Bastian Waidelich committed Sep 5, 2018
1 parent 65fd0a5 commit 5e436ee
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion Classes/Job/JobManager.php
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ public function waitAndExecute($queueName, $timeout = null)
$queueSettings = $this->queueManager->getQueueSettings($queueName);
try {
if (isset($queueSettings['executeIsolated']) && $queueSettings['executeIsolated'] === true) {
Scripts::executeCommand('flowpack.jobqueue.common:job:execute', $this->flowSettings, false, [$queue->getName(), base64_encode(serialize($message))]);
Scripts::executeCommand('flowpack.jobqueue.common:job:execute', $this->flowSettings, false, ['queue' => $queue->getName(), 'serializedMessage' => base64_encode(serialize($message))]);
} else {
$this->executeJobForMessage($queue, $message);
}
Expand Down
2 changes: 1 addition & 1 deletion Classes/Queue/FakeQueue.php
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ public function submit($payload, array $options = [])
{
$messageId = Algorithms::generateUUID();
$message = new Message($messageId, $payload);
$commandArguments = [$this->name, base64_encode(serialize($message))];
$commandArguments = ['queue' => $this->name, 'serializedMessage' => base64_encode(serialize($message))];
if ($this->async) {
if (!method_exists(Scripts::class, 'executeCommandAsync')) {
throw new \RuntimeException('The "async" flag is set, but the currently used Flow version doesn\'t support this (Flow 3.3+ is required)', 1469116604);
Expand Down

0 comments on commit 5e436ee

Please sign in to comment.