Breadcrumb issues

Problem/Motivation

I'm experiencing issues on my site related to breadcrumbs, and tracked down the issue to your module.

The MailJet module has code which duplicates the last breadcrumb path. I'm guessing the intention is so that the breadcrumbs will display the full path including the current page.

The code is in mailjet.module

/**
 * Implements hook_menu_breadcrumb_alter().
 */
function mailjet_menu_breadcrumb_alter(&$active_trail, $item) {
  global $base_url;
  $active_trail = array_merge(array_slice($active_trail, 0, count($active_trail)), array($active_trail[count($active_trail) - 1]));
}

Unfortunately, this code has a few of issues:
1 ) It does not check if the last path has a title, so empty breadcrumb paths are printed. You can see this if you visit a path like http://example.com/node
2 ) This code interferes with modules that control the display of breadcrumbs such as crumbs. With crumbs module installed, I see the last breadcrumb item twice.

Ultimately, the mailjet module should not be altering breadcrumbs. That behavior is beyond the scope of the module, and there is no necessity by the mailjet module to make such a broad change to an entire website. If webmasters want to see the current page title in the breadcrumb, they can install one of the many other breadcrumb modules.

Proposed resolution

This code should be removed entirely, and a new release should be made ASAP since it has such a wide-sweeping impact across all websites where the MailJet module is installed.