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.Currently, TAC's database tables are installed with CREATE TABLE queries specific to PostgreSQL and MySQL. Using the Schema API instead will improve portability, make a D7 port easier to accomplish, and allow for cleaner INSERT query syntax.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | taxonomy_access-723668-1.patch | 9.47 KB | xjm |
Comments
Comment #1
xjmThis patch implements
hook_schema()intaxonomy_access.install. (There's no need for ahook_update_N()because the CREATE queries themselves aren't changing, just the source of the calls.)I've tested this with MySQL and it works fine, but I haven't tested it with postgres.
Comment #2
xjmAccess permissions seem to need to be rebuilt frequently with this patch in both MySQL and PostgreSQL. Needs more testing.
Comment #3
xjmActually, the issue of the need for repeated rebuilds I mention in #2 (specifically after modifying TAC configurations) seems to occur in the unpatched module as well. So, it's unrelated to the patch. I'll open a new issue for that. Edit: refer to #727648: Node access not updated on default change for that bug.
Comment #4
xjmI've now tested this fairly thoroughly on both MySQL and PostgreSQL; the behavior is the same as the current dev build in every situation I tested.
Comment #5
xjmCommitted to 6.x-1.x-dev.