Looking great Quicksketch.

Found an issue in fboauth.fboath.inc. Line 74 calls:

$fbuser = fboauth_graph_query('me', $access_token);

but the fboauth_graph_query function wants 3 arguments:

function fboauth_graph_query($id, $access_token, $parameters) {}

The $parameters argument doesn't seem to be used in the function so I just removed it and all works well. Not sure if this is correct way to approach this as you may have intentions for that arg that just haven't been implemented yet.

CommentFileSizeAuthor
#1 fboauth_parameters.patch843 bytesquicksketch

Comments

quicksketch’s picture

Status: Active » Fixed
StatusFileSize
new843 bytes

Thanks for your review JaceRider. Nice to have some feedback on this not-quite-official module. I've fixed it with this patch. Committed and pushed.

Status: Fixed » Closed (fixed)

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