Re: [development] (no subject)

[Brian Vuyk]

Oops, I realize I misspoke. I would set up the columns with hook_install() / hook_uninstall(), but would also add them to the schema with hook_schema_alter(). On 10-07-12 12:30 PM, Brian Vuyk wrote: I don't see any problem with it, although I would add / remove the columns with hook_install() / hook_uninstall() rather than hook_schema_alter(). Brian On 10-07-12 12:19 PM, nan wich wrote: Is there an official stance on using hook_schema_alter to add columns to core tables? For example, we collect additional data on anonymous comments and want to actually save that data. Rather than creating another table (and subsequent JOINs), I'd just as soon stuff that data into the comments table, where it belongs. Should we ever disable comments (unlikely), we probably wouldn't mind losing that data too.   Nancy E. Wichmann, PMP Injustice anywhere is a threat to justice everywhere. -- Dr. Martin L. King, Jr.