mmucklo / DtcQueueBundle

Model/RetryableJob.php

Doc Comments +4 added lines, -4 removed lines

@@ -27,7 +27,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return integer
      */
     public function getMaxFailures()
     {
@@ -35,7 +35,7 @@ 
     }
 
     /**
-     * @param mixed $maxFailure
+     * @param mixed $maxFailures
      */
     public function setMaxFailures($maxFailures)
     {
@@ -121,7 +121,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return integer
      */
     public function getMaxExceptions()
     {
@@ -129,7 +129,7 @@ 
     }
 
     /**
-     * @param mixed $maxException
+     * @param mixed $maxExceptions
      */
     public function setMaxExceptions($maxExceptions)
     {

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;

Manager/StallableJobManager.php

Doc Comments +1 added lines, -1 removed lines

@@ -83,7 +83,7 @@
     }
 
     /**
-     * @param int|null $defaultMaxStalled
+     * @param integer|null $defaultMaxStalls
      */
     public function setDefaultMaxStalls($defaultMaxStalls)
     {

Manager/RetryableJobManager.php

Doc Comments +7 added lines, -3 removed lines

@@ -20,7 +20,7 @@ 
 
     /**
      * @param RetryableJob $job
-     * @param $retry bool
+     * @param boolean $retry bool
      *
      * @return
      */
@@ -65,6 +65,10 @@ 
         return $this->updateJobMax($job, 'Exceptions', RetryableJob::STATUS_MAX_EXCEPTIONS, $this->autoRetryOnException);
     }
 
+    /**
+     * @param string $type
+     * @param boolean $autoRetry
+     */
     protected function updateJobMax(RetryableJob $job, $type, $maxStatus, $autoRetry)
     {
         $setMethod = 'set'.$type;
@@ -171,7 +175,7 @@ 
     }
 
     /**
-     * @param int|null $defaultMaxFailure
+     * @param integer|null $defaultMaxFailures
      */
     public function setDefaultMaxFailures($defaultMaxFailures)
     {
@@ -219,7 +223,7 @@ 
     }
 
     /**
-     * @param int|null $defaultMaxException
+     * @param integer|null $defaultMaxExceptions
      */
     public function setDefaultMaxExceptions($defaultMaxExceptions)
     {

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;
@@ -107,6 +113,10 @@ 
         return null;
     }
 
+    /**
+     * @param string $foundJobCacheKey
+     * @param boolean $foundJobMessage
+     */
     protected function batchFoundJob(\Dtc\QueueBundle\Redis\Job $job, $foundJobCacheKey, $foundJobMessage)
     {
         $when = $job->getWhenUs();
@@ -132,6 +142,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?
@@ -151,6 +165,9 @@ 
         return $this->addFoundJob($adjust, $foundJob, $foundJobCacheKey, $crcCacheKey);
     }
 
+    /**
+     * @param boolean $adjust
+     */
     protected function addFoundJob($adjust, Job $foundJob, $foundJobCacheKey, $crcCacheKey)
     {
         $whenQueue = $this->getWhenQueueCacheKey();
@@ -168,6 +185,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) {
@@ -188,7 +208,7 @@ 
     /**
      * @param \Dtc\QueueBundle\Model\Job $job
      *
-     * @return \Dtc\QueueBundle\Model\Job
+     * @return Job|null
      *
      * @throws ClassNotSubclassException
      */
@@ -307,6 +327,9 @@ 
         }
     }
 
+    /**
+     * @param string $workerName
+     */
     protected function verifyGetJobArgs($workerName = null, $methodName = null, $prioritize = true)
     {
         if (null !== $workerName || null !== $methodName || (null !== $this->maxPriority && true !== $prioritize)) {