It appears that this will do what you want. Have you tried it yet? Since it's not doing any data modification (INSERT/UPDATE) I'd imagine that this would be easy to test. (I love code like that :-)

You should really turn on RaiseError in your assignment to %attr, since you have zero error checking in all the DBI calls you have here. Otherwise something could fail silently and you'd have a difficult time finding out where/why.

Also, you might want to look at the code you pasted in and get rid of the extra spaces trailing each line. It's quite distracting.

Good luck!

Chris
M-x auto-bs-mode