[Fixes #14374] Add deletion of temporary files in case of rollback (#14375)

* [Fixes #14374] Add deletion of temporary files in case of rollback

* [Fixes #14374] Fix PR suggestions

Author: mattiagiupponi

geonode/upload/handlers/common/vector.py

@@ -420,6 +420,12 @@ def perform_last_step(execution_id):
             for asset in assets:
                 asset.delete()
 
+        BaseVectorFileHandler.__remove_temporary_file(_exec)
+
+    @staticmethod
+    def __remove_temporary_file(_exec):
+        if not _exec:
+            return
         tmp_data = _exec.input_params.get("temporary_files")
         if tmp_data:
             # Delete at the end of the operations, the temporary files created at the beginning
@@ -1091,6 +1097,12 @@ def _import_resource_rollback(self, exec_id, instance_name=None, *args, **kwargs
         We use the schema editor directly, because the model itself is not managed
         on creation, but for the delete since we are going to handle, we can use it
         """
+        logger.info(f"Rollback temporary file uploaded for execid: {exec_id} resource published was: {instance_name}")
+        try:
+            BaseVectorFileHandler.__remove_temporary_file(orchestrator.get_execution_object(exec_id=exec_id))
+        except Exception as e:
+            logger.warning(f"Failed to remove temporary files during rollback: {e}")
+
         logger.info(
             f"Rollback dynamic model & ogr2ogr step in progress for execid: {exec_id} resource published was: {instance_name}"
         )