diff options
| author | 2024-06-29 17:10:49 +0900 | |
|---|---|---|
| committer | 2024-06-29 10:10:49 +0200 | |
| commit | 74deec6fb2aefa0bdaf6bb427f34d56ac3544cc3 (patch) | |
| tree | 338ea4c9ae7e76a5756c2bf862253836fbb33c99 | |
| parent | 7a9a24a0d0f2e5cc674995ce3174116803acde09 (diff) | |
Add quiet option to cli/db-backup.php (#6593)
* Add quiet option to cli/db-backup.php
* Fix CI test error
* Fix README
| -rw-r--r-- | app/Models/DatabaseDAO.php | 8 | ||||
| -rw-r--r-- | cli/README.md | 1 | ||||
| -rwxr-xr-x | cli/db-backup.php | 20 |
3 files changed, 23 insertions, 6 deletions
diff --git a/app/Models/DatabaseDAO.php b/app/Models/DatabaseDAO.php index b57138093..363225cdb 100644 --- a/app/Models/DatabaseDAO.php +++ b/app/Models/DatabaseDAO.php @@ -269,7 +269,7 @@ SQL; public const SQLITE_EXPORT = 1; public const SQLITE_IMPORT = 2; - public function dbCopy(string $filename, int $mode, bool $clearFirst = false): bool { + public function dbCopy(string $filename, int $mode, bool $clearFirst = false, bool $verbose = true): bool { if (!extension_loaded('pdo_sqlite')) { return self::stdError('PHP extension pdo_sqlite is missing!'); } @@ -354,7 +354,7 @@ SQL; $idMaps = []; - if (defined('STDERR')) { + if (defined('STDERR') && $verbose) { fwrite(STDERR, "Start SQL copy…\n"); } @@ -397,11 +397,11 @@ SQL; return self::stdError($error); } } - if ($n % 100 === 1 && defined('STDERR')) { //Display progression + if ($n % 100 === 1 && defined('STDERR') && $verbose) { //Display progression fwrite(STDERR, "\033[0G" . $n . '/' . $nbEntries); } } - if (defined('STDERR')) { + if (defined('STDERR') && $verbose) { fwrite(STDERR, "\033[0G" . $n . '/' . $nbEntries . "\n"); } $entryTo->commit(); diff --git a/cli/README.md b/cli/README.md index 366f456d5..22db88957 100644 --- a/cli/README.md +++ b/cli/README.md @@ -123,6 +123,7 @@ cd /usr/share/FreshRSS ./cli/db-backup.php # Back-up all users respective database to `data/users/*/backup.sqlite` +# -q, --quiet suppress non-error messages ./cli/db-restore.php --delete-backup --force-overwrite # Restore all users respective database from `data/users/*/backup.sqlite` diff --git a/cli/db-backup.php b/cli/db-backup.php index 290b5cc7b..8f8fda2b1 100755 --- a/cli/db-backup.php +++ b/cli/db-backup.php @@ -6,15 +6,31 @@ require(__DIR__ . '/_cli.php'); performRequirementCheck(FreshRSS_Context::systemConf()->db['type'] ?? ''); $ok = true; +$cliOptions = new class extends CliOptionsParser { + public string $quiet; + + public function __construct() { + $this->addOption('quiet', (new CliOption('quiet', 'q'))->withValueNone()); + parent::__construct(); + } +}; + +if (!empty($cliOptions->errors)) { + fail('FreshRSS error: ' . array_shift($cliOptions->errors) . "\n" . $cliOptions->usage); +} + foreach (listUsers() as $username) { $username = cliInitUser($username); $filename = DATA_PATH . '/users/' . $username . '/backup.sqlite'; @unlink($filename); + $verbose = !isset($cliOptions->quiet); - echo 'FreshRSS backup database to SQLite for user “', $username, "”…\n"; + if ($verbose) { + echo 'FreshRSS backup database to SQLite for user “', $username, "”…\n"; + } $databaseDAO = FreshRSS_Factory::createDatabaseDAO($username); - $ok &= $databaseDAO->dbCopy($filename, FreshRSS_DatabaseDAO::SQLITE_EXPORT); + $ok &= $databaseDAO->dbCopy($filename, FreshRSS_DatabaseDAO::SQLITE_EXPORT, false, $verbose); } done((bool)$ok); |
