not completed but commiting anyway

Same thing here, You're already getting a decimal from elsewhere so you don't have to recast everything as a decimal but you should prepare for what happens if None is returned.