fix: replace certain fs calls to use workaround

since you can't get file metadata in msfs, certain functions don't work. a few utility functions were added to replace those, and work properly

Author: pepperoni505

PackageSources/SimObjects/Airplanes/Navigraph_Navdata_Updater_Aircraft/panel/navdata_updater.wasm

@@ -1,4 +1,8 @@
-Building:
+# Building:
 
 - Install Docker
 - Run `.\build.bat` to build, and `.\run_cargo_cmd.bat` followed by a Cargo command to run a specified command (e.g `.\run_cargo_cmd.bat clippy`)
+
+
+# Warning
+The only file system functions that properly work are `fs::remove_file`, `fs::File::create`, `fs::create_dir`, `fs::remove_dir`, `fs::remove_file`. All the other functions regarding the filesystem do not work due to the MSFS implementation of WASI

Sources/Code/WASM/navdata_updater/README.md

@@ -50,6 +50,14 @@ impl<'a> Dispatcher<'a> {
                 })
                 .expect("Failed to register NAVIGRAPH_SetDownloadOptions");
         }
+        {
+            let captured_downloader = self.downloader.clone();
+            self.commbus
+                .register("NAVIGRAPH_DeleteAllNavdata", move |_| {
+                    captured_downloader.delete_all_navdata()
+                })
+                .expect("Failed to register NAVIGRAPH_DeleteAllNavdata");
+        }
     }
 
     fn handle_update(&mut self) {

Sources/Code/WASM/navdata_updater/src/dispatcher.rs

@@ -2,12 +2,12 @@ use std::cell::RefCell;
 use std::collections::HashMap;
 use std::fs;
 use std::io::Cursor;
-use std::path::PathBuf;
+use std::path::{Path, PathBuf};
 use std::rc::Rc;
 
 use msfs::{commbus::*, network::*};
 
-use crate::download::zip_handler::ZipFileHandler;
+use crate::{download::zip_handler::ZipFileHandler, util};
 
 pub struct DownloadOptions {
     batch_size: usize,
@@ -158,8 +158,20 @@ impl NavdataDownloader {
             ));
             return;
         }
-        // Create the directory if it doesn't exist
+        
         let path = PathBuf::from(format!("\\work/navdata/{}", folder));
+        // If the directory exists, delete it
+        if util::path_exists(&path) {
+            match util::delete_folder_recursively(&path) {
+                Ok(_) => (),
+                Err(e) => {
+                    let mut status = self.status.borrow_mut();
+                    *status = DownloadStatus::Failed(format!("Failed to delete directory: {}", e));
+                    return;
+                }
+            }
+        }
+        // Re create the directory
         if let Err(e) = fs::create_dir_all(&path) {
             let mut status = self.status.borrow_mut();
             *status = DownloadStatus::Failed(format!("Failed to create directory: {}", e));
@@ -260,4 +272,11 @@ impl NavdataDownloader {
         }
         self.update_and_get_status();
     }
+
+    pub fn delete_all_navdata(&self) {
+        let path = Path::new("\\work/navdata");
+        if util::path_exists(path) {
+            let _ = util::delete_folder_recursively(path);
+        }
+    }
 }

Sources/Code/WASM/navdata_updater/src/download/downloader.rs

@@ -2,6 +2,8 @@ use std::fs;
 use std::io;
 use std::path::PathBuf;
 
+use crate::util;
+
 pub struct ZipFileHandler<R: io::Read + io::Seek> {
     pub zip_archive: Option<zip::ZipArchive<R>>,
     path_buf: PathBuf,
@@ -46,14 +48,10 @@ impl<R: io::Read + io::Seek> ZipFileHandler<R> {
                 fs::create_dir_all(outpath).unwrap();
             } else {
                 if let Some(p) = outpath.parent() {
-                    if !p.exists() {
+                    if !util::path_exists(p) {
                         fs::create_dir_all(p).unwrap();
                     }
                 }
-                // If file already exists, delete it so we can overwrite it
-                if outpath.exists() {
-                    fs::remove_file(&outpath).unwrap();
-                }
                 let mut outfile = fs::File::create(outpath).unwrap();
                 io::copy(&mut file, &mut outfile).unwrap();
             }

Sources/Code/WASM/navdata_updater/src/download/zip_handler.rs

@@ -21,3 +21,26 @@ pub fn get_path_type(path: &Path) -> PathType {
     }
     PathType::DoesNotExist
 }
+
+pub fn path_exists(path: &Path) -> bool {
+    get_path_type(path) != PathType::DoesNotExist
+}
+
+pub fn delete_folder_recursively(path: &Path) -> io::Result<()> {
+    // Make sure we are deleting a directory (and in turn that it exists)
+    if get_path_type(path) != PathType::Directory {
+        return Ok(());
+    }
+    // We need to collect the entries into a vector since we can't iterate over them while deleting them
+    for entry in fs::read_dir(path)?.collect::<Vec<_>>() {
+        let entry = entry?;
+        let path = entry.path();
+        if get_path_type(&path) == PathType::Directory {
+            delete_folder_recursively(&path)?;
+        } else {
+            fs::remove_file(&path)?;
+        }
+    }
+    fs::remove_dir(path)?;
+    Ok(())
+}