fixup! fix: use appropriate methods to convert the numbers in XML

halestudio · Jun 5, 2024 · af96e9e · af96e9e
1 parent 979ce54
commit af96e9e
Show file tree

Hide file tree

Showing 4 changed files with 26 additions and 4 deletions.
diff --git a/io/plugins/eu.esdihumboldt.hale.io.gml.test/src/data/numbers/numbers.xsd b/io/plugins/eu.esdihumboldt.hale.io.gml.test/src/data/numbers/numbers.xsd
@@ -15,6 +15,8 @@
 			<xs:element name="longNumber" type="xs:long" minOccurs="0"/>
 			<xs:element name="floatNumber" type="xs:float" minOccurs="0"/>
 			<xs:element name="shortNumber" type="xs:short" minOccurs="0"/>
+
+			<element name="doubleSimpleNumber" type="double" minOccurs="0"/>
         </sequence>
       </extension>
     </complexContent>

diff --git a/...e.io.gml.test/src/eu/esdihumboldt/hale/io/gml/writer/internal/StreamGmlWriter2Test.groovy b/...e.io.gml.test/src/eu/esdihumboldt/hale/io/gml/writer/internal/StreamGmlWriter2Test.groovy
@@ -384,12 +384,20 @@ class StreamGmlWriter2Test {
 
 	@Test
 	void testNumbers() throws Exception {
+		//		testNumber("doubleSimpleNumber", 123456789.123456789)
+
 		testNumber("integerNumber", 12345)
 		testNumber("doubleNumber", 12345.123456789)
 		testNumber("floatNumber", 12345.6789F)
 		testNumber("longNumber", 1234567890123456789L)
 		testNumber("shortNumber", 12345 as short)
 		testNumber("decimalNumber", new BigDecimal("1234567890.123456789"))
+
+		double doubleNumber = 123456789.123456789
+		testNumber("decimalNumber", new BigDecimal(doubleNumber));
+
+		doubleNumber = 1.23456789123456789E8
+		testNumber("decimalNumber", new BigDecimal(doubleNumber));
 	}
 
 	void testNumber(String elementName, number) throws Exception {
@@ -403,10 +411,18 @@ class StreamGmlWriter2Test {
 
 		// Write file, validate and load it
 		def writer = new GmlInstanceWriter()
-		def xmlFile = writeFile(instance, schema, writer).getAbsolutePath()
+		File outFile = writeFile(instance, schema, writer)
+		def xmlFile = outFile.getAbsolutePath()
 		def xml = new XmlSlurper().parse(xmlFile)
 
+		println number.toString()
+		println xml.featureMember.PrimitiveTest."$elementName".text()
+
 		assertEquals(number.toString(), xml.featureMember.PrimitiveTest."$elementName".text())
+
+		if (DEL_TEMP_FILES) {
+			//			outFile.delete()
+		}
 	}
 
 	@CompileStatic

diff --git a/...boldt.hale.io.gml/src/eu/esdihumboldt/hale/io/gml/internal/simpletype/SimpleTypeUtil.java b/...boldt.hale.io.gml/src/eu/esdihumboldt/hale/io/gml/internal/simpletype/SimpleTypeUtil.java
@@ -125,9 +125,8 @@ else if (simpleTypeValue instanceof XmlDateTime) {
 
 					xmlDateTime.setGDateValue(gdate);
 				}
-
-				// Numbers should be handled here
-				if (simpleTypeValue != null) {
+				else if (simpleTypeValue != null && simpleTypeValue instanceof XmlAnySimpleType) {
+					// Numbers should be handled here
 					return simpleTypeValue.getStringValue();
 				}
 			} catch (ConversionException e) {

diff --git a/...gml/src/eu/esdihumboldt/hale/io/gml/internal/simpletype/converters/DoubleToXmlDouble.java b/...gml/src/eu/esdihumboldt/hale/io/gml/internal/simpletype/converters/DoubleToXmlDouble.java
@@ -15,6 +15,8 @@
 
 package eu.esdihumboldt.hale.io.gml.internal.simpletype.converters;
 
+import java.math.BigDecimal;
+
 import org.apache.xmlbeans.XmlDouble;
 import org.springframework.core.convert.converter.Converter;
 
@@ -31,6 +33,9 @@ public XmlDouble convert(Double value) {
 		if (value == null) {
 			return null;
 		}
+		if (value.toString().contains("E")) {
+			value = Double.parseDouble(new BigDecimal(value).toPlainString());
+		}
 		return XmlDouble.Factory.newValue(value);
 	}