visforms 3.2.1 suggested fix

I am using visforms 3.2.1 with Joomla 3.3.6, visforms is a fine product. I applied a small fix to two files with the same issue which was a warning that an index had not been initialized.

The same fix can be applied to both frontend files - the fix is:
if(!isset($attributeArray)) $attributeArray = ""; // initialize so that concatenate will work

The files are:
/com_visforms/lib/html/field/select - apply fix after line 35 before the foreach

/com_visforms/lib/html/field/text - apply fix after line 50 before the foreach

Hi,

thanks for that post.
Actually $attributeArray is initialized as an empty arrray just 4 lines above the line you mention.
And because it is an array I would like it to be initialized as an array.

So, could you please post me the initial warning you received?

Regards,
Aicha

Thanks for getting back to me so quickly Aicha. I experienced the same issue with radio.php yesterday, here are the warning messages:

Notice: Undefined index: class in C:\xampp\htdocs\blankets_3\components\com_visforms\lib\html\field\radio.php on line 83

Notice: Undefined index: class in C:\xampp\htdocs\blankets_3\components\com_visforms\lib\html\field\radio.php on line 83

Line 83 is concantenating a value to the array but there is nothing to concatenate to for the index class.

I applied the same fix and it works fine.

Regards,
Byron

Hi,

thanks for your help. The text of the original notice is helpful.

Did you make any changes to the visfield.xml file in administrator/components/com_visforms/models/forms?
Which form layout (advanced form settings) do you use with the form?

Could you please try if you replace the first line in the getFieldAttributeArray which at the moment is with the following code that fixes the problem, too.

Regards,
Aicha

Hi Aicha,

I haven't made any changes to your code or files other than my suggested fixes.
I tried your fix and it works as well.
The form layout I am using is Bootstrap horizontal layout with Load Bootstrap form CSS turned on.

Regards,
Byron

Hi Byron,

thanks for the information!

Actually it is very weird, that you get the Notice after all, because the visfields.xml makes sure, that the index class is set for each field.

I've tested extensively on my developing systems with all debug messages turned on and never receive that notice.
I think you are on a local xampp because on a productive system you would not turn message on a notice level on, is that correct? Which php version are you on? Mabe there is a problem with the xampp settings?

The behaviour of the select box, you have described in the other post, is not as expected too.

I would really like to take a look at the form, is there any chance?
Regards,
Aicha