Merodiro\Settings\Commands\SettingsCache

Complexity

Total Complexity

3

Size/Duplication

Total Lines	57
Duplicated Lines	15.79 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
B	handle()	9	25	2

<?php

namespace Merodiro\Settings\Commands;

use Illuminate\Console\Command;
use Illuminate\Support\Facades\Cache;
use Merodiro\Settings\Facades\Settings;

class SettingsCache extends Command
{
    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'settings:cache {--model=}';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Cache all settings';

    /**
     * Create a new command instance.
     *
     * @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()
    {
        parent::__construct();
    }

    /**
     * Execute the console command.
     *
     * @return mixed
     */
    public function handle()
    {
        $this->comment('Caching all settings');
        $duration = config('settings.cache_duration');

        Settings::all()->each(function ($value, $key) use ($duration) {
            $cache_key = Settings::cacheKey($key);

            Cache::put($cache_key, $value, $duration);
        });

        $model = $this->option('model');

        if (isset($model)) {

This code seems to be duplicated across your project.

            (new $model)->all()->each(function ($user) use ($duration) {
                $user->allSettings()->each(function ($value, $key) use ($user, $duration) {
                    $cache_key = $user->settingsCacheKey($key);

                    Cache::put($cache_key, $value, $duration);
                });
            });
        }

        $this->info('Cached all settings');
    }
}