Completed
Push — develop ( c61736...9994ff )
by Neil
7s
created

MigrateSettings::__construct()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 4
Code Lines 2

Duplication

Lines 0
Ratio 0 %

Importance

Changes 1
Bugs 0 Features 1
Metric Value
c 1
b 0
f 1
dl 0
loc 4
rs 10
cc 1
eloc 2
nc 1
nop 0
1
<?php
2
/**
3
 * MigrateSettings.php
4
 *
5
 * Command line to migrate settings from LibreNMS v1
6
 *
7
 * This program is free software: you can redistribute it and/or modify
8
 * it under the terms of the GNU General Public License as published by
9
 * the Free Software Foundation, either version 3 of the License, or
10
 * (at your option) any later version.
11
 *
12
 * This program is distributed in the hope that it will be useful,
13
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
14
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.See the
15
 * GNU General Public License for more details.
16
 *
17
 * You should have received a copy of the GNU General Public License
18
 * along with this program.  If not, see <http://www.gnu.org/licenses/>.
19
 *
20
 * @package    LibreNMS
21
 * @link       http://librenms.org
22
 * @copyright  2016 Tony Murray
23
 * @author     Tony Murray <[email protected]>
24
 */
25
26
namespace App\Console\Commands;
27
28
use App\Models\DbConfig;
29
use Config;
30
use Illuminate\Console\Command;
31
use Settings;
32
33
class MigrateSettings extends Command
34
{
35
    /**
36
     * The name and signature of the console command.
37
     *
38
     * @var string
39
     */
40
    protected $signature = 'librenms:migrate-settings
41
    {--force} : Force the migration. WARNING: May lose some of your settings.
42
    {--dry-run} : Print changes that would be made, but do not commit them.';
43
44
    /**
45
     * The console command description.
46
     *
47
     * @var string
48
     */
49
    protected $description = 'Import settings from config.php that existed in the database previously into the database.  (The database now overrides config.php)';
50
51
    /**
52
     * Create a new command instance.
53
     */
54
    public function __construct()
55
    {
56
        parent::__construct();
57
    }
58
59
    /**
60
     * Execute the console command.
61
     *
62
     * @return mixed
63
     */
64
    public function handle()
65
    {
66
        if (Settings::get('settings.migrated')) {
67
            $this->info(trans('commands.migrate-settings.migrated'));
1 ignored issue
show
Bug introduced by
It seems like trans('commands.migrate-settings.migrated') targeting trans() can also be of type object<Symfony\Component...on\TranslatorInterface>; however, Illuminate\Console\Command::info() does only seem to accept string, maybe add an additional type check?

This check looks at variables that are passed out again to other methods.

If the outgoing method call has stricter type requirements than the method itself, an issue is raised.

An additional type check may prevent trouble.

Loading history...
68
            if (!$this->option('force')) {
69
                return;
70
            }
71
            $this->warn(trans('commands.migrate-settings.migrated_warning'));
1 ignored issue
show
Bug introduced by
It seems like trans('commands.migrate-...ings.migrated_warning') targeting trans() can also be of type object<Symfony\Component...on\TranslatorInterface>; however, Illuminate\Console\Command::warn() does only seem to accept string, maybe add an additional type check?

This check looks at variables that are passed out again to other methods.

If the outgoing method call has stricter type requirements than the method itself, an issue is raised.

An additional type check may prevent trouble.

Loading history...
72
        }
73
74
        $settings = DbConfig::pluck('config_name');
75
76
        foreach ($settings as $key) {
77
            $config_key = '_config.'.$key;
78
79
            if (Config::has($config_key)) {
80
                $old = Settings::get($key);
81
                $new = Config::get($config_key);
82
83
                if ($old !== $new) {
84
                    $this->info(trans('commands.migrate-settings.migrating', ['setting' => $key, 'old' => $old, 'new' => $new]));
1 ignored issue
show
Bug introduced by
It seems like trans('commands.migrate-...> $old, 'new' => $new)) targeting trans() can also be of type object<Symfony\Component...on\TranslatorInterface>; however, Illuminate\Console\Command::info() does only seem to accept string, maybe add an additional type check?

This check looks at variables that are passed out again to other methods.

If the outgoing method call has stricter type requirements than the method itself, an issue is raised.

An additional type check may prevent trouble.

Loading history...
85
86
                    if (!$this->option('dry-run')) {
87
                        // we aren't authenticated so we can't write through the Settings facade
88
                        DbConfig::updateOrCreate(['config_name' => $key], ['config_value' => $new]);
89
                    }
90
                }
91
            }
92
        }
93
94
        if ($this->option('dry-run')) {
95
            $this->warn(trans('commands.migrate-settings.nochanges'));
1 ignored issue
show
Bug introduced by
It seems like trans('commands.migrate-settings.nochanges') targeting trans() can also be of type object<Symfony\Component...on\TranslatorInterface>; however, Illuminate\Console\Command::warn() does only seem to accept string, maybe add an additional type check?

This check looks at variables that are passed out again to other methods.

If the outgoing method call has stricter type requirements than the method itself, an issue is raised.

An additional type check may prevent trouble.

Loading history...
96
        }
97
        else {
98
            Settings::flush();  // clear the settings cache
99
            DbConfig::updateOrCreate(['config_name' => 'settings.migrated'], ['config_value' => true]);
100
        }
101
    }
102
}
103