(sedona-db) branch main updated: Fix missing SRID on EPSG:4326 (#79)

[jiayu]

This is an automated email from the ASF dual-hosted git repository.

jiayu pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/sedona-db.git


The following commit(s) were added to refs/heads/main by this push:
     new 827232c  Fix missing SRID on EPSG:4326 (#79)
827232c is described below

commit 827232c5fdf4b1241652b51dbafd2618253816bf
Author: jp <j...@wherobots.com>
AuthorDate: Sat Sep 13 21:03:25 2025 -0700

    Fix missing SRID on EPSG:4326 (#79)
---
 rust/sedona-schema/src/crs.rs | 15 ++++++++++++---
 1 file changed, 12 insertions(+), 3 deletions(-)

diff --git a/rust/sedona-schema/src/crs.rs b/rust/sedona-schema/src/crs.rs
index 5c81cb1..fef41b7 100644
--- a/rust/sedona-schema/src/crs.rs
+++ b/rust/sedona-schema/src/crs.rs
@@ -122,6 +122,10 @@ impl LngLat {
     pub fn is_authority_code_lnglat(string_value: &str) -> bool {
         string_value == "OGC:CRS84" || string_value == "EPSG:4326"
     }
+
+    pub fn srid() -> Option<u32> {
+        Some(4326)
+    }
 }
 
 /// Implementation of an authority:code CoordinateReferenceSystem
@@ -213,6 +217,8 @@ impl CoordinateReferenceSystem for AuthorityCode {
     fn srid(&self) -> Result<Option<u32>> {
         if self.authority.eq_ignore_ascii_case("EPSG") {
             Ok(self.code.parse::<u32>().ok())
+        } else if LngLat::is_lnglat(self) {
+            Ok(LngLat::srid())
         } else {
             Ok(None)
         }
@@ -287,9 +293,8 @@ impl CoordinateReferenceSystem for ProjJSON {
         let authority_code_opt = self.to_authority_code()?;
         if let Some(authority_code) = authority_code_opt {
             if LngLat::is_authority_code_lnglat(&authority_code) {
-                return Ok(Some(4326));
-            }
-            if let Some((_, code)) = 
AuthorityCode::split_auth_code(&authority_code) {
+                return Ok(LngLat::srid());
+            } else if let Some((_, code)) = 
AuthorityCode::split_auth_code(&authority_code) {
                 return Ok(code.parse::<u32>().ok());
             }
         }
@@ -391,5 +396,9 @@ mod test {
             Some("EPSG:4269".to_string())
         );
         assert_eq!(new_crs.unwrap().srid().unwrap(), Some(4269));
+
+        let value: Value = serde_json::from_str("\"EPSG:4326\"").unwrap();
+        let new_crs = deserialize_crs(&value).unwrap();
+        assert_eq!(new_crs.clone().unwrap().srid().unwrap(), Some(4326));
     }
 }