THEMA:

Hi all,

I've been running into a bug where mailback mail is not sent if the related "send a copy to my email" checkbox is named with an underscore (which is permitted as indicated by the backend).

This happens in current version "BreezingForms 1.8.3 Stable (build 831)" in line 5335 with the following code:

$yesno = explode('_', $accept[$row->name]);

where the intention was to split the first part (which seems to be 'true') from the checkbox name defined for the "send a copy to my email" checkbox where the delimiter is an underscore. The code assumes something like 'true_myCheckboxName'.

The second part of the string ('myCheckboxName') is used to get the submitted value of the form.

This fails if the checkbox name contains underscores (e.g. 'my_checkbox_name') because the explode statement then splits the string into multiple parts leading the code to search after the submitted checkbox with name 'my' and not as it should be 'my_checkbox_name'.

The fix is to not use explore but something like preg_split which allows to split on the first occurance of an underscore (line 5335 of facileforms.process.php):

$yesno = preg_split('/_/', $accept[$row->name], 2);

Find a (svn -Naur) patch attached to this subject fixing the bug.

Regards
Dietrich

Hi,

yeah thank you! We have a similar bug for html5/flash uploads and will fix that quick and dirty solution.

Regards,
Markus

Hi Markus,

when can we expect a new version containing the fix?

Regards.

Hello,

not sure how long it will take. Markus is working on it and this fix is on his todo list.