It may or may not help in this case, but it is a good idea in general to 'use strict', and also check the return value of all DBI statements (connect, prepare, and execute at the very least), or better yet, use 'RaiseError=>1' in the connect %attr hash.

I also notice 'connect' is not spelled correctly, is this a typo here, or something 'use strict' would have caught?

(Update:Nevermind, it ought to be obvious if this was the problem even without 'use strict').

You also have no 'where' clause on your second SQL statement. This would have been caught with RaiseError or checking the return value of the prepare.