From 26e96644e5862eebdba4f2e6bda0769a70966278 Mon Sep 17 00:00:00 2001 From: Olivier Leger Date: Thu, 20 Jun 2024 17:47:28 -0400 Subject: [PATCH] Fix export deletion on KC storage --- kobo/apps/openrosa/apps/viewer/signals.py | 4 +++- kobo/apps/openrosa/apps/viewer/tests/test_exports.py | 5 +++-- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/kobo/apps/openrosa/apps/viewer/signals.py b/kobo/apps/openrosa/apps/viewer/signals.py index 9c0eb3fe74..a021621e0f 100644 --- a/kobo/apps/openrosa/apps/viewer/signals.py +++ b/kobo/apps/openrosa/apps/viewer/signals.py @@ -1,5 +1,4 @@ from django.conf import settings -from django.core.files.storage import default_storage from django.db.models.signals import post_delete, post_save, pre_delete from django.dispatch import receiver from guardian.shortcuts import assign_perm, get_perms_for_model @@ -8,6 +7,9 @@ from kobo.apps.openrosa.apps.viewer.models.data_dictionary import DataDictionary from kobo.apps.openrosa.apps.viewer.models.export import Export from kobo.apps.openrosa.apps.viewer.models.parsed_instance import ParsedInstance +from kpi.deployment_backends.kc_access.storage import ( + default_kobocat_storage as default_storage, +) @receiver(post_delete, sender=Export) diff --git a/kobo/apps/openrosa/apps/viewer/tests/test_exports.py b/kobo/apps/openrosa/apps/viewer/tests/test_exports.py index 5667c9ed5f..26dc7e6185 100644 --- a/kobo/apps/openrosa/apps/viewer/tests/test_exports.py +++ b/kobo/apps/openrosa/apps/viewer/tests/test_exports.py @@ -89,8 +89,9 @@ def test_create_export(self): def test_delete_file_on_export_delete(self): self._publish_transportation_form() self._submit_transport_instance() - export = generate_export(Export.XLS_EXPORT, 'xls', self.user.username, - self.xform.id_string) + export = generate_export( + Export.XLS_EXPORT, 'xls', self.user.username, self.xform.id_string + ) self.assertTrue(default_storage.exists(export.filepath)) # delete export object export.delete()