Fix money on hand boolean PDF mapping #286
Conversation
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| } | ||
| Map<String, Object> inputData = submission.getInputData(); | ||
| if (inputData.containsKey("expeditedMoneyOnHandAmount")) { | ||
| double expeditedMoneyOnHandAmount = Double.parseDouble(inputData.get("expeditedMoneyOnHandAmount").toString()); |
Check notice
Code scanning / CodeQL
Missing catch of NumberFormatException Note
There was a problem hiding this comment.
expeditedMoneyOnHandAmount may be set to "". We need to account for that here. This is what I did in one of the actions that uses it:
Funny, though, I was just looking at the MdBenefitsFlow.java file and I see that it is required, so now I'm wondering what I was thinking about with that logic I linked to above. I wonder if people can enter " " and move forward.
There was a problem hiding this comment.
Actually, I sorted this out and it was unrelated to this. I think you're change is fine. The field is required and and you check to see that it is set before grabbing it. I think we will not see NumberFormatExceptions as a result of this code.
No description provided.