Merge pull request #1474 from Duet3D/hotfix-filechecks

Updating file read.ts

Author: yasasw

nodescriptsTS/src/functions_platform_stats.ts

@@ -92,16 +92,9 @@ const getPlatformVersionPerRelease = async (browser_download_url) => {
 const getPluginPlatformVersionList = async (gituser, gitrepo, parent_folder = 'plugin_versions', is_new_plugin = false) => {
     let file_path = `${parent_folder}/${gitrepo}.json`;
     console.log("getPluginPlatformVersionList for", gituser, gitrepo, file_path, is_new_plugin)
-    let current_file_temp = [];
-    try{
-        current_file_temp = is_new_plugin ? [] : await readFile.JSON(file_path);
-    }
-    catch(e){
-        console.log("Error - Reading file ", file_path);
-        console.log(e);
-    }
 
-    const current_file = current_file_temp;
+    const current_file = is_new_plugin ? [] : await readFile.JSON(file_path) || [];
+    
     const latest_tagName = ((current_file || [])[0] || {})['tagName'];
     const releases = await getReleases(gituser, gitrepo, latest_tagName) || [];
     console.log(gitrepo, ': New Releases');

nodescriptsTS/src/util.ts

@@ -141,7 +141,15 @@ const readFile = {
         return new Promise((resolve, reject) => {
             fs.readFile(path, (err, data) => {
                 if (err) reject(err);
-                resolve(JSON.parse((data||"{}").toString()));
+                try{
+                    resolve(JSON.parse((data).toString()));
+                }
+                catch(e){
+                    console.log("Error readFile");
+                    console.log(e);
+                    resolve(false);
+                }
+
             });
         });    
     },