Skip to content

Commit

Permalink
Fix export deletion on KC storage
Browse files Browse the repository at this point in the history
  • Loading branch information
noliveleger committed Jun 20, 2024
1 parent da7706e commit 26e9664
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
4 changes: 3 additions & 1 deletion kobo/apps/openrosa/apps/viewer/signals.py
Original file line number Diff line number Diff line change
@@ -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
Expand All @@ -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)
Expand Down
5 changes: 3 additions & 2 deletions kobo/apps/openrosa/apps/viewer/tests/test_exports.py
Original file line number Diff line number Diff line change
Expand Up @@ -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()
Expand Down

0 comments on commit 26e9664

Please sign in to comment.