Merge pull request #617 from paperless-ngx/fix-move-to-trash

uses shutil.move instead of rename
This commit is contained in:
Quinn Casey 2022-04-02 15:17:43 +07:00 committed by GitHub
commit 805c17565b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -1,5 +1,6 @@
import logging import logging
import os import os
import shutil
from django.conf import settings from django.conf import settings
from django.contrib.admin.models import ADDITION from django.contrib.admin.models import ADDITION
@ -252,7 +253,7 @@ def cleanup_document_deletion(sender, instance, using, **kwargs):
logger.debug(f"Moving {instance.source_path} to trash at {new_file_path}") logger.debug(f"Moving {instance.source_path} to trash at {new_file_path}")
try: try:
os.rename(instance.source_path, new_file_path) shutil.move(instance.source_path, new_file_path)
except OSError as e: except OSError as e:
logger.error( logger.error(
f"Failed to move {instance.source_path} to trash at " f"Failed to move {instance.source_path} to trash at "