Chore: Code style tweaks

Author: adil192

lib/data/file_manager/file_manager.dart

@@ -176,10 +176,18 @@ class FileManager {
   static Directory getRootDirectory() => Directory(documentsDirectory);
 
   /// Writes [toWrite] to [filePath].
-  static Future<void> writeFile(String filePath, List<int> toWrite,
-      {bool awaitWrite = false, bool alsoUpload = true, DateTime? lastModified}) async {
-        // if lastModified  is used, then last modified time stamp is set after file write
-        // it is used when downloading remote file, to set the same time stamp as has the file
+  ///
+  /// The file at [toPath] will have its last modified timestamp set to
+  /// [lastModified], if specified.
+  /// This is useful when downloading remote files, to make sure that the
+  /// timestamp is the same locally and remotely.
+  static Future<void> writeFile(
+    String filePath,
+    List<int> toWrite, {
+    bool awaitWrite = false,
+    bool alsoUpload = true,
+    DateTime? lastModified,
+  }) async {
     filePath = _sanitisePath(filePath);
     log.fine('Writing to $filePath');
 
@@ -201,9 +209,7 @@ class FileManager {
     ]);
 
     void afterWrite() {
-      if (lastModified != null){
-        // want to set timestamp of last modification. Used when downloading remote file, to put the same time stamp
-        // to avoid synchronization of newly created file back to remote.
+      if (lastModified != null) {
         file.setLastModified(lastModified);
       }
       broadcastFileWrite(FileOperationType.write, filePath);

lib/data/nextcloud/saber_syncer.dart

@@ -129,13 +129,10 @@ class SaberSyncInterface
 
   @override
   Future<SaberSyncFile> getSyncFileFromLocalFile(File localFile) async {
-    String relativePath =
-        localFile.path.substring(FileManager.documentsDirectory.length);
-    String separator=Platform.pathSeparator;
-    if (separator != "/") {
-      // I am on windows - replace \ with /
-      relativePath=relativePath.replaceAll("\\","/");
-    }
+    final relativePath = localFile.path
+        .substring(FileManager.documentsDirectory.length)
+        // Compensate for Windows using backslashes
+        .replaceAll(Platform.pathSeparator, '/');
 
     assert(relativePath.startsWith('/'));
     final encryptedName = await encryptPath(client, relativePath);
@@ -229,8 +226,13 @@ class SaberSyncInterface
     );
     assert(decryptedData.isNotEmpty,
         'Decrypted data is empty but encryptedBytes.length is ${encryptedBytes.length}');
-    await FileManager.writeFile(file.relativeLocalPath, decryptedData,
-        alsoUpload: false, lastModified: file.remoteFile?.lastModified);  // write remote file to local and set time stamp from remote file!
+    await FileManager.writeFile(
+      file.relativeLocalPath,
+      decryptedData,
+      alsoUpload: false,
+      // Local file should have the same last modified date as remote
+      lastModified: file.remoteFile?.lastModified,
+    );
   }
 
   @override