GitHub/Anki
1
0
Fork 0
mirror of https://github.com/ankitects/anki.git synced 2025-09-18 22:12:21 -04:00
Code Issues Projects Releases Packages 1 Wiki Activity Actions

save filters and field name separately

Browse source
This commit is contained in:
Damien Elmes 2020-01-16 17:44:26 +10:00
parent bdac937802
commit cddfddf1c8
2 changed files with 18 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
rslib/src/err.rs
View file

@ -26,7 +26,7 @@ pub enum TemplateError {
NoClosingBrackets(String),
ConditionalNotClosed(String),
ConditionalNotOpen(String),
FieldNotFound(String),
FieldNotFound { filters: String, field: String },
}
impl From<TemplateError> for AnkiError {
@ -40,9 +40,9 @@ impl From<TemplateError> for AnkiError {
TemplateError::ConditionalNotOpen(tag) => {
format!("missing '{{{{#{}}}}}' or '{{{{^{}}}}}'", tag, tag)
}
TemplateError::FieldNotFound(field) => format!(
"found '{{{{{}}}}}', but there is no field called '{}'",
field, field
TemplateError::FieldNotFound { field, filters } => format!(
"found '{{{{{}{}}}}}', but there is no field called '{}'",
filters, field, field
),
},
}

19
rslib/src/template.rs
View file

@ -330,14 +330,17 @@ fn render_into(
Some(text) => apply_filters(text, filters, key, context),
None => {
// unknown field encountered
let name_including_filters = filters
let filters_str = filters
.iter()
.rev()
.cloned()
.chain(iter::once(*key))
.chain(iter::once(""))
.collect::<Vec<_>>()
.join(":");
return Err(TemplateError::FieldNotFound(name_including_filters));
return Err(TemplateError::FieldNotFound {
field: (*key).to_string(),
filters: filters_str,
});
}
};
@ -726,14 +729,20 @@ mod test {
tmpl = PT::from_text("{{X}}").unwrap();
assert_eq!(
tmpl.render(&ctx).unwrap_err(),
TemplateError::FieldNotFound("X".to_owned())
TemplateError::FieldNotFound {
field: "X".to_owned(),
filters: "".to_owned()
}
);
// unknown field with filters
tmpl = PT::from_text("{{foo:text:X}}").unwrap();
assert_eq!(
tmpl.render(&ctx).unwrap_err(),
TemplateError::FieldNotFound("foo:text:X".to_owned())
TemplateError::FieldNotFound {
field: "X".to_owned(),
filters: "foo:text:".to_owned()
}
);
// a blank field is allowed if it has filters