Explicitly convert bind_params to list or dict

Earlier, we were returning an OrderedDict. But some sql drivers explicitly require a dict object Similarly, previously, in case of list, we were returning OrderDict.values(). Now, we explicitly convert that to a list
sripathikrishnan · May 27, 2020 · 543810a · 543810a
1 parent 5827c76
commit 543810a
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/jinjasql/core.py b/jinjasql/core.py
@@ -184,9 +184,10 @@ def _prepare_query(self, template, data):
             query = template.render(data)
             bind_params = _thread_local.bind_params
             if self.param_style in ('named', 'pyformat'):
-                return query, bind_params
+                bind_params = dict(bind_params)
             else:
-                return query, bind_params.values()
+                bind_params = list(bind_params.values())
+            return query, bind_params
         finally:
             del _thread_local.bind_params
             del _thread_local.param_style