PHP errors in the logs [MERGED]

[Mahesh Slaria]

Hi,

Please make sure that you set "allow_call_time_pass_reference ON" in your php.ini on your server.

Please let me know if issue persists.

Thanks.

Regards,

[bear]

Is it not possible instead to fix the issue instead of having me change a setting that affects the entire server this is installed on?

Asking all Kayako users to change thier ini setting instead of the script author correcting the use of a deprecated element in thier scripting seems the wrong way around to me, and making this change is simply hiding the error instead of correcting it.
Won't you fix it instead?

[Link]

I agree with Bear - update the script instead of having us allow deprecated features; it may have been deprecated due to a security vulnerability so it's not in our interest to enable this legacy feature.

What I'm curious about, is what is happening at the moment on the server? If the errors are being reported, is something important not being completed? What is it?? Presumably, we have to edit the php.ini to make the script work correctly - so what is it that isn't working? We don't know how important this is.

---

FYI, here's my setup and the two similar errors (always line 815 and 977). When it occurs, there'll be a run of about 8 to 15 of them over about 2 or 3 seconds.

Windows 2003 SP1
PHP 5.2 build 3790
Zend Engine v2.2.0
Zend Optimizer v3.2.2

[19-Mar-2007 11:26:47] PHP Warning: Call-time pass-by-reference has been deprecated; If you would like to pass it by reference, modify the declaration of parseCFData(). 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 D:\inetpub\...\includes\functions_customfields.php on line 815

[19-Mar-2007 11:26:47] PHP Warning: Call-time pass-by-reference has been deprecated; If you would like to pass it by reference, modify the declaration of parseCFData(). 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 D:\inetpub\...\includes\functions_customfields.php on line 977

[bear]

As a followup, Kayako fixed this once before when it came up, why not this time?
allow_call_time_pass_reference PHP setting

[Jamie Edwards]

Hi everyone,

If you do a Google for the problem, you will the function deprecation has caught many developers by surprise. Fortunately, it has not depreciated due to any immediate security issue and is still intended for mainstream use for some time.

The methods causing the issue are not so much obsoleted. In it's time, use of the method would have improved script efficiency.

When the function was deprecated the php.ini setting
allow_call_time_pass_reference
was added and set to ON by default. Only on systems where this setting is missing (due to php.ini preservation) or set to OFF, will the warnings be produced.

There will be ways around it and the developers are looking at it. In the mean time, however, the recommended course of action is to change this php.ini setting to ON. These notes will also be added to the setup and install documentation for the products.

[bear]

Instead of changing the setting for the whole server, I found the use of an .htaccess file seems to work. I don't see any errors since adding this in the working Kayako directory root:

php_flag allow_call_time_pass_reference On

I'll check later today to make certain, but so far this is working for me.
Be nice to have it fixed for real.

[Jamie Edwards]

Thanks for the suggestion bear - it's a good one. I believe this is an Apache configuration dependant feature (PHP setting permissions), but it works on most web servers it seems.