user/{uid}/edit generated as fbml gives the runtime error:

fb:tab-item: You can use this tag only from inside of tags: fb:tabs

although your function looks ok...

function fb_fbml_menu_item_link($link) {
  global $_fb_fbml_state;

  if ($_fb_fbml_state == 'tabs') {
	// Theme an FBML tab
	$output .= "<fb:tab-item href=\"".url($link['href'])."\" title=\"".$link['title']."\" selected=\"false\">".$item['title']."</fb:tab-item>\n";
	return $output;
  }
  else
	return theme_menu_item_link($link, $active);
}

shrug...

Comments

psi-borg’s picture

psi-borg commented

for user account tabs,

first set is: {view, edit, ...}

if you click edit for the next layer in the menu tree, only the account tab-items are displayed, with no link to any other profile.module created tabs...

only way to get to these is by creating a separate link for them. any remedy in the near future floating this way? =)

Dave Cohen’s picture

Dave Cohen commented

Yeah that's a bug. I don't have a fix on hand. If you submit patch I will commit!

psi-borg’s picture

psi-borg commented

Dave, all i have to say is =P and =P ...oh yeah and mebe =P

your template.php comments weren't very encouraging so i gave myself even lower odds for figuring it out.

psi-borg’s picture

psi-borg commented

@ http://www.drupalforfacebook.org/node/1234

how about ummmm

1. drupalpwnsfarceborg.org
2. farceborgsucksdingleberries.org
3. zuckerborgisgay.org
4. zuckerborghasnoanus.org
5. zuckerborghashisheadsofarupgatesasswhenbillbrusheshisteethzuckerborggivesthecourtesyreacharoundfloss.org

psi-borg’s picture

psi-borg commented

ooooh mebe ummmmmmm

EDIT: wastingmytimeatdrupalwhilezuckerborglaughsonhispileofcash.org?

Dave Cohen’s picture

Dave Cohen commented

All good ideas. :)

Dave Cohen’s picture

Dave Cohen commented
Status: Active » Fixed

Ok this tabs thing was bothering me so I worked around the problem. The problem will not be optimally fixed until either #521852: Local tasks lack semantic markup to indicate an active task or #371453: theme_menu_local_task does not differentiate between primary and secondary local tasks is fixed. In the meantime my fix use a preg_replace hack which I think will be pretty sturdy.

Let me know if it works for you.

Status: Fixed » Closed (fixed)

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