Rather see a nested if to change the date and add it to the done list if it was actually done, instead of mixing control logic and action result into the same if.