Remove -settings from Search settings page path

Created on 7 May 2025, 10 days ago

Problem/Motivation

For some reason, there is an extra -settings bit in the "Admin Toolbar Search settings" path, which should probably be removed, since that's not the convention:

  • /admin/config/user-interface/admin-toolbar
  • /admin/config/user-interface/admin-toolbar-search-settings
  • /admin/config/user-interface/admin-toolbar-tools

Steps to reproduce

Visit the three Admin Toolbar pages, and see an extra -settings in /admin/config/user-interface/admin-toolbar-search-settings.

Proposed resolution

Remove -settings from this path:

Now:
/admin/config/user-interface/admin-toolbar-search-settings

Proposal:
/admin/config/user-interface/admin-toolbar-search

Remaining tasks

User interface changes

API changes

Data model changes

🐛 Bug report
Status

Active

Version

3.0

Component

Code

Created by

🇩🇰Denmark ressa Copenhagen

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @ressa
  • Pipeline finished with Failed
    5 days ago
    Total: 379s
    #494575
  • 🇮🇳India chhavi.sharma

    Removed -settings from Search settings page path. The issue needs review.
    Previous Path:

    Altered Path:

  • 🇩🇰Denmark ressa Copenhagen

    Thanks @chhavi.sharma that looks much better. It looks like the test needs to be updated as well :)

    It's this one:

    admin_toolbar_search/tests/src/Functional/AdminToolbarSearchSettingsFormTest.php

  • First commit to issue fork.
  • 🇫🇷France dydave

    Indeed, great catch @ressa! 👍

    I've just made the change to the MR and it is back to passing 🟢
    Glad we have tests in place 🙂

    Moving issue back to Needs review, for a final confirmation, before it gets merged 👌

    Thanks again for all the great help!

  • 🇩🇰Denmark ressa Copenhagen

    Thanks @dydave, for a speedy reply and update as always!

    The path is fixed and the tests are green (I agree, tests really are great, when they catch these small, easily overlooked details) so let's ship it! 🚀

  • 🇫🇷France dydave

    Thanks a lot @ressa and @chhavi.sharma for your great help on this issue! 🙏

    Following your confirmation at #8, I went ahead and merged the changes above at #9 👌

    Another issue to scratch off our list, included in the next release! 🥳

    Getting closer to the release date ... Let's see what else we could squeeze in 😅

    Thanks again @ressa for suggesting these issues and @chhavi.sharma for your help with the code changes!

  • 🇩🇰Denmark ressa Copenhagen

    Thanks @chhavi.sharma and @dydave for fixing this so fast, with lightning speed! ⚡

    Awesome to get another little improvement included, before release. Thanks!

Production build 0.71.5 2024