Code review comment for lp:~therp-nl/therp-backports/server-6.1-lp923645-unlink_timesheet_lines_fails

Approve, explanation by Stefan is clear enough.

Just wondering why in the original code toremove is not simply an array that is filled with all id's to remove, but a dictionary that is later reduced to an array with the keys() method....