Fixed regression which prevented a null value from being set on DbObject

attributes or used as elements of collections (#273).

Author: anthony-tuininga

doc/src/release_notes.rst

@@ -19,6 +19,9 @@ Thin Mode Changes
 Common Changes
 ++++++++++++++
 
+#)  Fixed regression which prevented a null value from being set on DbObject
+    attributes or used as elements of collections
+    (`issue 273 <https://github.com/oracle/python-oracledb/issues/273>`__).
 #)  Corrected typing declarations.
 
 

src/oracledb/impl/base/dbobject.pyx

@@ -1,5 +1,5 @@
 #------------------------------------------------------------------------------
-# Copyright (c) 2020, 2022, Oracle and/or its affiliates.
+# Copyright (c) 2020, 2024, Oracle and/or its affiliates.
 #
 # This software is dual-licensed to you under the Universal Permissive License
 # (UPL) 1.0 as shown at https://oss.oracle.com/licenses/upl and Apache License
@@ -37,7 +37,7 @@ cdef class BaseDbObjectImpl:
         Checks to see if the maximum size has been violated.
         """
         violated[0] = False
-        if max_size > 0:
+        if max_size > 0 and value is not None:
             if isinstance(value, str):
                 actual_size[0] = len((<str> value).encode())
             else:

tests/test_2300_object_var.py

@@ -1,5 +1,5 @@
 # -----------------------------------------------------------------------------
-# Copyright (c) 2020, 2023, Oracle and/or its affiliates.
+# Copyright (c) 2020, 2024, Oracle and/or its affiliates.
 #
 # This software is dual-licensed to you under the Universal Permissive License
 # (UPL) 1.0 as shown at https://oss.oracle.com/licenses/upl and Apache License
@@ -771,6 +771,12 @@ def test_2334_validate_string_element_value(self):
             "C" * 101,
         )
 
+    def test_2335_validate_string_attr_null(self):
+        "2335 - test validating a string attribute with null value"
+        typ = self.conn.gettype("UDT_OBJECT")
+        obj = typ.newobject()
+        obj.STRINGVALUE = None
+
 
 if __name__ == "__main__":
     test_env.run_test_cases()