League\Geotools\CLI\Command\Geohash\Decode

Complexity

Total Complexity

2

Size/Duplication

Total Lines	26
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	11	1
A	execute()	0	11	1

<?php

/*
 * This file is part of the Geotools library.
 *
 * (c) Antoine Corcy <contact@sbin.dk>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace League\Geotools\CLI\Command\Geohash;

use League\Geotools\Geotools;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;

/**
 * Command-line geohash:decode class
 *
 * @author Antoine Corcy <contact@sbin.dk>
 */
class Decode extends \Symfony\Component\Console\Command\Command
{
    protected function configure()
    {
        $this
            ->setName('geohash:decode')
            ->setDescription('Decode a geo hash string to a coordinate')
            ->addArgument('geohash', InputArgument::REQUIRED, 'The geo hash to decode to coordinate')
            ->setHelp(<<<EOT
<info>Example</info>:              %command.full_name% spey61y
EOT
            );
    }

    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $geotools   = new Geotools;
        $coordinate = $geotools->geohash()->decode($input->getArgument('geohash'))->getCoordinate();

It seems like $input->getArgument('geohash') targeting Symfony\Component\Console\Input\InputInterface::getArgument() can also be of type array<integer,string> or null; however, League\Geotools\Geohash\GeohashInterface::decode() does only seem to accept string, maybe add an additional type check?

        $output->writeln(sprintf(
            '<value>%s, %s</value>',
            $coordinate->getLatitude(), $coordinate->getLongitude()
        ));
        return 0;
    }
}