ConsoleHelpers\JiraCLI\Command\ChangeLogCloneCommand

Complexity

Total Complexity

15

Size/Duplication

Total Lines	192
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	4
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	prepareDependencies()	0	7	1
A	completeOptionValues()	0	9	2
A	configure()	0	21	1
B	execute()	0	94	8
A	expandProjectKeys()	0	14	3

<?php
/**
 * This file is part of the Jira-CLI library.
 * For the full copyright and license information, please view
 * the LICENSE file that was distributed with this source code.
 *
 * @copyright Alexander Obuhovich <aik.bold@gmail.com>
 * @link      https://github.com/console-helpers/jira-cli
 */

namespace ConsoleHelpers\JiraCLI\Command;


use chobie\Jira\Issue;
use ConsoleHelpers\ConsoleKit\Exception\CommandException;
use ConsoleHelpers\JiraCLI\Issue\ChangeLogIssueCloner;
use ConsoleHelpers\JiraCLI\JiraApi;
use Stecman\Component\Symfony\Console\BashCompletion\CompletionContext;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;

class ChangeLogCloneCommand extends AbstractCommand
{

	/**
	 * Issue cloner.
	 *
	 * @var ChangeLogIssueCloner
	 */
	protected $issueCloner;

	/**
	 * {@inheritdoc}
	 */
	protected function configure()
	{
		$this
			->setName('changelog-clone')
			->setDescription('Clones issue for changelog into another project')
			->addArgument(
				'issue_key',
				InputArgument::REQUIRED,
				'Issue key, e.g. <comment>JRA-1234</comment>'
			)
			->addArgument(
				'project_keys',
				InputArgument::REQUIRED | InputArgument::IS_ARRAY,
				'Project keys, e.g. <comment>PRJ</comment>'
			)
			->addOption(
				'link-name',
				null,
				InputOption::VALUE_REQUIRED,
				'Link name between issues',
				'Blocks'
			);
	}

	/**
	 * Prepare dependencies.
	 *
	 * @return void
	 */
	protected function prepareDependencies()
	{
		parent::prepareDependencies();

		$container = $this->getContainer();

		$this->issueCloner = $container['changelog_issue_cloner'];
	}

	/**
	 * Return possible values for the named option
	 *
	 * @param string            $optionName Option name.
	 * @param CompletionContext $context    Completion context.
	 *
	 * @return array
	 */
	public function completeOptionValues($optionName, CompletionContext $context)
	{
		$ret = parent::completeOptionValues($optionName, $context);

		if ( $optionName === 'link-name' ) {
			return $this->jiraApi->getIssueLinkTypeNames();
		}

		return $ret;
	}

	/**
	 * {@inheritdoc}
	 *
	 * @throws CommandException When no backportable issues were found.
	 */
	protected function execute(InputInterface $input, OutputInterface $output)
	{
		$issue_key = $this->io->getArgument('issue_key');
		$link_name = $this->io->getOption('link-name');

		if ( !in_array($link_name, $this->jiraApi->getIssueLinkTypeNames()) ) {
			throw new CommandException('The "' . $link_name . '" link name doesn\'t exist.');
		}

		$project_keys = $this->expandProjectKeys($this->io->getArgument('project_keys'));

It seems like $this->io->getArgument('project_keys') can also be of type null and string; however, parameter $project_keys of ConsoleHelpers\JiraCLI\Command\ChangeLogCloneCommand::expandProjectKeys() does only seem to accept array, maybe add an additional type check?

		$non_existing_projects = array_diff($project_keys, $this->jiraApi->getProjectKeys());

		if ( $non_existing_projects ) {

The expression $non_existing_projects of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

			throw new CommandException(
				'These projects doesn\'t exist: "' . implode('", "', $non_existing_projects) . '".'
			);
		}

		$issues = $this->issueCloner->getIssues(
			'key = ' . $issue_key,

Are you sure $issue_key of type null|string|string[] can be used in concatenation?

			$link_name,

It seems like $link_name can also be of type string[]; however, parameter $link_name of ConsoleHelpers\JiraCLI\Issue\IssueCloner::getIssues() does only seem to accept string, maybe add an additional type check?

			ChangeLogIssueCloner::LINK_DIRECTION_OUTWARD,
			$project_keys
		);
		$issue_count = count($issues);

		if ( !$issue_count ) {
			throw new CommandException('The "' . $issue_key . '" issue not found.');
		}

		foreach ( $issues as $issue_pair ) {
			/** @var Issue $issue */
			/** @var Issue $linked_issue */
			list($issue, $linked_issue, $link_project_key) = $issue_pair;

			$issue_project = $issue->get('project');

			if ( $issue_project['key'] === $link_project_key ) {
				throw new CommandException('Creating of linked issue in same project is not supported.');
			}

			if ( is_object($linked_issue) ) {
				$this->io->writeln(sprintf(
					'The "<info>%s</info>" issue already has "<info>%s</info>" link to "<info>%s</info>" issue.',
					$issue->getKey(),
					$link_name,

It seems like $link_name can also be of type string[]; however, parameter $values of sprintf() does only seem to accept double|integer|string, maybe add an additional type check?

					$linked_issue->getKey()
				));

				continue;
			}

			$components = array();
			$project_components = $this->jiraApi->getProjectComponentMapping(
				$link_project_key,
				JiraApi::CACHE_DURATION_ONE_MONTH
			);

			if ( $project_components ) {

The expression $project_components of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

				$component_name = $this->io->choose(
					'Select linked issue component in "' . $link_project_key . '" project:',
					$project_components,
					'',
					'The component isn\'t valid'
				);

				$components[] = array_search($component_name, $project_components);
			}

			$start = microtime(true);

			$linked_issue_key = $this->issueCloner->createLinkedIssue(
				$issue,
				$link_project_key,
				$link_name,

It seems like $link_name can also be of type string[]; however, parameter $link_name of ConsoleHelpers\JiraCLI\Issue\IssueCloner::createLinkedIssue() does only seem to accept string, maybe add an additional type check?

				ChangeLogIssueCloner::LINK_DIRECTION_OUTWARD,
				$components
			);

			$this->statistics[] = sprintf(
				'Issue %s creation time: %ss',
				$linked_issue_key,
				round(microtime(true) - $start, 3)
			);

			$this->io->writeln(sprintf(
				'The "<info>%s</info>" issue now has "<info>%s</info>" link to "<info>%s</info>" issue.',
				$issue->getKey(),
				$link_name,
				$linked_issue_key
			));
		}

		$this->showStatistics();
	}

	/**
	 * Expands project keys.
	 *
	 * @param array $project_keys Project keys.
	 *
	 * @return array
	 */
	protected function expandProjectKeys(array $project_keys)
	{
		$ret = array();

		foreach ( $project_keys as $project_key ) {
			if ( strpos($project_key, ',') === false ) {
				$ret[] = $project_key;
			}
			else {
				$ret = array_merge($ret, explode(',', $project_key));
			}
		}

		return $ret;
	}

}