From f6744d97a82ebde915c2980a53571b91b39a416d Mon Sep 17 00:00:00 2001 From: Jaspreet-singh-1032 Date: Sat, 5 Aug 2023 21:34:38 +0530 Subject: [PATCH] added try except for import file delete --- import_export_celery/models/importjob.py | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/import_export_celery/models/importjob.py b/import_export_celery/models/importjob.py index 1ba0f56..74e545e 100644 --- a/import_export_celery/models/importjob.py +++ b/import_export_celery/models/importjob.py @@ -16,6 +16,10 @@ from ..fields import ImportExportFileField from ..tasks import run_import_job +import logging + +logger = logging.getLogger(__name__) + @with_author class ImportJob(models.Model): @@ -103,5 +107,10 @@ def auto_delete_file_on_delete(sender, instance, **kwargs): Deletes file related to the import job """ if instance.file: - instance.file.delete() - instance.delete() + try: + instance.file.delete() + except Exception as e: + logger.error( + "Some error occurred while deleting ImportJob file: {0}".format(e) + ) + ImportJob.objects.filter(id=instance.id).delete()