mmucklo / DtcQueueBundle

RabbitMQ/JobManager.php

Doc Comments +1 added lines, -1 removed lines

@@ -210,7 +210,7 @@
     }
 
     /**
-     * @param null $workerName
+     * @param string|null $workerName
      * @param null $methodName
      * @param bool $prioritize
      *

Redis/JobManager.php

Doc Comments +13 added lines, -1 removed lines

@@ -27,6 +27,9 @@ 
     protected $hostname;
     protected $pid;
 
+    /**
+     * @param string $cacheKeyPrefix
+     */
     public function __construct(RunManager $runManager, JobTimingManager $jobTimingManager, $jobClass, $cacheKeyPrefix)
     {
         $this->cacheKeyPrefix = $cacheKeyPrefix;
@@ -46,6 +49,9 @@ 
         return $this->cacheKeyPrefix.'_job_'.$jobId;
     }
 
+    /**
+     * @param string $jobCrc
+     */
     protected function getJobCrcHashKey($jobCrc)
     {
         return $this->cacheKeyPrefix.'_job_crc_'.$jobCrc;
@@ -103,6 +109,9 @@ 
         return null;
     }
 
+    /**
+     * @param string $foundJobCacheKey
+     */
     protected function batchFoundJob(\Dtc\QueueBundle\Redis\Job $job, $foundJobCacheKey, $foundJobMessage)
     {
         $when = $job->getWhenUs();
@@ -174,7 +183,7 @@ 
     /**
      * @param \Dtc\QueueBundle\Model\Job $job
      *
-     * @return \Dtc\QueueBundle\Model\Job
+     * @return null|Job
      *
      * @throws ClassNotSubclassException
      */
@@ -288,6 +297,9 @@ 
         }
     }
 
+    /**
+     * @param string $workerName
+     */
     protected function verifyGetJobArgs($workerName = null, $methodName = null, $prioritize = true)
     {
         if (null !== $workerName || null !== $methodName || (null !== $this->maxPriority && true !== $prioritize)) {

Redis/Predis.php

Spacing +2 added lines, -2 removed lines

@@ -70,7 +70,7 @@ 
         ];
 
         try {
-            $this->predis->transaction($options, function ($tx) use ($key, &$element) {
+            $this->predis->transaction($options, function($tx) use ($key, &$element) {
                 @list($element) = $tx->zrange($key, 0, 0);
 
                 if (isset($element)) {
@@ -95,7 +95,7 @@ 
         ];
 
         try {
-            $this->predis->transaction($options, function ($tx) use ($key, $max, &$element) {
+            $this->predis->transaction($options, function($tx) use ($key, $max, &$element) {
                 @list($element) = $tx->zrangebyscore($key, 0, $max, ['LIMIT' => [0, 1]]);
 
                 if (isset($element)) {

DependencyInjection/Configuration.php

Indentation +6 added lines, -6 removed lines

@@ -522,13 +522,13 @@
 
                 return $node;
             })->end()
-           ->validate()->ifTrue(function ($node) {
-               if (isset($node['ssl_options']) && !$node['ssl']) {
-                   return true;
-               }
+            ->validate()->ifTrue(function ($node) {
+                if (isset($node['ssl_options']) && !$node['ssl']) {
+                    return true;
+                }
 
-               return false;
-           })->thenInvalid('ssl must be true in order to set ssl_options')->end()
+                return false;
+            })->thenInvalid('ssl must be true in order to set ssl_options')->end()
         ->end();
 
         return $rootNode;

Spacing +8 added lines, -8 removed lines

@@ -301,7 +301,7 @@ 
                         ->scalarNode('alias')
                             ->defaultNull()->end()
                     ->end()
-                    ->validate()->ifTrue(function ($node) {
+                    ->validate()->ifTrue(function($node) {
                         if (isset($node['type']) && !isset($node['alias'])) {
                             return false;
                         }
@@ -317,7 +317,7 @@ 
                         ->scalarNode('dsn')->defaultNull()->end()
                         ->append($this->addPredisArgs())
                     ->end()
-                    ->validate()->ifTrue(function ($node) {
+                    ->validate()->ifTrue(function($node) {
                         if (isset($node['dsn']) && (isset($node['connection_parameters']['host']) || isset($node['connection_parameters']['port']))) {
                             return false;
                         }
@@ -327,7 +327,7 @@ 
                 ->end()
                 ->append($this->addPhpRedisArgs())
             ->end()
-            ->validate()->ifTrue(function ($node) {
+            ->validate()->ifTrue(function($node) {
                 if ((isset($node['predis']['dsn']) || isset($node['predis']['connection_parameters']['host'])) &&
                     (isset($node['snc_redis']['type']) || isset($node['phpredis']['host']))) {
                     return false;
@@ -362,7 +362,7 @@ 
                 ->floatNode('read_timeout')->defaultValue(0)->end()
                 ->scalarNode('auth')->end()
             ->end()
-            ->validate()->ifTrue(function ($node) {
+            ->validate()->ifTrue(function($node) {
                 if (!empty($node) && !isset($node['host'])) {
                     return false;
                 }
@@ -395,7 +395,7 @@ 
                 ->booleanNode('iterable_multibulk')->defaultFalse()->end()
                 ->booleanNode('throw_errors')->defaultTrue()->end()
             ->end()
-            ->validate()->ifTrue(function ($node) {
+            ->validate()->ifTrue(function($node) {
                 if (isset($node['host']) && !isset($node['port'])) {
                     return false;
                 }
@@ -435,7 +435,7 @@ 
         $rootNode
             ->prototype('variable')->end()
             ->validate()
-                ->ifTrue(function ($node) {
+                ->ifTrue(function($node) {
                     if (!is_array($node)) {
                         return true;
                     }
@@ -512,7 +512,7 @@ 
                 ->append($this->addRabbitMqArgs())
                 ->append($this->addRabbitMqExchange())
             ->end()
-            ->validate()->always(function ($node) {
+            ->validate()->always(function($node) {
                 if (empty($node['ssl_options'])) {
                     unset($node['ssl_options']);
                 }
@@ -522,7 +522,7 @@ 
 
                 return $node;
             })->end()
-           ->validate()->ifTrue(function ($node) {
+           ->validate()->ifTrue(function($node) {
                if (isset($node['ssl_options']) && !$node['ssl']) {
                    return true;
                }

Controller/QueueController.php

Spacing +2 added lines, -2 removed lines

@@ -60,13 +60,13 @@
         $methodName = $request->get('method');
 
         $jobManager = $this->get('dtc_queue.manager.job');
-        $callback = function ($count) {
+        $callback = function($count) {
             echo json_encode(['count' => $count]);
             echo "\n";
             flush();
         };
 
-        return new StreamedResponse(function () use ($jobManager, $callback, $workerName, $methodName) {
+        return new StreamedResponse(function() use ($jobManager, $callback, $workerName, $methodName) {
             $total = $jobManager->countLiveJobs($workerName, $methodName);
             echo json_encode(['total' => $total]);
             echo "\n";

Tests/Doctrine/DoctrineJobManagerTest.php

Spacing +1 added lines, -1 removed lines

@@ -286,7 +286,7 @@
         self::assertEquals(1, $count);
         $allCount = $this->runCountQuery($jobManager->getJobClass());
         $counter = 0;
-        $countJobs = function ($count) use (&$counter) {
+        $countJobs = function($count) use (&$counter) {
             $counter += $count;
         };
         $jobManager->archiveAllJobs(null, null, $countJobs);