Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Berdir created an issue. See original summary.

Berdir’s picture

Status: Active » Needs review
FileSize
1.3 KB
Berdir’s picture

Status: Needs review » Needs work

Somehow not quite working yet.

Berdir’s picture

Status: Needs work » Needs review
FileSize
1.6 KB

Ok, I think it did work, but somehow I have some weird caching issues on my site. But I did simplify it to use not the trait but the actual class name and using bas field override entity query directly.

Could possibly do it in a single loop for both, but was too lazy for that :)

kristiaanvandeneynde’s picture

Don't think we should use the entity type manager in update hooks, right?

Berdir’s picture

Yeah, I was too lazy for that, looks good.

kristiaanvandeneynde’s picture

Status: Needs review » Fixed

Cheers, fixed.

Status: Fixed » Closed (fixed)

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