Search module is to be deprecated.
function hook_menu_links_discovered_alter(&$links) {
// Change the weight and title of the user.logout link.
$links['user.logout']['weight'] = -10;
$links['user.logout']['title'] = new \Drupal\Core\StringTranslation\TranslatableMarkup('Logout');
// Conditionally add an additional link with a title that's not translated.
if (\Drupal::moduleHandler()->moduleExists('search')) {
$links['menu.api.search'] = [
'title' => \Drupal::config('system.site')->get('name'),
'route_name' => 'menu.api.search',
'description' => new \Drupal\Core\StringTranslation\TranslatableMarkup('View popular search phrases for this site.'),
'parent' => 'system.admin_reports',
];
}
}| Comment | File | Size | Author |
|---|
Show commandsStart within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
changes, plain diff MR !16452
Comments
Comment #4
danielvezaSwapped the Search reference over to Navigation. Rather than coming up with an example I used one of the links that the Navigation module creates.
Comment #5
smustgrave commentedLike the example replacement. Know navigation is safe in core for a while.
Comment #6
needs-review-queue-bot commentedThe Needs Review Queue Bot tested this issue. It fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide to find step-by-step guides for working with issues.
Comment #7
danielvezaPipeline is passing, assuming thats a false positive
Comment #8
quietone commentedYea, this is a doc only change.
Comment #10
catchCommit checks failed for me locally but it's because phpstan doesn't have any files to verify. I think we have an issue for that already so that it silently does nothing instead of blowing up, but explains the false positive.
Committed/pushed to main and 11.x, thanks!