fix(spp_hazard): address gemini-code-assist review findings

- _parse_datetime_string accepts datetime instances (avoids TypeError on
  programmatic values).
- Area-linking query wrapped in a savepoint so an invalid geometry cannot
  leave the transaction aborted for subsequent operations.
- Migration column check scoped to current_schema() to avoid false positives
  from same-named tables in other schemas.

Author: gonzalesedwin1123

spp_hazard/migrations/19.0.2.1.0/post-migration.py

@@ -94,6 +94,7 @@ def _column_exists(cr, table, column):
         SELECT 1
         FROM information_schema.columns
         WHERE table_name = %s AND column_name = %s
+          AND table_schema = current_schema()
         """,
         (table, column),
     )

spp_hazard/models/hazard_incident.py

@@ -28,9 +28,12 @@ def _parse_datetime_string(value):
     Returns:
         datetime object
     """
-    # Replace 'Z' with '+00:00' for fromisoformat compatibility
-    normalized = value.replace("Z", "+00:00")
-    dt = datetime.fromisoformat(normalized)
+    if isinstance(value, datetime):
+        dt = value
+    else:
+        # Replace 'Z' with '+00:00' for fromisoformat compatibility
+        normalized = value.replace("Z", "+00:00")
+        dt = datetime.fromisoformat(normalized)
     # Odoo requires naive (UTC) datetimes
     if dt.tzinfo is not None:
         dt = dt.astimezone(UTC).replace(tzinfo=None)
@@ -525,17 +528,20 @@ def _link_areas_from_geometry(self, geometry_dict):
         geojson_str = json.dumps(geometry_dict) if isinstance(geometry_dict, dict) else geometry_dict
 
         try:
-            self.env.cr.execute(
-                """
-                SELECT id FROM spp_area
-                WHERE geo_polygon IS NOT NULL
-                AND ST_Intersects(
-                    geo_polygon::geometry,
-                    ST_SetSRID(ST_GeomFromGeoJSON(%s), 4326)
+            # Savepoint: a failed query (e.g. invalid geometry) would otherwise
+            # abort the whole transaction, not just this optional linking step.
+            with self.env.cr.savepoint():
+                self.env.cr.execute(
+                    """
+                    SELECT id FROM spp_area
+                    WHERE geo_polygon IS NOT NULL
+                    AND ST_Intersects(
+                        geo_polygon::geometry,
+                        ST_SetSRID(ST_GeomFromGeoJSON(%s), 4326)
+                    )
+                    """,
+                    (geojson_str,),
                 )
-                """,
-                (geojson_str,),
-            )
             area_ids = [row[0] for row in self.env.cr.fetchall()]
             if area_ids:
                 self.write({"area_ids": [Command.set(area_ids)]})