From: Florian Brandes <florian.brandes@posteo.de>
Date: Sat, 2 Apr 2022 05:23:31 +0000 (+0200)
Subject: uses shutil.move instead of rename
X-Git-Tag: beta-1.6.1~3^2
X-Git-Url: http://git.ipfire.org/gitweb/gitweb.cgi?a=commitdiff_plain;h=refs%2Fpull%2F617%2Fhead;p=thirdparty%2Fpaperless-ngx.git

uses shutil.move instead of rename

fixes issues with cross-filesystem movement

Signed-off-by: Florian Brandes <florian.brandes@posteo.de>
---

diff --git a/src/documents/signals/handlers.py b/src/documents/signals/handlers.py
index 1b3f3b5772..71f2cdc554 100644
--- a/src/documents/signals/handlers.py
+++ b/src/documents/signals/handlers.py
@@ -1,5 +1,6 @@
 import logging
 import os
+import shutil
 
 from django.conf import settings
 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}")
             try:
-                os.rename(instance.source_path, new_file_path)
+                shutil.move(instance.source_path, new_file_path)
             except OSError as e:
                 logger.error(
                     f"Failed to move {instance.source_path} to trash at "