console-helpers / jira-cli

src/JiraCLI/Command/VersionsCommand.php

<?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 ConsoleHelpers\JiraCLI\JiraApi;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;

class VersionsCommand extends AbstractCommand
{

	/**
	 * {@inheritdoc}
	 */
	protected function configure()
	{
		$this
			->setName('versions')
			->setDescription('Shows version usage across projects');
	}

	/**
	 * {@inheritdoc}
	 */
	protected function execute(InputInterface $input, OutputInterface $output)
	{
		$versions = array();

		$this->io->write('Getting projects ... ');
		$project_keys = $this->jiraApi->getProjectKeys();
		$this->io->writeln('done (' . count($project_keys) . ' found)');

		foreach ( $project_keys as $project_key ) {
			$this->io->write('Getting project <info>' . $project_key . '</info> versions ... ');
			$project_versions = $this->jiraApi->getVersions($project_key, JiraApi::CACHE_DURATION_ONE_MONTH);
			$this->io->writeln('done (' . count($project_versions) . ' found)');

It seems like $project_versions can also be of type false; however, parameter $value of count() does only seem to accept Countable|array, maybe add an additional type check?

			foreach ( $project_versions as $project_version_data ) {
				$project_version = $project_version_data['name'];

				// Interested only in final releases.
				if ( strpos($project_version, '-') === false ) {
					$versions[$project_version] = true;
				}
			}
		}

		$versions = array_keys($versions);
		usort($versions, 'version_compare');

		$this->io->writeln(array('', 'Versions:'));

		foreach ( $versions as $version ) {
			$this->io->writeln(' * ' . $version);
		}

		$this->showStatistics();
	}

}