App\Modules\Core\Console\Commands\GenerateDocCommand

Complexity

Total Complexity

50

Size/Duplication

Total Lines	306
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes

0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
B	handle()	0	41	5
A	getRoutes()	0	14	2
A	getHeaders()	0	14	3
B	processDocBlock()	0	28	7
B	getPostData()	0	24	8
A	getErrors()	0	22	4
A	getMethodBody()	0	12	1
C	getModels()	0	41	12
B	getResponseObject()	0	8	7

<?php

namespace App\Modules\Core\Console\Commands;

use Illuminate\Console\Command;
use Illuminate\Support\Arr;
use App\Modules\Reporting\Services\ReportService;

class GenerateDocCommand extends Command
{
    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'doc:generate';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Generate api documentation';

    /**
     * @var ReprotService
     */
    protected $reportService;

    /**
     * Init new object.
     *
     * @return  void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct(ReportService $reportService)
    {
        $this->reportService = $reportService;

It seems like $reportService of type object<App\Modules\Reporting\Services\ReportService> is incompatible with the declared type object<App\Modules\Core\Console\Commands\ReprotService> of property $reportService.

        parent::__construct();
    }

    /**
     * Execute the console command.
     *
     * @return mixed
     */
    public function handle()
    {
        $docData           = [];
        $docData['models'] = [];
        $routes            = $this->getRoutes();
        foreach ($routes as $route) {
            if ($route) {
                $actoinArray = explode('@', $route['action']);
                if (Arr::get($actoinArray, 1, false)) {
                    $prefix = $route['prefix'];
                    $module = \Str::camel(str_replace('/', '_', str_replace('api', '', $prefix)));
                    if ($prefix === 'telescope') {
                        continue;
                    }

                    $controller       = $actoinArray[0];
                    $method           = $actoinArray[1];
                    $route['name']    = $method;
                    $reflectionClass  = new \ReflectionClass($controller);
                    $reflectionMethod = $reflectionClass->getMethod($method);
                    $classProperties  = $reflectionClass->getDefaultProperties();
                    $skipLoginCheck   = Arr::get($classProperties, 'skipLoginCheck', false);
                    $modelName        = explode('\\', $controller);
                    $modelName        = lcfirst(str_replace('Controller', '', end($modelName)));

                    $this->processDocBlock($route, $reflectionMethod);
                    $this->getHeaders($route, $method, $skipLoginCheck);
                    $this->getPostData($route, $reflectionMethod, explode('\\', $reflectionClass->getName())[2], $modelName);

Consider using $reflectionClass->name. There is an issue with getName() and APC-enabled PHP versions.

                    $route['response'] = $this->getResponseObject($modelName, $route['name'], $route['returnDocBlock']);
                    $docData['modules'][$module][] = $route;

                    $this->getModels($modelName, $docData, $reflectionClass);
                }
            }
        }
        
        $docData['errors']  = $this->getErrors();
        $docData['reports'] = $this->reportService->all();
        \File::put(app_path('Modules/Core/Resources/api.json'), json_encode($docData));
    }

    /**
     * Get list of all registered routes.
     *
     * @return collection
     */
    protected function getRoutes()
    {
        return collect(\Route::getRoutes())->map(function ($route) {
            if (strpos($route->uri(), 'api/') !== false) {
                return [
                    'method' => $route->methods()[0],
                    'uri'    => $route->uri(),
                    'action' => $route->getActionName(),
                    'prefix' => $route->getPrefix()
                ];
            }
            return false;
        })->all();
    }

    /**
     * Generate headers for the given route.
     *
     * @param  array  &$route
     * @param  string $method
     * @param  array  $skipLoginCheck
     * @return void
     */
    protected function getHeaders(&$route, $method, $skipLoginCheck)
    {
        $route['headers'] = [
        'Accept'         => 'application/json',
        'Content-Type'   => 'application/json',
        'Accept-Language' => 'The language of the returned data: ar, en or all.',
        'time-zone'       => 'Your locale time zone',
        ];


        if (! $skipLoginCheck || ! in_array($method, $skipLoginCheck)) {

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

            $route['headers']['Authorization'] = 'Bearer {token}';
        }
    }

    /**
     * Generate description and params for the given route
     * based on the docblock.
     *
     * @param  array  &$route
     * @param  \ReflectionMethod $reflectionMethod
     * @return void
     */
    protected function processDocBlock(&$route, $reflectionMethod)
    {
        $factory                 = \phpDocumentor\Reflection\DocBlockFactory::createInstance();
        $docblock                = $factory->create($reflectionMethod->getDocComment());
        $route['description']    = trim(preg_replace('/\s+/', ' ', $docblock->getSummary()));
        $params                  = $docblock->getTagsByName('param');
        $route['returnDocBlock'] = $docblock->getTagsByName('return')[0]->getType()->getFqsen()->getName();

        foreach ($params as $param) {
            $name = $param->getVariableName();
            if ($name == 'perPage') {
                $route['parametars'][$name] = 'perPage? number of records per page default 15';
            } elseif ($name == 'sortBy') {
                $route['parametars'][$name] = 'sortBy? sort column default created_at';
            } elseif ($name == 'desc') {
                $route['parametars'][$name] = 'desc? sort descending or ascending default is descending';
            } elseif ($name !== 'request') {
                $route['parametars'][$name] = $param->getDescription()->render();
            }
        }

        if ($route['name'] === 'index') {
            $route['parametars']['perPage'] = 'perPage? number of records per page default 15';
            $route['parametars']['sortBy']  = 'sortBy? sort column default created_at';
            $route['parametars']['desc']    = 'desc? sort descending or ascending default is descending';
            $route['parametars']['trashed'] = 'trashed? retreive trashed or not default not';
        }
    }

    /**
     * Generate post body for the given route.
     *
     * @param  array  &$route
     * @param  \ReflectionMethod $reflectionMethod
     * @param  string $module
     * @param  string $modelName
     * @return void
     */
    protected function getPostData(&$route, $reflectionMethod, $module, $modelName)
    {
        $parameters = $reflectionMethod->getParameters();
        $className = optional(optional(\Arr::get($parameters, 0))->getType())->getName();
        if ($className) {
            $reflectionClass  = new \ReflectionClass($className);
        } elseif (in_array($reflectionMethod->getName(), ['store', 'update'])) {

Consider using $reflectionMethod->name. There is an issue with getName() and APC-enabled PHP versions.

            $className = 'App\\Modules\\' . ucfirst($module) . '\\Http\\Requests\\Store'  . ucfirst($modelName);
            $reflectionClass  = new \ReflectionClass($className);
        }

        if (isset($reflectionClass) && $reflectionClass->hasMethod('rules')) {
            $reflectionMethod = $reflectionClass->getMethod('rules');
            $route['body'] = $reflectionMethod->invoke(new $className);

            foreach ($route['body'] as &$rule) {
                if (strpos($rule, 'unique')) {
                    $rule = substr($rule, 0, strpos($rule, 'unique') + 6);
                } elseif (strpos($rule, 'exists')) {
                    $rule = substr($rule, 0, strpos($rule, 'exists') - 1);
                }
            }
        }
    }

    /**
     * Generate application errors.
     *
     * @return array
     */
    protected function getErrors()
    {
        $errors = [];
        foreach (\Module::all() as $module) {
            $nameSpace = 'App\\Modules\\' . $module['basename'];
            $class = $nameSpace . '\\Errors\\'  . $module['basename'] . 'Errors';
            $reflectionClass = new \ReflectionClass($class);
            foreach ($reflectionClass->getMethods() as $method) {
                $methodName       = $method->name;
                $reflectionMethod = $reflectionClass->getMethod($methodName);
                $body             = $this->getMethodBody($reflectionMethod);

                preg_match('/abort\(([^#]+)\,/iU', $body, $match);

                if (count($match)) {
                    $errors[$match[1]][] = $methodName;
                }
            }
        }

        return $errors;
    }

    /**
     * Get the given method body code.
     *
     * @param  object $reflectionMethod
     * @return string
     */
    protected function getMethodBody($reflectionMethod)
    {
        $filename   = $reflectionMethod->getFileName();
        $start_line = $reflectionMethod->getStartLine() - 1;
        $end_line   = $reflectionMethod->getEndLine();
        $length     = $end_line - $start_line;
        $source     = file($filename);
        $body       = implode("", array_slice($source, $start_line, $length));
        $body       = trim(preg_replace('/\s+/', '', $body));

        return $body;
    }

    /**
     * Get example object of all availble models.
     *
     * @param  string $modelName
     * @param  array  $docData
     * @return string
     */
    protected function getModels($modelName, &$docData, $reflectionClass)
    {
        if ($modelName && ! Arr::has($docData['models'], $modelName)) {
            $repo = call_user_func_array("\Core::{$modelName}", []);
            if (! $repo) {
                return;
            }
            
            $modelClass = get_class($repo->model);
            $model      = factory($modelClass)->make();

            $property = $reflectionClass->getProperty('modelResource');
            $property->setAccessible(true);
            $modelResource = $property->getValue(\App::make($reflectionClass->getName()));

            $relations = [];
            $relationMethods = ['hasOne', 'hasMany', 'belongsTo', 'belongsToMany', 'morphToMany', 'morphTo'];
            $reflector = new \ReflectionClass($model);
            foreach ($reflector->getMethods() as $reflectionMethod) {
                $body = $this->getMethodBody($reflectionMethod);
                foreach ($relationMethods as $relationMethod) {
                    $relation = $reflectionMethod->getName();

Consider using $reflectionMethod->name. There is an issue with getName() and APC-enabled PHP versions.

                    if (strpos($body, '$this->' . $relationMethod) && $relation !== 'morphedByMany') {
                        $relations[] = $relation;
                        break;
                    }
                }
            }

            $modelResource = new $modelResource($model->load($relations));
            $modelArr      = $modelResource->toArray([]);

            foreach ($modelArr as $key => $attr) {
                if (is_object($attr) && property_exists($attr, 'resource') && $attr->resource instanceof \Illuminate\Http\Resources\MissingValue) {
                    unset($modelArr[$key]);
                }
            }

            $docData['models'][$modelName] = json_encode($modelArr, JSON_PRETTY_PRINT);
        }
    }

    /**
     * Get the route response object type.
     *
     * @param  string $modelName
     * @param  string $method
     * @param  string $returnDocBlock
     * @return array
     */
    protected function getResponseObject($modelName, $method, $returnDocBlock)
    {
        $relations = config('core.relations');
        $relations = Arr::has($relations, $modelName) ? Arr::has($relations[$modelName], $method) ? $relations[$modelName] : false : false;
        $modelName = call_user_func_array("\Core::{$returnDocBlock}", []) ? $returnDocBlock : $modelName;

        return $relations ? [$modelName => $relations && $relations[$method] ? $relations[$method] : []] : false;
    }
}