I've attached, for discussion, a patch to get nodewords to apply the taxonomy settings to ubercart catalog pages.
We are happily using nodewords to on other types of pages, but it doesn't work for ubercart category pages, which are basically fancy taxonomy term pages.
So my questions are:
* Is this the correct way to go about getting nodewords to work?
* If it is, would it be better to have a separate setting for taxonomy pages and for ubercart category pages?
* If it is isn't, what would be the correct way about getting this to work correctly? Are there any plans to make some sort of hook system to extend the nodewords module?
I believe there would be some demand for this feature from ubercart users.
| Comment | File | Size | Author |
|---|---|---|---|
| nodewords-ubercart-catalog-20091119.patch | 578 bytes | dougp |
Comments
Comment #1
avpadernoThat line is redundant, as the value of
$arg[0]is already checked with the switch statement.Patches to the code must be against the development snapshot.
To change the code I need to know the exact name of the module that implement that menu callback. Is it ubercart.module?
Comment #2
avpadernoI think that the module name is uc_catalog.module.
The code has been changed, and committed in CVS.
Thanks for your report.
Comment #3
dougp commentedYeah, it's uc_catalog. Thanks for that!