Agreed, maybe we don't need to update if nil. It was put in for completeness but the commit_avatax_invoice is only called at the order completion, unlike the calculator which is called over and over again.