According to https://api.drupal.org/api/drupal/core!modules!system!entity.api.php/gro...
> For content entities, the annotation will refer to a number of database tables and their fields. These annotation properties, such as 'base_table', 'data_table', 'entity_keys', etc., are documented on \Drupal\Core\Entity\EntityType. Your module will also need to set up its database tables using hook_schema().
This is no longer true, as can be seen with Node module for instance. Entity tables are no longer defined in hook_schema().
Novice task/proposed resolution
Take out the sentence that says you will need to set up the database tables.
Comment | File | Size | Author |
---|---|---|---|
#7 | Fix_entity_api_doc_about_hook_schema_2491691-7.patch | 831 bytes | marcoscano |
Comments
Comment #1
jhodgdonYeah, that changed and the docs need to be fixed.
Comment #2
jmolivas CreditAttribution: jmolivas at Blink Reaction (now part of FFW) commentedFixing broken link to the docs
Comment #3
cilefen CreditAttribution: cilefen commented@jmolivas Please note in the issue summary if there are any tasks a novice can perform after you have looked it over.
Comment #4
cilefen CreditAttribution: cilefen commented@jmolivas Sorry, I misunderstood. Thank you for fixing the link.
Comment #5
jhodgdonI think all we need in the patch is for the sentence about hook_schema to be taken out.
Comment #6
cilefen CreditAttribution: cilefen commentedComment #7
marcoscanoGood novice task :)
Hope I didn't mess up with anything
Comment #8
jhodgdonThat's it, thanks!
Comment #9
webchickCommitted and pushed to 8.0.x. Thanks!
Comment #12
xjmHah. :) Legacy bit left over from our second-to-last beta blocker. This issue only changes documentation, so per https://www.drupal.org/core/beta-changes, this can be completed any time during the Drupal 8 beta phase. Committed and pushed to 8.0.x. Thanks everyone!
Comment #13
xjmlol. Looks like @webchick won that crosspost.