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

awolfey’s picture

Sub

gagarine’s picture

tracking it's also a problem with views tab.

yhahn’s picture

Status: Active » Fixed

The menu_set_active_item() has been removed as part of this commit: http://drupal.org/cvs?commit=406162

rdeboer’s picture

That's great!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.