Call-time pass-by-reference has been deprecated

This issue was identified in http://drupal.org/node/269145 and I wrote a patch, but apparently my patch was incomplete. On that line you mention, the ampersand just needs to be removed. I'll post a new patch shortly.

I'm changing the priority on this bug to normal because the module still operates properly with the warning.

peterpoe: This issue is indeed a duplicate of http://drupal.org/node/269145, but by my mistake the patch for that issue only fixed one out of two warnings. That's why this issue was created. This line still needs its ampersand removed:
conditional_fields_set_required_for_render(&$item);

Can you make this change directly, or should I make a new patch?

Required fields in d5 are still buggy (see also #292044: Required fields within conditional groups not indicated as required). In d6 after many changes (in d6 the function conditional_fields_set_required_for_render does not even exist!) they work correctly, but the fixes have not been yet backported to d5. Will backport the fixes from d6 as soon as d6 is ready for release (it should not take long), but if you want to help with this you are welcome.

Just a reminder that this issue is still alive and well :)

abramo: In the meantime until the fix gets backported, you can just remove the ampersand in the two instances of the following text in conditional_fields.module:
conditional_fields_set_required_for_render(&$item);

many thanks - I just mentioned this as a mere reminder ;-)

Postponing like all 5.x-1.x-dev issues already fixed in v6.

Closing all 5.x issues.