Gemini review fix

Author: mattiagiupponi

geonode/upload/handlers/common/vector.py

@@ -608,12 +608,11 @@ def setup_dynamic_model(
         """
 
         layer_name = self.fixup_name(layer.GetName() if isinstance(layer, ogr.Layer) else layer)
-        is_dynamic_model_managed = orchestrator.get_execution_object(execution_id).input_params.get(
-            "is_dynamic_model_managed", False
-        )
-        workspace = DataPublisher(None).workspace
+        _exec_obj = orchestrator.get_execution_object(execution_id)
 
-        if resource_pk := orchestrator.get_execution_object(execution_id).input_params.get("resource_pk", None):
+        is_dynamic_model_managed = _exec_obj.input_params.get("is_dynamic_model_managed", False)
+        workspace = DataPublisher(None).workspace
+        if resource_pk := _exec_obj.input_params.get("resource_pk", None):
             user_datasets = Dataset.objects.filter(owner=username, pk=resource_pk)
         else:
             user_datasets = Dataset.objects.filter(owner=username, alternate__iexact=f"{workspace.name}:{layer_name}")

geonode/upload/handlers/tiles3d/handler.py

@@ -277,6 +277,13 @@ def create_geonode_resource(
         return resource
 
     def create_asset_and_link(self, resource, files, action=None, asset_name=None, asset_type=None, **kwargs):
+        """
+        Overrides the base handler to prevent double asset creation for 3D tiles.
+
+        This handler's `create_geonode_resource` method already creates the necessary asset.
+        This override ensures that the subsequent generic call from the celery task does not create a duplicate asset.
+        The asset is only created by calling the parent method if both `asset_type` and `extension` are explicitly provided.
+        """
         if asset_type and kwargs.get("extension", None):
             return super().create_asset_and_link(resource, files, action, asset_name, asset_type, **kwargs)