I've been seeing this error periodically:

Notice: Undefined index: destination_error in _hybridauth_window_close() (line 97 of /sites/all/modules/hybridauth/hybridauth.pages.inc).

Not sure what's causing it, but in _hybridauth_window_close(), $destination_error is currently only pulled from $_GET, with no fallback if it's been messed up and not set. The attached patch uses $_GET if it's there, and falls back to the variable otherwise.

CommentFileSizeAuthor
hybridauth-destination_error.patch747 byteschellman

Comments

chellman created an issue. See original summary.

chellman’s picture

chellman commented
Issue summary: View changes

  • duozersk committed 2eb3632 on 7.x-2.x authored by chellman 
    Issue #2578641 by chellman: Destination error not always set in $_GET
duozersk’s picture

duozersk
he/him
Primary language Russian
Location Moscow
commented
Status: Needs review » Fixed

Joe,

Thank you for using the module and for the report.

I have just committed your patch. It's really strange that the destination_error is missing at this place in code, but we definitely can check it the way you did in the patch.

Thanks again, have a nice day!
AndyB

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.