Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
This one might need some explanation though. ;)
We, as a collective, had issues with @fago's usual way of "over-abstraction" in the past. However, this concern was communicated, and he welcomed the feedback and even started to "simplify" some stuff in contrib already.
At this point, partially due to entity.module in contrib, @fago is probably the most experienced API developer in the entity system field.
His recent replies in the core queue attest that. (like #1042822-46: Developers need an $entity->entity_type property)
Comment | File | Size | Author |
---|---|---|---|
#9 | d7_entity.patch | 416 bytes | fago |
drupal8.entity-fago-maintainer.0.patch | 365 bytes | sun | |
Comments
Comment #1
fagolol - yep, that sort of feedback is invaluable!
I'll continue working on the entity API and moving it forward while making sure it works for developers. So yep, I'd be happy to help out as maintainer!
Comment #2
yched CreditAttribution: yched commentedSounds only logical.
Comment #3
webchickDoes this need backport to D7 too? :)
Comment #4
Lars Toomre CreditAttribution: Lars Toomre commentedOnly if this meets the D7 coding standards webchick. Now where were those documented? LOL :)
Comment #5
webchick??
Right here: http://drupal.org/coding-standards
If people want to add D8-only rules to the coding standards, it's fine with me, and up to them to document them as such. :)
Comment #6
catchCommitted and pushed to 8.x.
Moving CNR for 7.x, not sure if fago volunteered for that too or not ;)
Comment #7
fagoYay :)
@d7: Yep, I'd be happy to cover d7 too.
Comment #8
webchickyay. :)
Comment #9
fagoRe-rolled patch for d7 to apply without fuzz.
Comment #11
tstoeckler#9: d7_entity.patch queued for re-testing.
Comment #12
amateescu CreditAttribution: amateescu commentedHey, how did we forget about this? :)
Comment #13
webchickWoohoo! :) Committed and pushed to 7.x. Thanks!
Comment #14
fagoOh, I had forgotten that it's still open for D7 too. :-)