Fix SQL list generation

vega · Feb 12, 2024 · 681ff18 · 681ff18
1 parent 48bc143
commit 681ff18
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/vegafusion-sql/src/compile/scalar.rs b/vegafusion-sql/src/compile/scalar.rs
@@ -151,11 +151,12 @@ impl ToSqlScalar for ScalarValue {
                     value: "make_list".to_string(),
                     quote_style: None,
                 };
-
-                let args = (0..array.len())
-                    .map(|i| {
-                        let v = array.value(0).to_scalar_vec()?;
-                        let sql_expr = v[i].to_sql(dialect)?;
+                let args = array
+                    .value(0)
+                    .to_scalar_vec()?
+                    .into_iter()
+                    .map(|v| {
+                        let sql_expr = v.to_sql(dialect)?;
                         Ok(SqlFunctionArg::Unnamed(FunctionArgExpr::Expr(sql_expr)))
                     })
                     .collect::<Result<Vec<_>>>()?;