4.2.1 = Broken incoming Emails

[alsutton]

Did the update, got the following from the cron job;

PHP Fatal error: Call to a member function on a non-object in /home/httpd/vhosts/argosytelcrest.co.uk/httpdocs/crm/data/SugarBean.php on line 779

Thought, Oh well, thats Sugar Quality for ya, so reinstaled with the full package, guess what, output from cron;

PHP Fatal error: Call to a member function on a non-object in /home/httpd/vhosts/argosytelcrest.co.uk/httpdocs/crm/data/SugarBean.php on line 779

My schedulers list is empty apart from the job to poll monitored incoming mailboxes. If I take that out the error goes away.

So much for managing our inbound mail using it.

[sugarchris]

line 779 is a curly brace for me - what's the code that is erroring?

[alsutton]

In the 4.2.0 -> 4.2.1 patch SugarBean.php has the following on line 779;

$query .= $field."='".$GLOBALS['db']->quote(from_html($this->$field))."'";

[sugarchris]

ok, I think I know what the error is. we refactored quite a bit of DB layer code for preparation for SQL Server and Postgres/EDB integration, and there may be a missing link there.

I can't be exactly sure where the fix should go, so i'll forward this to the guy most responsible for the code changes. He should get in here and offer a solution once the holiday is done.

[mmgareth]

I got the same problem I look forward to testing the fix


Gareth

[lbarth]

Hi,

I've just upgraded to 4.2.1 and had this problem. To fix it replace:

File: data/SugarBean.php
Line:799

 $query .= $field."='".$GLOBALS['db']->quote(from_html($this->$field))."'"; 
  

with:

 $query .= $field."='".$this->db->quote(from_html($this->$field))."'"; 
  

This is working for me here. I did not work with sugar befor so i might be wrong but it look like that $GlOBALS['db'] got relaced with $this->db

Hope this helps

Bye
Lorenz

[lukemilton1]

Seems to be working for me as well. Thanks

[mmgareth]

File: data/SugarBean.php
Line:799

I think this is line 779, but it works for me too Thanks a lot!

Anyone know the official verdict from the Sugar team?



Gareth

[alsutton]

I think this is line 779, but it works for me too Thanks a lot!

Anyone know the official verdict from the Sugar team?



Gareth

The official response seems to be the usual.... tell it's going to be fixed then tell us nothing, not even confirmation that Lorenz's fix worked

And yup.. it worked for me....

[jstevensr]

Worked for me On win XP with XAMPP.
SugarBean.php file located here:
C:\Program Files\xampp\htdocs\SugarSuite-Full-4.0.0\data
Line to change: 779, not 799.
also, make a copy of the SugarBean.php from the 4.21 zip archive, then modify it, then rename old file then rename changed file to SugarBean.php.

-Steve