Raise errors instead of unwrapping in document (#106)
parent
50809a186d
commit
bbcd1f2aef
|
@ -427,10 +427,14 @@ impl Document {
|
||||||
/// Args:
|
/// Args:
|
||||||
/// field_name (str): The field for which we are adding the bytes.
|
/// field_name (str): The field for which we are adding the bytes.
|
||||||
/// value (str): The json object that will be added to the document.
|
/// value (str): The json object that will be added to the document.
|
||||||
fn add_json(&mut self, field_name: String, json: &str) {
|
///
|
||||||
|
/// Raises a ValueError if the json is invalid.
|
||||||
|
fn add_json(&mut self, field_name: String, json: &str) -> PyResult<()> {
|
||||||
let json_object: serde_json::Value =
|
let json_object: serde_json::Value =
|
||||||
serde_json::from_str(json).unwrap();
|
serde_json::from_str(json).map_err(to_pyerr)?;
|
||||||
self.add_value(field_name, json_object);
|
self.add_value(field_name, json_object);
|
||||||
|
|
||||||
|
Ok(())
|
||||||
}
|
}
|
||||||
|
|
||||||
/// Returns the number of added fields that have been added to the document
|
/// Returns the number of added fields that have been added to the document
|
||||||
|
|
Loading…
Reference in New Issue