Warning message on a good install

tony2365 wrote on Friday, April 20, 2012:

After installing on a shared server everything seems to work fine except I am getting these messages:

Warning: Call-time pass-by-reference has been deprecated in /home/tonyrug/tonyruggiero-lmt.com/openemr/library/freeb/xmlrpcs.inc on line 498

Warning: Call-time pass-by-reference has been deprecated in /home/tonyrug/tonyruggiero-lmt.com/openemr/library/freeb/xmlrpcs.inc on line 501

Warning: Call-time pass-by-reference has been deprecated in /home/tonyrug/tonyruggiero-lmt.com/openemr/library/freeb/xmlrpcs.inc on line 538

tmccormi wrote on Friday, April 20, 2012:

You need to set your php.ini error reporting like this…

error_reporting = E_ALL & ~E_DEPRECATED & ~E_NOTICE

-Tony

tony2365 wrote on Friday, April 20, 2012:

Thanks…contacting the hosting company to see if they can fix that for me.

kevmccor wrote on Tuesday, April 24, 2012:

What about the php.ini directive:

allow_call_time_pass_reference = Off/On

Mine is currently “Off” and I have tons of “Call-time pass-by-reference” errors.  The ~E_DEPRECATED option is not in the php.ini file comment on error handling and logging.

Thanks

yehster wrote on Wednesday, April 25, 2012:

http://www.php.net/manual/en/ini.core.php#ini.allow-call-time-pass-reference

allow_call_time_pass_reference On will get rid of the deprecated warnings, but won’t really change functionality.

If we want to be compatible with php 5.4, we need to try and clean them up,
http://www.php.net/manual/en/language.references.pass.php
But if folks are running 5.3 or earlier and have display_errors off, the warnings don’t change functionality, they just tell us that us a PHP language feature which is going away was used.

kevmccor wrote on Wednesday, April 25, 2012:

setting allow_call_time_pass_reference = On seems to have helped.  The httpd/error_log is a much smaller file.  I really ignored it for the longest time, but it was an annoyance. The  /library/freeb/xmlrpcs.inc produced literally gigabytes of error messages. 

yehster wrote on Wednesday, April 25, 2012:

https://github.com/yehster/openemr/commit/fc9b0da37ac9e0a2d6ca045c4590f9ab4bd88a2e
This fixes the deprecated syntax that shows up on initial login.

I’m sure there are other instances, but I don’t plan on tracking them down right now.

yehster wrote on Wednesday, April 25, 2012:

https://github.com/yehster/openemr/commit/8b8b2d7cbc939a8fd3ee1fa9e70489552e1444f5
Fixed the syntax in gen_hcfa too.

yehster wrote on Thursday, April 26, 2012:

I’ve pushed these changes into official.