I looked back at some of your past posts before I made this post. I am not sure what your immediate problem is, but you have some serious scoping issues in your scripts. I am also not sure why you are so verbose with your execute statements. The delete could be rewritten as:

$sth = $dbh->prepare(qq!DELETE 
FROM $holding_table 
WHERE year=? AND
month=? AND
day=? AND
time_start=?
AND time_end=?
AND title=?");

$sth->execute(@key);
[download]

If there is a reason to use bind_param vs. this syntax someone please enlighten me, this is the way I have been doing it and it works for me, but TIMTOWTDI What are you doing at the bottom with @success_message? You assign a single value to it, but then does it get magically used by success_message? I say magicly because that is dangerous. You want to pass as much information into a sub as new each time. I would rewrite this as:

sucess_message( h2({-align=>'center'},"Database(s) updated. Thanks!") 
+);
[download]

or at the very least:

my @success_message = h2({-align=>'center'},"Database(s) updated. Than
+ks!");
sucess_message( @success_message );
[download]

Where is $sth getting scoped at?
What is your DBI connect string? (does it have RaiseError?)

Please don't misread my curiosity as critizism.
UPDATE: This was funny, Ovid and I were talking about the similarity of another post with his last meditation and now I have posted what sounds similar to Ovid's post on this node. Very odd day.