Sugarbean.php deprecated

[xviperforcex]

I just found while debugging another problem a log message that appears every minute in log file of sugar.
We are using Sugar 6.0.1.

That's the message:

Fri Nov 5 16:54:02 2010 [27652][1][DEPRECATED] SugarBean.php: preprocess_fields_on_save() is deprecated
Fri Nov 5 16:54:02 2010 [27652][1][DEPRECATED] Formatting correction: SchedulersJobs->date_entered had formatting automatically corrected. This will be removed in the future, please upgrade your external code
Fri Nov 5 16:54:02 2010 [27652][1][DEPRECATED] Formatting correction: SchedulersJobs->date_modified had formatting automatically corrected. This will be removed in the future, please upgrade your external code
Fri Nov 5 16:54:02 2010 [27652][1][DEPRECATED] Formatting correction: SchedulersJobs->execute_time had formatting automatically corrected. This will be removed in the future, please upgrade your external code
Fri Nov 5 16:54:02 2010 [27652][1][DEPRECATED] SugarBean.php: preprocess_fields_on_save() is deprecated

I did no changes in SugarBean.php. What is meant with external code? Shall I upgrade Sugar or some other code?

Thx in advance!

[robertbmirth]

I've had similar date/time formatting issues. Unless you're calling these functions directly I wouldn't worry too much - if the deprecated call is coming from somewhere inside the sugar core, then its Sugar Inc's job to modify these calls to remove the deprecated call when (if) they remove the actual function.

[xviperforcex]

I've had similar date/time formatting issues. Unless you're calling these functions directly I wouldn't worry too much - if the deprecated call is coming from somewhere inside the sugar core, then its Sugar Inc's job to modify these calls to remove the deprecated call when (if) they remove the actual function.

You're right but this minute-by-minute log entry avoids catching other more important messages. We don't want to increase the number of log files or their size just because of this message. Any ideas to solve the problem at its root?

[robertbmirth]

Hmm... this is kind of interesting.

So, if you want to kill it at its source, you're going to have to modify <sugar_root>/data/SugarBean.php. On line 2052 you'll find the function you're looking for:

     function preprocess_fields_on_save()
    {
        $GLOBALS['log']->deprecated('SugarBean.php: preprocess_fields_on_save() is deprecated');
    } 
  

The interesting thing is that this function is doing nothing. Its already completely deprecated, so idk why Sugar is still calling it. Maybe someone didn't call housekeeping? Anyway, if you just comment out the log call in this function it will/should stop logging it.

I say will/should because I haven't done it; I'm very weary of making any changes to core sugar files. SugarBean.php is the core of cores - its like the Java Object class of SugarCRM. That being said, if your other, more important log calls are of type 'error' or higher, rather than changing this you should go to Admin->System Settings, scroll down to Logger settings and switch your logging level to 'error' or higher. If they're on a lower level, then if you make this change to SugarBean.php remember that it isn't upgrade safe so you may have to make it again every time you upgrade until they stop calling the function.

*edit*
I totally forgot about the second deprecated call. its coming from line 2174 in the same file. Looks like this:

             if ( $reformatted ) {
                $GLOBALS['log']->deprecated('Formatting correction: '.$this->module_dir.'->'.$field.' had formatting automatically corrected. This will be removed in the future, please upgrade your external code');
            } 
  

Do the same thing, comment out the log call, to get rid of it if you decide not to change your logger settings.

[russells]

Just a quick note that in that same file on line 1219 is the call to that empty deprecated function which you can comment out.