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.
entity.api.php is in core/includes. Surely it should be in core/modules/entity?
Comment | File | Size | Author |
---|---|---|---|
#5 | entity-2088775-5.patch | 187 bytes | tim.plunkett |
Comments
Comment #1
jhodgdonIt seems to have a bunch of probably obsolete hooks in it, like I'm pretty sure hook_entity_info() does not exist any more. It used to be in includes because there was no entity module... anyway yeah.
Comment #2
tim.plunkettNo, none of those hooks are relevant to the entity module, which might be renamed soon to entity_ui module. All of these are part of Drupal\Core\Entity, and we've left those sort of api.php in core/includes.
So either we leave it, start putting api.php files in core/lib/Drupal/Core/*, or move them to system.module
Comment #3
jhodgdonAh. I think in the past we've put other ones in the system directory. Isn't there a theme.api.php file there and some others? Alternatively, maybe all of them should be in core/includes, as that is really where the hooks in them are defined, rather than actually in the System module? or in core/lib? ??
Comment #4
joachim CreditAttribution: joachim commentedAll other api.php files live in module folders. Theme.api.php lives in system along with a few others:
modules/system/language.api.php
modules/system/system.api.php
modules/system/theme.api.php
Comment #5
tim.plunkettGreat.
Comment #6
jhodgdonyeah.
Comment #7
jhodgdonThanks again - committed to 8.x.