librenms / librenmsv2

app/Console/Commands/MigrateSettings.php

<?php
/**
 * MigrateSettings.php
 *
 * Command line to migrate settings from LibreNMS v1
 *
 * This program is free software: you can redistribute it and/or modify
 * it under the terms of the GNU General Public License as published by
 * the Free Software Foundation, either version 3 of the License, or
 * (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License
 * along with this program.  If not, see <http://www.gnu.org/licenses/>.
 *
 * @package    LibreNMS
 * @link       http://librenms.org
 * @copyright  2016 Tony Murray
 * @author     Tony Murray <murraytony@gmail.com>
 */

namespace App\Console\Commands;

use App\Models\DbConfig;
use Config;
use Illuminate\Console\Command;
use Settings;

class MigrateSettings extends Command
{
    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'librenms:migrate-settings
    {--force} : Force the migration. WARNING: May lose some of your settings.
    {--dry-run} : Print changes that would be made, but do not commit them.';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Import settings from config.php that existed in the database previously into the database.  (The database now overrides config.php)';

    /**
     * Create a new command instance.
     */
    public function __construct()
    {
        parent::__construct();
    }

    /**
     * Execute the console command.
     *
     * @return mixed
     */
    public function handle()
    {
        if (Settings::get('settings.migrated')) {
            $this->info(trans('commands.migrate-settings.migrated'));

It seems like trans('commands.migrate-settings.migrated') targeting trans() can also be of type object<Illuminate\Contracts\Translation\Translator>; however, Illuminate\Console\Command::info() does only seem to accept string, maybe add an additional type check?

            if (!$this->option('force')) {
                return;
            }
            $this->warn(trans('commands.migrate-settings.migrated_warning'));

It seems like trans('commands.migrate-settings.migrated_warning') targeting trans() can also be of type object<Illuminate\Contracts\Translation\Translator>; however, Illuminate\Console\Command::warn() does only seem to accept string, maybe add an additional type check?

        }

        $settings = DbConfig::pluck('config_name');

        foreach ($settings as $key) {
            $config_key = '_config.'.$key;

            if (Config::has($config_key)) {
                $old = Settings::get($key);
                $new = Config::get($config_key);

                if ($old !== $new) {
                    $this->info(trans('commands.migrate-settings.migrating', ['setting' => $key, 'old' => $old, 'new' => $new]));

It seems like trans('commands.migrate-settings.migrating', array('setting' => $key, 'old' => $old, 'new' => $new)) targeting trans() can also be of type object<Illuminate\Contracts\Translation\Translator>; however, Illuminate\Console\Command::info() does only seem to accept string, maybe add an additional type check?

                    if (!$this->option('dry-run')) {
                        // we aren't authenticated so we can't write through the Settings facade
                        DbConfig::updateOrCreate(['config_name' => $key], ['config_value' => $new]);
                    }
                }
            }
        }

        if ($this->option('dry-run')) {
            $this->warn(trans('commands.migrate-settings.nochanges'));

It seems like trans('commands.migrate-settings.nochanges') targeting trans() can also be of type object<Illuminate\Contracts\Translation\Translator>; however, Illuminate\Console\Command::warn() does only seem to accept string, maybe add an additional type check?

        } else {
            Settings::flush(); // clear the settings cache
            DbConfig::updateOrCreate(['config_name' => 'settings.migrated'], ['config_value' => true]);
        }
    }
}