WP_Queue\Worker::process()

Complexity

Conditions	7
Paths	19

Size

Total Lines	32
Code Lines	19

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	11
CRAP Score	9.8821

Importance

Changes

0

<?php
	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;
	}
}