Can you please put a print statement just prior to the for loop, and let us know how many times the this new print statement fires?
(Also, with your connections thing - it is easier if you just use :Once in place of :Connect, as it will automatically disconnect it after a single activation)