Merge pull request #513 from ApptiveGrid/revert-scaled-decimal

reverted fix as no conversion is in place yet
ApptiveGrid · Nov 20, 2023 · 3948543 · 3948543
2 parents 05cae06 + 7c455f7
commit 3948543
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/src/Soil-Serializer-Tests/SoilPrimitiveSerializationTest.class.st b/src/Soil-Serializer-Tests/SoilPrimitiveSerializationTest.class.st
@@ -513,7 +513,7 @@ SoilPrimitiveSerializationTest >> testSerializationScaledDecimal [
 	serialized := SoilSerializer serializeToBytes: object.
 
 	self assert: (serialized at: 1) equals: TypeCodeScaledDecimal.
-	self assert: serialized equals: #[9 5 10 1 2].
+	self assert: serialized equals: #[9 10 1 2].
 	materialized := SoilMaterializer materializeFromBytes: serialized.
 	self assert: materialized class equals: ScaledDecimal.
 	self assert: materialized equals: object

diff --git a/src/Soil-Serializer/SoilBasicMaterializer.class.st b/src/Soil-Serializer/SoilBasicMaterializer.class.st
@@ -104,7 +104,7 @@ SoilBasicMaterializer >> nextScaledDecimal: aClass [
 
 	| scaledDecimal |
 	scaledDecimal := aClass new
-		                 setNumerator: self nextSoilObject
+		                 setNumerator: self nextLengthEncodedInteger
 		                 denominator: self nextLengthEncodedInteger
 		                 scale: self nextLengthEncodedInteger.
 	self registerObject: scaledDecimal.

diff --git a/src/Soil-Serializer/SoilBasicSerializer.class.st b/src/Soil-Serializer/SoilBasicSerializer.class.st
@@ -121,7 +121,7 @@ SoilBasicSerializer >> nextPutScaledDecimal: aNumber [
 	"we know that numerator, denominator and scale are SmallIntegers"
 	self
 		nextPutByte: TypeCodeScaledDecimal;
-		nextPutInteger: aNumber numerator;
+		nextPutLengthEncodedInteger: aNumber numerator;
 		nextPutLengthEncodedInteger: aNumber denominator;
 		nextPutLengthEncodedInteger: aNumber scale 
 ]