Contrib.social

Duplicate path aliases can be added

Open on Drupal.org →
Created on 3 October 2014, over 9 years ago
Updated 3 April 2024, 3 months 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 6 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.
  • Merge request !7290Issue #2350135: Duplicate path aliases can be added. → (Open) created by AjitS
  • Assigned to AjitS
  • Status changed to Needs review 3 months ago
  • Comment 3 months ago →
    🇮🇳India AjitS Pune

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

  • Pipeline finished with Failed
    3 months ago
    Total: 206s
    #135486
  • Status changed to Needs work 3 months ago
  • Comment 3 months ago →
    🇮🇳India AjitS Pune

    Failures reported.

  • Pipeline finished with Failed
    3 months ago
    Total: 632s
    #135498
  • Pipeline finished with Failed
    3 months ago
    Total: 664s
    #136375
  • Pipeline finished with Failed
    3 months ago
    Total: 194s
    #136421
  • Pipeline finished with Success
    3 months ago
    Total: 752s
    #136425
  • Issue was unassigned.
  • Status changed to Needs review 3 months ago
  • Comment 3 months ago →
    🇮🇳India AjitS Pune

    Fixed the tests.

  • Status changed to Needs work 3 months ago
  • Comment 3 months 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

contrib.social Blog FAQ Discussions
Production build 0.69.0 2024