Block Title

Okay I have a fix to this issue:

I am not sure how to create patches as I am green to module development. However, I looked at the API and a tutorial on creating module and found a fix to this.

On line 88 of facebook_pull.module the block is declared with this code

function facebook_pull_block_view($delta = '') {
  $block = array();
  
  switch ($delta) {
    case 'facebook_pull_default':
      $block = array(
        'title' => t('Facebook stream'),
        'content' => _facebook_pull_render_block(),
      );
      break;
  }
  
  return $block;
}

The line that sets the title is:

'title' => t('Facebook stream'),

When it is set, we will not be able to change the title in the configuration screen. However, if we change 'title' to 'subject' , we will be able to change the title. The block will use the subject as the title if we do not set a title in the configuration screen.

Here is the change:

'subject' => t('Facebook stream'),

After making this change, my new code looks like this:

function facebook_pull_block_view($delta = '') {
  $block = array();
  
  switch ($delta) {
    case 'facebook_pull_default':
      $block = array(
        'subject' => t('Facebook stream'),
        'content' => _facebook_pull_render_block(),
      );
      break;
  }
  
  return $block;
}

It seems to be working fine. Does anyone see any issues with this change that will effect the module?