Problem/Motivation

Followup to #2511968: Path field should fall back to language neutral aliases (also makes this happen for the form widget!)

I just realized language neutral aliases are not removed when deleting parent entty.

Proposed resolution

Adjust PathFieldItemList::delete() in order to remove aliases belonging to non-existing entities.

Remaining tasks

Create test
Create patch
Review

User interface changes

n/a

API changes

n/a

Data model changes

n/a

Release notes snippet

CommentFileSizeAuthor
#12 Screenshot from 2023-06-27 12-44-06.png25.28 KBzaporylie
#12 Screenshot from 2023-06-27 12-38-04.png42.25 KBzaporylie

Comments

zaporylie created an issue. See original summary.

anisha.challa’s picture

anisha.challa commented
Assigned: Unassigned » anisha.challa
anisha.challa’s picture

anisha.challa commented
Assigned: anisha.challa » Unassigned

Version: 8.6.x-dev » 8.8.x-dev

Drupal 8.6.x will not receive any further development aside from security fixes. Bug reports should be targeted against the 8.8.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.9.x-dev branch. For more information see the Drupal 8 and 9 minor version schedule and the Allowed changes during the Drupal 8 and 9 release cycles.

Version: 8.8.x-dev » 8.9.x-dev

Drupal 8.8.7 was released on June 3, 2020 and is the final full bugfix release for the Drupal 8.8.x series. Drupal 8.8.x will not receive any further development aside from security fixes. Sites should prepare to update to Drupal 8.9.0 or Drupal 9.0.0 for ongoing support.

Bug reports should be targeted against the 8.9.x-dev branch from now on, and new development or disruptive changes should be targeted against the 9.1.x-dev branch. For more information see the Drupal 8 and 9 minor version schedule and the Allowed changes during the Drupal 8 and 9 release cycles.

Version: 8.9.x-dev » 9.2.x-dev

Drupal 8 is end-of-life as of November 17, 2021. There will not be further changes made to Drupal 8. Bugfixes are now made to the 9.3.x and higher branches only. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

Version: 9.2.x-dev » 9.3.x-dev
larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
commented
Status: Active » Postponed (maintainer needs more info)
Issue tags: +Needs steps to reproduce, +Bug Smash Initiative

Can you provide steps to reproduce this?

What is the impact if these items aren't removed?

Version: 9.3.x-dev » 9.4.x-dev

Drupal 9.3.15 was released on June 1st, 2022 and is the final full bugfix release for the Drupal 9.3.x series. Drupal 9.3.x will not receive any further development aside from security fixes. Drupal 9 bug reports should be targeted for the 9.4.x-dev branch from now on, and new development or disruptive changes should be targeted for the 9.5.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

Version: 9.4.x-dev » 9.5.x-dev

Drupal 9.4.9 was released on December 7, 2022 and is the final full bugfix release for the Drupal 9.4.x series. Drupal 9.4.x will not receive any further development aside from security fixes. Drupal 9 bug reports should be targeted for the 9.5.x-dev branch from now on, and new development or disruptive changes should be targeted for the 10.1.x-dev branch. For more information see the Drupal core minor version schedule and the Allowed changes during the Drupal core release cycle.

smustgrave’s picture

smustgrave commented
Status: Postponed (maintainer needs more info) » Closed (outdated)

If still a valid bug please reopen updating issue summary with steps to reproduce per #8

Thanks!

zaporylie’s picture

zaporylie
he/him
Primary language Polish
commented
StatusFileSize
new Screenshot from 2023-06-27 12-38-04.png42.25 KB
new Screenshot from 2023-06-27 12-44-06.png25.28 KB

Interestingly I wasn't able to reproduce this particular bug anymore but when doing some manual testing I managed to end up with 2 aliases created for the same content - with the same path and language. Is that a known bug or should I report it separately and add steps to reproduce?