fix: fix some api about create directory

ShrBox · ShrBox · commit 3379c55cbc6c · 2024-03-14T19:32:04.000+08:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,6 +5,12 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.4.11] - 2024-03-14
+
+### Fixed
+
+- Fix some api about create directory
+
 ## [0.4.10] - 2024-03-14
 
 ### Changed
diff --git a/manifest.json b/manifest.json
@@ -4,7 +4,7 @@
     "type": "native",
     "description": "A plugin engine for running LLSE plugins on LeviLamina",
     "author": "LiteLDev",
-    "version": "0.4.10",
+    "version": "0.4.11",
     "dependencies": [
         {
             "name": "LegacyMoney"
diff --git a/src/legacy/api/DataAPI.cpp b/src/legacy/api/DataAPI.cpp
@@ -291,7 +291,7 @@ ConfIniClass* ConfIniClass::constructor(const Arguments& args) {
         if (args.size() >= 2) return new ConfIniClass(args.thiz(), path, args[1].toStr());
         else return new ConfIniClass(args.thiz(), path, "");
     }
-    CATCH_C("Fail in Open JsonConfigFile!");
+    CATCH_C("Fail in Open IniConfigFile!");
 }
 
 bool ConfIniClass::flush() { return iniConf->SaveFile(iniConf->filePath.c_str(), true); }
diff --git a/src/legacy/api/FileSystemAPI.cpp b/src/legacy/api/FileSystemAPI.cpp
@@ -11,6 +11,7 @@
 #include <fstream>
 #include <ll/api/io/FileUtils.h>
 #include <string>
+#include <system_error>
 
 using namespace std::filesystem;
 
@@ -113,7 +114,7 @@ FileClass* FileClass::constructor(const Arguments& args) {
         string                     path    = args[0].toStr();
         std::optional<std::string> dirPath = getDirectoryPath(path);
         if (dirPath.has_value()) {
-            std::filesystem::create_directory(dirPath.value());
+            std::filesystem::create_directories(dirPath.value());
         } else {
             LOG_ERROR_WITH_SCRIPT_INFO("Fail to create directory " + dirPath.value() + "!\n");
             return nullptr;
@@ -520,7 +521,7 @@ Local<Value> DirCreate(const Arguments& args) {
     CHECK_ARG_TYPE(args[0], ValueKind::kString);
 
     try {
-        return Boolean::newBoolean(std::filesystem::create_directory(args[0].toStr()));
+        return Boolean::newBoolean(std::filesystem::create_directories(args[0].toStr()));
     } catch (const filesystem_error& e) {
         LOG_ERROR_WITH_SCRIPT_INFO("Fail to Create Dir " + args[0].asString().toString() + "!\n");
         return Boolean::newBoolean(false);
@@ -678,7 +679,12 @@ Local<Value> FileWriteTo(const Arguments& args) {
         string                     path    = args[0].toStr();
         std::optional<std::string> dirPath = getDirectoryPath(path);
         if (dirPath.has_value()) {
-            std::filesystem::create_directory(dirPath.value());
+            std::error_code code;
+            std::filesystem::create_directories(dirPath.value(), code);
+            if (code) {
+                LOG_ERROR_WITH_SCRIPT_INFO("Fail to create directory " + dirPath.value() + "!\n");
+                return Boolean::newBoolean(false);
+            }
         } else {
             LOG_ERROR_WITH_SCRIPT_INFO("Fail to create directory " + dirPath.value() + "!\n");
             return Boolean::newBoolean(false);
@@ -697,7 +703,12 @@ Local<Value> FileWriteLine(const Arguments& args) {
         string                     path    = args[0].toStr();
         std::optional<std::string> dirPath = getDirectoryPath(path);
         if (dirPath.has_value()) {
-            std::filesystem::create_directory(dirPath.value());
+            std::error_code code;
+            std::filesystem::create_directories(dirPath.value(), code);
+            if (code) {
+                LOG_ERROR_WITH_SCRIPT_INFO("Fail to create directory " + dirPath.value() + "!\n");
+                return Boolean::newBoolean(false);
+            }
         } else {
             LOG_ERROR_WITH_SCRIPT_INFO("Fail to create directory " + dirPath.value() + "!\n");
             return Boolean::newBoolean(false);
@@ -723,7 +734,12 @@ Local<Value> OpenFile(const Arguments& args) {
         string                     path    = args[0].toStr();
         std::optional<std::string> dirPath = getDirectoryPath(path);
         if (dirPath.has_value()) {
-            std::filesystem::create_directory(dirPath.value());
+            std::error_code code;
+            std::filesystem::create_directories(dirPath.value(), code);
+            if (code) {
+                LOG_ERROR_WITH_SCRIPT_INFO("Fail to create directory " + dirPath.value() + "!\n");
+                return Boolean::newBoolean(false);
+            }
         } else {
             LOG_ERROR_WITH_SCRIPT_INFO("Fail to create directory " + dirPath.value() + "!\n");
             return {};
diff --git a/src/legacy/utils/JsonHelper.h b/src/legacy/utils/JsonHelper.h
@@ -19,13 +19,13 @@ inline ordered_json CreateJson(const std::string& path, const std::string& defCo
             std::size_t pos = path.find_last_of('/');
             if (pos != std::string::npos) {
                 std::string dirPath = path.substr(0, pos);
-                std::filesystem::create_directory(dirPath);
+                std::filesystem::create_directories(dirPath);
             }
         } else if (path.find('\\') != std::string::npos) { // e.g. plugins\\LeviLamina\\LeviLamina.json
             std::size_t pos = path.find_last_of('\\');
             if (pos != std::string::npos) {
                 std::string dirPath = path.substr(0, pos);
-                std::filesystem::create_directory(dirPath);
+                std::filesystem::create_directories(dirPath);
             }
         } else {
             lse::getSelfPluginInstance().getLogger().error("Fail in create json file!");
diff --git a/tooth.json b/tooth.json
@@ -1,7 +1,7 @@
 {
     "format_version": 2,
     "tooth": "github.com/LiteLDev/LegacyScriptEngine",
-    "version": "0.4.10",
+    "version": "0.4.11",
     "info": {
         "name": "LegacyScriptEngine",
         "description": "A plugin engine for running LLSE plugins on LeviLamina",
@@ -12,7 +12,7 @@
         ]
     },
     "dependencies": {
-        "gitea.litebds.com/LiteLDev/legacy-script-engine-lua": "0.4.10",
-        "gitea.litebds.com/LiteLDev/legacy-script-engine-quickjs": "0.4.10"
+        "gitea.litebds.com/LiteLDev/legacy-script-engine-lua": "0.4.11",
+        "gitea.litebds.com/LiteLDev/legacy-script-engine-quickjs": "0.4.11"
     }
 }
diff --git a/tooth.lua.json b/tooth.lua.json
@@ -1,7 +1,7 @@
 {
     "format_version": 2,
     "tooth": "gitea.litebds.com/LiteLDev/legacy-script-engine-lua",
-    "version": "0.4.10",
+    "version": "0.4.11",
     "info": {
         "name": "LegacyScriptEngine with Lua backend",
         "description": "A plugin engine for running LLSE plugins on LeviLamina",
diff --git a/tooth.nodejs.json b/tooth.nodejs.json
@@ -1,7 +1,7 @@
 {
     "format_version": 2,
     "tooth": "gitea.litebds.com/LiteLDev/legacy-script-engine-nodejs",
-    "version": "0.4.10",
+    "version": "0.4.11",
     "info": {
         "name": "LegacyScriptEngine with NodeJs backend",
         "description": "A plugin engine for running LLSE plugins on LeviLamina",
diff --git a/tooth.python.json b/tooth.python.json
@@ -1,7 +1,7 @@
 {
     "format_version": 2,
     "tooth": "gitea.litebds.com/LiteLDev/legacy-script-engine-python",
-    "version": "0.4.10",
+    "version": "0.4.11",
     "info": {
         "name": "LegacyScriptEngine with Python backend",
         "description": "A plugin engine for running LLSE plugins on LeviLamina",
diff --git a/tooth.quickjs.json b/tooth.quickjs.json
@@ -1,7 +1,7 @@
 {
     "format_version": 2,
     "tooth": "gitea.litebds.com/LiteLDev/legacy-script-engine-quickjs",
-    "version": "0.4.10",
+    "version": "0.4.11",
     "info": {
         "name": "LegacyScriptEngine with QuickJs backend",
         "description": "A plugin engine for running LLSE plugins on LeviLamina",

Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@`
`4`	`4`	`"type": "native",`
`5`	`5`	`"description": "A plugin engine for running LLSE plugins on LeviLamina",`
`6`	`6`	`"author": "LiteLDev",`
`7`		`- "version": "0.4.10",`
	`7`	`+ "version": "0.4.11",`
`8`	`8`	`"dependencies": [`
`9`	`9`	`{`
`10`	`10`	`"name": "LegacyMoney"`
Original file line number	Diff line number	Diff line change
`@@ -291,7 +291,7 @@ ConfIniClass* ConfIniClass::constructor(const Arguments& args) {`
`291`	`291`	`if (args.size() >= 2) return new ConfIniClass(args.thiz(), path, args[1].toStr());`
`292`	`292`	`else return new ConfIniClass(args.thiz(), path, "");`
`293`	`293`	`}`
`294`		`- CATCH_C("Fail in Open JsonConfigFile!");`
	`294`	`+ CATCH_C("Fail in Open IniConfigFile!");`
`295`	`295`	`}`
`296`	`296`
`297`	`297`	`bool ConfIniClass::flush() { return iniConf->SaveFile(iniConf->filePath.c_str(), true); }`
Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,7 @@`
`1`	`1`	`{`
`2`	`2`	`"format_version": 2,`
`3`	`3`	`"tooth": "github.com/LiteLDev/LegacyScriptEngine",`
`4`		`- "version": "0.4.10",`
	`4`	`+ "version": "0.4.11",`
`5`	`5`	`"info": {`
`6`	`6`	`"name": "LegacyScriptEngine",`
`7`	`7`	`"description": "A plugin engine for running LLSE plugins on LeviLamina",`
`@@ -12,7 +12,7 @@`
`12`	`12`	`]`
`13`	`13`	`},`
`14`	`14`	`"dependencies": {`
`15`		`- "gitea.litebds.com/LiteLDev/legacy-script-engine-lua": "0.4.10",`
`16`		`- "gitea.litebds.com/LiteLDev/legacy-script-engine-quickjs": "0.4.10"`
	`15`	`+ "gitea.litebds.com/LiteLDev/legacy-script-engine-lua": "0.4.11",`
	`16`	`+ "gitea.litebds.com/LiteLDev/legacy-script-engine-quickjs": "0.4.11"`
`17`	`17`	`}`
`18`	`18`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,7 @@`
`1`	`1`	`{`
`2`	`2`	`"format_version": 2,`
`3`	`3`	`"tooth": "gitea.litebds.com/LiteLDev/legacy-script-engine-lua",`
`4`		`- "version": "0.4.10",`
	`4`	`+ "version": "0.4.11",`
`5`	`5`	`"info": {`
`6`	`6`	`"name": "LegacyScriptEngine with Lua backend",`
`7`	`7`	`"description": "A plugin engine for running LLSE plugins on LeviLamina",`