- ๐ฌ๐งUnited Kingdom alexpott ๐ช๐บ๐
So I think we only need the StatisticsLastUpdated field, filter and sort to entity types which implement EntityChangedInterface and have a changed field. That way we avoid quite a few issues. If you want to filter on the last_updated time in comment_entity_statistics then you can use the last comment timestamp field, filter and sort... you do not need this special implementation.
- ๐ฌ๐งUnited Kingdom alexpott ๐ช๐บ๐
@pandepoulus yep assigning the return $this->query->orderBy() to a property is pointless. I've updated the MR to fix this for 11.x.
We still need to add test coverage for the comments on entity types that do not implement EntityChangedInterface
- First commit to issue fork.
- ๐ฌ๐งUnited Kingdom catch
Moving this to Drupal core because the ideas queue is deprecated, however I think we need concrete suggestions for changes to be laid out a bit more. The only active issue I can think of is โจ Add tracking for when recipes from Drupal.org are applied Active .