Bmatovu\Ussd\Commands\Validate

Complexity

Total Complexity

9

Size/Duplication

Total Lines	94
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	validate()	0	26	3
A	getElement()	0	7	1
A	handle()	0	19	4
A	getMessage()	0	5	1

<?php

namespace Bmatovu\Ussd\Commands;

use Illuminate\Console\Command;

class Validate extends Command
{
    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'ussd:validate
                                {--f|file=menus.xml : Main menu file.}
                                {--s|schema=menus.xsd : XSD to validate against.}';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Validate ussd menus.';

    /**
     * Execute the console command.
     */
    public function handle(): void
    {
        $xml = menus_path($this->option('file'));
        $xsd = menus_path($this->option('schema'));

        if(! file_exists($xsd)) {
            $xsd = __DIR__.'/../../menus/menus.xsd';
        }

        $errors = $this->validate($xml, $xsd);

        if(! $errors) {

The expression $errors 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.

$errors is an empty array, thus ! $errors is always true.

            $this->info('Cheers, you have valid menus.');
            return;
        }

        foreach($errors as $file => $messages) {
            $this->error('File: '.$file);
            $this->table(['Line', 'Element', 'Message'], $messages);
        }
    }

    /**
     * @see https://www.php.net/manual/en/class.simplexmlelement.php#107869
     */
    public function validate(string $xmlFile, string $xsdFile, int $flags = 0): array
    {
        libxml_use_internal_errors(true);

        $domDocument = new \DOMDocument();

        $domDocument->load($xmlFile);

        $errors = [];

        if ($domDocument->schemaValidate($xsdFile, $flags)) {
            return $errors;
        }

        foreach (libxml_get_errors() as $error) {
            // level, code, column, message, file, line
            $errors[$error->file][] = [
                $error->line,
                $this->getElement($error->message),
                $this->getMessage($error->message),
            ];
        }

        libxml_clear_errors();

        return $errors;
    }

    /**
     * Get element from message.
     */
    protected function getElement(string $message): ?string
    {
        $matches = [];

        preg_match("/'([-_\\w]+)'/", $message, $matches);

        return array_pop($matches);
    }

    /**
     * Get refined message.
     */
    protected function getMessage(string $message): string
    {
        $parts = explode(':', $message);

        return trim(array_pop($parts));
    }
}