HEADER

There is a bug with the redirection: if I set a redirection url, autogenerated code is not executed.

I corrected it by moving the following lines of chronocontact.php:

Code: Select all

/**
* Redirect the page if requested
*/
if ( !empty($rows[0]->redirecturl) ) {
   $mainframe->redirect($rows[0]->redirecturl);
}


out of the for($ixx = 1 ; $ixx <= 3; $ixx++){[..]} statement.

Hi Rohan,

If you do a redirect ChronoForms assumes that you will handle all the form processing yourself - no email is sent. no data is saved, no thank-you page will be shown.

If you want the form processed and then to redirect, add a little redirect code at the end of the Autogenerated Code box.

Bob

I thought that the redirect url was for a redirection after processing, and that the submit url was here in case I wanted to handle all the processing.

anyway, i think it's strange that the redirection is made inside this for(){}, because it will do the first action of the "runorder" tab then redirect, and just drop the two others?

Hi Ronan,

Sorry, you are right. Mental aberration on my part. But I don't recognise this

Code: Select all

for($ixx = 1 ; $ixx <= 3; $ixx++){[..]}

Which version are you using?

Bob

I use ChronoForms V2.5 J1.5 RC3.1.

The for loop is lines 535 to 581.

Hi Ronan,

Thanks, I need to install 3.1 and see what's happening.

Bob

Hi Rohan,

Thank you, bug confirmed. I've fixed the installer and upgrade packages and there's an updated chronocontact.php posted in this thread.

Sorry it took a while for me to get to it.

Bob

I can confirm this bug in ChronoForms V2.5 J1.5 RC3.1.

This one really made me sweat for a while...

Ronan, thanks for the fix! :)

Hi Bob,

we bought the stuff, and the installer contained the chronocontact.php with the same bug...
I upgrade it to the good version, so the stuff is very good, and useful.

thx

tamas