Problem with conditional_fields [#269145]

CRON says

PHP Warning: Call-time pass-by-reference has been deprecated; If you would like to pass it by reference, modify the declaration of conditional_fields_set_required_for_render(). If you would like to enable call-time pass-by-reference, you can set allow_call_time_pass_reference to true in your INI file. in /u/c20/ххх/htdocs/sites/all/modules/conditional_fields/conditional_fields.module on line 893

Warning: Call-time pass-by-reference has been deprecated; If you would like to pass it by reference, modify the declaration of conditional_fields_set_required_for_render(). If you would like to enable call-time pass-by-reference, you can set allow_call_time_pass_reference to true in your INI file. in /u/c20/ххх/htdocs/sites/all/modules/conditional_fields/conditional_fields.module on line 893

Help to correct a problem and Explain on what it influences.

Comment	File	Size	Author
#4	fix_required_display_2.patch	1014 bytes	threexk
#4

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

threexk CreditAttribution: threexk commented 11 June 2008 at 16:25

Attached is a patch that should fix the problem. It is just a call-by-reference where I improperly used the '&'; the module should probably still work fine with this warning. It was introduced by my recent patch to fix the placement of the required-field indication (asterisk).

Could you please tell me
1. Whether the patch works
2. Which PHP version you are using
3. Where you see this warning

I could not replicate this problem on my end. Maybe my PHP version is too old.

Comment #2

threexk CreditAttribution: threexk commented 11 June 2008 at 21:14

Status:

Active

» Needs review

Comment #3

deole CreditAttribution: deole commented 15 June 2008 at 02:01

1 Patch did not put
2 version PHP 5
3 This warning appears after start CRON comes on email

The support service of a hosting writes the following "In the beginning of a script put a command ini_set ("call_time_pass_reference", "1"); or esteem about this on the Internet."

Has made changes in conditional_fields.module

else if ($id = $item['#controlled_field']) {
ini_set("call_time_pass_reference","1"); - added
conditional_fields_set_required_for_render(&$item);
if ($item['#type'] == 'markup') {
/* Avoid unnecessary divs in multiple fields */
$item['#prefix'] = '

';
$item['#suffix'] = '

';
$output = drupal_render($item);
}
else {
$output = '

'. drupal_render($item) .'

';
}
}

Nothing has changed. Can it is not correct has changed a code