Fixed bug causing a leading zero to be returned when certain numbers are

converted to strings (one example noted here:
sqlalchemy/sqlalchemy#8744 (comment)).

Author: anthony-tuininga

doc/src/release_notes.rst

@@ -24,6 +24,8 @@ Thin Mode Changes
     cursor.
 #)  Fixed bug preventing a cursor from being reused after it was bound as a
     REF CURSOR to a PL/SQL block that closes it.
+#)  Fixed bug causing a leading zero to be returned when certain numbers are
+    converted to strings.
 
 Thick Mode Changes
 ++++++++++++++++++

src/oracledb/impl/thin/buffer.pyx

@@ -570,7 +570,7 @@ cdef class Buffer:
 
             # process the first digit; leading zeroes are ignored
             digit = <uint8_t> byte // 10
-            if digit == 0 and i == 0:
+            if digit == 0 and num_digits == 0:
                 decimal_point_index -= 1
             elif digit == 10:
                 digits[num_digits] = 1

tests/test_2200_number_var.py

@@ -44,6 +44,11 @@ def output_type_handler_decimal(self, cursor, name, default_type, size,
             return cursor.var(str, 255, outconverter=decimal.Decimal,
                               arraysize=cursor.arraysize)
 
+    def output_type_handler_str(self, cursor, name, default_type, size,
+                                precision, scale):
+        return cursor.var(str, 255, arraysize=cursor.arraysize)
+
+
     def setUp(self):
         super().setUp()
         self.raw_data = []
@@ -471,5 +476,13 @@ def test_2237_fetch_number_with_lobs_default_false(self):
             result, = self.cursor.fetchone()
             self.assertEqual(type(result), int)
 
+    def test_2238_fetch_small_constant_with_decimal_point(self):
+        "2238 - fetch a small constant with a decimal point"
+        self.cursor.outputtypehandler = self.output_type_handler_str
+        self.cursor.execute("select 3 / 2 from dual")
+        result, = self.cursor.fetchone()
+        self.assertTrue(len(result) == 3 and result[0] == "1" \
+                        and result[-1] == "5")
+
 if __name__ == "__main__":
     test_env.run_test_cases()