mmucklo / DtcQueueBundle

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

@@ -272,7 +272,7 @@ 
                         ->scalarNode('alias')
                             ->defaultNull()->end()
                     ->end()
-                    ->validate()->ifTrue(function ($node) {
+                    ->validate()->ifTrue(function($node) {
                         if (isset($node['type']) && !isset($node['alias'])) {
                             return true;
                         }
@@ -288,7 +288,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 true;
                         }
@@ -298,7 +298,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 true;
@@ -334,7 +334,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 true;
                 }
@@ -367,7 +367,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 true;
                 }
@@ -407,7 +407,7 @@ 
         $rootNode
             ->prototype('variable')->end()
             ->validate()
-                ->ifTrue(function ($node) {
+                ->ifTrue(function($node) {
                     if (!is_array($node)) {
                         return true;
                     }
@@ -484,7 +484,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']);
                 }
@@ -494,7 +494,7 @@ 
 
                 return $node;
             })->end()
-           ->validate()->ifTrue(function ($node) {
+           ->validate()->ifTrue(function($node) {
                if (isset($node['ssl_options']) && !$node['ssl']) {
                    return true;
                }

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);

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();
         };
 
-        $streamingResponse = new StreamedResponse(function () use ($jobManager, $callback, $workerName, $methodName) {
+        $streamingResponse = new StreamedResponse(function() use ($jobManager, $callback, $workerName, $methodName) {
             $total = $jobManager->countLiveJobs($workerName, $methodName);
             echo json_encode(['total' => $total]);
             echo "\n";

Redis/JobManager.php

Doc Comments +24 added lines, -1 removed lines

@@ -26,6 +26,9 @@ 
     protected $hostname;
     protected $pid;
 
+    /**
+     * @param string $cacheKeyPrefix
+     */
     public function __construct(RunManager $runManager, JobTimingManager $jobTimingManager, $jobClass, $cacheKeyPrefix)
     {
         $this->cacheKeyPrefix = $cacheKeyPrefix;
@@ -45,6 +48,9 @@ 
         return $this->cacheKeyPrefix.'_job_'.$jobId;
     }
 
+    /**
+     * @param string $jobCrc
+     */
     protected function getJobCrcHashKey($jobCrc)
     {
         return $this->cacheKeyPrefix.'_job_crc_'.$jobCrc;
@@ -102,6 +108,10 @@ 
         return null;
     }
 
+    /**
+     * @param string $foundJobCacheKey
+     * @param boolean $foundJobMessage
+     */
     protected function batchFoundJob(\Dtc\QueueBundle\Redis\Job $job, $foundJobCacheKey, $foundJobMessage)
     {
         $when = $job->getWhenUs();
@@ -127,6 +137,10 @@ 
         return $this->finishBatchFoundJob($foundJob, $foundJobCacheKey, $crcCacheKey, $newFoundWhen, $newFoundPriority);
     }
 
+    /**
+     * @param string $crcCacheKey
+     * @param integer|null $newFoundPriority
+     */
     protected function finishBatchFoundJob(Job $foundJob, $foundJobCacheKey, $crcCacheKey, $newFoundWhen, $newFoundPriority)
     {
         // Now how do we adjust this job's priority or time?
@@ -146,6 +160,9 @@ 
         return $this->addFoundJob($adjust, $foundJob, $foundJobCacheKey, $crcCacheKey);
     }
 
+    /**
+     * @param boolean $adjust
+     */
     protected function addFoundJob($adjust, Job $foundJob, $foundJobCacheKey, $crcCacheKey)
     {
         $whenQueue = $this->getWhenQueueCacheKey();
@@ -163,6 +180,9 @@ 
         return $result ?: false;
     }
 
+    /**
+     * @param string $queue
+     */
     private function adjustJob($adjust, $queue, Job $foundJob, $foundJobCacheKey, $crcCacheKey, $zScore)
     {
         if ($adjust && $this->redis->zRem($queue, $foundJob->getId()) > 0) {
@@ -183,7 +203,7 @@ 
     /**
      * @param \Dtc\QueueBundle\Model\Job $job
      *
-     * @return \Dtc\QueueBundle\Model\Job
+     * @return null|Job
      *
      * @throws ClassNotSubclassException
      */
@@ -302,6 +322,9 @@ 
         }
     }
 
+    /**
+     * @param string $workerName
+     */
     protected function verifyGetJobArgs($workerName = null, $methodName = null, $prioritize = true)
     {
         if (null !== $workerName || null !== $methodName || (null !== $this->maxPriority && true !== $prioritize)) {