In version 6.x-2.x-dev, function diff_nodeapi(..) contains the following @TODO:
// @TODO: This is a bad hack that should instead be a patch against the node module.
menu_set_active_item("node/{$node->nid}");
I was wondering whether diff_nodeapi(..) could be rewired to not use this statement, as it effectively hijacks the original page, preventing other modules from doing something with it, see for instance #658550: Edit tab not displayed when Diff module installed and "Inline diffs" ticked for content type and http://drupal.org/node/586764#comment-2353816.
Comments
Comment #1
awolfey commentedSub
Comment #2
gagarine commentedtracking it's also a problem with views tab.
Comment #3
yhahn commentedThe
menu_set_active_item()has been removed as part of this commit: http://drupal.org/cvs?commit=406162Comment #4
rdeboerThat's great!