Mirrored_Repos/crafty-4
1
0
Fork 0
mirror of https://gitlab.com/crafty-controller/crafty-4.git synced 2024-08-30 18:23:09 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix bug where del fail would give no feedback

Browse Source
This commit is contained in:
amcmanu3 2024-01-28 16:21:27 -05:00
parent 27678e93ca
commit ec5e291a98
2 changed files with 11 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
app/classes/shared/file_helpers.py
View File

@ -35,8 +35,9 @@ class FileHelpers:
try:
# This removes the top-level folder:
path.rmdir()
except:
except Exception as e:
logger.error("Unable to remove top level")
return e
return True
@staticmethod
@ -47,9 +48,9 @@ class FileHelpers:
# Remove the file
os.remove(path)
return True
except FileNotFoundError:
except (FileNotFoundError, PermissionError) as e:
logger.error(f"Path specified is not a file or does not exist. {path}")
return False
return e
@staticmethod
def copy_dir(src_path, dest_path, dirs_exist_ok=False):

10
app/classes/web/routes/api/servers/server/files.py
View File

@ -237,10 +237,14 @@ class ApiServersServerFilesIndexHandler(BaseApiHandler):
)
if os.path.isdir(data["filename"]):
FileHelpers.del_dirs(data["filename"])
proc = FileHelpers.del_dirs(data["filename"])
else:
FileHelpers.del_file(data["filename"])
return self.finish_json(200, {"status": "ok"})
proc = FileHelpers.del_file(data["filename"])
# disabling pylint because return value could be truthy
# but not a true boolean value
if proc == True: # pylint: disable=singleton-comparison
return self.finish_json(200, {"status": "ok"})
return self.finish_json(500, {"status": "error", "error": str(proc)})
def patch(self, server_id: str):
auth_data = self.authenticate_user()