fboauth_block_view() checks if the $app_id variable exists but this is not defined in the current scope. This probably is some leftover cruft.

Also $delta is not used, but as this is part of the standard hook implementation this should perhaps not be removed.

function fboauth_block_view($delta) {
  $block = array();
  $app_id = isset($app_id) ? $app_id : variable_get('fboauth_id', '');
  if ($app_id && !fboauth_fbid_load()) {
    $block['content'] = fboauth_action_display('connect');
  }
  return $block;
}

Comments

pfrenssen’s picture

Status: Active » Needs review
StatusFileSize
new846 bytes

Here's a patch. I've left the $delta in the function declaration.

quicksketch’s picture

Status: Needs review » Fixed

Thanks, committed!

Status: Fixed » Closed (fixed)

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