Update the module admin forms for D7.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | taxonomy-access-admin-forms-1097782-1.patch | 12.49 KB | David_Rothstein |
Update the module admin forms for D7.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | taxonomy-access-admin-forms-1097782-1.patch | 12.49 KB | David_Rothstein |
Comments
Comment #1
David_Rothstein commentedHere's a first pass. The admin forms aren't fully working after this (not by any means) but they do whitescreen less often than they did before :)
I tried to focus mostly on the code in taxonomy_access_admin.inc (even though many API functions called from there are still broken). So it might be at a pretty good committable state.
Comment #2
xjmWell, global defaults can be configured, at least. Looks like some of the helper functions need to have their new queries committed before configuring terms or defaults will work. I'll migrate those.
Comment #3
xjmComment #4
xjmhttp://drupalcode.org/project/taxonomy_access.git/commit/b07689d
Comment #5
David_Rothstein commentedGreat, glad you were able to finish those up. I guess I didn't fully appreciate that if I signed up to "make the admin forms work in D7", doing that completely would basically be the equivalent of porting the entire module to D7 :)
Comment #6
David_Rothstein commentedSkimming through the committed patch, in _taxonomy_access_get_nodes_for_vocabulary():
Is this change actually right? Looks like it switched from NOT IN to IN...
This doesn't specify what columns to join on - I assume it needs to?
Comment #7
xjmWhoops. That probably explains why node access was corrupted a few times while I was testing the patch; I'd assumed it had something to do with the as-yet-unfinished node API changes. I thought the second bit (without an explicit "ON") was supposed to automatically use a matching key name if one was present, but I tested it and it doesn't actually return the same records.
http://drupalcode.org/project/taxonomy_access.git/commit/a3d65f2
In light of this I think #903120: Add test suite should probably be completed before a 7.x-1.0 release.