App\Handler

Complexity

Total Complexity

11

Size/Duplication

Total Lines	92
Duplicated Lines	28.26 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes	2
Bugs	1	Features	1

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	handleDeploy()	0	17	3
A	handlePush()	13	13	2
A	handleTag()	13	13	2
A	extractBranchName()	0	4	1
A	extractTagName()	0	4	1
A	executeCommand()	0	4	1

<?php

namespace App;

use Interop\Container\ContainerInterface;


class Handler
{

	/**
	 * @var array
	 */
	private $scripts;

	/**
	 * @var Executor
	 */
	private $executor;


	public function __construct(ContainerInterface $ci, Executor $executor)
	{
		$this->executor = $executor;
		$this->scripts = $ci->get('scripts');

It seems like $ci->get('scripts') of type * is incompatible with the declared type array of property $scripts.

	}


	public function handleDeploy(array $event, array $build)
	{
		if ($build['status'] === 'created') {
			$projectName = $event['project']['path_with_namespace'];
			$buildId = $build['id'];
			$commitId = $event['commit']['id'];
			$name = $build['name'];
			if (isset($this->scripts[$projectName]['deploy'][$name])) {
				$this->executeCommand($this->scripts[$projectName]['deploy'][$name], [
					'HOOK_PROJECT_PATH' => $projectName,
					'HOOK_ENV_NAME' => $name,
					'HOOK_BUILD_ID' => $buildId,
					'HOOK_BUILD_REF' => $commitId
				]);
			}
		}
	}


	public function handlePush(array $event)

This method seems to be duplicated in your project.

	{
		$projectName = $event['project']['path_with_namespace'];
		$ref = $event['ref'];
		if (isset($this->scripts[$projectName]['push'][$ref])) {
			$this->executeCommand($this->scripts[$projectName]['push'][$ref], [
				'HOOK_PROJECT_PATH' => $projectName,
				'HOOK_REF' => $ref,
				'HOOK_BRANCH'=> $this->extractBranchName($ref),
				'HOOK_BUILD_REF' => $event['after']
			]);
		}
	}


	public function handleTag(array $event)

This method seems to be duplicated in your project.

	{
		$projectName = $event['project']['path_with_namespace'];
		$ref = $event['ref'];
		if (isset($this->scripts[$projectName]['tag'])) {
			$this->executeCommand($this->scripts[$projectName]['tag'], [
				'HOOK_PROJECT_PATH' => $projectName,
				'HOOK_REF' => $ref,
				'HOOK_TAG'=> $this->extractTagName($ref),
				'HOOK_BUILD_REF' => $event['after']
			]);
		}
	}


	private function extractBranchName($ref)
	{
		return substr($ref, strlen('refs/heads/'));
	}


	private function extractTagName($ref)
	{
		return substr($ref, strlen('refs/tags/'));
	}


	/**
	 * @param string|array $scriptPath
	 * @param array $env
	 */
	private function executeCommand($scriptPath, array $env)
	{
		$this->executor->executeCommand($scriptPath, $env);
	}

}