fix: display nested geo values (#734)

* fix display nested geo values

* fix

---------

Co-authored-by: Yang Xiufeng <yangxiufeng.c@gmail.com>

Author: b41sh

cli/tests/00-base.result

@@ -37,4 +37,5 @@ NULL	"{""k1"":""v1"",""k2"":""v2""}"	(2,NULL)
 1	NULL	1	ab
 NULL	v1	2	NULL
 "{""k1"":""v1"",""k2"":""v2""}"	[6162,78797A]	"([1,2],""2024-04-10"")"
+"[{""type"": ""Point"", ""coordinates"": [-122.35,37.55]}]"	"[{""type"": ""LineString"", ""coordinates"": [[0.75,0.75],[-10,20]]}]"
 bye

cli/tests/00-base.sql

@@ -64,6 +64,9 @@ select a[1], b['k1'], c:x, c:y from test_nested;
 
 select {'k1':'v1','k2':'v2'}, [to_binary('ab'), to_binary('xyz')], (parse_json('[1,2]'), to_date('2024-04-10'));
 
+set geometry_output_format='GEOJSON';
+select [to_geometry('POINT(-122.35 37.55)')], [st_geographyfromewkt('LINESTRING(0.75 0.75, -10 20)')];
+
 select 'bye';
 drop table test;
 drop table test_decimal;

sql/src/value/string_decoder.rs

@@ -354,26 +354,28 @@ impl ValueDecoder {
 
     fn read_geometry<R: AsRef<[u8]>>(&self, reader: &mut Cursor<R>) -> Result<Value> {
         let mut buf = Vec::new();
-        if reader.read_quoted_text(&mut buf, b'"').is_err() {
-            if let Ok(val) = self.read_json(reader) {
-                return Ok(Value::Variant(val));
-            }
-            reader.read_quoted_text(&mut buf, b'\'')?;
+        if reader.read_quoted_text(&mut buf, b'"').is_ok()
+            || reader.read_quoted_text(&mut buf, b'\'').is_ok()
+        {
+            Ok(Value::Geometry(unsafe { String::from_utf8_unchecked(buf) }))
+        } else {
+            let val = self.read_json(reader)?;
+            Ok(Value::Geometry(val))
         }
-        Ok(Value::Geometry(unsafe { String::from_utf8_unchecked(buf) }))
     }
 
     fn read_geography<R: AsRef<[u8]>>(&self, reader: &mut Cursor<R>) -> Result<Value> {
         let mut buf = Vec::new();
-        if reader.read_quoted_text(&mut buf, b'"').is_err() {
-            if let Ok(val) = self.read_json(reader) {
-                return Ok(Value::Variant(val));
-            }
-            reader.read_quoted_text(&mut buf, b'\'')?;
+        if reader.read_quoted_text(&mut buf, b'"').is_ok()
+            || reader.read_quoted_text(&mut buf, b'\'').is_ok()
+        {
+            Ok(Value::Geography(unsafe {
+                String::from_utf8_unchecked(buf)
+            }))
+        } else {
+            let val = self.read_json(reader)?;
+            Ok(Value::Geography(val))
         }
-        Ok(Value::Geography(unsafe {
-            String::from_utf8_unchecked(buf)
-        }))
     }
 
     fn read_nullable<R: AsRef<[u8]>>(