Better output on dryrun, make clear what we would delete now

This commit is contained in:
Sebastian Spaeth 2022-11-08 18:55:23 +01:00
parent a00ffc258f
commit 36d392e9b1

View File

@ -151,7 +151,7 @@ def parse_options():
logging.basicConfig(level=loglevel, format= '%(levelname)s - %(message)s') logging.basicConfig(level=loglevel, format= '%(levelname)s - %(message)s')
return args return args
#-----------------------------------------------------------------------
if __name__ == '__main__': if __name__ == '__main__':
args = parse_options() args = parse_options()
(MEDIA_PATH, CONN_STR) = read_config(args.config) (MEDIA_PATH, CONN_STR) = read_config(args.config)
@ -160,7 +160,10 @@ if __name__ == '__main__':
files = mr.get_remote_media() files = mr.get_remote_media()
for file in files: for file in files:
if file.create_date < cleantime: if file.create_date < cleantime:
if not file.exists(): if args.dryrun: # the great pretender
if file.exists():
logging.info(f"Pretending to delete file id {file.media_id} on path {file.fullpath}.")
else:
logging.info(f"file id {file.media_id} does not physically exist (path {file.fullpath})") logging.info(f"file id {file.media_id} does not physically exist (path {file.fullpath})")
if not args.dryrun: else:
file.delete() file.delete()