hohn/codeql
1
0
Fork 0
mirror of https://github.com/github/codeql.git synced 2025-12-17 01:03:14 +01:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #11815 from aibaars/too-many-fields

Browse Source 
Ruby: use record_parse_error_for_node to report extractor error
This commit is contained in:
Arthur Baars
2023-01-09 15:40:19 +01:00
committed by GitHub
parent 5157d4df7b 799e0c1bcc
commit 664fdc3b2a
1 changed files with 13 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
ruby/extractor/src/extractor.rs
View File

@@ -500,13 +500,22 @@ impl<'a> Visitor<'a> {
} => { } => {
for (index, child_value) in child_values.iter().enumerate() { for (index, child_value) in child_values.iter().enumerate() {
if !*has_index && index > 0 { if !*has_index && index > 0 {
error!( let error_message = format!(
"{}:{}: too many values for field: {}::{}", "too many values for field: {}::{}",
&self.path,
node.start_position().row + 1,
node.kind(), node.kind(),
table_name, table_name,
); );
let full_error_message = format!(
"{}:{}: {}",
&self.path,
node.start_position().row + 1,
error_message
);
self.record_parse_error_for_node(
error_message,
full_error_message,
*node,
);
break; break;
} }
let mut args = vec![trap::Arg::Label(parent_id)]; let mut args = vec![trap::Arg::Label(parent_id)];