Admin Breadcrumbs broken (site recently upgraded to 1.6.1)

[tinhat]

This site was recently upgraded from 1.5.4 to 1.6.1

[edited - slightly solved, slightly different problem]

The problem is that the second level breadcrumbs are not displaying a hyperlink. This kind of breaks breadcrumb navigation under modules such as Newsletter Made Simple (NMS). For example,  if I went deep into a modules admin pages (lets say I am creating a job under the NMS module) then the breadcrumb will be:

Extensions » Newsletter Made Simple

Previously the "Newsletter Made Simple" would be hyperlinked but now it isn't.

Cheers
Mark
---------------------------------------------
Site Admin » System Information

"System Information" hyperlink will be "http://domain/admin"

Same problem for all modules breadcrumbs.

Any suggestions?

Thanks
Mark

Cms Version: 1.6.1

Installed Modules:
CMSMailer: 1.73.14
FileManager: 1.0
MenuManager: 1.6.1
ModuleManager: 1.3.1
News: 2.10.1
nuSOAP: 1.0.1
Printing: 1.0
Search: 1.6.1
ThemeManager: 1.1.1
TinyMCE: 2.5.1
CodePress: 0.1.2
CGExtensions: 1.16
SiteMapMadeSimple: 1.1.4
Album: 0.9.3
TinyMCE: 2.5.0beta3
NMS: 2.2.2


Config Information:
php_memory_limit:
process_whole_template: true
max_upload_size: 2000000
default_upload_permission: 664
assume_mod_rewrite: true
page_extension:
internal_pretty_urls: false
use_hierarchy: true


Php Information:
phpversion: 5.2.6
md5_function: On (True)
gd_version: 2
tempnam_function: On (True)
magic_quotes_runtime: Off (False)
memory_limit: 32M
max_execution_time: 60
safe_mode: Off (False)
session_save_path: /var/lib/php/session (1777)


Server Information:
Server Api: apache2handler
Server Db Type: MySQL (mysql)
Server Db Version: 5.0.22


----------------------------------------------

[tinhat]

OK - I read that in the change log for 1.6 we see:

" Fixed issue with breadcrumbs in admin theme"

I am assuming that along the way something has been rewritten that broke the second level hyperlinks in the admin pages?

I notice that in NCleanGreyTheme.php line 166 has been changed from:
[php]
if (str_replace('&', '&', $crumb['url']) != basename($_SERVER['REQUEST_URI']))
[/php]
to
[php]
if (isset($crumb['url']) && str_replace('&', '&', $crumb['url']) != basename($_SERVER['REQUEST_URI']))
[/php]
and I note that if I revert to the old code it would appear that $crumb['url'] isn't set properly - just points to /admin directory.

[Dr.CSS]

Like the breadcrumb on a site it doesn't have a link to the page you are on and most modules use tabs so if you are in module admin and go to another tab your still on the same page according to breadcrumb so it not broken is just the way it works...

[Franck]

Isn't there a way to modify this behavior and always have a link to the module's main page in the breadcrumb? It's really tiring (and slow), and not user friendly at all to always navigate in the submenu to get back to that page.