EloquentJs\ScriptGenerator\Model\Inspector

Complexity

Total Complexity

5

Size/Duplication

Total Lines	70
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	1
Bugs	0	Features	1

4 Methods

Rating	Name	Duplication	Size	Complexity
A	inspect()	0	9	1
A	findEndpoint()	0	4	1
A	findDateColumns()	0	6	1
A	findScopeMethods()	0	16	2

<?php

namespace EloquentJs\ScriptGenerator\Model;

use EloquentJs\Model\AcceptsEloquentJsQueries;

class Inspector
{
    /**
     * @var array
     */
    protected $excludeScopes = ['scopeScope', 'scopeEloquentJs'];

    /**
     * Inspect a model class and return its metadata.
     *
     * @param AcceptsEloquentJsQueries $instance
     * @return Metadata
     */
    public function inspect(AcceptsEloquentJsQueries $instance)
    {
        return new Metadata(
            class_basename($instance),
            $this->findEndpoint($instance),
            $this->findDateColumns($instance),
            $this->findScopeMethods($instance)
        );
    }

    /**
     * Find the endpoint for this model.
     *
     * @param AcceptsEloquentJsQueries $instance
     * @return string
     */
    protected function findEndpoint(AcceptsEloquentJsQueries $instance)
    {
        return $instance->getEndpoint();
    }

    /**
     * Get any additional date columns for this model.
     *
     * @param AcceptsEloquentJsQueries $instance
     * @return array
     */
    protected function findDateColumns(AcceptsEloquentJsQueries $instance)
    {
        return array_values(
            array_diff($instance->getDates(), ['created_at', 'updated_at', 'deleted_at'])

The method getDates() does not seem to exist on object<EloquentJs\Model\AcceptsEloquentJsQueries>.

        );
    }

    /**
     * Get the scope methods for this model with 'scope' prefix removed.
     *
     * @param AcceptsEloquentJsQueries $instance
     * @return array
     */
    protected function findScopeMethods(AcceptsEloquentJsQueries $instance)
    {
        return array_map(
            function($method) {
                return lcfirst(substr($method, 5));
            },
            array_values(
                array_filter(
                    get_class_methods($instance),
                    function($method) {
                        return substr($method, 0, 5) === 'scope' and ! in_array($method, $this->excludeScopes);

Using logical operators such as and instead of && is generally not recommended.

                    }
                )
            )
        );
    }
}