WP_Queue\Worker

Complexity

Total Complexity

8

Size/Duplication

Total Lines	60
Duplicated Lines	0 %

Test Coverage

Coverage

68.18%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
C	process()	0	32	7

<?php

namespace WP_Queue;

use Exception;
use WP_Queue\Connections\ConnectionInterface;
use WP_Queue\Exceptions\WorkerAttemptsExceededException;

class Worker {

	/**
	 * @var ConnectionInterface
	 */
	protected $connection;

	/**
	 * @var int
	 */
	protected $attempts;

	/**
	 * Worker constructor.
	 *
	 * @param ConnectionInterface $connection
	 * @param int                 $attempts
	 */
	public function __construct( $connection, $attempts = 3 ) {
		$this->connection = $connection;
		$this->attempts   = $attempts;
	}

	/**
	 * Process a job on the queue.
	 *
	 * @return bool
	 */
	public function process() {
		$job = $this->connection->pop();

		if ( ! $job ) {
			return false;
		}

		$exception = null;

		try {
			$job->handle();
		} catch ( Exception $exception ) {
			$job->release();
		}

		if ( $job->attempts() >= $this->attempts ) {
			if ( empty( $exception ) ) {

The condition empty($exception) is always false.

				$exception = new WorkerAttemptsExceededException();
			}
			
			$job->fail();
		}

		if ( $job->failed() ) {
			$this->connection->failure( $job, $exception );

It seems like $job can also be of type true; however, parameter $job of WP_Queue\Connections\ConnectionInterface::failure() does only seem to accept WP_Queue\Job, maybe add an additional type check?

		} else if ( $job->released() ) {
			$this->connection->release( $job );

It seems like $job can also be of type true; however, parameter $job of WP_Queue\Connections\ConnectionInterface::release() does only seem to accept WP_Queue\Job, maybe add an additional type check?

		} else {
			$this->connection->delete( $job );

It seems like $job can also be of type true; however, parameter $job of WP_Queue\Connections\ConnectionInterface::delete() does only seem to accept WP_Queue\Job, maybe add an additional type check?

		}

		return true;
	}

}