Commit e1812b33 authored by webchick's avatar webchick
Browse files

#678628 follow-up by jhodgdon: Remove buggy and incorrect 'More Help'...

#678628 follow-up by jhodgdon: Remove buggy and incorrect 'More Help' functionality from Help module.
parent 26417f9d
......@@ -1495,12 +1495,6 @@ function menu_get_active_help() {
// Lookup help for this path.
if ($help = $function($router_path, $arg)) {
$output .= $help . "\n";
// Add "more help" link on admin pages if the module provides a
// standalone help page.
if ($arg[0] == "admin" && user_access('access administration pages') && module_exists('help') && $function('admin/help#' . $module, $empty_arg)) {
$output .= theme("more_help_link", array('url' => url('admin/help/' . $module)));
}
}
}
return $output;
......
......@@ -53,11 +53,6 @@ class HelpTestCase extends DrupalWebTestCase {
foreach ($this->modules as $module => $name) {
$this->assertLink($name, 0, t('Link properly added to @name (admin/help/@module)', array('@module' => $module, '@name' => $name)));
}
// Verify that the "More help" links generated by menu.inc work properly.
// Test on an out-of-the-way page.
$this->drupalGet('admin/config/people/roles');
$this->assertRaw(theme("more_help_link", array('url' => url('admin/help/user'))), t('"More help" link is functional.'));
}
/**
......
......@@ -98,10 +98,6 @@ function node_help($path, $arg) {
$output .= '</dl>';
return $output;
case 'admin/content':
// Return a non-null value so that the 'more help' link is shown.
return ' ';
case 'admin/structure/types/add':
return '<p>' . t('Individual content types can have different fields, behaviors, and permissions assigned to them.') . '</p>';
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment