Remove references to the Laravel backup manager
As proposed in https://github.com/CachetHQ/Cachet/pull/3150 the current backup solution produces too many edge cases when configured with the non-default options. Cachet should not be held responsible for backups since it's unable to complete this task properly. Feel free to re-add a laravel backup solution to your custom cachet instance.
This commit is contained in:
@@ -128,7 +128,7 @@ class CommandSubscriber
|
||||
}
|
||||
|
||||
/**
|
||||
* Handle the main bulk of the command, clear the settings and backup the database.
|
||||
* Handle the main bulk of the command, clear the settings
|
||||
*
|
||||
* @param \Illuminate\Console\Command $command
|
||||
*
|
||||
@@ -141,27 +141,6 @@ class CommandSubscriber
|
||||
$this->cache->clear();
|
||||
|
||||
$command->line('Settings cache cleared!');
|
||||
|
||||
// SQLite does not backup.
|
||||
if ($this->config->get('database.default') === 'sqlite') {
|
||||
$command->line('Backup skipped: SQLite is not supported.');
|
||||
|
||||
return;
|
||||
}
|
||||
|
||||
$command->line('Backing up database...');
|
||||
|
||||
try {
|
||||
$command->call('backup:run', [
|
||||
'--only-db' => true,
|
||||
'--no-interaction' => true,
|
||||
]);
|
||||
} catch (Exception $e) {
|
||||
$command->error($e->getMessage());
|
||||
$command->line('Backup skipped!');
|
||||
}
|
||||
|
||||
$command->line('Backup completed!');
|
||||
}
|
||||
|
||||
/**
|
||||
|
||||
Reference in New Issue
Block a user