Hi!
I was in need to override some of meta tag permissions as I didn't want to give end user full access of all meta tags. So I made this module which is basically rewrite of override node options module.
You might be interested in including this with your code as it might be pretty handy for lots of people.
| Comment | File | Size | Author |
|---|---|---|---|
| #13 | metatag-n1961354-13.patch | 4.39 KB | damienmckenna |
| #12 | metatag-n1961354-12.patch | 4.34 KB | damienmckenna |
| #10 | metatag_remove_empty_groups.patch | 2.67 KB | thesame- |
| #8 | remove_empty_goups.patch | 4.98 KB | thesame- |
| #5 | remove_empty_goups.patch | 1.31 KB | thesame- |
Comments
Comment #1
damienmckennaThat's an interesting idea, thanks for the code. I know Nodewords (D6) had separate permissions for each meta tag, while not every meta tag needs it, it might be worth including as a submodule with a disclaimer that "this will make your Permissions page even more crazy."
Comment #2
thesame- commentedHa ha! :D I like your idea of disclaimer!
Comment #3
damienmckennaHow about this alternative approach? It provides a hidden variable, metatag_extended_permissions, which can be set to TRUE to turn on the Extended Permissions option. When this is enabled every single meta tag will have its own permission, users will have to have the 'edit meta tags' permission to see the fieldset and then their access to individual meta tags can be assigned. Food for thought, and I haven't tested it fully yet.
Comment #4
thesame- commentedThis seams to be a great idea. And code looks cleaner as well. Good job. :)
Comment #5
thesame- commentedI realized my sub-module didn't work for taxonomy terms. I gave a shot to your patch.
Got Notice: Undefined variable: metatag_name in metatag_metatags_form() (line 901 of /sites/all/modules/metatag/metatag.module).
I also noticed empty meta tag groups are not removed and nether is Meta tags tab, if user doesn't have permission to edit any of meta tags.
Patch in attachment.
Comment #7
thesame- commented#5: remove_empty_goups.patch queued for re-testing.
Comment #8
thesame- commentedI have no idea how to edit patch for re-testing. I'll just upload it again.
Comment #10
thesame- commentedOk something went seriously wrong. This patch should be fine. I hope...
Comment #11
damienmckennaComment #12
damienmckennaThis merges together the changes from #10 with the README.txt changes from #3.
Comment #13
damienmckennaSome small improvements to the README.txt file.
Comment #14
damienmckennaTag.
Comment #15
damienmckennaI've tested this and it works as intended. Thanks for the help, thesame!
Comment #16
thesame- commentedGlad to be helpful! ;]
Comment #17.0
(not verified) commentedtypo