This worked ok for me on 5.6.1. One observation, though:

You should take the calls to run_on_start and run_on_finish and move them before the foreach loop. This way they are not being redefined with the same thing in each iteration.