Contrib.social

Redirects for path aliases not shown on node edit page

Open on Drupal.org →
Created on 21 February 2021, almost 4 years ago
Updated 4 April 2024, 8 months ago

Problem/Motivation

As a user if I'm creating redirects to a particular node using its path alias, I want to see them on node edit page.

Steps to reproduce

1. Create a node of any content type with a path alias.
2. Create 3 redirects, one with /node/nid, one with /path-alias and the last one with entity reference.
3. Go to node edit page, and check the URL Redirects section.
Actual:
Only redirects for internal:/node/nid and entity:node/nid are shown.
Expected:
Path alias redirect should also be shown in the list.

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

🐛 Bug report
Status

Needs work

Version

1.0

Component

Code

Created by

🇮🇳India amangrover90

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

Comments & Activities

Not all content is available!

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

  • Comment over 1 year ago →
    🇺🇸United States DamienMcKenna NH, USA

    The correct status for a patch / merge request that you believe covers what's needed is "Reviews and tested by the community"; "Patch (to be ported)" means that a change was committed to one branch and then needs to be ported to work with another branch, e.g. a fix was committed to 7.x-1.x and then needs to be rewritten to work on 8.x-1.x.

  • Comment about 1 year ago →
    🇮🇳India vishal prasad

    Vishal Prasad → made their first commit to this issue’s fork.

  • Open in Jenkins → Open on Drupal.org →
    Core: 9.5.x + Environment: PHP 7.4 & MySQL 5.7
    last update about 1 year ago
    63 pass
  • @vishal-prasad opened merge request.
  • Status changed to RTBC about 1 year ago
  • Comment about 1 year ago →
    🇮🇳India diwakar07

    I have reviewed the merge request !61 raised by Vishal Prasad as hence confirm that the patch is working fine to solve the problem.

    I have attached the screenshots of the test case and the desired output.

  • Comment about 1 year ago →
    🇬🇧United Kingdom Alina Basarabeanu

    The changes from the merge request !6 fixed the issue on Drupal 10.1.5 and Redirect 1.9.0.

  • Status changed to Needs work 8 months ago
  • Comment 8 months ago →
    🇨🇭Switzerland berdir Switzerland

    Such redirects are currently not supported, so merging this is misleading and will not be done until that changes. In fact, it should probably be part of that patch/issue. FWIW, I do not agree with the current implementation in said issue.

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024