Problem/Motivation

This bug was introduced in #2543726: Make $term->parent behave like any other entity reference field, to fix REST and Migrate support and de-customize its Views integration, which added a new \Drupal\taxonomy\TermStorage::$ancestors property but forgot to unset it in the __sleep() method.

Proposed resolution

Unset it.

Remaining tasks

Review.

User interface changes

Nope.

API changes

Nope.

Data model changes

Nope.

CommentFileSizeAuthor
#2 2994830.patch659 bytesamateescu

Comments

amateescu created an issue. See original summary.

amateescu’s picture

amateescu commented
Status: Active » Needs review
StatusFileSize
new 2994830.patch659 bytes

No idea how to test this, but the fix is quite obvious..

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Needs review » Reviewed & tested by the community

Nice catch

  • catch committed 39d22bf on 8.7.x 
    Issue #2994830 by amateescu: TermStorage::__sleep() does not unset the '...

  • catch committed e3dd5d5 on 8.6.x 
    Issue #2994830 by amateescu: TermStorage::__sleep() does not unset the '...
catch’s picture

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

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

Status: Fixed » Closed (fixed)

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