stream post : Notice: Undefined index: q in fb_stream_admin_settings() (line 64 of /home/xxx/domains/xxx.com/public_html/sites/all/modules/fb/fb_stream.admin.inc).

line 64 : $redirect_uri = url($_REQUEST['q'], array(
'absolute' => TRUE,
'query' => array(
'client_id' => $fb_app->id,
),
));

CommentFileSizeAuthor
#14 fb_1715310.diff2.91 KBDave Cohen
#4 fb_stream.admin_.inc_.patch1.77 KBinterlated

Comments

Dave Cohen’s picture

Dave Cohen commented
Category: bug » task

fb_stream.module needs upgrading to D7.

wanyen’s picture

wanyen commented

i use 7.x-3.3-beta3 and drupal 7.15

cimo75’s picture

cimo75 commented

Same issue here

interlated’s picture

interlated commented
StatusFileSize
new fb_stream.admin_.inc_.patch1.77 KB

As Dave says - fb_stream doesn't work in drupal-7. The $_REQUEST['q'] items need to be changed to request_path() for this specific bug. See attached.

cimo75’s picture

cimo75 commented

Sure, I ve misunderstood Dave s words. I could not apply the patch for fb-7.x-3.x-dev so i did it manually and the error disappeared but you mention in the patch

The $_REQUEST['q'] items need to be changed to request_path()

and then in patch 

-    $redirect_uri = url(request_path(), array(
+    $redirect_uri = url($_REQUEST['q'], array(

isnt it the other way round then?

Anyway, the post to Facebook checkbox is not appearing in my node add form.

tx
Simone

noahott’s picture

noahott commented

Has anyone found a solution in getting the "post to Facebook" options to show up when adding nodes?

cimo75’s picture

cimo75 commented

+1

Mackee’s picture

Mackee commented

Subscribe

Dave Cohen’s picture

Dave Cohen commented
Status: Active » Fixed

See if the .dev build works for you. fb_stream.module upgraded to D7.

Mackee’s picture

Mackee commented

Upgraded to latest dev, error still exist. Notice: Undefined index: q in fb_stream_admin_settings() (line 64 of /.../.../public_html/new/sites/all/modules/fb/fb_stream.admin.inc).

Status: Fixed » Closed (fixed)

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

Daniel Schaefer’s picture

Daniel Schaefer commented

same issue, using beta4

Dave Cohen’s picture

Dave Cohen commented
Assigned: Unassigned » Dave Cohen
Status: Closed (fixed) » Active

looking into it

Dave Cohen’s picture

Dave Cohen commented
Status: Active » Needs review
StatusFileSize
new fb_1715310.diff2.91 KB

I think this will finally get all the references to $_GET[q] upgraded to D7.

Dave Cohen’s picture

Dave Cohen commented
Version: 7.x-3.3-beta3 » 7.x-3.x-dev
Status: Needs review » Fixed

pushed changes.

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

(not verified) commented
Issue summary: View changes

insert code