Please cut down on (comment out or delete) most of these debug statements unless you expect them to be useful for debugging later. This one should definitely be removed, and if you do, then I don't think there's any reason to have the "itemDone" handler at all