joomla / statistics-server

src/EventListener/ErrorSubscriber.php

Indentation +123 added lines, -123 removed lines

@@ -27,127 +27,127 @@
  */
 class ErrorSubscriber implements SubscriberInterface, LoggerAwareInterface
 {
-    use LoggerAwareTrait;
-
-    /**
-     * Returns an array of events this subscriber will listen to.
-     *
-     * @return  array
-     */
-    public static function getSubscribedEvents(): array
-    {
-        return [
-            ApplicationEvents::ERROR         => 'handleWebError',
-            ConsoleEvents::APPLICATION_ERROR => 'handleConsoleError',
-        ];
-    }
-
-    /**
-     * Handle console application errors.
-     *
-     * @param   ConsoleApplicationErrorEvent  $event  Event object
-     *
-     * @return  void
-     */
-    public function handleConsoleError(ConsoleApplicationErrorEvent $event): void
-    {
-        $this->logger->error(
-            sprintf('Uncaught Throwable of type %s caught.', \get_class($event->getError())),
-            ['exception' => $event->getError()]
-        );
-
-        (new SymfonyStyle($event->getApplication()->getConsoleInput(), $event->getApplication()->getConsoleOutput()))
-            ->error(sprintf('Uncaught Throwable of type %s caught: %s', \get_class($event->getError()), $event->getError()->getMessage()));
-    }
-
-    /**
-     * Handle web application errors.
-     *
-     * @param   ApplicationErrorEvent  $event  Event object
-     *
-     * @return  void
-     */
-    public function handleWebError(ApplicationErrorEvent $event): void
-    {
-        $app = $event->getApplication();
-
-        switch (true) {
-            case $event->getError() instanceof MethodNotAllowedException:
-                // Log the error for reference
-                $this->logger->error(
-                    sprintf('Route `%s` not supported by method `%s`', $app->get('uri.route'), $app->getInput()->getMethod()),
-                    ['exception' => $event->getError()]
-                );
-
-                $this->prepareResponse($event);
-
-                $app->setHeader('Allow', implode(', ', $event->getError()->getAllowedMethods()));
-
-                break;
-
-            case $event->getError() instanceof RouteNotFoundException:
-                // Log the error for reference
-                $this->logger->error(
-                    sprintf('Route `%s` not found', $app->get('uri.route')),
-                    ['exception' => $event->getError()]
-                );
-
-                $this->prepareResponse($event);
-
-                break;
-
-            default:
-                $this->logger->error(
-                    sprintf('Uncaught Throwable of type %s caught.', \get_class($event->getError())),
-                    ['exception' => $event->getError()]
-                );
-
-                $this->prepareResponse($event);
-
-                break;
-        }
-    }
-
-    /**
-     * Prepare the response for the event
-     *
-     * @param   ApplicationErrorEvent  $event  Event object
-     *
-     * @return  void
-     */
-    private function prepareResponse(ApplicationErrorEvent $event): void
-    {
-        /** @var WebApplication $app */
-        $app = $event->getApplication();
-
-        $app->allowCache(false);
-
-        $data = [
-            'code'    => $event->getError()->getCode(),
-            'message' => $event->getError()->getMessage(),
-            'error'   => true,
-        ];
-
-        $response = new JsonResponse($data);
-
-        switch ($event->getError()->getCode()) {
-            case 404:
-                $response = $response->withStatus(404);
-
-                break;
-
-            case 405:
-                $response = $response->withStatus(405);
-
-                break;
-
-            case 500:
-            default:
-                $response = $response->withStatus(500);
-
-                break;
-        }
-
-        $app->setResponse($response);
-    }
+	use LoggerAwareTrait;
+
+	/**
+	 * Returns an array of events this subscriber will listen to.
+	 *
+	 * @return  array
+	 */
+	public static function getSubscribedEvents(): array
+	{
+		return [
+			ApplicationEvents::ERROR         => 'handleWebError',
+			ConsoleEvents::APPLICATION_ERROR => 'handleConsoleError',
+		];
+	}
+
+	/**
+	 * Handle console application errors.
+	 *
+	 * @param   ConsoleApplicationErrorEvent  $event  Event object
+	 *
+	 * @return  void
+	 */
+	public function handleConsoleError(ConsoleApplicationErrorEvent $event): void
+	{
+		$this->logger->error(
+			sprintf('Uncaught Throwable of type %s caught.', \get_class($event->getError())),
+			['exception' => $event->getError()]
+		);
+
+		(new SymfonyStyle($event->getApplication()->getConsoleInput(), $event->getApplication()->getConsoleOutput()))
+			->error(sprintf('Uncaught Throwable of type %s caught: %s', \get_class($event->getError()), $event->getError()->getMessage()));
+	}
+
+	/**
+	 * Handle web application errors.
+	 *
+	 * @param   ApplicationErrorEvent  $event  Event object
+	 *
+	 * @return  void
+	 */
+	public function handleWebError(ApplicationErrorEvent $event): void
+	{
+		$app = $event->getApplication();
+
+		switch (true) {
+			case $event->getError() instanceof MethodNotAllowedException:
+				// Log the error for reference
+				$this->logger->error(
+					sprintf('Route `%s` not supported by method `%s`', $app->get('uri.route'), $app->getInput()->getMethod()),
+					['exception' => $event->getError()]
+				);
+
+				$this->prepareResponse($event);
+
+				$app->setHeader('Allow', implode(', ', $event->getError()->getAllowedMethods()));
+
+				break;
+
+			case $event->getError() instanceof RouteNotFoundException:
+				// Log the error for reference
+				$this->logger->error(
+					sprintf('Route `%s` not found', $app->get('uri.route')),
+					['exception' => $event->getError()]
+				);
+
+				$this->prepareResponse($event);
+
+				break;
+
+			default:
+				$this->logger->error(
+					sprintf('Uncaught Throwable of type %s caught.', \get_class($event->getError())),
+					['exception' => $event->getError()]
+				);
+
+				$this->prepareResponse($event);
+
+				break;
+		}
+	}
+
+	/**
+	 * Prepare the response for the event
+	 *
+	 * @param   ApplicationErrorEvent  $event  Event object
+	 *
+	 * @return  void
+	 */
+	private function prepareResponse(ApplicationErrorEvent $event): void
+	{
+		/** @var WebApplication $app */
+		$app = $event->getApplication();
+
+		$app->allowCache(false);
+
+		$data = [
+			'code'    => $event->getError()->getCode(),
+			'message' => $event->getError()->getMessage(),
+			'error'   => true,
+		];
+
+		$response = new JsonResponse($data);
+
+		switch ($event->getError()->getCode()) {
+			case 404:
+				$response = $response->withStatus(404);
+
+				break;
+
+			case 405:
+				$response = $response->withStatus(405);
+
+				break;
+
+			case 500:
+			default:
+				$response = $response->withStatus(500);
+
+				break;
+		}
+
+		$app->setResponse($response);
+	}
 }

src/EventListener/AnalyticsSubscriber.php

Indentation +65 added lines, -65 removed lines

@@ -23,77 +23,77 @@
  */
 class AnalyticsSubscriber implements SubscriberInterface, LoggerAwareInterface
 {
-    use LoggerAwareTrait;
+	use LoggerAwareTrait;
 
-    /**
-     * Application analytics object.
-     *
-     * @var  Analytics
-     */
-    private $analytics;
+	/**
+	 * Application analytics object.
+	 *
+	 * @var  Analytics
+	 */
+	private $analytics;
 
-    /**
-     * Constructor.
-     *
-     * @param   Analytics  $analytics  Application analytics object.
-     */
-    public function __construct(Analytics $analytics)
-    {
-        $this->analytics = $analytics;
-    }
+	/**
+	 * Constructor.
+	 *
+	 * @param   Analytics  $analytics  Application analytics object.
+	 */
+	public function __construct(Analytics $analytics)
+	{
+		$this->analytics = $analytics;
+	}
 
-    /**
-     * Returns an array of events this subscriber will listen to.
-     *
-     * @return  array
-     */
-    public static function getSubscribedEvents(): array
-    {
-        return [
-            ApplicationEvents::BEFORE_EXECUTE => 'onBeforeExecute',
-        ];
-    }
+	/**
+	 * Returns an array of events this subscriber will listen to.
+	 *
+	 * @return  array
+	 */
+	public static function getSubscribedEvents(): array
+	{
+		return [
+			ApplicationEvents::BEFORE_EXECUTE => 'onBeforeExecute',
+		];
+	}
 
-    /**
-     * Logs the visit to analytics if able.
-     *
-     * @param   ApplicationEvent  $event  Event object
-     *
-     * @return  void
-     */
-    public function onBeforeExecute(ApplicationEvent $event): void
-    {
-        $app = $event->getApplication();
+	/**
+	 * Logs the visit to analytics if able.
+	 *
+	 * @param   ApplicationEvent  $event  Event object
+	 *
+	 * @return  void
+	 */
+	public function onBeforeExecute(ApplicationEvent $event): void
+	{
+		$app = $event->getApplication();
 
-        if (!($app instanceof WebApplicationInterface)) {
-            return;
-        }
+		if (!($app instanceof WebApplicationInterface)) {
+			return;
+		}
 
-        // On a GET request to the live domain, submit analytics data
-        if (
-            $app->getInput()->getMethod() !== 'GET'
-            || strpos($app->getInput()->server->getString('HTTP_HOST', ''), 'developer.joomla.org') !== 0
-        ) {
-            return;
-        }
+		// On a GET request to the live domain, submit analytics data
+		if (
+			$app->getInput()->getMethod() !== 'GET'
+			|| strpos($app->getInput()->server->getString('HTTP_HOST', ''), 'developer.joomla.org') !== 0
+		) {
+			return;
+		}
 
-        $this->analytics->setAsyncRequest(true)
-            ->setProtocolVersion('1')
-            ->setTrackingId('UA-544070-16')
-            ->setClientId(Uuid::uuid4()->toString())
-            ->setDocumentPath($app->get('uri.base.path'))
-            ->setIpOverride($app->getInput()->server->getString('REMOTE_ADDR', '127.0.0.1'))
-            ->setUserAgentOverride($app->getInput()->server->getString('HTTP_USER_AGENT', 'JoomlaStats/1.0'));
+		$this->analytics->setAsyncRequest(true)
+			->setProtocolVersion('1')
+			->setTrackingId('UA-544070-16')
+			->setClientId(Uuid::uuid4()->toString())
+			->setDocumentPath($app->get('uri.base.path'))
+			->setIpOverride($app->getInput()->server->getString('REMOTE_ADDR', '127.0.0.1'))
+			->setUserAgentOverride($app->getInput()->server->getString('HTTP_USER_AGENT', 'JoomlaStats/1.0'));
 
-        // Don't allow sending Analytics data to cause a failure
-        try {
-            $this->analytics->sendPageview();
-        } catch (\Exception $e) {
-            // Log the error for reference
-            $this->logger->error(
-                'Error sending analytics data.',
-                ['exception' => $e]
-            );
-        }
-    }
+		// Don't allow sending Analytics data to cause a failure
+		try {
+			$this->analytics->sendPageview();
+		} catch (\Exception $e) {
+			// Log the error for reference
+			$this->logger->error(
+				'Error sending analytics data.',
+				['exception' => $e]
+			);
+		}
+	}
 }

src/Controllers/DisplayStatisticsController.php

Indentation +34 added lines, -34 removed lines

@@ -20,45 +20,45 @@
  */
 class DisplayStatisticsController extends AbstractController
 {
-    /**
-     * JSON view for displaying the statistics.
-     *
-     * @var  StatsJsonView
-     */
-    private $view;
+	/**
+	 * JSON view for displaying the statistics.
+	 *
+	 * @var  StatsJsonView
+	 */
+	private $view;
 
-    /**
-     * Constructor.
-     *
-     * @param   StatsJsonView  $view  JSON view for displaying the statistics.
-     */
-    public function __construct(StatsJsonView $view)
-    {
-        $this->view = $view;
-    }
+	/**
+	 * Constructor.
+	 *
+	 * @param   StatsJsonView  $view  JSON view for displaying the statistics.
+	 */
+	public function __construct(StatsJsonView $view)
+	{
+		$this->view = $view;
+	}
 
-    /**
-     * Execute the controller.
-     *
-     * @return  boolean
-     */
-    public function execute()
-    {
-        // Check if we are allowed to receive the raw data
-        $authorizedRaw = $this->getInput()->server->getString('HTTP_JOOMLA_RAW', 'fail') === $this->getApplication()->get('stats.rawdata', false);
+	/**
+	 * Execute the controller.
+	 *
+	 * @return  boolean
+	 */
+	public function execute()
+	{
+		// Check if we are allowed to receive the raw data
+		$authorizedRaw = $this->getInput()->server->getString('HTTP_JOOMLA_RAW', 'fail') === $this->getApplication()->get('stats.rawdata', false);
 
-        // Check if a single data source is requested
-        $source = $this->getInput()->getString('source', '');
+		// Check if a single data source is requested
+		$source = $this->getInput()->getString('source', '');
 
-        // Check if a timeframe is requested
-        $timeframe = (int) $this->getInput()->getInt('timeframe', 0);
+		// Check if a timeframe is requested
+		$timeframe = (int) $this->getInput()->getInt('timeframe', 0);
 
-        $this->view->isAuthorizedRaw($authorizedRaw);
-        $this->view->setSource($source);
-        $this->view->setTimeframe($timeframe);
+		$this->view->isAuthorizedRaw($authorizedRaw);
+		$this->view->setSource($source);
+		$this->view->setTimeframe($timeframe);
 
-        $this->getApplication()->setBody($this->view->render());
+		$this->getApplication()->setBody($this->view->render());
 
-        return true;
-    }
+		return true;
+	}
 }

src/Controllers/SubmitDataController.php

Indentation +233 added lines, -233 removed lines

@@ -25,237 +25,237 @@
  */
 class SubmitDataController extends AbstractController
 {
-    use ValidateVersion;
-
-    /**
-     * Statistics repository.
-     *
-     * @var  StatisticsRepository
-     */
-    private $repository;
-
-    /**
-     * Influxdb repository.
-     *
-     * @var  InfluxdbRepository
-     */
-    private $influxdbRepository;
-
-    /**
-     * Filesystem adapter for the snapshots space.
-     *
-     * @var  Filesystem
-     */
-    private $filesystem;
-
-    /**
-     * Allowed Database Types.
-     *
-     * @var  array
-     */
-    private $databaseTypes = [
-        'mysql',
-        'mysqli',
-        'pgsql',
-        'pdomysql',
-        'postgresql',
-        'sqlazure',
-        'sqlsrv',
-    ];
-
-    /**
-     * Constructor.
-     *
-     * @param   StatisticsRepository  $repository  Statistics repository.
-     * @param   Filesystem            $filesystem  Filesystem adapter for the versions space.
-     */
-    public function __construct(StatisticsRepository $repository, Filesystem $filesystem, InfluxdbRepository $influxdbRepository = null)
-    {
-        $this->repository         = $repository;
-        $this->filesystem         = $filesystem;
-        $this->influxdbRepository = $influxdbRepository;
-    }
-
-    /**
-     * Execute the controller.
-     *
-     * @return  boolean
-     */
-    public function execute()
-    {
-        $input = $this->getInput();
-
-        $data = [
-            'php_version' => $input->getRaw('php_version', ''),
-            'db_version'  => $input->getRaw('db_version', ''),
-            'cms_version' => $input->getRaw('cms_version', ''),
-            'unique_id'   => $input->getString('unique_id'),
-            'db_type'     => $input->getString('db_type', ''),
-            'server_os'   => $input->getString('server_os'),
-        ];
-
-        // Backup the original POST before manipulating/validating data
-        $originalData = $data;
-
-        // Validate the submitted data
-        $data['php_version'] = $this->checkPHPVersion($data['php_version']);
-        $data['cms_version'] = $this->checkCMSVersion($data['cms_version']);
-        $data['db_type']     = $this->checkDatabaseType($data['db_type']);
-        $data['db_version']  = $this->validateVersionNumber($data['db_version']);
-
-        // We require at a minimum a unique ID and the CMS version
-        if (empty($data['unique_id']) || (empty($data['cms_version']) && $data['cms_version'] !== false)) {
-            $this->getApplication()->getLogger()->info(
-                'Missing required data from request.',
-                ['postData' => $originalData]
-            );
-
-            /** @var JsonResponse $response */
-            $response = $this->getApplication()->getResponse();
-            $response = $response->withPayload(
-                [
-                    'error'   => true,
-                    'message' => 'There was an error storing the data.',
-                ]
-            );
-            $response = $response->withStatus(500);
-
-            $this->getApplication()->setResponse($response);
-
-            return true;
-        }
-
-        // If the below data does not pass tests, we do not accept the POST
-        if ($data['php_version'] === false || $data['cms_version'] === false || $data['db_type'] === false || $data['db_version'] === false) {
-            /** @var JsonResponse $response */
-            $response = $this->getApplication()->getResponse();
-            $response = $response->withPayload(
-                [
-                    'error'   => true,
-                    'message' => 'Invalid data submission.',
-                ]
-            );
-            $response = $response->withStatus(500);
-
-            $this->getApplication()->setResponse($response);
-
-            return true;
-        }
-
-        // Account for configuration differences with 4.0
-        if (version_compare($data['cms_version'], '4.0', 'ge')) {
-            // For 4.0 and later, we map `mysql` to the `pdomysql` option to correctly track the database type
-            if ($data['db_type'] === 'mysql') {
-                $data['db_type'] = 'pdomysql';
-            }
-        }
-
-        $this->repository->save((object) $data);
-        if ($this->influxdbRepository instanceof InfluxdbRepository) {
-            $this->influxdbRepository->save((object) $data);
-        }
-
-        /** @var JsonResponse $response */
-        $response = $this->getApplication()->getResponse();
-        $response = $response->withPayload(
-            [
-                'error'   => false,
-                'message' => 'Data saved successfully',
-            ]
-        );
-
-        $this->getApplication()->setResponse($response);
-
-        return true;
-    }
-
-    /**
-     * Check the CMS version.
-     *
-     * @param   string  $version  The version number to check.
-     *
-     * @return  string|boolean  The version number on success or boolean false on failure.
-     */
-    private function checkCMSVersion(string $version)
-    {
-        $version = $this->validateVersionNumber($version);
-
-        // If the version number is invalid, don't go any further
-        if ($version === false) {
-            return false;
-        }
-
-        // Joomla only uses major.minor.patch so everything else is invalid
-        $explodedVersion = explode('.', $version);
-
-        if (\count($explodedVersion) > 3) {
-            return false;
-        }
-
-        try {
-            $validVersions = json_decode($this->filesystem->read('joomla.json'), true);
-        } catch (FileNotFoundException $exception) {
-            throw new \RuntimeException('Missing Joomla! release listing', 500, $exception);
-        }
-
-        // Check that the version is in our valid release list
-        if (!\in_array($version, $validVersions)) {
-            return false;
-        }
-
-        return $version;
-    }
-
-    /**
-     * Check the database type
-     *
-     * @param   string  $database  The database type to check.
-     *
-     * @return  string|boolean  The database type on success or boolean false on failure.
-     */
-    private function checkDatabaseType(string $database)
-    {
-        if (!\in_array($database, $this->databaseTypes)) {
-            return false;
-        }
-
-        return $database;
-    }
-
-    /**
-     * Check the PHP version
-     *
-     * @param   string  $version  The version number to check.
-     *
-     * @return  string|boolean  The version number on success or boolean false on failure.
-     */
-    private function checkPHPVersion(string $version)
-    {
-        $version = $this->validateVersionNumber($version);
-
-        // If the version number is invalid, don't go any further
-        if ($version === false) {
-            return false;
-        }
-
-        // We only track versions based on major.minor.patch so everything else is invalid
-        $explodedVersion = explode('.', $version);
-
-        if (\count($explodedVersion) > 3) {
-            return false;
-        }
-
-        try {
-            $validVersions = json_decode($this->filesystem->read('php.json'), true);
-        } catch (FileNotFoundException $exception) {
-            throw new \RuntimeException('Missing PHP release listing', 500, $exception);
-        }
-
-        // Check that the version is in our valid release list
-        if (!\in_array($version, $validVersions)) {
-            return false;
-        }
-
-        return $version;
-    }
+	use ValidateVersion;
+
+	/**
+	 * Statistics repository.
+	 *
+	 * @var  StatisticsRepository
+	 */
+	private $repository;
+
+	/**
+	 * Influxdb repository.
+	 *
+	 * @var  InfluxdbRepository
+	 */
+	private $influxdbRepository;
+
+	/**
+	 * Filesystem adapter for the snapshots space.
+	 *
+	 * @var  Filesystem
+	 */
+	private $filesystem;
+
+	/**
+	 * Allowed Database Types.
+	 *
+	 * @var  array
+	 */
+	private $databaseTypes = [
+		'mysql',
+		'mysqli',
+		'pgsql',
+		'pdomysql',
+		'postgresql',
+		'sqlazure',
+		'sqlsrv',
+	];
+
+	/**
+	 * Constructor.
+	 *
+	 * @param   StatisticsRepository  $repository  Statistics repository.
+	 * @param   Filesystem            $filesystem  Filesystem adapter for the versions space.
+	 */
+	public function __construct(StatisticsRepository $repository, Filesystem $filesystem, InfluxdbRepository $influxdbRepository = null)
+	{
+		$this->repository         = $repository;
+		$this->filesystem         = $filesystem;
+		$this->influxdbRepository = $influxdbRepository;
+	}
+
+	/**
+	 * Execute the controller.
+	 *
+	 * @return  boolean
+	 */
+	public function execute()
+	{
+		$input = $this->getInput();
+
+		$data = [
+			'php_version' => $input->getRaw('php_version', ''),
+			'db_version'  => $input->getRaw('db_version', ''),
+			'cms_version' => $input->getRaw('cms_version', ''),
+			'unique_id'   => $input->getString('unique_id'),
+			'db_type'     => $input->getString('db_type', ''),
+			'server_os'   => $input->getString('server_os'),
+		];
+
+		// Backup the original POST before manipulating/validating data
+		$originalData = $data;
+
+		// Validate the submitted data
+		$data['php_version'] = $this->checkPHPVersion($data['php_version']);
+		$data['cms_version'] = $this->checkCMSVersion($data['cms_version']);
+		$data['db_type']     = $this->checkDatabaseType($data['db_type']);
+		$data['db_version']  = $this->validateVersionNumber($data['db_version']);
+
+		// We require at a minimum a unique ID and the CMS version
+		if (empty($data['unique_id']) || (empty($data['cms_version']) && $data['cms_version'] !== false)) {
+			$this->getApplication()->getLogger()->info(
+				'Missing required data from request.',
+				['postData' => $originalData]
+			);
+
+			/** @var JsonResponse $response */
+			$response = $this->getApplication()->getResponse();
+			$response = $response->withPayload(
+				[
+					'error'   => true,
+					'message' => 'There was an error storing the data.',
+				]
+			);
+			$response = $response->withStatus(500);
+
+			$this->getApplication()->setResponse($response);
+
+			return true;
+		}
+
+		// If the below data does not pass tests, we do not accept the POST
+		if ($data['php_version'] === false || $data['cms_version'] === false || $data['db_type'] === false || $data['db_version'] === false) {
+			/** @var JsonResponse $response */
+			$response = $this->getApplication()->getResponse();
+			$response = $response->withPayload(
+				[
+					'error'   => true,
+					'message' => 'Invalid data submission.',
+				]
+			);
+			$response = $response->withStatus(500);
+
+			$this->getApplication()->setResponse($response);
+
+			return true;
+		}
+
+		// Account for configuration differences with 4.0
+		if (version_compare($data['cms_version'], '4.0', 'ge')) {
+			// For 4.0 and later, we map `mysql` to the `pdomysql` option to correctly track the database type
+			if ($data['db_type'] === 'mysql') {
+				$data['db_type'] = 'pdomysql';
+			}
+		}
+
+		$this->repository->save((object) $data);
+		if ($this->influxdbRepository instanceof InfluxdbRepository) {
+			$this->influxdbRepository->save((object) $data);
+		}
+
+		/** @var JsonResponse $response */
+		$response = $this->getApplication()->getResponse();
+		$response = $response->withPayload(
+			[
+				'error'   => false,
+				'message' => 'Data saved successfully',
+			]
+		);
+
+		$this->getApplication()->setResponse($response);
+
+		return true;
+	}
+
+	/**
+	 * Check the CMS version.
+	 *
+	 * @param   string  $version  The version number to check.
+	 *
+	 * @return  string|boolean  The version number on success or boolean false on failure.
+	 */
+	private function checkCMSVersion(string $version)
+	{
+		$version = $this->validateVersionNumber($version);
+
+		// If the version number is invalid, don't go any further
+		if ($version === false) {
+			return false;
+		}
+
+		// Joomla only uses major.minor.patch so everything else is invalid
+		$explodedVersion = explode('.', $version);
+
+		if (\count($explodedVersion) > 3) {
+			return false;
+		}
+
+		try {
+			$validVersions = json_decode($this->filesystem->read('joomla.json'), true);
+		} catch (FileNotFoundException $exception) {
+			throw new \RuntimeException('Missing Joomla! release listing', 500, $exception);
+		}
+
+		// Check that the version is in our valid release list
+		if (!\in_array($version, $validVersions)) {
+			return false;
+		}
+
+		return $version;
+	}
+
+	/**
+	 * Check the database type
+	 *
+	 * @param   string  $database  The database type to check.
+	 *
+	 * @return  string|boolean  The database type on success or boolean false on failure.
+	 */
+	private function checkDatabaseType(string $database)
+	{
+		if (!\in_array($database, $this->databaseTypes)) {
+			return false;
+		}
+
+		return $database;
+	}
+
+	/**
+	 * Check the PHP version
+	 *
+	 * @param   string  $version  The version number to check.
+	 *
+	 * @return  string|boolean  The version number on success or boolean false on failure.
+	 */
+	private function checkPHPVersion(string $version)
+	{
+		$version = $this->validateVersionNumber($version);
+
+		// If the version number is invalid, don't go any further
+		if ($version === false) {
+			return false;
+		}
+
+		// We only track versions based on major.minor.patch so everything else is invalid
+		$explodedVersion = explode('.', $version);
+
+		if (\count($explodedVersion) > 3) {
+			return false;
+		}
+
+		try {
+			$validVersions = json_decode($this->filesystem->read('php.json'), true);
+		} catch (FileNotFoundException $exception) {
+			throw new \RuntimeException('Missing PHP release listing', 500, $exception);
+		}
+
+		// Check that the version is in our valid release list
+		if (!\in_array($version, $validVersions)) {
+			return false;
+		}
+
+		return $version;
+	}
 }

src/Decorators/ValidateVersion.php

Indentation +11 added lines, -11 removed lines

@@ -14,15 +14,15 @@
  */
 trait ValidateVersion
 {
-    /**
-     * Validates and filters the version number
-     *
-     * @param   string  $version  The version string to validate.
-     *
-     * @return  string|boolean  A validated version number on success or boolean false.
-     */
-    protected function validateVersionNumber(string $version)
-    {
-        return preg_match('/\d+(?:\.\d+)+/', $version, $matches) ? $matches[0] : false;
-    }
+	/**
+	 * Validates and filters the version number
+	 *
+	 * @param   string  $version  The version string to validate.
+	 *
+	 * @return  string|boolean  A validated version number on success or boolean false.
+	 */
+	protected function validateVersionNumber(string $version)
+	{
+		return preg_match('/\d+(?:\.\d+)+/', $version, $matches) ? $matches[0] : false;
+	}
 }

src/Commands/UpdateCommand.php

Indentation +69 added lines, -69 removed lines

@@ -22,73 +22,73 @@
  */
 class UpdateCommand extends AbstractCommand
 {
-    /**
-     * The default command name
-     *
-     * @var  string|null
-     */
-    protected static $defaultName = 'update:server';
-
-    /**
-     * Internal function to execute the command.
-     *
-     * @param   InputInterface   $input   The input to inject into the command.
-     * @param   OutputInterface  $output  The output to inject into the command.
-     *
-     * @return  integer  The command exit code
-     */
-    protected function doExecute(InputInterface $input, OutputInterface $output): int
-    {
-        $symfonyStyle = new SymfonyStyle($input, $output);
-
-        $symfonyStyle->title('Update Server');
-        $symfonyStyle->comment('Updating server to git HEAD');
-
-        if (!$this->getHelperSet()) {
-            $symfonyStyle->error('The helper set has not been registered to the update command.');
-
-            return 1;
-        }
-
-        /** @var ProcessHelper $processHelper */
-        $processHelper = $this->getHelperSet()->get('process');
-
-        // Pull from remote repo
-        try {
-            $processHelper->mustRun($output, new Process(['git', 'pull'], APPROOT));
-        } catch (ProcessFailedException $e) {
-            $this->getApplication()->getLogger()->error('Could not execute `git pull`', ['exception' => $e]);
-
-            $symfonyStyle->error('Error running `git pull`: ' . $e->getMessage());
-
-            return 1;
-        }
-
-        $symfonyStyle->comment('Updating Composer resources');
-
-        // Run Composer install
-        try {
-            $processHelper->mustRun($output, new Process(['composer', 'install', '--no-dev', '-o', '-a'], APPROOT));
-        } catch (ProcessFailedException $e) {
-            $this->getApplication()->getLogger()->error('Could not update Composer resources', ['exception' => $e]);
-
-            $symfonyStyle->error('Error updating Composer resources: ' . $e->getMessage());
-
-            return 1;
-        }
-
-        $symfonyStyle->success('Update complete');
-
-        return 0;
-    }
-
-    /**
-     * Configures the current command.
-     *
-     * @return  void
-     */
-    protected function configure(): void
-    {
-        $this->setDescription('Update the server to the current git HEAD');
-    }
+	/**
+	 * The default command name
+	 *
+	 * @var  string|null
+	 */
+	protected static $defaultName = 'update:server';
+
+	/**
+	 * Internal function to execute the command.
+	 *
+	 * @param   InputInterface   $input   The input to inject into the command.
+	 * @param   OutputInterface  $output  The output to inject into the command.
+	 *
+	 * @return  integer  The command exit code
+	 */
+	protected function doExecute(InputInterface $input, OutputInterface $output): int
+	{
+		$symfonyStyle = new SymfonyStyle($input, $output);
+
+		$symfonyStyle->title('Update Server');
+		$symfonyStyle->comment('Updating server to git HEAD');
+
+		if (!$this->getHelperSet()) {
+			$symfonyStyle->error('The helper set has not been registered to the update command.');
+
+			return 1;
+		}
+
+		/** @var ProcessHelper $processHelper */
+		$processHelper = $this->getHelperSet()->get('process');
+
+		// Pull from remote repo
+		try {
+			$processHelper->mustRun($output, new Process(['git', 'pull'], APPROOT));
+		} catch (ProcessFailedException $e) {
+			$this->getApplication()->getLogger()->error('Could not execute `git pull`', ['exception' => $e]);
+
+			$symfonyStyle->error('Error running `git pull`: ' . $e->getMessage());
+
+			return 1;
+		}
+
+		$symfonyStyle->comment('Updating Composer resources');
+
+		// Run Composer install
+		try {
+			$processHelper->mustRun($output, new Process(['composer', 'install', '--no-dev', '-o', '-a'], APPROOT));
+		} catch (ProcessFailedException $e) {
+			$this->getApplication()->getLogger()->error('Could not update Composer resources', ['exception' => $e]);
+
+			$symfonyStyle->error('Error updating Composer resources: ' . $e->getMessage());
+
+			return 1;
+		}
+
+		$symfonyStyle->success('Update complete');
+
+		return 0;
+	}
+
+	/**
+	 * Configures the current command.
+	 *
+	 * @return  void
+	 */
+	protected function configure(): void
+	{
+		$this->setDescription('Update the server to the current git HEAD');
+	}
 }

Spacing +2 added lines, -2 removed lines

@@ -59,7 +59,7 @@ 
         } catch (ProcessFailedException $e) {
             $this->getApplication()->getLogger()->error('Could not execute `git pull`', ['exception' => $e]);
 
-            $symfonyStyle->error('Error running `git pull`: ' . $e->getMessage());
+            $symfonyStyle->error('Error running `git pull`: '.$e->getMessage());
 
             return 1;
         }
@@ -72,7 +72,7 @@ 
         } catch (ProcessFailedException $e) {
             $this->getApplication()->getLogger()->error('Could not update Composer resources', ['exception' => $e]);
 
-            $symfonyStyle->error('Error updating Composer resources: ' . $e->getMessage());
+            $symfonyStyle->error('Error updating Composer resources: '.$e->getMessage());
 
             return 1;
         }

src/Commands/Tags/FetchJoomlaTagsCommand.php

Indentation +103 added lines, -103 removed lines

@@ -20,107 +20,107 @@
  */
 class FetchJoomlaTagsCommand extends AbstractTagCommand
 {
-    use ValidateVersion;
-
-    /**
-     * The default command name
-     *
-     * @var  string|null
-     */
-    protected static $defaultName = 'tags:joomla';
-
-    /**
-     * Constructor.
-     *
-     * @param   GitHub      $github      GitHub API object.
-     * @param   Filesystem  $filesystem  Filesystem adapter for the versions space.
-     */
-    public function __construct(GitHub $github, Filesystem $filesystem)
-    {
-        parent::__construct($github, $filesystem);
-
-        $this->repoName  = 'joomla-cms';
-        $this->repoOwner = 'joomla';
-    }
-
-    /**
-     * Internal function to execute the command.
-     *
-     * @param   InputInterface   $input   The input to inject into the command.
-     * @param   OutputInterface  $output  The output to inject into the command.
-     *
-     * @return  integer  The command exit code
-     */
-    protected function doExecute(InputInterface $input, OutputInterface $output): int
-    {
-        $this->io->title('Fetching Joomla Releases');
-
-        $versions    = [];
-        $highVersion = '0.0.0';
-
-        foreach ($this->getTags() as $tag) {
-            $version = $this->validateVersionNumber($tag->name);
-
-            // Only process if the tag name looks like a version number
-            if ($version === false) {
-                continue;
-            }
-
-            // Joomla only uses major.minor.patch so everything else is invalid
-            $explodedVersion = explode('.', $version);
-
-            if (\count($explodedVersion) != 3) {
-                continue;
-            }
-
-            // Version collection is valid for the 3.x series and later
-            if (version_compare($version, '3.0.0', '<')) {
-                continue;
-            }
-
-            // We have a valid version number, great news... add it to our array if it isn't already present
-            if (!\in_array($version, $versions)) {
-                $versions[] = $version;
-
-                // If this version is higher than our high version, replace it
-                // TODO - When 4.0 is stable adjust this logic
-                if (version_compare($version, '4.0', '<') && version_compare($version, $highVersion, '>')) {
-                    $highVersion = $version;
-                }
-            }
-        }
-
-        // If the high version is not the default then let's add some (arbitrary) allowed versions based on the repo's dev structure
-        if ($highVersion !== '0.0.0') {
-            $explodedVersion = explode('.', $highVersion);
-
-            // Allow the next patch release after this one
-            $nextPatch  = $explodedVersion[2] + 1;
-            $versions[] = $explodedVersion[0] . '.' . $explodedVersion[1] . '.' . $nextPatch;
-
-            // And allow the next minor release after this one
-            $nextMinor  = $explodedVersion[1] + 1;
-            $versions[] = $explodedVersion[0] . '.' . $nextMinor . '.0';
-        }
-
-        if (!$this->filesystem->put('joomla.json', json_encode($versions))) {
-            $this->io->error('Failed writing version data to the filesystem.');
-
-            return 1;
-        }
-
-        $this->io->success('Joomla! versions updated.');
-
-        return 0;
-    }
-
-    /**
-     * Configures the current command.
-     *
-     * @return  void
-     */
-    protected function configure(): void
-    {
-        $this->setDescription('Parses the release tags for the Joomla! CMS GitHub repository.');
-    }
+	use ValidateVersion;
+
+	/**
+	 * The default command name
+	 *
+	 * @var  string|null
+	 */
+	protected static $defaultName = 'tags:joomla';
+
+	/**
+	 * Constructor.
+	 *
+	 * @param   GitHub      $github      GitHub API object.
+	 * @param   Filesystem  $filesystem  Filesystem adapter for the versions space.
+	 */
+	public function __construct(GitHub $github, Filesystem $filesystem)
+	{
+		parent::__construct($github, $filesystem);
+
+		$this->repoName  = 'joomla-cms';
+		$this->repoOwner = 'joomla';
+	}
+
+	/**
+	 * Internal function to execute the command.
+	 *
+	 * @param   InputInterface   $input   The input to inject into the command.
+	 * @param   OutputInterface  $output  The output to inject into the command.
+	 *
+	 * @return  integer  The command exit code
+	 */
+	protected function doExecute(InputInterface $input, OutputInterface $output): int
+	{
+		$this->io->title('Fetching Joomla Releases');
+
+		$versions    = [];
+		$highVersion = '0.0.0';
+
+		foreach ($this->getTags() as $tag) {
+			$version = $this->validateVersionNumber($tag->name);
+
+			// Only process if the tag name looks like a version number
+			if ($version === false) {
+				continue;
+			}
+
+			// Joomla only uses major.minor.patch so everything else is invalid
+			$explodedVersion = explode('.', $version);
+
+			if (\count($explodedVersion) != 3) {
+				continue;
+			}
+
+			// Version collection is valid for the 3.x series and later
+			if (version_compare($version, '3.0.0', '<')) {
+				continue;
+			}
+
+			// We have a valid version number, great news... add it to our array if it isn't already present
+			if (!\in_array($version, $versions)) {
+				$versions[] = $version;
+
+				// If this version is higher than our high version, replace it
+				// TODO - When 4.0 is stable adjust this logic
+				if (version_compare($version, '4.0', '<') && version_compare($version, $highVersion, '>')) {
+					$highVersion = $version;
+				}
+			}
+		}
+
+		// If the high version is not the default then let's add some (arbitrary) allowed versions based on the repo's dev structure
+		if ($highVersion !== '0.0.0') {
+			$explodedVersion = explode('.', $highVersion);
+
+			// Allow the next patch release after this one
+			$nextPatch  = $explodedVersion[2] + 1;
+			$versions[] = $explodedVersion[0] . '.' . $explodedVersion[1] . '.' . $nextPatch;
+
+			// And allow the next minor release after this one
+			$nextMinor  = $explodedVersion[1] + 1;
+			$versions[] = $explodedVersion[0] . '.' . $nextMinor . '.0';
+		}
+
+		if (!$this->filesystem->put('joomla.json', json_encode($versions))) {
+			$this->io->error('Failed writing version data to the filesystem.');
+
+			return 1;
+		}
+
+		$this->io->success('Joomla! versions updated.');
+
+		return 0;
+	}
+
+	/**
+	 * Configures the current command.
+	 *
+	 * @return  void
+	 */
+	protected function configure(): void
+	{
+		$this->setDescription('Parses the release tags for the Joomla! CMS GitHub repository.');
+	}
 }

Spacing +2 added lines, -2 removed lines

@@ -96,11 +96,11 @@
 
             // Allow the next patch release after this one
             $nextPatch  = $explodedVersion[2] + 1;
-            $versions[] = $explodedVersion[0] . '.' . $explodedVersion[1] . '.' . $nextPatch;
+            $versions[] = $explodedVersion[0].'.'.$explodedVersion[1].'.'.$nextPatch;
 
             // And allow the next minor release after this one
             $nextMinor  = $explodedVersion[1] + 1;
-            $versions[] = $explodedVersion[0] . '.' . $nextMinor . '.0';
+            $versions[] = $explodedVersion[0].'.'.$nextMinor.'.0';
         }
 
         if (!$this->filesystem->put('joomla.json', json_encode($versions))) {

src/Commands/Tags/FetchPhpTagsCommand.php

Indentation +125 added lines, -125 removed lines

@@ -20,129 +20,129 @@
  */
 class FetchPhpTagsCommand extends AbstractTagCommand
 {
-    use ValidateVersion;
-
-    /**
-     * The default command name
-     *
-     * @var  string|null
-     */
-    protected static $defaultName = 'tags:php';
-
-    /**
-     * Constructor.
-     *
-     * @param   GitHub      $github      GitHub API object.
-     * @param   Filesystem  $filesystem  Filesystem adapter for the versions space.
-     */
-    public function __construct(GitHub $github, Filesystem $filesystem)
-    {
-        parent::__construct($github, $filesystem);
-
-        $this->repoName  = 'php-src';
-        $this->repoOwner = 'php';
-    }
-
-    /**
-     * Internal function to execute the command.
-     *
-     * @param   InputInterface   $input   The input to inject into the command.
-     * @param   OutputInterface  $output  The output to inject into the command.
-     *
-     * @return  integer  The command exit code
-     */
-    protected function doExecute(InputInterface $input, OutputInterface $output): int
-    {
-        $this->io->title('Fetching PHP Releases');
-
-        $versions          = [];
-        $supportedBranches = [
-            '7.2' => '',
-            '7.3' => '',
-            '7.4' => '',
-        ];
-
-        foreach ($this->getTags() as $tag) {
-            // Replace 'php-' from the tag to get our version number; skip if the segment doesn't exist
-            if (strpos($tag->name, 'php-') !== 0) {
-                continue;
-            }
-
-            $version = substr($tag->name, 4);
-
-            $version = $this->validateVersionNumber($version);
-
-            // Only process if the tag name looks like a version number
-            if ($version === false) {
-                continue;
-            }
-
-            // We only track versions based on major.minor.patch so everything else is invalid
-            $explodedVersion = explode('.', $version);
-
-            if (\count($explodedVersion) != 3) {
-                continue;
-            }
-
-            // Joomla collects stats for the 3.x branch and later, the minimum PHP version for 3.0.0 was 5.3.1
-            if (version_compare($version, '5.3.1', '<')) {
-                continue;
-            }
-
-            // We have a valid version number, great news... add it to our array if it isn't already present
-            if (!\in_array($version, $versions)) {
-                $versions[] = $version;
-
-                // If this version is higher than our branch's high version, replace it
-                $branch = substr($version, 0, 3);
-
-                if (isset($supportedBranches[$branch]) && version_compare($version, $supportedBranches[$branch], '>')) {
-                    $supportedBranches[$branch] = $version;
-                }
-            }
-        }
-
-        // For each supported branch, also add the next patch release
-        foreach ($supportedBranches as $branch => $version) {
-            $explodedVersion = explode('.', $version);
-
-            $nextPatch  = $explodedVersion[2] + 1;
-            $versions[] = $explodedVersion[0] . '.' . $explodedVersion[1] . '.' . $nextPatch;
-        }
-
-        // Use $branch from the previous loop to allow the next minor version (PHP's master branch)
-        $explodedVersion = explode('.', $branch);
-
-        $nextMinor   = $explodedVersion[1] + 1;
-        $nextRelease = $explodedVersion[0] . '.' . $nextMinor . '.0';
-
-        // There won't be a PHP 7.5, change next release to 8.0 if needed
-        if ($nextRelease === '7.5.0') {
-            $nextRelease = '8.0.0';
-        }
-
-        if (!\in_array($nextRelease, $versions, true)) {
-            $versions[] = $nextRelease;
-        }
-
-        if (!$this->filesystem->put('php.json', json_encode($versions))) {
-            $this->io->error('Failed writing version data to the filesystem.');
-
-            return 1;
-        }
-
-        $this->io->success('PHP versions updated.');
-
-        return 0;
-    }
-
-    /**
-     * Configures the current command.
-     *
-     * @return  void
-     */
-    protected function configure(): void
-    {
-        $this->setDescription('Parses the release tags for the PHP GitHub repository.');
-    }
+	use ValidateVersion;
+
+	/**
+	 * The default command name
+	 *
+	 * @var  string|null
+	 */
+	protected static $defaultName = 'tags:php';
+
+	/**
+	 * Constructor.
+	 *
+	 * @param   GitHub      $github      GitHub API object.
+	 * @param   Filesystem  $filesystem  Filesystem adapter for the versions space.
+	 */
+	public function __construct(GitHub $github, Filesystem $filesystem)
+	{
+		parent::__construct($github, $filesystem);
+
+		$this->repoName  = 'php-src';
+		$this->repoOwner = 'php';
+	}
+
+	/**
+	 * Internal function to execute the command.
+	 *
+	 * @param   InputInterface   $input   The input to inject into the command.
+	 * @param   OutputInterface  $output  The output to inject into the command.
+	 *
+	 * @return  integer  The command exit code
+	 */
+	protected function doExecute(InputInterface $input, OutputInterface $output): int
+	{
+		$this->io->title('Fetching PHP Releases');
+
+		$versions          = [];
+		$supportedBranches = [
+			'7.2' => '',
+			'7.3' => '',
+			'7.4' => '',
+		];
+
+		foreach ($this->getTags() as $tag) {
+			// Replace 'php-' from the tag to get our version number; skip if the segment doesn't exist
+			if (strpos($tag->name, 'php-') !== 0) {
+				continue;
+			}
+
+			$version = substr($tag->name, 4);
+
+			$version = $this->validateVersionNumber($version);
+
+			// Only process if the tag name looks like a version number
+			if ($version === false) {
+				continue;
+			}
+
+			// We only track versions based on major.minor.patch so everything else is invalid
+			$explodedVersion = explode('.', $version);
+
+			if (\count($explodedVersion) != 3) {
+				continue;
+			}
+
+			// Joomla collects stats for the 3.x branch and later, the minimum PHP version for 3.0.0 was 5.3.1
+			if (version_compare($version, '5.3.1', '<')) {
+				continue;
+			}
+
+			// We have a valid version number, great news... add it to our array if it isn't already present
+			if (!\in_array($version, $versions)) {
+				$versions[] = $version;
+
+				// If this version is higher than our branch's high version, replace it
+				$branch = substr($version, 0, 3);
+
+				if (isset($supportedBranches[$branch]) && version_compare($version, $supportedBranches[$branch], '>')) {
+					$supportedBranches[$branch] = $version;
+				}
+			}
+		}
+
+		// For each supported branch, also add the next patch release
+		foreach ($supportedBranches as $branch => $version) {
+			$explodedVersion = explode('.', $version);
+
+			$nextPatch  = $explodedVersion[2] + 1;
+			$versions[] = $explodedVersion[0] . '.' . $explodedVersion[1] . '.' . $nextPatch;
+		}
+
+		// Use $branch from the previous loop to allow the next minor version (PHP's master branch)
+		$explodedVersion = explode('.', $branch);
+
+		$nextMinor   = $explodedVersion[1] + 1;
+		$nextRelease = $explodedVersion[0] . '.' . $nextMinor . '.0';
+
+		// There won't be a PHP 7.5, change next release to 8.0 if needed
+		if ($nextRelease === '7.5.0') {
+			$nextRelease = '8.0.0';
+		}
+
+		if (!\in_array($nextRelease, $versions, true)) {
+			$versions[] = $nextRelease;
+		}
+
+		if (!$this->filesystem->put('php.json', json_encode($versions))) {
+			$this->io->error('Failed writing version data to the filesystem.');
+
+			return 1;
+		}
+
+		$this->io->success('PHP versions updated.');
+
+		return 0;
+	}
+
+	/**
+	 * Configures the current command.
+	 *
+	 * @return  void
+	 */
+	protected function configure(): void
+	{
+		$this->setDescription('Parses the release tags for the PHP GitHub repository.');
+	}
 }

Spacing +2 added lines, -2 removed lines

@@ -107,14 +107,14 @@
             $explodedVersion = explode('.', $version);
 
             $nextPatch  = $explodedVersion[2] + 1;
-            $versions[] = $explodedVersion[0] . '.' . $explodedVersion[1] . '.' . $nextPatch;
+            $versions[] = $explodedVersion[0].'.'.$explodedVersion[1].'.'.$nextPatch;
         }
 
         // Use $branch from the previous loop to allow the next minor version (PHP's master branch)
         $explodedVersion = explode('.', $branch);
 
         $nextMinor   = $explodedVersion[1] + 1;
-        $nextRelease = $explodedVersion[0] . '.' . $nextMinor . '.0';
+        $nextRelease = $explodedVersion[0].'.'.$nextMinor.'.0';
 
         // There won't be a PHP 7.5, change next release to 8.0 if needed
         if ($nextRelease === '7.5.0') {

src/Commands/Tags/AbstractTagCommand.php

Indentation +96 added lines, -96 removed lines

@@ -21,100 +21,100 @@
  */
 abstract class AbstractTagCommand extends AbstractCommand
 {
-    /**
-     * GitHub API object.
-     *
-     * @var  GitHub
-     */
-    protected $github;
-
-    /**
-     * Filesystem adapter for the snapshots space.
-     *
-     * @var  Filesystem
-     */
-    protected $filesystem;
-
-    /**
-     * Output helper.
-     *
-     * @var  SymfonyStyle
-     */
-    protected $io;
-
-    /**
-     * The GitHub repository to query.
-     *
-     * @var  string
-     */
-    protected $repoName;
-
-    /**
-     * The owner of the GitHub repository to query.
-     *
-     * @var  string
-     */
-    protected $repoOwner;
-
-    /**
-     * Constructor.
-     *
-     * @param   GitHub      $github      GitHub API object.
-     * @param   Filesystem  $filesystem  Filesystem adapter for the versions space.
-     */
-    public function __construct(GitHub $github, Filesystem $filesystem)
-    {
-        $this->github     = $github;
-        $this->filesystem = $filesystem;
-
-        parent::__construct();
-    }
-
-    /**
-     * Internal hook to initialise the command after the input has been bound and before the input is validated.
-     *
-     * @param   InputInterface   $input   The input to inject into the command.
-     * @param   OutputInterface  $output  The output to inject into the command.
-     *
-     * @return  void
-     */
-    protected function initialise(InputInterface $input, OutputInterface $output): void
-    {
-        $this->io = new SymfonyStyle($input, $output);
-    }
-
-    /**
-     * Get the tags for a repository
-     *
-     * @return  array
-     */
-    protected function getTags(): array
-    {
-        $tags = [];
-
-        $this->io->comment('Fetching page 1 of tags.');
-
-        // Get the first page so we can process the headers to figure out how many times we need to do this
-        $tags = array_merge($tags, $this->github->repositories->getTags($this->repoOwner, $this->repoName, 1));
-
-        $response = $this->github->repositories->getApiResponse();
-
-        if ($response->hasHeader('Link')) {
-            preg_match('/(\?page=[0-9]+>; rel=\"last\")/', $response->getHeader('Link')[0], $matches);
-
-            if ($matches && isset($matches[0])) {
-                preg_match('/\d+/', $matches[0], $pages);
-
-                $lastPage = $pages[0];
-
-                for ($page = 2; $page <= $lastPage; $page++) {
-                    $this->io->comment(sprintf('Fetching page %d of %d pages of tags.', $page, $lastPage));
-
-                    $tags = array_merge($tags, $this->github->repositories->getTags($this->repoOwner, $this->repoName, $page));
-                }
-            }
-        }
-
-        return $tags;
-    }
+	/**
+	 * GitHub API object.
+	 *
+	 * @var  GitHub
+	 */
+	protected $github;
+
+	/**
+	 * Filesystem adapter for the snapshots space.
+	 *
+	 * @var  Filesystem
+	 */
+	protected $filesystem;
+
+	/**
+	 * Output helper.
+	 *
+	 * @var  SymfonyStyle
+	 */
+	protected $io;
+
+	/**
+	 * The GitHub repository to query.
+	 *
+	 * @var  string
+	 */
+	protected $repoName;
+
+	/**
+	 * The owner of the GitHub repository to query.
+	 *
+	 * @var  string
+	 */
+	protected $repoOwner;
+
+	/**
+	 * Constructor.
+	 *
+	 * @param   GitHub      $github      GitHub API object.
+	 * @param   Filesystem  $filesystem  Filesystem adapter for the versions space.
+	 */
+	public function __construct(GitHub $github, Filesystem $filesystem)
+	{
+		$this->github     = $github;
+		$this->filesystem = $filesystem;
+
+		parent::__construct();
+	}
+
+	/**
+	 * Internal hook to initialise the command after the input has been bound and before the input is validated.
+	 *
+	 * @param   InputInterface   $input   The input to inject into the command.
+	 * @param   OutputInterface  $output  The output to inject into the command.
+	 *
+	 * @return  void
+	 */
+	protected function initialise(InputInterface $input, OutputInterface $output): void
+	{
+		$this->io = new SymfonyStyle($input, $output);
+	}
+
+	/**
+	 * Get the tags for a repository
+	 *
+	 * @return  array
+	 */
+	protected function getTags(): array
+	{
+		$tags = [];
+
+		$this->io->comment('Fetching page 1 of tags.');
+
+		// Get the first page so we can process the headers to figure out how many times we need to do this
+		$tags = array_merge($tags, $this->github->repositories->getTags($this->repoOwner, $this->repoName, 1));
+
+		$response = $this->github->repositories->getApiResponse();
+
+		if ($response->hasHeader('Link')) {
+			preg_match('/(\?page=[0-9]+>; rel=\"last\")/', $response->getHeader('Link')[0], $matches);
+
+			if ($matches && isset($matches[0])) {
+				preg_match('/\d+/', $matches[0], $pages);
+
+				$lastPage = $pages[0];
+
+				for ($page = 2; $page <= $lastPage; $page++) {
+					$this->io->comment(sprintf('Fetching page %d of %d pages of tags.', $page, $lastPage));
+
+					$tags = array_merge($tags, $this->github->repositories->getTags($this->repoOwner, $this->repoName, $page));
+				}
+			}
+		}
+
+		return $tags;
+	}
 }