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.
Description keys were removed from the schema cache in Drupal 7. This broke the describe tab which no longer shows any descriptions for tables and fields.
Proposed solution is to get the full schema including descriptions without using drupal_get_schema().
Comment | File | Size | Author |
---|---|---|---|
#7 | 1792608-schema-missing-descriptions_7.patch | 4.44 KB | greggles |
| |||
#3 | 1792608-schema-missing-descriptions.patch | 4.46 KB | Dave Reid |
#1 | schema-construct-schema-manually-1792608-1.patch | 702 bytes | Antti J. Salminen |
Comments
Comment #1
Antti J. Salminen CreditAttribution: Antti J. Salminen commentedHere's an attached patch that removes drupal_get_schema() usage and constructs the schema in a way similar to what drupal_get_complete_schema() does.
Comment #2
jneubert CreditAttribution: jneubert commentedWorked fine for me - thanks!
Comment #3
Dave ReidMoved this to a re-usable function that could be used by all.
Comment #4
Liam MorlandThe comment before the call to _drupal_schema_initialize() needs to be updated.
Comment #5
Liam MorlandThis could also be fixed by #2369097: Allow descriptions to be returned by drupal_get_schema().
Comment #6
Liam MorlandComment #7
gregglesI've rerolled this to updated the comment as suggested by Liam Morland, but it's basically the same as Dave's work.
Moving to RTBC since I only changed the comment.
I also noticed some other issues in the queue that seem to be related and/or dups of this issue, so it seems worthwhile to commit and release soon.
Comment #8
gregglesIMO this is at least major since it affects a pretty basic use of the module.
Comment #10
joachim CreditAttribution: joachim as a volunteer commentedThanks for the reroll!
Committed.
Comment #11
joachim CreditAttribution: joachim as a volunteer commentedThis warrants a new release (though it's clearly been broken for ages!). I'll check the issue queue for any low-hanging fruit and make a new release soon.
Comment #12
gregglesAgreed about a new release. Thanks for your recent work on this module!