Here for the community? On May 9th, we'll be in New Orleans. Don’t miss out!
... because it's the nice thing to do.
Here we go.
Manually checked all classes inherited from Entity - all complete!
Yep, makes sense.
Curious if NOT doing this causes any kind of issues that we could potentially test for? Anyone who's ported their entity-related module to D8 at this point probably has missed doing this in their own methods.
We don't have anything to test right now because all the parent methods are empty. #1605290: Enable entity render caching with cache tag support is the issue where we put some logic in two of those empty methods.
Cool, works for me.
Committed and pushed to 8.x. Thanks!
Needs a quick change notice.
Updated the code example in https://drupal.org/node/1400186 and added a sentence to the one that explains those methods.
I don't think this needs a separate change notice, we didn't even add a new one when we moved those methods to the Entity classes. This just applies a common OO pattern.
Makes sense. Thanks, Berdir!
Automatically closed -- issue fixed for 2 weeks with no activity.
Drupal is a registered trademark of Dries Buytaert.