Category Breadcrumb Exclude bug

Going to check later today (or maybe tomorrow). On first glance, I'm unsure what's the point of just moving the condition somewhere else, but maybe I'm missing something. Proper review coming soon :-)

Ah, yes... Now I see, that this is about the additional "add node title to breadcrumb" feature, that wasn't suppressed by the exclude setting. To be honest, this was initially somehow by design (I'm the author of category_breadcrumb initial code), as I wanted to have consistent format of breadcrumbs on all pages, even the excluded types, so I really did want to add title even to these. But such a setup is quite problematic anyway, as there is a bunch of other pages (non-node) not processed by category_breadcrumb. Breadcrumb consistency on these is left to theme layer already (and my theme have the needed workaround already, making the "feature" unneeded even on my site). Really, when it comes to page titles and breadcrumbs, Drupal is somehow unsolvable :-/

So, in the end I agree with this patch, because the "feature" intended here doesn't really work anyway, and an Exclude setting is quite probably expected to stop really all activity in the specified cases. That said, the patch is correct, works as desired. I tested it (and installed on my site like all the other tested Category patches) and found no problem.

Attaching the same patch rerolled from Category package root directory, per Drupal common practice. No new changes in the code.

Thanks. Committed to HEAD.