Now that #12201: Show table descriptions in PHPMyAdmin and other tools has been committed, update functions that use db_add_table, db_add/change_field etc. should include description in the schema specification.

CommentFileSizeAuthor
#1 description.patch877 bytesmfb
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mfb’s picture

mfb
they or he
CreditAttribution: mfb commented
Component: update system » documentation
Status: Active » Needs review
FileSize
description.patch877 bytes

I patched the hook_update_N() documentation to reflect this. Not clear that it's worth patching all the hook_update_7*() functions but maybe a good idea?

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Issue tags: +Novice

If we're going to enforce it in the API docs, core should follow it for it's own updates I think. This seems like a decent patch to work on for a new contributor - so tagging as Novice.

mfb’s picture

mfb
they or he
CreditAttribution: mfb commented

In the meantime, it might be a good idea to just commit this patch on hook_update_N() so at least the documentation is correct.

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Needs review » Reviewed & tested by the community

Good idea.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

I committed this patch to CVS HEAD yesterday but forgot to update the issue status. Thanks.

Status: Fixed » Closed (fixed)
Issue tags: -Novice

Automatically closed -- issue fixed for 2 weeks with no activity.

Issue tags: +Novice

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.