fix: issue with wrong output for boolean list

HISTORY.rst

@@ -1,6 +1,12 @@
 History
 =======
 
+3.17.1 / 2022-04-20
+===================
+
+  * fix: issue with wrong output for boolean list
+  * fix: pull requests also trigger action runs
+
 3.17.0 / 2022-04-18
 ===================
 

examples/booleanjson2.json

@@ -0,0 +1,5 @@
+{
+      "boolean_list": [true, false],
+      "number_array": [1, 2, 3],
+      "string_array": ["a", "b", "c"]
+}

json2xml/__init__.py

@@ -2,4 +2,4 @@
 
 __author__ = """Vinit Kumar"""
 __email__ = "mail@vinitkumar.me"
-__version__ = "3.17.0"
+__version__ = "3.17.1"

json2xml/dicttoxml.py

@@ -137,6 +137,7 @@ def convert(obj, ids, attr_type, item_func, cdata, item_wrap, parent="root"):
 
     LOG.info(f'Inside convert(). obj type is: "{type(obj).__name__}", obj="{str(obj)}"')
 
+
     item_name = item_func(parent)
 
     # since bool is also a subtype of number.Number and int, the check for bool
@@ -265,7 +266,11 @@ def convert_list(items, ids, parent, attr_type, item_func, cdata, item_wrap):
             f'Looping inside convert_list(): item="{str(item)}", item_name="{item_name}", type="{type(item).__name__}"'
         )
         attr = {} if not ids else {"id": f"{this_id}_{i + 1}"}
-        if isinstance(item, (numbers.Number, str)):
+
+        if isinstance(item, bool):
+            addline(convert_bool(item_name, item, attr_type, attr, cdata))
+
+        elif isinstance(item, (numbers.Number, str)):
             if item_wrap:
                 addline(
                     convert_kv(
@@ -298,8 +303,6 @@ def convert_list(items, ids, parent, attr_type, item_func, cdata, item_wrap):
                 )
             )
 
-        elif isinstance(item, bool):
-            addline(convert_bool(item_name, item, attr_type, attr, cdata))
 
         elif isinstance(item, dict):
             item_dict_str = convert_dict(

tests/test_json2xml.py

@@ -184,3 +184,12 @@ def test_read_boolean_data_from_json(self):
         dict_from_xml = xmltodict.parse(result)
         assert dict_from_xml["all"]["boolean"]["#text"] != 'True'
         assert dict_from_xml["all"]["boolean"]["#text"] == 'true'
+
+
+    def test_read_boolean_data_from_json2(self):
+        """Test correct return for boolean types."""
+        data = readfromjson("examples/booleanjson2.json")
+        result = json2xml.Json2xml(data).to_xml()
+        dict_from_xml = xmltodict.parse(result)
+        assert dict_from_xml["all"]["item"][0]["#text"] != 'True'
+        assert dict_from_xml["all"]["item"][0]["#text"] == 'true'