Problem/Motivation

A follow up to ensure that d7_url_alias migration correctly handles paths that do not have a slash. It is expected the path will be similar to the one from #2927227: d6_url_alias migration fails when the src path in D6 contains no slashes. Note that it will likely include a change to the d7 test fixture.

Proposed resolution

Using the d6 patch as guide, make one for d7.

Remaining tasks

Write a patch
Review
Commit

CommentFileSizeAuthor
#2 2942388-2.patch2.15 KBalexpott
#2 2942388-2.test-only.patch1.68 KBalexpott

Comments

quietone created an issue. See original summary.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Active » Needs review
StatusFileSize
new 2942388-2.test-only.patch1.68 KB
new 2942388-2.patch2.15 KB

Here's a patch and a failing test.

The last submitted patch, 2: 2942388-2.test-only.patch, failed testing. View results

jofitz’s picture

jofitz
he/him
commented
Status: Needs review » Reviewed & tested by the community

LGTM. Matches the functionality of the equivalent d6 patch. Test only patch fails, complete patch corrects the failure.

  • catch committed 132a42b on 8.6.x 
    Issue #2942388 by alexpott, quietone: Ensure d7_url_alias migration...
catch’s picture

catch
he/him
Primary language English
commented
Version: 8.6.x-dev » 8.5.x-dev
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.6.x and cherry-picked to 8.5.x. Thanks!

Status: Fixed » Closed (fixed)

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