More specific breadcrumbs with tasks

I've implemented this :
Storm > Projects > My Specific Project > Tasks

Sorry to re-open this but I still have issue with task breadcrumb.

When I'm viewing a task (or editing it), I just have :
Storm > tasks

When I had a look at the code, I found 2 issues :
1. a first trail is set in stormtask_form which is just using $_GET['project_nid'] which is only set when creating a new task.
2. there is no project part in the trail set in stormtask_view

I fixed this by adding this function :

function _storamtask_build_breadcrumb($node) {
  $breadcrumb = array();
  $breadcrumb[] = l(t('Storm'), 'storm');
  
  if (array_key_exists('project_nid', $_GET)) {
    $breadcrumb[] = l(t('Projects'), 'storm/projects');
    $project=node_load($_GET['project_nid']);
    $project_nid = $project->nid;
    $project_title = $project->title;
  } else {
    $project_nid = $node->project_nid;
    $project_title = $node->project_title;
  }
    
  $breadcrumb[] = l(t('Projects'), 'storm/projects');
  $breadcrumb[] = l(t('Project : !project', array('!project' => $project_title)), "node/{$project_nid}");
  $breadcrumb[] = l(t('Tasks'), "node/{$project_nid}/tasks");
  
  drupal_set_breadcrumb($breadcrumb);
}

And replacing both breadcrumb building by a call to it.

This way it works in every situation.

Please could you post this as a patch? (http://drupal.org/patch/create)

Also, it will need to be against the current -dev version.

Here it is

#5: stormtask-6.x-dev-breadcrumb.patch queued for re-testing.

#5: stormtask-6.x-dev-breadcrumb.patch queued for re-testing.

I'd love to see "Home" added to the breadcrumbs.