#148677 by pwolanin: set active menu item on 403/404 pages (and on the user page)

includes/common.inc

@@ -340,6 +340,9 @@ function drupal_not_found() {
 
   $path = drupal_get_normal_path(variable_get('site_404', ''));
   if ($path && $path != $_GET['q']) {
+    // Set the active item in case there are tabs to display, or other
+    // dependencies on the path.
+    menu_set_active_item($path);
     $return = menu_execute_active_handler($path);
   }
 
@@ -365,6 +368,9 @@ function drupal_access_denied() {
 
   $path = drupal_get_normal_path(variable_get('site_403', ''));
   if ($path && $path != $_GET['q']) {
+    // Set the active item in case there are tabs to display, or other
+    // dependencies on the path.
+    menu_set_active_item($path);
     $return = menu_execute_active_handler($path);
   }
 

includes/menu.inc

@@ -1240,7 +1240,18 @@ function menu_get_active_menu_name() {
   return menu_set_active_menu_name();
 }
 
-function menu_set_active_item() {
+/**
+ * Set the active path, which determines which page is loaded.
+ *
+ * @param $path
+ *   A Drupal path - not a path alias.
+ *
+ * Note that this may not have the desired effect unless invoked very early
+ * in the page load, such as during hook_boot, or unless you call
+ * menu_execute_active_handler() to generate your page output.
+ */
+function menu_set_active_item($path) {
+  $_GET['q'] = $path;
 }
 
 /**

modules/user/user.module

@@ -1138,7 +1138,7 @@ function user_set_authmaps($account, $authmaps) {
 function user_page() {
   global $user;
   if ($user->uid) {
-    $_GET['q'] = 'user/'. $user->uid;
+    menu_set_active_item('user/'. $user->uid);
     return menu_execute_active_handler();
   }
   else {