Skip to content

Commit 19e88f7

Browse files
authored
Merge pull request #106 from filecoin-project/parsing-cases
Parsing datacap- On parsing error
2 parents 85a2f0b + 814e402 commit 19e88f7

File tree

1 file changed

+19
-14
lines changed

1 file changed

+19
-14
lines changed

fplus-lib/src/parsers.rs

Lines changed: 19 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -290,23 +290,26 @@ impl From<IssueValidData> for Datacap {
290290
datacap.total_requested_amount =
291291
format!("{}{}", datacap.total_requested_amount, value.0);
292292
}
293-
// Modified TotalRequestedAmount case
294-
ParsedApplicationDataFields::TotalRequestedAmount => {
295-
if let Ok(num) = value.0.parse::<f64>() {
296-
datacap.total_requested_amount = format!("{}{}", num, datacap.total_requested_amount);
297-
} else {
298-
log::error!("Failed to parse TotalRequestedAmount: {}", value.0)
293+
ParsedApplicationDataFields::TotalRequestedAmount => {
294+
match value.0.parse::<f64>() {
295+
Ok(num) => datacap.total_requested_amount = format!("{}{}", num, datacap.total_requested_amount),
296+
Err(_) => {
297+
log::debug!("Failed to parse TotalRequestedAmount: {}", value.0);
298+
datacap.total_requested_amount = "0".to_string();
299+
}
299300
}
300301
}
301302
ParsedApplicationDataFields::UnitSingleSizeDataset => {
302303
datacap.single_size_dataset =
303304
format!("{}{}", datacap.single_size_dataset, value.0);
304305
}
305306
ParsedApplicationDataFields::SingleSizeDataset => {
306-
if let Ok(num) = value.0.parse::<f64>() {
307-
datacap.single_size_dataset = num.to_string();
308-
} else {
309-
log::error!("Failed to parse SingleSizeDataset: {}", value.0)
307+
match value.0.parse::<f64>() {
308+
Ok(num) => datacap.single_size_dataset = num.to_string(),
309+
Err(_) => {
310+
log::debug!("Failed to parse SingleSizeDataset: {}", value.0);
311+
datacap.single_size_dataset = "0".to_string();
312+
}
310313
}
311314
}
312315
ParsedApplicationDataFields::Replicas => {
@@ -316,10 +319,12 @@ impl From<IssueValidData> for Datacap {
316319
datacap.weekly_allocation = format!("{}{}", datacap.weekly_allocation, value.0);
317320
}
318321
ParsedApplicationDataFields::WeeklyAllocation => {
319-
if let Ok(num) = value.0.parse::<f64>() {
320-
datacap.weekly_allocation = num.to_string();
321-
} else {
322-
log::error!("Failed to parse WeeklyAllocation: {}", value.0)
322+
match value.0.parse::<f64>() {
323+
Ok(num) => datacap.weekly_allocation = num.to_string(),
324+
Err(_) => {
325+
log::debug!("Failed to parse WeeklyAllocation: {}", value.0);
326+
datacap.weekly_allocation = "0".to_string();
327+
}
323328
}
324329
}
325330
ParsedApplicationDataFields::CustomMultisig => {

0 commit comments

Comments
 (0)