I'd look to exceptional cases as the problem since the logic looks ok (especially with the added parens).

What happens if there are no rows returned?

What happens if a DBI error occurs?

I'll try to take more detailed look later, but it's been my experience that DBI can throw you some curves that are less than obvious.