LaravelDatabaseRepository::getCached() - Code Metrics - Inspection of "Add database management feature" - nWidart/laravel-modules - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#1163)

unknown

created 2021-02-02 03:42 UTC

LaravelDatabaseRepository::getCached() A

↳ Parent: LaravelDatabaseRepository

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
nc	1
nop	0
dl	0
loc	6
rs	10
c	0
b	0
f	0

<?php

namespace Nwidart\Modules\Laravel;

use Illuminate\Contracts\Filesystem\FileNotFoundException;
use Illuminate\Support\Facades\Schema;
use Illuminate\Support\Str;
use Nwidart\Modules\Collection;
use Nwidart\Modules\Contracts\ActivatorInterface;
use Nwidart\Modules\Contracts\DatabaseRepositoryInterface;
use Nwidart\Modules\Entities\ModuleEntity;
use Nwidart\Modules\Exceptions\ModuleNotFoundException;
use Nwidart\Modules\Generators\DatabaseModuleGenerator;
use Nwidart\Modules\Json;
use Nwidart\Modules\Process\Updater;

/**
 * Class LaravelDatabaseRepository
 * @package Nwidart\Modules\Laravel
 * @method DatabaseModule findOrFail(string $name)
 */
class LaravelDatabaseRepository extends LaravelFileRepository implements DatabaseRepositoryInterface
{
    /**
     * Creates a new Module instance.
     *
     * @param mixed ...$args
     *
     * @return DatabaseModule
     */
    protected function createModule(...$args)
    {
        return new DatabaseModule(...$args);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
    }

    /**
     * @return ModuleEntity
     */
    public function getModel()
    {
        return new ModuleEntity();
    }

    /**
     * Scan & get all available modules.
     */
    public function scan()
    {
        /**
         * @var ModuleEntity[] $rows
         */
        $rows = $this->getModel()->get();
        $modules = [];
        if (!empty($rows)) {
            foreach ($rows as $row) {
                if (file_exists($row->path)) {
                    $modules[$row->name] = $this->createModule($this->app, $row->name, $row->path);
                }
            }
        }

        return $modules;
    }

    /**
     * Get all modules as laravel collection instance.
     *
     * @param $status
     *
     * @return Collection
     */
    public function collections($status = 1): Collection
    {
        return new Collection($this->getByStatus($status));
    }

    /**
     * Get module path for a specific module.
     *
     * @param $name
     *
     * @return string
     */
    public function getModulePath($name)
    {
        $module = $this->find($name);
        if ($module) {
            return $module->getPath() . '/';
        }

        return $this->getPath() . '/' . Str::studly($name) . '/';
    }

    /**
     * Get modules by status.
     *
     * @param $status
     *
     * @return array
     */
    public function getByStatus($status): array
    {
        $modules = [];

        foreach ($this->all() as $name => $module) {
            if ($module->isStatus($status) == $status) {
                $modules[$name] = $module;
            }
        }

        return $modules;
    }

    /**
     * Format the cached data as array of modules.
     *
     * @param array $cached
     *
     * @return array
     */
    protected function formatCached($cached)
    {
        $modules = [];

        foreach ($cached as $moduleEntity) {
            $module = $this->createModule($this->app, $moduleEntity['name'], $moduleEntity['path']);
            $module->setAttributes($moduleEntity->toArray());
            $modules[$moduleEntity['name']] = $module;
        }

        return $modules;
    }

    /**
     * Get cached modules from database.
     *
     * @return ModuleEntity[]
     */
    public function getCached()
    {
        return $this->app['cache']->remember($this->config('cache.key'), $this->config('cache.lifetime'), function () {
            return $this->getModel()->all();
        });
    }

    public function all(): array
    {
        // Do not load or register if there are no modules table yet.
        if (!Schema::hasTable('modules')) {
            return [];
        }

        return parent::all();
    }

    public function create($params, $force = true, $isApi = true, $isPlain = true)
    {
        $moduleType = $this->getModuleType($isApi, $isPlain); // Custom later.
        /** @var DatabaseModuleGenerator $generator */
        $generator = with(new DatabaseModuleGenerator($params['name']));
        $code = $generator
            ->setFilesystem(app('files'))
            ->setModule($this)
            ->setConfig(app('config'))
            ->setActivator(app(ActivatorInterface::class))
            ->setForce($force)
            ->setType($moduleType)
            ->setActive($params['is_active'])
            ->setSilentOutput(true) // Don't use console output
            ->generate();

        return $code ? $this->find($params['name']) : false;
    }

    /**
     * Get module type .
     *
     * @param bool $isApi
     * @param bool $isPlain
     *
     * @return string
     */
    public function getModuleType($isApi = true, $isPlain = true)
    {
        if ($isPlain && $isApi) {
            return 'web';
        }
        if ($isPlain) {
            return 'plain';
        } elseif ($isApi) {
            return 'api';
        } else {
            return 'web';
        }
    }

    /**
     * Get module used for cli session.
     * @return string
     * @throws ModuleNotFoundException|FileNotFoundException
     */
    public function getUsedNow(): string
    {
        $module = $this->getFiles()->get($this->getUsedStoragePath());
        if (!$module) {
            return '';
        }

        return $this->findOrFail($module);
    }

    public function migrateFileToDatabase($forceUpdate = false)
    {
        $paths = $this->getScanPaths();
        $modules = [];

        foreach ($paths as $key => $path) {
            $manifests = $this->getFiles()->glob("{$path}/module.json");

            is_array($manifests) || $manifests = [];

            foreach ($manifests as $manifest) {
                $json = Json::make($manifest);
                $data = $json->getAttributes();
                $data['path'] = str_replace('module.json', '', $json->getPath());
                if (!isset($data['version'])) {
                    $data['version'] = '1.0.0';
                }
                $module = $this->find($data['name']);
                $data = $this->validateAttributes($data);
                if (!$module) {
                    $modules[] = $this->getModel()->create($data);
                } else {
                    // Check version, if version is higher then update module.json into database.
                    // Can use force update here.
                    if (version_compare($module->getVersion(), $data['version'], '<') || $forceUpdate) {
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
                        $modules[] = $this->getModel()->where(['name' => $data['name']])->update($data);
                    }
                }
            }
        }

        $this->register();
        $this->boot();

        return $modules;
    }

    public function update($name)
    {
        return $this->findOrFail($name)->update(new Updater($this));
    }

    /**
     * Validate array attributes before insert/update into database.
     *
     * @param array $attributes
     * @param array $allows
     *
     * @return array
     */
    protected function validateAttributes(array $attributes, array $allows = [])
    {
        if (empty($allows)) {
            $allows = $this->getModel()->getFillable();
        }

        return array_filter($attributes, function ($k) use ($allows) {
            return in_array($k, $allows);
        }, ARRAY_FILTER_USE_KEY);
    }
}


1		<?php
2
3		namespace Nwidart\Modules\Laravel;
4
5		use Illuminate\Contracts\Filesystem\FileNotFoundException;
6		use Illuminate\Support\Facades\Schema;
7		use Illuminate\Support\Str;
8		use Nwidart\Modules\Collection;
9		use Nwidart\Modules\Contracts\ActivatorInterface;
10		use Nwidart\Modules\Contracts\DatabaseRepositoryInterface;
11		use Nwidart\Modules\Entities\ModuleEntity;
12		use Nwidart\Modules\Exceptions\ModuleNotFoundException;
13		use Nwidart\Modules\Generators\DatabaseModuleGenerator;
14		use Nwidart\Modules\Json;
15		use Nwidart\Modules\Process\Updater;
16
17		/**
18		* Class LaravelDatabaseRepository
19		* @package Nwidart\Modules\Laravel
20		* @method DatabaseModule findOrFail(string $name)
21		*/
22		class LaravelDatabaseRepository extends LaravelFileRepository implements DatabaseRepositoryInterface
23		{
24		/**
25		* Creates a new Module instance.
26		*
27		* @param mixed ...$args
28		*
29		* @return DatabaseModule
30		*/
31		protected function createModule(...$args)
32		{
33		return new DatabaseModule(...$args);
		0 ignored issues – show Bug introduced 2021-01-06 04:27 UTC by Report Bug Copy Issue Report The call to `DatabaseModule::__construct()` misses some required arguments starting with `$name`. Loading history... Documentation introduced 2021-01-06 07:38 UTC by Report Bug Copy Issue Report `$args` is of type `array<integer,*>`, but the function expects a `object<Illuminate\Container\Container>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
34		}
35
36		/**
37		* @return ModuleEntity
38		*/
39		public function getModel()
40		{
41		return new ModuleEntity();
42		}
43
44		/**
45		* Scan & get all available modules.
46		*/
47		public function scan()
48		{
49		/**
50		* @var ModuleEntity[] $rows
51		*/
52		$rows = $this->getModel()->get();
53		$modules = [];
54		if (!empty($rows)) {
55		foreach ($rows as $row) {
56		if (file_exists($row->path)) {
57		$modules[$row->name] = $this->createModule($this->app, $row->name, $row->path);
58		}
59		}
60		}
61
62		return $modules;
63		}
64
65		/**
66		* Get all modules as laravel collection instance.
67		*
68		* @param $status
69		*
70		* @return Collection
71		*/
72		public function collections($status = 1): Collection
73		{
74		return new Collection($this->getByStatus($status));
75		}
76
77		/**
78		* Get module path for a specific module.
79		*
80		* @param $name
81		*
82		* @return string
83		*/
84		public function getModulePath($name)
85		{
86		$module = $this->find($name);
87		if ($module) {
88		return $module->getPath() . '/';
89		}
90
91		return $this->getPath() . '/' . Str::studly($name) . '/';
92		}
93
94		/**
95		* Get modules by status.
96		*
97		* @param $status
98		*
99		* @return array
100		*/
101	View Code Duplication	public function getByStatus($status): array
102		{
103		$modules = [];
104
105		foreach ($this->all() as $name => $module) {
106		if ($module->isStatus($status) == $status) {
107		$modules[$name] = $module;
108		}
109		}
110
111		return $modules;
112		}
113
114		/**
115		* Format the cached data as array of modules.
116		*
117		* @param array $cached
118		*
119		* @return array
120		*/
121		protected function formatCached($cached)
122		{
123		$modules = [];
124
125		foreach ($cached as $moduleEntity) {
126		$module = $this->createModule($this->app, $moduleEntity['name'], $moduleEntity['path']);
127		$module->setAttributes($moduleEntity->toArray());
128		$modules[$moduleEntity['name']] = $module;
129		}
130
131		return $modules;
132		}
133
134		/**
135		* Get cached modules from database.
136		*
137		* @return ModuleEntity[]
138		*/
139		public function getCached()
140		{
141		return $this->app['cache']->remember($this->config('cache.key'), $this->config('cache.lifetime'), function () {
142		return $this->getModel()->all();
143		});
144		}
145
146		public function all(): array
147		{
148		// Do not load or register if there are no modules table yet.
149		if (!Schema::hasTable('modules')) {
150		return [];
151		}
152
153		return parent::all();
154		}
155
156		public function create($params, $force = true, $isApi = true, $isPlain = true)
157		{
158		$moduleType = $this->getModuleType($isApi, $isPlain); // Custom later.
159		/** @var DatabaseModuleGenerator $generator */
160		$generator = with(new DatabaseModuleGenerator($params['name']));
161		$code = $generator
162		->setFilesystem(app('files'))
163		->setModule($this)
164		->setConfig(app('config'))
165		->setActivator(app(ActivatorInterface::class))
166		->setForce($force)
167		->setType($moduleType)
168		->setActive($params['is_active'])
169		->setSilentOutput(true) // Don't use console output
170		->generate();
171
172		return $code ? $this->find($params['name']) : false;
173		}
174
175		/**
176		* Get module type .
177		*
178		* @param bool $isApi
179		* @param bool $isPlain
180		*
181		* @return string
182		*/
183		public function getModuleType($isApi = true, $isPlain = true)
184		{
185		if ($isPlain && $isApi) {
186		return 'web';
187		}
188		if ($isPlain) {
189		return 'plain';
190		} elseif ($isApi) {
191		return 'api';
192		} else {
193		return 'web';
194		}
195		}
196
197		/**
198		* Get module used for cli session.
199		* @return string
200		* @throws ModuleNotFoundException\|FileNotFoundException
201		*/
202		public function getUsedNow(): string
203		{
204		$module = $this->getFiles()->get($this->getUsedStoragePath());
205		if (!$module) {
206		return '';
207		}
208
209		return $this->findOrFail($module);
210		}
211
212		public function migrateFileToDatabase($forceUpdate = false)
213		{
214		$paths = $this->getScanPaths();
215		$modules = [];
216
217		foreach ($paths as $key => $path) {
218		$manifests = $this->getFiles()->glob("{$path}/module.json");
219
220		is_array($manifests) \|\| $manifests = [];
221
222		foreach ($manifests as $manifest) {
223		$json = Json::make($manifest);
224		$data = $json->getAttributes();
225		$data['path'] = str_replace('module.json', '', $json->getPath());
226		if (!isset($data['version'])) {
227		$data['version'] = '1.0.0';
228		}
229		$module = $this->find($data['name']);
230		$data = $this->validateAttributes($data);
231		if (!$module) {
232		$modules[] = $this->getModel()->create($data);
233		} else {
234		// Check version, if version is higher then update module.json into database.
235		// Can use force update here.
236		if (version_compare($module->getVersion(), $data['version'], '<') \|\| $forceUpdate) {
		0 ignored issues – show Documentation Bug introduced 2021-02-01 03:38 UTC by Report Bug Copy Issue Report The method `getVersion` does not exist on `object<Nwidart\Modules\Module>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
237		$modules[] = $this->getModel()->where(['name' => $data['name']])->update($data);
238		}
239		}
240		}
241		}
242
243		$this->register();
244		$this->boot();
245
246		return $modules;
247		}
248
249		public function update($name)
250		{
251		return $this->findOrFail($name)->update(new Updater($this));
252		}
253
254		/**
255		* Validate array attributes before insert/update into database.
256		*
257		* @param array $attributes
258		* @param array $allows
259		*
260		* @return array
261		*/
262		protected function validateAttributes(array $attributes, array $allows = [])
263		{
264		if (empty($allows)) {
265		$allows = $this->getModel()->getFillable();
266		}
267
268		return array_filter($attributes, function ($k) use ($allows) {
269		return in_array($k, $allows);
270		}, ARRAY_FILTER_USE_KEY);
271		}
272		}
273

nWidart / laravel-modules

Pull Request — master (#1163)

LaravelDatabaseRepository::getCached() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like