Problem/Motivation

#2968082: Breadcrumb paths with redirects/aliases introduced breadcrumb links for redirects. However, the redirect lookup includes the language prefix whereas the Redirect module expects only the path without the prefix. As a result, the breadcrumb does not include any links for redirects on a page that has a language prefix.

Proposed resolution

Strip the language prefix from the redirect lookup, so the breadcrumb includes links for redirects on a page that has a language prefix.

Remaining tasks

  1. Write a patch
  2. Review
  3. Commit

User interface changes

The breadcrumb includes links for redirects on a page that has a language prefix.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#2 3154473-2.patch739 bytesidebr

Comments

idebr created an issue. See original summary.

idebr’s picture

idebr commented
Status: Active » Needs review
StatusFileSize
new 3154473-2.patch739 bytes

Attached patch strips the language prefix from the redirect lookup, so the breadcrumb includes links for redirects on a page that has a language prefix.

anneke_vde’s picture

anneke_vde commented
Status: Needs review » Reviewed & tested by the community

Patch works as described. I tested this with a redirect on a page that has a language prefix and a page without a language prefix. Both redirects are working correct.

neslee canil pinto’s picture

neslee canil pinto
he/him
Primary language English
commented

+1, works for me as well.

neslee canil pinto’s picture

neslee canil pinto
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed to branch, Thanks.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.