Runorder and savedata code bug

I have found a bug in the code of chronocontact.php (version 2.5 J1.5 RC3.1).

Inside function uploadandmail() at about line 528 (the end of the function) the runorder is executed.
At the end of this for-loop the code checks if there is a redirect url filled in.
If this is the case the loop will end!
Only 1 of 3 runorder parts are executed.

I found this bug because my data suddenly wasn't stored anymore.
After a long debug session I found out that the Autogenerated block wasn't executed at all.

The sollution to this is to place this code outside the for-loop.

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

Hope it helps others to solve problems and the ChronoEngine team to improve their code.

Kind regard,

Erik Smit
TXO
The Netherlands

Hi txo,

Thanks, this is a known bug and there are many reports and several fixed versions of chronocontact.php here in the forums.

Hopefully Max will fix it soon in the downloadable version.

Bob