Garmelon/PFERD
1
0
Fork 0
mirror of https://github.com/Garmelon/PFERD.git synced 2026-04-12 07:25:04 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

Don't recursively walk local-only dirs with NO_DELETE

Browse source
This commit is contained in:
Mr-Pine 2025-06-08 12:36:21 +02:00
parent 56e3065950
commit 6dae86df7d
No known key found for this signature in database
GPG key ID: 5481659D1D983422
2 changed files with 12 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

11
PFERD/output_dir.py
View file

@ -501,6 +501,10 @@ class OutputDirectory:
await self._cleanup_file(path, pure)
async def _cleanup_dir(self, path: Path, pure: PurePath, delete_self: bool = True) -> None:
if self._on_conflict in {OnConflict.NO_DELETE, OnConflict.NO_DELETE_PROMPT_OVERWRITE} and not self.report.contains_marked(pure):
self._report_not_deleted(pure)
return
for child in sorted(path.iterdir()):
pure_child = pure / child.name
await self._cleanup(child, pure_child)
@ -523,8 +527,11 @@ class OutputDirectory:
except OSError:
pass
else:
log.not_deleted("[bold bright_magenta]", "Not deleted", fmt_path(pure))
self._report.not_delete_file(pure)
self._report_not_deleted(pure)
def _report_not_deleted(self, path: PurePath) -> None:
log.not_deleted("[bold bright_magenta]", "Not deleted", fmt_path(path))
self._report.not_delete_file(path)
def load_prev_report(self) -> None:
log.explain_topic(f"Loading previous report from {fmt_real_path(self._report_path)}")

3
PFERD/report.py
View file

@ -176,6 +176,9 @@ class Report:
def is_marked(self, path: PurePath) -> bool:
return path in self.marked
def contains_marked(self, path: PurePath) -> bool:
return any(marked_file.is_relative_to(path) for marked_file in self.marked)
def add_file(self, path: PurePath) -> None:
"""
Unlike mark(), this function accepts any paths.