Hi guys,
Thanks a lot for this cool implementation for integration with views.
I was wondering if it would be possible to add a default configuration instance for Views contexts.
Please find attached to this ticket, a patch that has been tested against the latest dev version metatag_views-7.x-1.x (2012-03-06).
(File named: metatag_views-default-configuration-views-meta-tags-0.patch)
It is just a simple implementation of a standard metatag config instance:
hook_metatag_config_default()
You will also notice it requires a hook_ctools_plugin_api to be implemented in the module file to be declared to ctools as a metatag hook/config object.
I would tend to think it would be a good thing to add a default config instance to the module.
Besides, it could be disabled as well, from the metatag config overview page, or within the module (change the line $config->disabled to TRUE), if you wanted to get the same result as what it is currently (no default config for views), for some particular reason.
Please let me know if you would have any questions on any points/code/aspects mentioned in this ticket, I would surely be glad to provide more information.
I would greatly appreciate if you could take a bit of time to take a look at the patch and give me your feedback on this feature request.
Feel free to let me know if you would have any questions about the code or any other aspect of the patch, I would be glad to explain in more details.
Thanks in advance.
| Comment | File | Size | Author |
|---|---|---|---|
| metatag_views-default-configuration-views-meta-tags-0.patch | 1.46 KB | dydave |
Comments
Comment #1
dydave commentedChanging status for review.
Comment #2
damienmckennaThe module is deprecated, so this is no longer needed.