EloquentJs\Laravel\Console\GenerateJavascript

Complexity

Total Complexity

6

Size/Duplication

Total Lines	93
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	handle()	0	16	2
A	printMapping()	0	8	1
A	promptUserForMissingEndpoints()	0	8	2

<?php

namespace EloquentJs\Laravel\Console;

use EloquentJs\Generator\Generator;
use EloquentJs\Generator\ModelInputParser;
use Illuminate\Console\Command;

class GenerateJavascript extends Command
{
    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'eloquentjs:generate
                            {--models= : Models to include in the generated javascript}
                            {--namespace=App : Namespace prefix to use with the --models option}
                            {--output=public/eloquent.js : Where to save the generated javascript file}';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Generate a javascript file for an Eloquent-ish API in the browser';

    /**
     * @var ModelInputParser
     */
    protected $inputParser;
    /**
     * @var Generator
     */
    protected $generator;

    /**
     * Create a new command instance.
     *
     * @param ModelInputParser $inputParser
     * @param Generator $generator
     */
    public function __construct(ModelInputParser $inputParser, Generator $generator)
    {
        parent::__construct();

        $this->inputParser = $inputParser;
        $this->generator = $generator;
    }

    /**
     * Execute the console command.
     *
     * @return mixed
     */
    public function handle()
    {
        $models = $this->inputParser
            ->searchIn(app_path())
            ->getEndpointMapping($this->option('models'), $this->option('namespace'));

It seems like $this->option('models') targeting Illuminate\Console\Command::option() can also be of type array; however, EloquentJs\Generator\ModelInputParser::getEndpointMapping() does only seem to accept string, maybe add an additional type check?

It seems like $this->option('namespace') targeting Illuminate\Console\Command::option() can also be of type array; however, EloquentJs\Generator\ModelInputParser::getEndpointMapping() does only seem to accept string|null, maybe add an additional type check?

        $this->promptUserForMissingEndpoints($models);

        $this->printMapping($models);

        if ($this->confirm('Generate javascript for these models and associated endpoints?', true)) {
            $output = $this->option('output');
            $this->generator->saveTo($output)->build($models);
            $this->info("Javascript written to {$output}");
        }
    }

    /**
     * Print the inferred Model => Endpoint mapping.
     *
     * @param $models
     * @return void
     */
    protected function printMapping($models)
    {
        $rows = array_map(function($model, $endpoint) {
            return [$model, $endpoint];
        }, array_keys($models), $models);

        $this->table(['Model', 'Endpoint'], $rows);
    }

    /**
     * Ask user to specify an endpoint for those we failed to determine automatically.
     *
     * @param array $models
     */
    protected function promptUserForMissingEndpoints(&$models)
    {
        array_walk($models, function(&$endpoint, $model) {
            if (empty($endpoint)) {
                $endpoint = $this->ask("Enter the endpoint to use for the '{$model}' model");
            }
        });
    }
}