Implement --sync-root-files when using a sync_list file (Issue #491) (#492)

* Implement --sync-root-files to sync all files in the OneDrive root when using a sync_list file that would normally exclude these files from being synced

config

@@ -28,3 +28,4 @@
 # min_notify_changes = "5"
 # monitor_log_frequency = "5"
 # monitor_fullscan_frequency = "10"
+# sync_root_files = "false"

src/config.d

@@ -46,6 +46,7 @@ final class Config
 		boolValues["debug_https"]        = false;
 		boolValues["skip_dotfiles"]      = false;
 		boolValues["dry_run"]            = false;
+		boolValues["sync_root_files"]	 = false;
 		longValues["verbose"]            = log.verbose; // might be initialized by the first getopt call!
 		longValues["monitor_interval"]   = 45,
 		longValues["min_notify_changes"]  = 5;
@@ -266,6 +267,9 @@ final class Config
 				"synchronize",
 					"Perform a synchronization",
 					&boolValues["synchronize"],
+				"sync-root-files",
+					"Sync all files in sync_dir root when using sync_list.",
+					&boolValues["sync_root_files"],
 				"upload-only",
 					"Only upload to OneDrive, do not sync changes from OneDrive locally",
 					&boolValues["upload_only"],

src/main.d

@@ -179,6 +179,7 @@ int main(string[] args)
 		
 		// Is sync_list configured?
 		if (exists(userSyncList)){
+			writeln("Config option 'sync_root_files'     = ", cfg.getValueBool("sync_root_files"));
 			writeln("Selective sync configured           = true");
 			writeln("sync_list contents:");
 			// Output the sync_list contents
@@ -189,6 +190,7 @@ int main(string[] args)
 				writeln(line);
 			}
 		} else {
+			writeln("Config option 'sync_root_files'     = ", cfg.getValueBool("sync_root_files"));
 			writeln("Selective sync configured           = false");
 		}
 		

src/sync.d

@@ -858,8 +858,19 @@ final class SyncEngine
 				// compute the item path to see if the path is excluded
 				path = itemdb.computePath(item.driveId, item.parentId) ~ "/" ~ item.name;
 				path = buildNormalizedPath(path);
-				unwanted = selectiveSync.isPathExcluded(path);
+				if (selectiveSync.isPathExcluded(path)) {
-				if (unwanted) log.vdebug("OneDrive change path is to be excluded by user configuration: ", path);
+					// selective sync advised to skip, however is this a file and are we configured to upload / download files in the root?
+					if ((isItemFile(driveItem)) && (cfg.getValueBool("sync_root_files")) && (rootName(path) == "") ) {
+						// This is a file
+						// We are configured to sync all files in the root
+						// This is a file in the logical root
+						unwanted = false;
+					} else {
+						// path is unwanted
+						unwanted = true;
+						log.vdebug("OneDrive change path is to be excluded by user configuration: ", path);
+					}
+				}
 			} else {
 				log.vdebug("Flagging as unwanted: item.driveId (", item.driveId,"), item.parentId (", item.parentId,") not in local database");
 				unwanted = true;
@@ -1638,8 +1649,12 @@ final class SyncEngine
 					}
 				}
 				if (selectiveSync.isPathExcluded(path)) {
-					log.vlog("Skipping item - path excluded by sync_list: ", path);
+					if ((isFile(path)) && (cfg.getValueBool("sync_root_files")) && (rootName(strip(path,"./")) == "")) {
-					return;
+						log.vdebug("Not skipping path due to sync_root_files inclusion: ", path);
+					} else {
+						log.vlog("Skipping item - path excluded by sync_list: ", path);
+						return;
+					}
 				}
 			}