deblan-mirror/abraunegg-onedrive
deblan-mirror/abraunegg-onedrive
1
0
Fork 0
mirror of https://github.com/abraunegg/onedrive synced 2026-03-14 14:35:46 +01:00
Code Issues Projects Releases Wiki Activity

Fix bug that 'items-dryrun.sqlite3' gets erroneously created when running a 'no sync' operation (#3325)

Browse source 
* Fix bug that 'items-dryrun.sqlite3' gets erroneously created when running a 'no sync' operation task such as --logout
This commit is contained in:
abraunegg 2025-06-09 08:00:46 +10:00 • committed by GitHub
commit 51c4392411
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 15 additions and 18 deletions
Download patch file Download diff file Expand all files Collapse all files

33
src/main.d
View file

@ -427,33 +427,30 @@ int main(string[] cliArgs) {
// If this has been requested, we need to ensure that all actions are performed against the dry-run database copy, and,
// no actual action takes place - such as deleting files if deleted online, moving files if moved online or local, downloading new & changed files, uploading new & changed files
if (dryRun || (noSyncTaskOperationRequested)) {
// Cleanup any existing dry-run elements ... these should never be left hanging around and should be cleaned up first
cleanupDatabaseFiles(appConfig.databaseFilePathDryRun);
// If --dry-run
if (dryRun) {
// This is a --dry-run operation
addLogEntry("DRY-RUN Configured. Output below shows what 'would' have occurred.");
}
// Cleanup any existing dry-run elements ... these should never be left hanging around and should be cleaned up first
cleanupDatabaseFiles(appConfig.databaseFilePathDryRun);
// Make a copy of the original items.sqlite3 for use as the dry run copy if it exists
if (exists(appConfig.databaseFilePath)) {
// In a --dry-run --resync scenario, we should not copy the existing database file
if (!appConfig.getValueBool("resync")) {
// Copy the existing DB file to the dry-run copy
if (dryRun) {
// Make a copy of the original items.sqlite3 for use as the dry run copy if it exists
if (exists(appConfig.databaseFilePath)) {
// In a --dry-run --resync scenario, we should not copy the existing database file
if (!appConfig.getValueBool("resync")) {
// Copy the existing DB file to the dry-run copy
addLogEntry("DRY-RUN: Copying items.sqlite3 to items-dryrun.sqlite3 to use for dry run operations");
}
copy(appConfig.databaseFilePath,appConfig.databaseFilePathDryRun);
} else {
// No database copy due to --resync
if (dryRun) {
copy(appConfig.databaseFilePath,appConfig.databaseFilePathDryRun);
} else {
// No database copy due to --resync - an empty DB file will be used for the resync operation
addLogEntry("DRY-RUN: No database copy created for --dry-run due to --resync also being used");
}
}
// update runtimeDatabaseFile now that we are using the dry run path
runtimeDatabaseFile = appConfig.databaseFilePathDryRun;
}
// update runtimeDatabaseFile now that we are using the dry run path
runtimeDatabaseFile = appConfig.databaseFilePathDryRun;
} else {
// Cleanup any existing dry-run elements ... these should never be left hanging around
cleanupDatabaseFiles(appConfig.databaseFilePathDryRun);