screenshot: http://prntscr.com/20g5u

after upgrading advanced forum to the latest Jun-7-2011 one i got this gravatar error. but gravatars still work on the site.. not sure what the error means:

Notice: Undefined property: stdClass::$mail in _gravatar_get_account_user_picture() (line 181 of /home/xx/public_html/dev/sites/all/modules/gravatar/gravatar.module).

Comments

asrob’s picture

subscribe

Narno’s picture

Title: Undefined property: stdClass::$mail in _gravatar_get_account_user_picture() (line 181 » Undefined property: stdClass::$mail in _gravatar_get_account_user_picture()
Assigned: Unassigned » Narno
Narno’s picture

Status: Active » Needs review
patoshi’s picture

awesome.. fixed...

go into gravatar.module

change line 183:
183 return gravatar_get_gravatar($mail);

to this:
183 return gravatar_get_gravatar($account->mail);

the problem was the $mail wasnt updated updated to the object reference.

BenK’s picture

The solution posted in #4 worked for me. With the change, the module works great.

It's a very small fix... does someone want to post a proper patch so that we can get this officially committed? Or since the change only involves one line, can the module maintainer just make the fix without a patch?

--Ben

asrob’s picture

Status: Needs review » Reviewed & tested by the community

If it works, then it is ready to rtbc.

coloryan’s picture

Agreed, #4 fixed the error for me.

Narno’s picture

Status: Reviewed & tested by the community » Closed (fixed)