Yep....
A little logic bug.
In the subroutine:
sub site_html_recc_success {
Make the last
"print $output;"
to:
"return $output;"
The reason is that with the "print" the subroutine returns a '1' indicating success, and the program should end. That works when the output of the form is the "last" thing you want to do --- ie: in the input form.
But, since we want the option of post-processing (sending the user somewhere else) with this form, "return" it and the program will generate it's own "1" for success on termination.
I _think_ that's what happening, but changing the "print" to "return" fixes it
I've fixed up all the files.