Duplicate path aliases can be added

Created on 3 October 2014, over 10 years ago
Updated 3 April 2024, about 1 year ago

When adding aliases through the admin interface, there is a validation check (function aliasExists) to make sure you are not reusing an existing alias. Strangely the function does not prevent you from making a duplicate alias.

To reproduce, go to /admin/config/search/path/add and add an alias with a source. Then do it again (same alias, same source).

If you use different sources you will be stopped from reusing an alias, but nothing stops you from adding a duplicate. Why allow that?

🐛 Bug report
Status

Needs work

Version

11.0 🔥

Component
Path 

Last updated 12 days ago

  • Maintained by
  • 🇬🇧United Kingdom @catch
Created by

🇺🇸United States Jody Lynn

Live updates comments and jobs are added and updated live.
  • Needs issue summary update

    Issue summaries save everyone time if they are kept up-to-date. See Update issue summary task instructions.

Sign in to follow issues

Merge Requests

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • First commit to issue fork.
  • Assigned to ajits
  • Status changed to Needs review about 1 year ago
  • 🇮🇳India ajits India

    Created reroll for 11.x of #62. I will work on test failures after testbot results.

  • Pipeline finished with Failed
    about 1 year ago
    Total: 206s
    #135486
  • Status changed to Needs work about 1 year ago
  • 🇮🇳India ajits India

    Failures reported.

  • Pipeline finished with Failed
    about 1 year ago
    Total: 632s
    #135498
  • Pipeline finished with Failed
    about 1 year ago
    Total: 664s
    #136375
  • Pipeline finished with Failed
    about 1 year ago
    Total: 194s
    #136421
  • Pipeline finished with Success
    about 1 year ago
    Total: 752s
    #136425
  • Issue was unassigned.
  • Status changed to Needs review about 1 year ago
  • 🇮🇳India ajits India

    Fixed the tests.

  • Status changed to Needs work about 1 year ago
  • 🇺🇸United States smustgrave

    Issue summary is incomplete.

    But think it needs to be documented why the json tests needed to be updated. Doesn't seem like something that should need to be fixed based on the title of duplicate aliases. Could be wrong

Production build 0.71.5 2024