Skip to content
Snippets Groups Projects
Commit 67e4f744 authored by Dries Buytaert's avatar Dries Buytaert
Browse files

- Usability improvement: the 'recent posts' feature on the user pages lead
  to a page with a confusing (wrong) title so I shuffled things around a bit
  and tab-ified it as 'track'.
parent 67925ce0
No related branches found
No related tags found
2 merge requests!7452Issue #1797438. HTML5 validation is preventing form submit and not fully...,!789Issue #3210310: Adjust Database API to remove deprecated Drupal 9 code in Drupal 10
...@@ -37,10 +37,24 @@ function tracker_menu($may_cache) { ...@@ -37,10 +37,24 @@ function tracker_menu($may_cache) {
'type' => MENU_LOCAL_TASK); 'type' => MENU_LOCAL_TASK);
} }
} }
else {
if (arg(0) == 'user' && is_numeric(arg(1))) {
$items[] = array('path' => 'user/'. arg(1) .'/track', 'title' => t('track posts'),
'callback' => 'tracker_track_user', 'access' => user_access('access content'),
'type' => MENU_LOCAL_TASK);
}
}
return $items; return $items;
} }
/**
* Menu callback. Prints a listing of active nodes on the site.
*/
function tracker_track_user() {
tracker_page(arg(1));
}
/** /**
* Menu callback. Prints a listing of active nodes on the site. * Menu callback. Prints a listing of active nodes on the site.
*/ */
...@@ -90,10 +104,4 @@ function tracker_page($uid = 0) { ...@@ -90,10 +104,4 @@ function tracker_page($uid = 0) {
print theme('page', $output); print theme('page', $output);
} }
function tracker_user($type, &$edit, &$user) {
if ($type == 'view' && user_access('access content')) {
return array(t('History') => form_item(t('Recent posts'), l(t('recent posts'), "tracker/$user->uid")));
}
}
?> ?>
...@@ -37,10 +37,24 @@ function tracker_menu($may_cache) { ...@@ -37,10 +37,24 @@ function tracker_menu($may_cache) {
'type' => MENU_LOCAL_TASK); 'type' => MENU_LOCAL_TASK);
} }
} }
else {
if (arg(0) == 'user' && is_numeric(arg(1))) {
$items[] = array('path' => 'user/'. arg(1) .'/track', 'title' => t('track posts'),
'callback' => 'tracker_track_user', 'access' => user_access('access content'),
'type' => MENU_LOCAL_TASK);
}
}
return $items; return $items;
} }
/**
* Menu callback. Prints a listing of active nodes on the site.
*/
function tracker_track_user() {
tracker_page(arg(1));
}
/** /**
* Menu callback. Prints a listing of active nodes on the site. * Menu callback. Prints a listing of active nodes on the site.
*/ */
...@@ -90,10 +104,4 @@ function tracker_page($uid = 0) { ...@@ -90,10 +104,4 @@ function tracker_page($uid = 0) {
print theme('page', $output); print theme('page', $output);
} }
function tracker_user($type, &$edit, &$user) {
if ($type == 'view' && user_access('access content')) {
return array(t('History') => form_item(t('Recent posts'), l(t('recent posts'), "tracker/$user->uid")));
}
}
?> ?>
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment