aljaz/turso
1
0
Fork 0
mirror of https://github.com/aljazceru/turso.git synced 2026-01-29 04:44:23 +01:00
Code Issues Packages Projects Releases Wiki Activity

Fix condition that checks table.cols against number of provided values

Browse Source
This commit is contained in:
Jussi Saurio
2025-08-21 15:02:13 +03:00
parent b5bd31a47b
commit cc28b8833e
1 changed files with 8 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
core/translate/insert.rs
View File

@@ -741,13 +741,16 @@ fn build_insertion<'a>(
if columns.is_none() {
// Case 1: No columns specified - map values to columns in order
if num_values != table_columns.iter().filter(|c| !c.hidden).count() {
crate::bail_parse_error!(
"table {} has {} columns but {} values were supplied",
&table.get_name(),
table_columns.len(),
num_values
);
}
let mut value_idx = 0;
for (i, col) in table_columns.iter().enumerate() {
if value_idx == num_values {
// If there are less values than columns, the rest will have value_index = None,
// meaning NULLs will be emitted for them.
break;
}
if col.hidden {
// Hidden columns are not taken into account.
continue;
@@ -763,15 +766,6 @@ fn build_insertion<'a>(
}
value_idx += 1;
}
if value_idx != num_values {
crate::bail_parse_error!(
"table {} has {} columns but {} values were supplied",
&table.get_name(),
num_values,
value_idx
);
}
} else {
// Case 2: Columns specified - map named columns to their values
// Map each named column to its value index